Jakub Kicinski
2020-Oct-14 00:39 UTC
[Bridge] [PATCH net-next v2 00/12] net: add and use function dev_fetch_sw_netstats for fetching pcpu_sw_netstats
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_GPLApplied, thank you!
Leon Romanovsky
2020-Oct-14 05:42 UTC
[Bridge] [PATCH net-next v2 00/12] net: add and use function dev_fetch_sw_netstats for fetching pcpu_sw_netstats
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. 44fa32f008ab ("net: add function dev_fetch_sw_netstats for fetching pcpu_sw_netstats") Thanks
Heiner Kallweit
2020-Oct-14 06:13 UTC
[Bridge] [PATCH net-next v2 00/12] net: add and use function dev_fetch_sw_netstats for fetching pcpu_sw_netstats
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 >