From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9B3B02DA75B; Wed, 13 May 2026 20:00:23 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=192.198.163.17 ARC-Seal:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778702425; cv=fail; b=KvJjBG1iaXmoZdajQct62uUfYRC+OFtsOI5nZBlMr0SUTsBsE/siGGx3O/Xk3n+SFDbT1T4/HmtbvcR1cZpc+tDkOp8Xv8SyhdprLBJuJRrhCZy5stS50c1EovOLWxc8d3NxrF5dvhiZu5x7slqwuFYW460DDDcps1TaichDRog= ARC-Message-Signature:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1778702425; c=relaxed/simple; bh=Thlx3Pj37/vzZGa/YzC7+OS57t+OnLvc5vLmvw45AoE=; h=Message-ID:Date:Subject:To:CC:References:From:In-Reply-To: Content-Type:MIME-Version; b=E12daQIO/twbTYY7sNz1ObICiWOma55OlDkho9MPLTcofAHlVJp/N60LBa9LVQZ/vIUYD7c5EVkE0FAmIghpYXn8Lzm7Q8XQLRaBIl6D/u4oyAxpQNyKTr04a6o3UMhVH4qT6UCGB+GtrxQBbmxmDjMSEuvahA3ptRycYpMUt2U= ARC-Authentication-Results:i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=SYvoF8nZ; arc=fail smtp.client-ip=192.198.163.17 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="SYvoF8nZ" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1778702424; x=1810238424; h=message-id:date:subject:to:cc:references:from: in-reply-to:content-transfer-encoding:mime-version; bh=Thlx3Pj37/vzZGa/YzC7+OS57t+OnLvc5vLmvw45AoE=; b=SYvoF8nZfgvKettXlFISSZ5p5XxR8K514hwaoekScwzaiA1rIemBztxj v0nYfkUdz106t9A5AKCTDxRlYXCCCMgc5uS0QNSMs14psy20SQ2vYoyaP zYXQ80n3hGnzjl+OfrrP02zyiada7i2yTEnOxdPeTBvEsstbWvFSGxy1d kRLZ0CN3O1+AnRXI7gBzbZnICVBQgagzItVO0CodAr0v0Bqq/Uf/kF8DB gZvu68fDkMKNWBtMII5Veo7LpwUdaEOkNo9NAG8tlJ6ArUsgVJBPaOx1C p2P14GXuLLU/3UxZoMgzZMz9PHz3LC00uFXblEJ7N0EupymZhI5jyrWLt w==; X-CSE-ConnectionGUID: /fLHhRoqTl29tIQ9ketrhQ== X-CSE-MsgGUID: g6GGwycJR/mFI7H0vYqhOg== X-IronPort-AV: E=McAfee;i="6800,10657,11785"; a="79497639" X-IronPort-AV: E=Sophos;i="6.23,233,1770624000"; d="scan'208";a="79497639" Received: from fmviesa007.fm.intel.com ([10.60.135.147]) by fmvoesa111.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 May 2026 13:00:22 -0700 X-CSE-ConnectionGUID: +wWBu5jvQGaMZGUiW6C6Fg== X-CSE-MsgGUID: dkShG1jzQOiolYgrMHhr6g== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.23,233,1770624000"; d="scan'208";a="235114324" Received: from orsmsx901.amr.corp.intel.com ([10.22.229.23]) by fmviesa007.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 May 2026 13:00:21 -0700 Received: from ORSMSX902.amr.corp.intel.com (10.22.229.24) by ORSMSX901.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.37; Wed, 13 May 2026 13:00:20 -0700 Received: from ORSEDG902.ED.cps.intel.com (10.7.248.12) by ORSMSX902.amr.corp.intel.com (10.22.229.24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.37 via Frontend Transport; Wed, 13 May 2026 13:00:20 -0700 Received: from MW6PR02CU001.outbound.protection.outlook.com (52.101.48.8) by edgegateway.intel.com (134.134.137.112) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.37; Wed, 13 May 2026 13:00:19 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=ch7Tlvwscs1NKqud74whJypFrgtsYbnMoUC/dqKMBJYIx9mHq9GYozoNxMOniKFzXqt1Ip3wT3xLlaUZTtrDdDyty3kq9PSYJ60JALT7Li8AFWcE5AdmTlaHN0Tuu5wqUn287/ufgSynW5sHJYJGJz9C68eIhw28vqmjSXQi3Xnrmc/sJ9+0Mvp+ViXCPR1r8JKgZ7PtyS+wXLXAB6ArgcYBFKwZVuIriYvZACUb4Itk/nCajaZtGXNIa5+O3yHfcNlPhUPMG9BF2QgklAOUBfJEYjvm7WNbiedoaltGBd33sLjpvsZdlvnZh5vkOkyaykyRsqze+oHYmSfx3CdcUw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Ql2BFZ6hIlaRWfrmaKY6b4ErsGDOpy8XiGPhWyw/HD0=; b=U3X+XNYhiL5RijGrFrN6b4pFPyrpUT0cRaiVvWv8FFOlRZ8hnZnI+5ItdO9Fx6KH02VIJt46tudjrnabVeAbYwRTMDIRSWizzhIIG77etgUDr2INsmFaFkBpAaIZZhn19WYGV2LImNESLVMOoLrk0TrAT/MS0JOst1gDexuKi6ak9CrFGPDtP5Vizr3vzEaWwcc6zL3RFU9rlJF/jcWBmEplOYy4P5VBDmlgcAzpRHVjadHvztzU7CCs3tRD+S/30xFU02uoZNveQnWxO9wqTYa7vCEAf02O4+/j4ZemrXic5+Mw8C9eQs9v20GxXib5hiJxoQjNMYApJ74PAZT86Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; Received: from SN7PR11MB7592.namprd11.prod.outlook.com (2603:10b6:806:343::16) by DS7PR11MB7860.namprd11.prod.outlook.com (2603:10b6:8:e9::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9913.12; Wed, 13 May 2026 20:00:11 +0000 Received: from SN7PR11MB7592.namprd11.prod.outlook.com ([fe80::3e09:8700:df72:37b6]) by SN7PR11MB7592.namprd11.prod.outlook.com ([fe80::3e09:8700:df72:37b6%6]) with mapi id 15.20.9891.021; Wed, 13 May 2026 20:00:09 +0000 Message-ID: <9fbf6682-b521-4b7e-b5b6-af91694ed051@intel.com> Date: Wed, 13 May 2026 13:00:05 -0700 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH net-next 1/2] net: ti: icssg: Derive stats array lengths from ARRAY_SIZE To: MD Danish Anwar , David CARLIER CC: "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Simon Horman , Jonathan Corbet , Shuah Khan , Roger Quadros , Andrew Lunn , Meghana Malladi , Kevin Hao , Vadim Fedorenko , , , , , Vignesh Raghavendra References: <20260512060627.3781329-1-danishanwar@ti.com> <20260512060627.3781329-2-danishanwar@ti.com> <6a1f411c-d7ed-463b-abf1-277d8cc0c184@ti.com> Content-Language: en-US From: Jacob Keller In-Reply-To: <6a1f411c-d7ed-463b-abf1-277d8cc0c184@ti.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-ClientProxiedBy: MW4PR03CA0189.namprd03.prod.outlook.com (2603:10b6:303:b8::14) To SN7PR11MB7592.namprd11.prod.outlook.com (2603:10b6:806:343::16) Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SN7PR11MB7592:EE_|DS7PR11MB7860:EE_ X-MS-Office365-Filtering-Correlation-Id: e421620d-6e37-447c-b636-08deb12a3c14 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|1800799024|366016|11063799003|4143699003|22082099003|18002099003|56012099003; X-Microsoft-Antispam-Message-Info: AYlc5/mZxv2BPJdveoKRjRSrdqPzvJCR0X+/Xp1ZfR/n3sMvoyWSrOVnD5BxEH4DwRhuDxREeN+LWbM3RxOwarhbidBrVxTv6LvsgTWt+Z/HmhzyfOeeQgOv7c8/PSAi5sxTTQ3qq3EoCm7PaQsFN9reerfwaH8ukW1eJ08cVPuMR0P0X+2Tz/V22E/C20s7zqRisNwQve0AyJIUzqjKjghbwGfClDye1LYWYgMnTcLmrqG/zTJ+O+9xOGe0yDpuhvLmEeyyX2w5+PH0r+noCy0AnbBK2T/k0ibT4bB8XpoR8Ql+Sun3emIXzAx8lk7Y03AJy+hsEd736cKojzzyZq+/YZTp9MxOxLXHfzlh9vmPe7QWwWlmyr8sXF9CKYMzvIsCDplMrvvhYTN0aSYI5ogdh2SFkg1M1ThmxaBy27pdI0sz4Y7AT2tpOnt+wXsLm8RJKsbhbC71vLp3lBPQKLYaEjfx0LIyXFo0Pwia7N370VYYaIKQkSC9OS8bqvHWkxYizvmgYKDk165lBzl5tjT+I3yWBKbBBl/vMeMZc3OzHsma5ozKn8qWEz19Gq1Q+GHyQsAdVphl7zou1ayQ3zGyefx+61OclbiJmxz5b5JsDbffoK9uc2sRlovGMAOaxGA0R6APIB4sXCaist+/iZBZGBernHc5FD2NORiVvPPfJnMGkV5gFDiYLHALGGIH X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SN7PR11MB7592.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(7416014)(1800799024)(366016)(11063799003)(4143699003)(22082099003)(18002099003)(56012099003);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?OEp6cXd3OEVLYWNJTFFVTXdoaC80L25HMm5XUm1odzNBUUVSaEhSN3AySW1Q?= =?utf-8?B?cUI1aXhTZmt1OGN1TGV4S1dVaEZkanl6Y2RYc2RzaHl5aFR2ZEhTakgyZ3Zq?= =?utf-8?B?Uis2VEwrR1kzeWJQNXdxYWhYdkplS2pNN21SWEtHVjRUWVpGSjdmeURxTWNu?= =?utf-8?B?YjUxdXVFRThGWmdabEpuc1U5R3BJRThEb3VzK2diSjkwZ05MWDNsdHZTY3Rt?= =?utf-8?B?ZU52OWgzam5CUC9XWEpRWExMaGhXYXdzWEt0aVRqSVJUbkxVdjNPays3VWNh?= =?utf-8?B?ZHUvRGpmek0xa1ZHNCtPVXdoeGZkeElZT1JDTkczdVBqTjljS3B4UXN6Mmdo?= =?utf-8?B?UzdzbjVlemNzWVg3bjBWWmhyZ1lFdlFwbThyb2h5cGRmTUhIcWRndU9xU1lr?= =?utf-8?B?SlAxc1BMSmhTZ3hQSUlMWC8wRTJRVWIzZzVLckt3UFBKMnIrazNCNnAvRGNH?= =?utf-8?B?dXpOWU42RmZlUm9yL1pMR2hYUmcvS0dDck5iZ1p6Q05XR09HcjAzaktYT1Zl?= =?utf-8?B?VEtqMkZ3Y1JQNGJWTElCVUhwdHBzUW5QeDgxdFVBNU9YOElYYk5zM1JxRHZZ?= =?utf-8?B?R0JWOVJJUnRTckpRRGUxZjNnSWxwNEs0SjNpeGFqbHFKSHBiS1c2VVg3ZFNP?= =?utf-8?B?Zmg4WDROU01CVzBZOTNObHRhNVVWd2pvanpCMUluS3MrWnNvYUY2V3BlaGgr?= =?utf-8?B?RHVpcGxKVkI1T3d5ZXBQSEp1ZzBYZXEzTGVYWjFUckhkN0dkSkJwT0RRSGUz?= =?utf-8?B?ditnNVFUZGtieXFmRG9ncXFUREJ1bUJDeDJxOXlmaEJ4WVVMT3ZvY0xCTFY0?= =?utf-8?B?ZWRLcVZYL1ZianpJblZRZWNZUW5nWWc0RnBaRHZiQ1BTdUdTLzVuWUlMRUFU?= =?utf-8?B?OXZxUEFwSE8vRk1DZVpjOHFTZTU0T2FmZ3lSdEZsbjN3alE0UFVsQnhhSFNH?= =?utf-8?B?aGppOXVING5OWmtLUUxKdXJhcGdUTUM2cGMxZVNxTmRLVEJTM2Z6azFEWCsz?= =?utf-8?B?b1hiU2svdkprY1NDT1Y3eDlUV283NW0vL0FTcCtIWlV0cENZUzNEbUUyWXNN?= =?utf-8?B?WXZCS1dmWSt5bVg4Y1k5SkRBekFpM3NyOE5ib2M4R2tUUVR0T2R1ZDJmVkcw?= =?utf-8?B?TWZPWVZ5T050Zm9Nd1pMZ21YbVRjYmYrNHJpbkJVV3RSVVlWMlAzUkJvTzZK?= =?utf-8?B?MlQ3b3hvcmN4WkVIVU9STjZMbXF0TlM3MjZUME1rYkdzRVJXYmt5aHlHMHZR?= =?utf-8?B?RC8xVXZyNzhvbkh5Y1B1RG1xaTFJWnZKeEtwUlg2VjkrK1FuQmgvdm9EZ0xw?= =?utf-8?B?Y05ObXBSQU1zM3BjR05kcTI1VDNRZFRscVplby80ME1mN0s3cHpqN29yV0F2?= =?utf-8?B?dVphaVF3NE9oa2txendnN0ZYN2ZFUHFKeGFqZ05sUVkwOC93V0ovaUhnT3Fr?= =?utf-8?B?ckt0Ym1zM3pOc3B4RVN4QUE4anFDVXplYk43QzRsUWkzWjR0NzBDQjUrbm9y?= =?utf-8?B?c0V1alBuaDh2c2VHcUI3ZnJBbUNSUnEyaVk2N1VTVnlVTXlWMXlGOTUzWENn?= =?utf-8?B?Q0hhV045endaWG9FYWs2Wlo4RzBmUTl1SGdNTUtUWGVnRmI2N0huano3TVg4?= =?utf-8?B?QkczZ3BDVUV0b0xSQitINU9lNVNweWpNeXRWSjVRMytxbVgvdk42aGI1UFBj?= =?utf-8?B?ZnBtRWJPTE15ZEl3aWl3YW85KzhiZVYvSUVkUnNmSHJzSDhzZ0kySTZDdTZt?= =?utf-8?B?eWFLajNzUkozVzN5QnFWM2x5N05NbnVUZTRLOS82UmozK292QVMvWjFsNzgy?= =?utf-8?B?QUV6WThCb1VYeFVrVmYzbDJjbUc5UVFjL3d3azhzNEp4ekhjcGFnOUp1c213?= =?utf-8?B?T1ZBeFR3bUw4STNSVmRRWnlSWTgwejdSZldEQWFNK0plK244ZW9Lc2VaejBl?= =?utf-8?B?VmZCS01xNVdSL3hvUmFwYWxEcEhla3lRNVNCWG4vc0dVTktOc2hZbmRFc0xv?= =?utf-8?B?OGlIcGtmbEdhUzBOWGxhR3k5TFk5eGxUNGZad3lyQURINjF2R3J2QjEvNEZU?= =?utf-8?B?TW9xZUVWN3AzUU01dUtVZmFWL2t0aWpiZ3pIeldpRC81ek9UVkhldlcrbVda?= =?utf-8?B?QkQwbzhlZGJpZ3NmTUQ5bm0xVmRCWkRrTWJyWk83SlRxQTVYa2l6eVhYZjJR?= =?utf-8?B?VVBZS3oxL0N2V0kwdzUycGFMeFB2TFhEWFA2MXp1eHE0TkM1REdzSHNZOTRi?= =?utf-8?B?dVBPZTFlS2t0N0NnWXZlaG9meGtSalpTbFJHLzcwZ0dKOTd3eGEyZGdMQkZr?= =?utf-8?B?eUVTZDZ5VnZiNjJVY2xockxlQVpacHA0UzlWQ3FOdzk3WVJXVVBCZWRwZkJs?= =?utf-8?Q?y2NH4X8vGEcHL1Ek=3D?= X-Exchange-RoutingPolicyChecked: hU/vzk57BNmPj9HtRQeOfhBQcIM6IeDkhD+1j0wkue/T/LZdoU65xtKWu9nfIHO3/41RjwPG8Im36Tri+k6PH7swxAxZtffsz1l2IHBBigfIZTD/bO4woiS7z2V1b5M4OTwxbmPpM6RxqlPjowNqpCwbg0CMBGUB0JqfILtm5XmPEyMKD2phmknhS0V5eSlAKzbu/oTqAGCt7qghMOQT1HYRg+cU+rNnQ6DGwBX2Bxp42R3ymwAdoYzjQbsjJzL6AHNuhIVih0k3nUbZrggJFh6QLa7cQXf3mbkPDhcPSqf8Ep+b+FLQ4RC90UftdFOev5rJPhHod7dpItibdAF+nw== X-MS-Exchange-CrossTenant-Network-Message-Id: e421620d-6e37-447c-b636-08deb12a3c14 X-MS-Exchange-CrossTenant-AuthSource: SN7PR11MB7592.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 May 2026 20:00:09.2619 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: IYvdXjFF4TuPBGLmGgsLXPFHncn0Q7PDPrk06k1rP0Nh7kjUtA+8gNYtKDR9Lu9yBh6hantJworlbfoWkqTmOnfkOYCNuU+olAvulQ9h/rQ= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR11MB7860 X-OriginatorOrg: intel.com On 5/12/2026 2:40 AM, MD Danish Anwar wrote: > Hi David, > > On 12/05/26 1:28 pm, David CARLIER wrote: >> Hi MD, >> >> On Tue, 12 May 2026 at 07:06, MD Danish Anwar wrote: >>> >>> Replace the manually maintained ICSSG_NUM_MIIG_STATS and >>> ICSSG_NUM_PA_STATS constants with ARRAY_SIZE() expressions derived >>> directly from the corresponding stat descriptor arrays, so that adding >>> new entries to icssg_all_miig_stats[] or icssg_all_pa_stats[] no longer >>> requires a separate update to a numeric constant. >>> >>> To make this self-contained, break the circular include dependency >>> between icssg_stats.h and icssg_prueth.h: >>> >>> - icssg_stats.h previously included icssg_prueth.h (transitively >>> pulling in icssg_switch_map.h and ETH_GSTRING_LEN). Replace that >>> with direct includes of , and >>> "icssg_switch_map.h". >>> >>> - icssg_prueth.h now includes icssg_stats.h, giving it access to >>> the ARRAY_SIZE-based ICSSG_NUM_MIIG_STATS and ICSSG_NUM_PA_STATS >>> before they are used in the prueth_emac struct and ICSSG_NUM_STATS. >>> >>> Signed-off-by: MD Danish Anwar >>> --- >>> drivers/net/ethernet/ti/icssg/icssg_prueth.h | 3 +-- >>> drivers/net/ethernet/ti/icssg/icssg_stats.h | 7 ++++++- >>> 2 files changed, 7 insertions(+), 3 deletions(-) >>> >>> diff --git a/drivers/net/ethernet/ti/icssg/icssg_prueth.h b/drivers/net/ethernet/ti/icssg/icssg_prueth.h >>> index df93d15c5b78..e2ccecb0a0dd 100644 >>> --- a/drivers/net/ethernet/ti/icssg/icssg_prueth.h >>> +++ b/drivers/net/ethernet/ti/icssg/icssg_prueth.h >>> @@ -43,6 +43,7 @@ >>> >>> #include "icssg_config.h" >>> #include "icss_iep.h" >>> +#include "icssg_stats.h" >>> #include "icssg_switch_map.h" >>> >>> #define PRUETH_MAX_MTU (2000 - ETH_HLEN - ETH_FCS_LEN) >>> @@ -57,8 +58,6 @@ >>> >>> #define ICSSG_MAX_RFLOWS 8 /* per slice */ >>> >>> -#define ICSSG_NUM_PA_STATS 32 >>> -#define ICSSG_NUM_MIIG_STATS 60 >>> /* Number of ICSSG related stats */ >>> #define ICSSG_NUM_STATS (ICSSG_NUM_MIIG_STATS + ICSSG_NUM_PA_STATS) >>> #define ICSSG_NUM_STANDARD_STATS 31 >>> diff --git a/drivers/net/ethernet/ti/icssg/icssg_stats.h b/drivers/net/ethernet/ti/icssg/icssg_stats.h >>> index 5ec0b38e0c67..b854eb587c1e 100644 >>> --- a/drivers/net/ethernet/ti/icssg/icssg_stats.h >>> +++ b/drivers/net/ethernet/ti/icssg/icssg_stats.h >>> @@ -8,10 +8,15 @@ >>> #ifndef __NET_TI_ICSSG_STATS_H >>> #define __NET_TI_ICSSG_STATS_H >>> >>> -#include "icssg_prueth.h" >>> +#include >>> +#include >>> +#include "icssg_switch_map.h" >>> >>> #define STATS_TIME_LIMIT_1G_MS 25000 /* 25 seconds @ 1G */ >>> >>> +#define ICSSG_NUM_MIIG_STATS ARRAY_SIZE(icssg_all_miig_stats) >>> +#define ICSSG_NUM_PA_STATS ARRAY_SIZE(icssg_all_pa_stats) >>> + >>> struct miig_stats_regs { >>> /* Rx */ >>> u32 rx_packets; >>> -- >>> 2.34.1 >>> >> >> One thing that caught my eye: icssg_all_miig_stats[] and >> icssg_all_pa_stats[] are 'static const' arrays in icssg_stats.h with >> ETH_GSTRING_LEN name buffers per entry. Right now only icssg_stats.c >> and icssg_ethtool.c pull them in. After this patch icssg_prueth.h >> includes icssg_stats.h, so every .c in the driver (classifier, >> common, config, mii_cfg, queues, switchdev, ...) ends up with its own >> static-const copy of both tables. >> >> Would a static_assert() work for what you're after? Something like: >> > > While adding more stats manually, The ARRAY_SIZE() approach was > explicitly requested by maintainer [1]: > > This patch is a direct response to that feedback. static_assert() would > still require updating the numeric constant on every array change. The > goal here is to eliminate the need of manually incrementing stats count > whenever new stats are added > > Your concern about multiple copies of table is noted and valid. Could > you advise on the preferred way to reconcile these two requirements? I > am happy to restructure if there is an approach that satisfies both. > The way we solved this in the Intel drivers is to use a single array which contains both the stat name as well as the offset from the structure where the stat resides. The stat string code just iterates over the stat list for the strings, while the stat value code iterates the array and computes the stat address from the offset and size and base structure pointer. Each object that has stats has its own stat array structure. This is probably overkill, but the advantage is that the strings and their values are stored together and adding a new stat is as simple as adding a new entry to that list. I.e. struct ice_stats { char stat_string[ETH_GSTRING_LEN]; int sizeof_stat; int stat_offset; }; #define ICE_STAT(_type, _name, _stat) { \ .stat_string = _name, \ .sizeof_stat = sizeof_field(_type, _stat), \ .stat_offset = offsetof(_type, _stat) \ } #define ICE_VSI_STAT(_name, _stat) \ ICE_STAT(struct ice_vsi, _name, _stat) #define ICE_PF_STAT(_name, _stat) \ ICE_STAT(struct ice_pf, _name, _stat) Then the stats for the individial arrays are defined like this: static const struct ice_stats ice_gstrings_vsi_stats[] = { ICE_VSI_STAT(ICE_RX_UNICAST, eth_stats.rx_unicast), ICE_VSI_STAT(ICE_TX_UNICAST, eth_stats.tx_unicast), ICE_VSI_STAT(ICE_RX_MULTICAST, eth_stats.rx_multicast), ICE_VSI_STAT(ICE_TX_MULTICAST, eth_stats.tx_multicast), ICE_VSI_STAT(ICE_RX_BROADCAST, eth_stats.rx_broadcast), ICE_VSI_STAT(ICE_TX_BROADCAST, eth_stats.tx_broadcast), ... }; (Note, ICE_RX_UNICAST is a macro that defines the string value.. I don't recall who changed this to macros or why vs just having the strings be directly in the definition...) This is probably a lot bigger refactor to make work, and may not be exactly suitable for your driver. I've considered "upgrading" these data structures and logic as helpers to the core ethtool code (or perhaps now, to libeth) but never got around to it.