linux-kernel - Re: [PATCH V2 net-next 1/8] net: hibmcge: Add dump statistics supported in this module

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <bf66c7a6-af2f-436f-a8eb-50dd46d014d6@huawei.com>
Date: Fri, 1 Nov 2024 14:08:26 +0800
From: Jijie Shao <shaojijie@...wei.com>
To: Jakub Kicinski <kuba@...nel.org>
CC: <shaojijie@...wei.com>, <davem@...emloft.net>, <edumazet@...gle.com>,
	<pabeni@...hat.com>, <andrew+netdev@...n.ch>, <horms@...nel.org>,
	<shenjian15@...wei.com>, <wangpeiyang1@...wei.com>, <liuyonglong@...wei.com>,
	<chenhao418@...wei.com>, <sudongming1@...wei.com>, <xujunsheng@...wei.com>,
	<shiyongbang@...wei.com>, <libaihan@...wei.com>,
	<jonathan.cameron@...wei.com>, <shameerali.kolothum.thodi@...wei.com>,
	<salil.mehta@...wei.com>, <netdev@...r.kernel.org>,
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH V2 net-next 1/8] net: hibmcge: Add dump statistics
 supported in this module


on 2024/11/1 9:45, Jakub Kicinski wrote:
> On Sat, 26 Oct 2024 19:57:33 +0800 Jijie Shao wrote:
>> +	HBG_STATS_REG_I(rx_framesize_64, HBG_REG_RX_PKTS_64OCTETS_ADDR),
>> +	HBG_STATS_REG_I(rx_framesize_65_127,
>> +			HBG_REG_RX_PKTS_65TO127OCTETS_ADDR),
>> +	HBG_STATS_REG_I(rx_framesize_128_255,
>> +			HBG_REG_RX_PKTS_128TO255OCTETS_ADDR),
>> +	HBG_STATS_REG_I(rx_framesize_256_511,
>> +			HBG_REG_RX_PKTS_256TO511OCTETS_ADDR),
>> +	HBG_STATS_REG_I(rx_framesize_512_1023,
>> +			HBG_REG_RX_PKTS_512TO1023OCTETS_ADDR),
>> +	HBG_STATS_REG_I(rx_framesize_1024_1518,
>> +			HBG_REG_RX_PKTS_1024TO1518OCTETS_ADDR),
>> +	HBG_STATS_REG_I(rx_framesize_bt_1518,
>> +			HBG_REG_RX_PKTS_1519TOMAXOCTETS_ADDR),
>> +	HBG_STATS_REG_I(rx_fcs_error_cnt, HBG_REG_RX_FCS_ERRORS_ADDR),
>> +	HBG_STATS_REG_I(rx_data_error_cnt, HBG_REG_RX_DATA_ERR_ADDR),
>> +	HBG_STATS_REG_I(rx_align_error_cnt, HBG_REG_RX_ALIGN_ERRORS_ADDR),
>> +	HBG_STATS_REG_I(rx_frame_long_err_cnt, HBG_REG_RX_LONG_ERRORS_ADDR),
> Please do not dump in ethtool statistics which can be accessed via
> standard APIs like ethtool -I, queue_stats or rtnl_stat.
> https://docs.kernel.org/next/networking/statistics.html

ok, in V3, I will modify this patch according to this document.

Thanks,
Jijie Shao