Re: [PATCH net-next 12/20] net: hns3: Add packet statistics of netdev

2018-01-08 Thread lipeng (Y)
On 2018/1/9 11:06, David Miller wrote: From: "lipeng (Y)" Date: Tue, 9 Jan 2018 10:48:04 +0800 So I think it is OK if you can revert [patch 12/20 ]("net: hns3: Add packet statistics of netdev"). I think it is OK if you send the revert patch, which is what I am asking for :-) . sure, i wi

Re: [PATCH net-next 12/20] net: hns3: Add packet statistics of netdev

2018-01-08 Thread David Miller
From: "lipeng (Y)" Date: Tue, 9 Jan 2018 10:48:04 +0800 > So I think it is OK if you can revert [patch 12/20 ]("net: hns3: Add > packet statistics of netdev"). I think it is OK if you send the revert patch, which is what I am asking for :-)

Re: [PATCH net-next 12/20] net: hns3: Add packet statistics of netdev

2018-01-08 Thread lipeng (Y)
On 2018/1/9 9:54, David Miller wrote: From: Jakub Kicinski Date: Mon, 8 Jan 2018 17:50:21 -0800 Oh, I only noticed this extra misleading comment now. Unless each queue has a netdev, I don't see how these are per-queue. If it isn't per-queue I want this change reverted. [patch 12/20 ] add

Re: [PATCH net-next 12/20] net: hns3: Add packet statistics of netdev

2018-01-08 Thread David Miller
From: Jakub Kicinski Date: Mon, 8 Jan 2018 17:50:21 -0800 > Oh, I only noticed this extra misleading comment now. Unless each queue > has a netdev, I don't see how these are per-queue. If it isn't per-queue I want this change reverted.

Re: [PATCH net-next 12/20] net: hns3: Add packet statistics of netdev

2018-01-08 Thread Jakub Kicinski
On Mon, 8 Jan 2018 17:46:02 -0800, Jakub Kicinski wrote: > On Mon, 08 Jan 2018 20:39:13 -0500 (EST), David Miller wrote: > > From: Jakub Kicinski > > Date: Mon, 8 Jan 2018 12:04:31 -0800 > > > > > Ugh, I so didn't review this in time :( I think there is a consensus > > > that we should avoid d

Re: [PATCH net-next 12/20] net: hns3: Add packet statistics of netdev

2018-01-08 Thread Jakub Kicinski
On Mon, 08 Jan 2018 20:39:13 -0500 (EST), David Miller wrote: > From: Jakub Kicinski > Date: Mon, 8 Jan 2018 12:04:31 -0800 > > > Ugh, I so didn't review this in time :( I think there is a consensus > > that we should avoid duplicating standard stats in ethtool. Especially > > those old ones.

Re: [PATCH net-next 12/20] net: hns3: Add packet statistics of netdev

2018-01-08 Thread David Miller
From: Jakub Kicinski Date: Mon, 8 Jan 2018 12:04:31 -0800 > Ugh, I so didn't review this in time :( I think there is a consensus > that we should avoid duplicating standard stats in ethtool. Especially > those old ones. Like "collisions", I assume this is a modern NIC, are > collisions still a

Re: [PATCH net-next 12/20] net: hns3: Add packet statistics of netdev

2018-01-08 Thread Jakub Kicinski
On Fri, 5 Jan 2018 18:18:16 +0800, Peng Li wrote: > +static const struct hns3_stats hns3_netdev_stats[] = { > + /* Rx per-queue statistics */ > + HNS3_NETDEV_STAT("rx_packets", rx_packets), > + HNS3_NETDEV_STAT("tx_packets", tx_packets), > + HNS3_NETDEV_STAT("rx_bytes", rx_bytes), >

[PATCH net-next 12/20] net: hns3: Add packet statistics of netdev

2018-01-05 Thread Peng Li
From: Jian Shen Add packet statistics of netdev for ethtool -S, in order to show the statistics data for current net device. Remove update_stats() calling because it has been completed in hns3_get_netdev_stats(). Signed-off-by: Jian Shen Signed-off-by: Peng Li --- drivers/net/ethernet/hisili