netdev - Re: [PATCH net-next v5 2/2] net: ti: icssg-prueth: Add support for PA Stats

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <e8aee03a-2141-4c96-968f-02e4c622c59c@ti.com>
Date: Mon, 19 Aug 2024 12:40:45 +0530
From: "Anwar, Md Danish" <a0501179@...com>
To: Simon Horman <horms@...nel.org>, MD Danish Anwar <danishanwar@...com>
CC: Suman Anna <s-anna@...com>, Sai Krishna <saikrishnag@...vell.com>,
        Jan
 Kiszka <jan.kiszka@...mens.com>,
        Dan Carpenter <dan.carpenter@...aro.org>, Andrew Lunn <andrew@...n.ch>,
        Diogo Ivo <diogo.ivo@...mens.com>,
        Kory
 Maincent <kory.maincent@...tlin.com>,
        Heiner Kallweit <hkallweit1@...il.com>,
        Paolo Abeni <pabeni@...hat.com>, Jakub Kicinski <kuba@...nel.org>,
        Eric
 Dumazet <edumazet@...gle.com>,
        "David S. Miller" <davem@...emloft.net>,
        Roger
 Quadros <rogerq@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
        Krzysztof
 Kozlowski <krzk+dt@...nel.org>,
        Rob Herring <robh@...nel.org>,
        Santosh
 Shilimkar <ssantosh@...nel.org>, Nishanth Menon <nm@...com>,
        <netdev@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>,
        <srk@...com>, Vignesh Raghavendra <vigneshr@...com>
Subject: Re: [PATCH net-next v5 2/2] net: ti: icssg-prueth: Add support for PA
 Stats



On 8/15/2024 9:31 PM, Simon Horman wrote:
> On Wed, Aug 14, 2024 at 02:50:33PM +0530, MD Danish Anwar wrote:
> 
> ...
> 
>> diff --git a/drivers/net/ethernet/ti/icssg/icssg_prueth.h b/drivers/net/ethernet/ti/icssg/icssg_prueth.h
>> index f678d656a3ed..ac2291d22c42 100644
>> --- a/drivers/net/ethernet/ti/icssg/icssg_prueth.h
>> +++ b/drivers/net/ethernet/ti/icssg/icssg_prueth.h
>> @@ -50,8 +50,10 @@
>>  
>>  #define ICSSG_MAX_RFLOWS	8	/* per slice */
>>  
>> +#define ICSSG_NUM_PA_STATS 4
>> +#define ICSSG_NUM_MII_G_RT_STATS 60
>>  /* Number of ICSSG related stats */
>> -#define ICSSG_NUM_STATS 60
>> +#define ICSSG_NUM_STATS (ICSSG_NUM_MII_G_RT_STATS + ICSSG_NUM_PA_STATS)
>>  #define ICSSG_NUM_STANDARD_STATS 31
>>  #define ICSSG_NUM_ETHTOOL_STATS (ICSSG_NUM_STATS - ICSSG_NUM_STANDARD_STATS)
>>  
>> @@ -263,6 +265,7 @@ struct prueth {
>>  	struct net_device *registered_netdevs[PRUETH_NUM_MACS];
>>  	struct regmap *miig_rt;
>>  	struct regmap *mii_rt;
>> +	struct regmap *pa_stats;
> 
> Please add an entry for pa_stats to the Kernel doc for this structure.
> 
>>  
>>  	enum pruss_pru_id pru_id[PRUSS_NUM_PRUS];
>>  	struct platform_device *pdev;
> 
> ...
> 
>> diff --git a/drivers/net/ethernet/ti/icssg/icssg_stats.h b/drivers/net/ethernet/ti/icssg/icssg_stats.h
>> index 999a4a91276c..e834316092c9 100644
>> --- a/drivers/net/ethernet/ti/icssg/icssg_stats.h
>> +++ b/drivers/net/ethernet/ti/icssg/icssg_stats.h
>> @@ -77,6 +77,20 @@ struct miig_stats_regs {
>>  	u32 tx_bytes;
>>  };
>>  
>> +/**
>> + * struct pa_stats_regs - ICSSG Firmware maintained PA Stats register
>> + * @u32 fw_rx_cnt: Number of valid packets sent by Rx PRU to Host on PSI
>> + * @u32 fw_tx_cnt: Number of valid packets copied by RTU0 to Tx queues
>> + * @u32 fw_tx_pre_overflow: Host Egress Q (Pre-emptible) Overflow Counter
>> + * @u32 fw_tx_exp_overflow: Host Egress Q (Express) Overflow Counter
>> + */
> 
> ./scripts/kernel-doc -none doesn't seem to like the syntax above.
> Perhaps s/u32 // ?

Sure, I will drop u32 from comment.

> 
>> +struct pa_stats_regs {
>> +	u32 fw_rx_cnt;
>> +	u32 fw_tx_cnt;
>> +	u32 fw_tx_pre_overflow;
>> +	u32 fw_tx_exp_overflow;
>> +};
>> +
>>  #define ICSSG_STATS(field, stats_type)			\
>>  {							\
>>  	#field,						\
> 
> ...

-- 
Thanks and Regards,
Md Danish Anwar