netdev - RE: [iwl-next v3 8/8] ice: count representor 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] [day] [month] [year] [list]

Message-ID: <PH0PR11MB5013FD8A98FF37FCDE6B358796272@PH0PR11MB5013.namprd11.prod.outlook.com>
Date: Fri, 8 Mar 2024 07:30:26 +0000
From: "Buvaneswaran, Sujai" <sujai.buvaneswaran@...el.com>
To: Michal Swiatkowski <michal.swiatkowski@...ux.intel.com>,
	"intel-wired-lan@...ts.osuosl.org" <intel-wired-lan@...ts.osuosl.org>
CC: "netdev@...r.kernel.org" <netdev@...r.kernel.org>, "Szycik, Marcin"
	<marcin.szycik@...el.com>, "Drewek, Wojciech" <wojciech.drewek@...el.com>,
	"Samudrala, Sridhar" <sridhar.samudrala@...el.com>, "Kitszel, Przemyslaw"
	<przemyslaw.kitszel@...el.com>, "horms@...nel.org" <horms@...nel.org>, Marcin
 Szycik <marcin.szycik@...ux.intel.com>
Subject: RE: [iwl-next v3 8/8] ice: count representor stats

> -----Original Message-----
> From: Michal Swiatkowski <michal.swiatkowski@...ux.intel.com>
> Sent: Friday, March 1, 2024 5:24 PM
> To: intel-wired-lan@...ts.osuosl.org
> Cc: netdev@...r.kernel.org; Szycik, Marcin <marcin.szycik@...el.com>;
> Drewek, Wojciech <wojciech.drewek@...el.com>; Samudrala, Sridhar
> <sridhar.samudrala@...el.com>; Kitszel, Przemyslaw
> <przemyslaw.kitszel@...el.com>; horms@...nel.org; Buvaneswaran, Sujai
> <sujai.buvaneswaran@...el.com>; Michal Swiatkowski
> <michal.swiatkowski@...ux.intel.com>; Marcin Szycik
> <marcin.szycik@...ux.intel.com>
> Subject: [iwl-next v3 8/8] ice: count representor stats
> 
> Removing control plane VSI result in no information about slow-path
> statistic. In current solution statistics need to be counted in driver.
> 
> Patch is based on similar implementation done by Simon Horman in nfp:
> commit eadfa4c3be99 ("nfp: add stats and xmit helpers for representors")
> 
> Add const modifier to netdev parameter in ice_netdev_to_repr(). It isn't (and
> shouldn't be) modified in the function.
> 
> Reviewed-by: Marcin Szycik <marcin.szycik@...ux.intel.com>
> Signed-off-by: Michal Swiatkowski <michal.swiatkowski@...ux.intel.com>
> ---
>  drivers/net/ethernet/intel/ice/ice_eswitch.c  |   7 +-
>  drivers/net/ethernet/intel/ice/ice_repr.c     | 103 +++++++++++++-----
>  drivers/net/ethernet/intel/ice/ice_repr.h     |  16 ++-
>  drivers/net/ethernet/intel/ice/ice_txrx_lib.c |   3 +
>  4 files changed, 99 insertions(+), 30 deletions(-)
> 
Tested-by: Sujai Buvaneswaran <sujai.buvaneswaran@...el.com>