From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=aeWyu0fhicZ35ndvB5G7CKdKzKpJv3Q8+/dMeny8r3E=; b=ngUoDnD3ez3wPljEH7V6ZRJPxi6VyKPo3NxCEGkokrtq2QA/+s/lV/gHCbtL1ccOZM y4RF2dEUkDYGhH4MNBxZg/1KDQafk1w8ITyco6/45YqH8gEgwjnp0z6zsFzKn4IOHj4s SEgx35DPjNeiQzLyJktDf8AGW15iQC8AEp6CWP5YKxFjWBgY54Ck35nwJlqfo9PVt8+y bOUQNnEc8FKh9al2GBvmeJd6AXmxiWSD5M3XIfYFKi7rr36blaxM8b/XEUh/rGL1Hk8R zvF+ytnIvMGQMQUTRgQmPC6Nm15fiVz3SDDpT9DWCqkOjC2BvUYWeMwsgd6Mu48AXDAk ln9Q== References: <20201013173951.25677bcc@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com> <20201014054250.GB6305@unreal> From: Heiner Kallweit Message-ID: <3be8fd19-1c7e-0e05-6039-e5404b2682b9@gmail.com> Date: Wed, 14 Oct 2020 08:13:47 +0200 MIME-Version: 1.0 In-Reply-To: <20201014054250.GB6305@unreal> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Subject: Re: [Bridge] [PATCH net-next v2 00/12] net: add and use function dev_fetch_sw_netstats for fetching pcpu_sw_netstats List-Id: Linux Ethernet Bridging List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Leon Romanovsky , Jakub Kicinski , David Miller Cc: Andrew Lunn , Linux USB Mailing List , Steffen Klassert , Florian Fainelli , Herbert Xu , Sergey Matyukevich , linux-rdma@vger.kernel.org, bridge@lists.linux-foundation.org, Pravin B Shelar , Nikolay Aleksandrov , Roopa Prabhu , Alexey Kuznetsov , Vivien Didelot , Igor Mitsyanko , Kalle Valo , Hideaki YOSHIFUJI , "netdev@vger.kernel.org" , Oliver Neukum , linux-wireless , Johannes Berg , Vladimir Oltean , =?UTF-8?Q?Bj=c3=b8rn_Mork?= On 14.10.2020 07:42, Leon Romanovsky wrote: > On Tue, Oct 13, 2020 at 05:39:51PM -0700, Jakub Kicinski wrote: >> On Mon, 12 Oct 2020 10:00:11 +0200 Heiner Kallweit wrote: >>> In several places the same code is used to populate rtnl_link_stats64 >>> fields with data from pcpu_sw_netstats. Therefore factor out this code >>> to a new function dev_fetch_sw_netstats(). >>> >>> v2: >>> - constify argument netstats >>> - don't ignore netstats being NULL or an ERRPTR >>> - switch to EXPORT_SYMBOL_GPL >> >> Applied, thank you! > > Jakub, > > Is it possible to make sure that changelogs are not part of the commit > messages? We don't store previous revisions in the git repo, so it doesn't > give too much to anyone who is looking on git log later. The lore link > to the patch is more than enough. > I remember that once I did it the usual way (changelog below the ---) David requested the changelog to be part of the commit message. So obviously he sees some benefit in doing so. > 44fa32f008ab ("net: add function dev_fetch_sw_netstats for fetching pcpu_sw_netstats") > > Thanks >