Received: by 10.213.65.68 with SMTP id h4csp273981imn; Sat, 17 Mar 2018 03:30:20 -0700 (PDT) X-Google-Smtp-Source: AG47ELu0dtG3t8DUdnZ99oLpsddWrf0+B9upbROWiXePgYrdQPYx8Z7i/ZOGQ8v16bOsnTnznzLy X-Received: by 2002:a17:902:a5c5:: with SMTP id t5-v6mr5309882plq.244.1521282620418; Sat, 17 Mar 2018 03:30:20 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521282620; cv=none; d=google.com; s=arc-20160816; b=lCMEZyAxuQ8+Gk7slxD8ELOH0R+spvKq7BlO/XeReULQNb+GegkJ7fiuEHu2rrJuUl YOMaXGQafGYM2L8Dy+xWnPfez0zyKzI3Rnoydk5mlQ/QFaUKp9nq+bAQpMkPoCyYiSj5 sQiICfT0p+d9pR9PhrOkDRM4kgafLlibofBCg/wt4vfVPR8sGvc/ndKSKhOQJuGwqhoJ WCEpyiru/goCDwFRDPqVdNGjVgl4uPSASi0YGsCATDYbqggTAoNBrsEYVqRwqf+aS8fa Nl5e6Zaf3VOsg0za2ZY88wYMDCysfJTXpDRLhJIfPfgMowkpJDWkxrjyOV1kP/roWROb OlXw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:arc-authentication-results; bh=fnDAh+kzlCZKeVTYtcR3JuwPew8WWrM7In26S056CPY=; b=etjJl/cMC5IQIUH9FZ2ll/VKaEAy8tvkUctFHvXaQ/5Rme/HnRJ0yUVkkyVt/pqoDe ZojcVBI46ytxBuNhtYN4rY2L5lk7LrPd+IZ2n49M5I7oSSKu6iDMSXzP9c9MzdKmtrfA 9RVDi+z6g6nLbugp1Ii6PCXVpB3gbnPlhOXnc5+S6SaJEOh5FqrTadkUZ3K1nzOth02g lH3/3E8BUCEOyECYy73kkJ36ZmpWBXGm9yFz33/RT1hNQT7ZZYf2DMhnOTe3MveGRxmR O3hwiCOHORU5PXRZICkAHkDEymgoGtiOihGjtpTqRknHztlYUWpM1R7Gihi2EjetoNtb Xwgw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=canonical.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l6si4340709pfg.218.2018.03.17.03.30.04; Sat, 17 Mar 2018 03:30:20 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=canonical.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752276AbeCQK3I (ORCPT + 99 others); Sat, 17 Mar 2018 06:29:08 -0400 Received: from youngberry.canonical.com ([91.189.89.112]:59398 "EHLO youngberry.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751677AbeCQK3H (ORCPT ); Sat, 17 Mar 2018 06:29:07 -0400 Received: from mail-wr0-f199.google.com ([209.85.128.199]) by youngberry.canonical.com with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.76) (envelope-from ) id 1ex95F-0005r2-VK for linux-kernel@vger.kernel.org; Sat, 17 Mar 2018 10:29:05 +0000 Received: by mail-wr0-f199.google.com with SMTP id k44so6889449wrc.3 for ; Sat, 17 Mar 2018 03:29:05 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=fnDAh+kzlCZKeVTYtcR3JuwPew8WWrM7In26S056CPY=; b=r/+jhrmC9deIrrTOSJlFH4iuMgeuNfm0kBbNN/JkfJsgrOU3hbEo7HRdifqrWYl2P3 nErnNJ8lT6td4Ba8HMkGLV4SWhGMm2UtkXn9D3txEkPDDM7v5BLDFlb/WaWIBZY8CJQa 9OVJJSsIFrx8mNbrrgEEmPZJBjG6mNDLAHi1R2YAjpLJXVXyR5ecURAfttsEcWOaUQ5M Mnx0EPxN3uo8nN1rBEWOgmImhmysG9riKV8Hx3+VzY0725Mix3IkwdbkBjubyQwcOvcq SG1EEy67a9bf5x1hkRx/k2eWjLbzRjBKBie9ZqrlEGjih0XVC4wEZjRHGdv+r6yWIkkf Hv9A== X-Gm-Message-State: AElRT7HqwvgVV/Z7Vern+76kl+hWMlmC3zJyX+N55X/gmN4MwP/rAvgc xQF+IfUTm4/09D+vHsStPKXDfocp/ftIIoaA9ok5JFS1IC0PMcSfC4AE1SdV65hVDL4tujM+Ngg nCHgQCqXYWAjnjjxSjYtnHE8XhOT7PBOMqZtcEiR25Q== X-Received: by 10.28.212.141 with SMTP id l135mr3585788wmg.122.1521282545705; Sat, 17 Mar 2018 03:29:05 -0700 (PDT) X-Received: by 10.28.212.141 with SMTP id l135mr3585774wmg.122.1521282545500; Sat, 17 Mar 2018 03:29:05 -0700 (PDT) Received: from gmail.com ([2a02:8070:8895:9700:4c9d:d656:8952:c1b2]) by smtp.gmail.com with ESMTPSA id t91sm16650717wrc.21.2018.03.17.03.29.04 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Sat, 17 Mar 2018 03:29:04 -0700 (PDT) Date: Sat, 17 Mar 2018 11:29:03 +0100 From: Christian Brauner To: David Miller Cc: christian.brauner@ubuntu.com, ebiederm@xmission.com, gregkh@linuxfoundation.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, serge@hallyn.com, avagin@virtuozzo.com, ktkhai@virtuozzo.com Subject: Re: [PATCH v2] netns: send uevent messages Message-ID: <20180317102902.GA4301@gmail.com> References: <20180316125030.23624-1-christian.brauner@ubuntu.com> <20180316.144145.1695554148709004295.davem@davemloft.net> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20180316.144145.1695554148709004295.davem@davemloft.net> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Mar 16, 2018 at 02:41:45PM -0400, David Miller wrote: > From: Christian Brauner > Date: Fri, 16 Mar 2018 13:50:30 +0100 > > > +static int uevent_net_broadcast(struct sock *usk, struct sk_buff *skb, > > + struct netlink_ext_ack *extack) > > +{ > > + int ret; > > + /* u64 to chars: 2^64 - 1 = 21 chars */ > > + char buf[sizeof("SEQNUM=") + 21]; > > + struct sk_buff *skbc; > > I hate to be difficult, but please use reverse christmas tree ordering > for local variables. No problem David, coding style is important. I'm about to send out another version with: /* u64 to chars: 2^64 - 1 = 21 chars */ char buf[sizeof("SEQNUM=") + 21]; struct sk_buff *skbc; int ret; Thanks! Christian > > > +static int uevent_net_rcv_skb(struct sk_buff *skb, struct nlmsghdr *nlh, > > + struct netlink_ext_ack *extack) > > +{ > > + int ret; > > + struct net *net; > > Likewise. > > Thank you.