2024-03-03 11:42:08

by 付元力 Jerry Fu

Subject: [PATCH net-next v2] net/nlmon: Cancel setting the fields of statistics to zero.

Since fields of rtnl_link_stats64 have been set to zero in the previous
dev_get_stats function, there is no need to set them again in the
ndo_get_stats64 function.

Signed-off-by: fuyuanli <[email protected]>
Link: https://lore.kernel.org/netdev/20240302105224.GA7223@didi-ThinkCentre-M920t-N000/
---
v2:
1) check and fix error of spelling and grammar in the commit message.
---
drivers/net/nlmon.c | 2 --
1 file changed, 2 deletions(-)

diff --git a/drivers/net/nlmon.c b/drivers/net/nlmon.c
index 5e19a6839dea..9b205b152734 100644
--- a/drivers/net/nlmon.c
+++ b/drivers/net/nlmon.c
@@ -56,10 +56,8 @@ nlmon_get_stats64(struct net_device *dev, struct rtnl_link_stats64 *stats)
dev_lstats_read(dev, &packets, &bytes);

stats->rx_packets = packets;
- stats->tx_packets = 0;

stats->rx_bytes = bytes;
- stats->tx_bytes = 0;
}

static u32 always_on(struct net_device *dev)
--
2.17.1

2024-03-03 13:51:15

by Eric Dumazet

Subject: Re: [PATCH net-next v2] net/nlmon: Cancel setting the fields of statistics to zero.

On Sun, Mar 3, 2024 at 12:42 PM fuyuanli <[email protected]> wrote:
>
> Since fields of rtnl_link_stats64 have been set to zero in the previous
> dev_get_stats function, there is no need to set them again in the
> ndo_get_stats64 function.
>
> Signed-off-by: fuyuanli <[email protected]>
> Link: https://lore.kernel.org/netdev/20240302105224.GA7223@didi-ThinkCentre-M920t-N000/
>

Reviewed-by: Eric Dumazet <[email protected]>

2024-03-04 07:05:46

by Jason Xing

Subject: Re: [PATCH net-next v2] net/nlmon: Cancel setting the fields of statistics to zero.

On Sun, Mar 3, 2024 at 7:43 PM fuyuanli <[email protected]> wrote:
>
> Since fields of rtnl_link_stats64 have been set to zero in the previous
> dev_get_stats function, there is no need to set them again in the
> ndo_get_stats64 function.
>
> Signed-off-by: fuyuanli <[email protected]>
> Link: https://lore.kernel.org/netdev/20240302105224.GA7223@didi-ThinkCentre-M920t-N000/

Suggested-by: Jason Xing <[email protected]>
See https://lore.kernel.org/all/CAL+tcoA=FVBJi2eJgAELhWG_f+N-kwmrHc+XRfKXhYk2RJcPKg@mail.gmail.com/

Reviewed-by: Jason Xing <[email protected]>

2024-03-04 11:15:07

by yuanli fu

Subject: Re: [PATCH net-next v2] net/nlmon: Cancel setting the fields of statistics to zero.

Jason Xing <[email protected]> 于2024年3月4日周一 15:05写道：
>
> On Sun, Mar 3, 2024 at 7:43 PM fuyuanli <[email protected]> wrote:
> >
> > Since fields of rtnl_link_stats64 have been set to zero in the previous
> > dev_get_stats function, there is no need to set them again in the
> > ndo_get_stats64 function.
> >
> > Signed-off-by: fuyuanli <[email protected]>
> > Link: https://lore.kernel.org/netdev/20240302105224.GA7223@didi-ThinkCentre-M920t-N000/
>
> Suggested-by: Jason Xing <[email protected]>
> See https://lore.kernel.org/all/CAL+tcoA=FVBJi2eJgAELhWG_f+N-kwmrHc+XRfKXhYk2RJcPKg@mail.gmail.com/
OK, I will submit a v3 patch which updating commit message, thanks.
>
> Reviewed-by: Jason Xing <[email protected]>
>

2024-03-04 13:26:15

by Jason Xing

Subject: Re: [PATCH net-next v2] net/nlmon: Cancel setting the fields of statistics to zero.

On Mon, Mar 4, 2024 at 7:14 PM yuanli fu <[email protected]> wrote:
>
> Jason Xing <[email protected]> 于2024年3月4日周一 15:05写道：
> >
> > On Sun, Mar 3, 2024 at 7:43 PM fuyuanli <[email protected]> wrote:
> > >
> > > Since fields of rtnl_link_stats64 have been set to zero in the previous
> > > dev_get_stats function, there is no need to set them again in the
> > > ndo_get_stats64 function.
> > >
> > > Signed-off-by: fuyuanli <[email protected]>
> > > Link: https://lore.kernel.org/netdev/20240302105224.GA7223@didi-ThinkCentre-M920t-N000/
> >
> > Suggested-by: Jason Xing <[email protected]>
> > See https://lore.kernel.org/all/CAL+tcoA=FVBJi2eJgAELhWG_f+N-kwmrHc+XRfKXhYk2RJcPKg@mail.gmail.com/
> OK, I will submit a v3 patch which updating commit message, thanks.

I don't think you need to send a new version of the patch unless
someone points out other changes that should be made.

Thanks,
Jason

> >
> > Reviewed-by: Jason Xing <[email protected]>
> >

2024-03-04 14:00:26

by Daniel Borkmann

Subject: Re: [PATCH net-next v2] net/nlmon: Cancel setting the fields of statistics to zero.

On 3/4/24 2:25 PM, Jason Xing wrote:
> On Mon, Mar 4, 2024 at 7:14 PM yuanli fu <[email protected]> wrote:
>> Jason Xing <[email protected]> 于2024年3月4日周一 15:05写道：
>>> On Sun, Mar 3, 2024 at 7:43 PM fuyuanli <[email protected]> wrote:
>>>>
>>>> Since fields of rtnl_link_stats64 have been set to zero in the previous
>>>> dev_get_stats function, there is no need to set them again in the
>>>> ndo_get_stats64 function.
>>>>
>>>> Signed-off-by: fuyuanli <[email protected]>
>>>> Link: https://lore.kernel.org/netdev/20240302105224.GA7223@didi-ThinkCentre-M920t-N000/
>>>
>>> Suggested-by: Jason Xing <[email protected]>
>>> See https://lore.kernel.org/all/CAL+tcoA=FVBJi2eJgAELhWG_f+N-kwmrHc+XRfKXhYk2RJcPKg@mail.gmail.com/
>> OK, I will submit a v3 patch which updating commit message, thanks.
>
> I don't think you need to send a new version of the patch unless
> someone points out other changes that should be made.

I think this patch is not needed anymore, see net-next :

4f41ce81a919 ("net: nlmon: Remove init and uninit functions")
26b5df99bf60 ("net: nlmon: Simplify nlmon_get_stats64")

Thanks,
Daniel