From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id DEDB0C30658 for ; Tue, 2 Jul 2024 12:50:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version: Content-Transfer-Encoding:Content-Type:In-Reply-To:From:References:Cc:To: Subject:Date:Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=14+c+GAFBEeax4OFoKirH2vJUORZLtnKCfXXmX7xGY0=; b=lwlATtlBMpQVznHgJcb3aN2jNK u0hz9/ndxExQF47ww8Avnfa1MI2l+rLKfIpafYYMEfdKqkBGBhC/ztdtHqW0PxhrzKtS+1sxTgb+2 32NlvRHvdS0Hm/xGwkkTHd539VBuI76/Eix/5uO1GMhEVbZKmFW/gRURAeW2WHvj7V+tQWhrpmSEU HtNK5O3uWT8+ggM2EwxseMnhZ8g7yCKuhDwwm3dIM69zi9U4awhAOS2Kylu5tZBe91fygglliDv0u tahO5oxouU3OBRiz5z1+FhXiFfGOpfaT9T///AjmQcVrd0UpvQWyZFTLKYSnzmDX8AAHZfpiagecf s34Htx8Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOcy3-00000006kEi-3PPE; Tue, 02 Jul 2024 12:50:44 +0000 Received: from mail-am6eur05on20620.outbound.protection.outlook.com ([2a01:111:f403:2612::620] helo=EUR05-AM6-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sOcxr-00000006kCk-0cQb for linux-arm-kernel@lists.infradead.org; Tue, 02 Jul 2024 12:50:33 +0000 ARC-Seal: i=2; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=pass; b=auAhtt2kDKLkVQ+/sQoMYAg2hdOQ7pPlL7Umb2+9ssDdWurtAsKvXPKLPnQqOJws1qZatczQSnoa0xQcqf6AXey77K93P624EQwK7J06EmtBpULdN3ZrIXJm7iALucfOYa1UpfbA1zCHoeBiNiP5VILwKlEd0SeBL8oYvUE/DjzAFO6b5uHbTqC0/u1ckpYmzumwz3NxixMtTRbxaJBfLbPECDHeuQgiXqcmk+YmSfhYx+whF0qnwIiphmim6rC6thTBxkDqyR5vkwfMsiddLLu0kTFIw1cNnYCYS2Az09MQvcGJOnTnCO5/Q59RTAx/7OmrBlIFk3g4XWYh0se6Kw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=14+c+GAFBEeax4OFoKirH2vJUORZLtnKCfXXmX7xGY0=; b=Qr1rNr0bKx7HWeajjvhizNwKAtYv3Bs9G6qxEBn8oc8V7r4p0n2YDiwQU92M0UoT51BQAF4BSdEFDlsfCEdoDXyyRD+cij9wp94/uqjeNiIiYVP4N90KUhciCL/hh8ThoMtrmEuGXsoHy/C+Mxn6oNB+97HODWLL1xTeaFmE8HubcQ7nkNA661AooAhlxOdjkDBCVImN19xTexvY2DB7JN03YWfXrUZvsDOY6kVE33PYAC9VhWhv3PWRuKLzVY3fOP011f6MDWkxfhQubOZ3uh8ZMNM9u0tXEQmT+ivqhoB/iYBlMfKtTuQ77GflXyjnhoYoEPV5IzcdbbQ/49OLNg== ARC-Authentication-Results: i=2; mx.microsoft.com 1; spf=pass (sender ip is 63.35.35.123) smtp.rcpttodomain=lists.infradead.org smtp.mailfrom=arm.com; dmarc=pass (p=none sp=none pct=100) action=none header.from=arm.com; dkim=pass (signature was verified) header.d=arm.com; arc=pass (0 oda=1 ltdi=1 spf=[1,1,smtp.mailfrom=arm.com] dkim=[1,1,header.d=arm.com] dmarc=[1,1,header.from=arm.com]) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arm.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=14+c+GAFBEeax4OFoKirH2vJUORZLtnKCfXXmX7xGY0=; b=Eofy/W6cVhJm3VGjx+kwH01HETymkyg2kj+wP+HKLrOIsaekreOSjQwITfXWocQvZ8Sl7KkZHqe7CHW6OXoO5Y1odQtv9I+R6M4abYtolLP5/dbvjBR1h2OSqHd05qr/Kgy4DF1JX4DN6mx5DPPXOM32gOJUTkNq2EvCuqkhCXk= Received: from DUZPR01CA0053.eurprd01.prod.exchangelabs.com (2603:10a6:10:469::11) by PAVPR08MB9186.eurprd08.prod.outlook.com (2603:10a6:102:30c::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7719.34; Tue, 2 Jul 2024 12:50:26 +0000 Received: from DB3PEPF0000885C.eurprd02.prod.outlook.com (2603:10a6:10:469:cafe::34) by DUZPR01CA0053.outlook.office365.com (2603:10a6:10:469::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7741.25 via Frontend Transport; Tue, 2 Jul 2024 12:50:26 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 63.35.35.123) smtp.mailfrom=arm.com; dkim=pass (signature was verified) header.d=arm.com;dmarc=pass action=none header.from=arm.com; Received-SPF: Pass (protection.outlook.com: domain of arm.com designates 63.35.35.123 as permitted sender) receiver=protection.outlook.com; client-ip=63.35.35.123; helo=64aa7808-outbound-1.mta.getcheckrecipient.com; pr=C Received: from 64aa7808-outbound-1.mta.getcheckrecipient.com (63.35.35.123) by DB3PEPF0000885C.mail.protection.outlook.com (10.167.242.7) with Microsoft SMTP Server (version=TLS1_3, cipher=TLS_AES_256_GCM_SHA384) id 15.20.7741.18 via Frontend Transport; Tue, 2 Jul 2024 12:50:25 +0000 Received: ("Tessian outbound 3a274c9b26ce:v357"); Tue, 02 Jul 2024 12:50:25 +0000 X-CheckRecipientChecked: true X-CR-MTA-CID: 46fc4fa3bb593851 X-CR-MTA-TID: 64aa7808 Received: from 9774f3dd5cae.2 by 64aa7808-outbound-1.mta.getcheckrecipient.com id 70394DB3-ACE2-48CA-B490-7931B201077B.1; Tue, 02 Jul 2024 12:50:19 +0000 Received: from EUR02-DB5-obe.outbound.protection.outlook.com by 64aa7808-outbound-1.mta.getcheckrecipient.com with ESMTPS id 9774f3dd5cae.2 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384); Tue, 02 Jul 2024 12:50:19 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=GtgsipR8+0XMNkab2M0Ga8CULTMvgpgDEe6uDHICWtqrgaRDkge4iCNvaeQlTZbp5C+QFYuILP31Q2TtB49e96zKcuB/znDskyCWG+jFQjTgHcJses8SahIZFwUF4miz+D1sZvBaw2vbGgETGfencdk0fVdN0idbvwF+tSApyHXy+JBJlJ+vf9FhTDbOP/q2VY2RZ36mBF+snHJHiImS6S4NC4orBs2Te+qD/TNNV/4P65+0md7tgPTdg6tOvPgxrsa3ea/FQzxhBzFR875d5Ilz7ywqJzEuMLXLZ2UvdlFEHsg5rDy7h8G6xJWcdFuQpbWp+6c5c14h+2NZfvvJww== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=14+c+GAFBEeax4OFoKirH2vJUORZLtnKCfXXmX7xGY0=; b=JLd5k8COAFn2FAWraqhHugmEEIWrcbw0SDwjW48vEDhfiPtnp6NqZepakCETYxZT/DR7DROJAHVghkI1W+Gpv63qFkuWFKnd6aeOiSj1CB2P+8lToZ9KZL8BmEYXNSQPIhuWBlcmHLVS0Di96mlzT3rOG4+6t8wcpEya0jref4xjijoLIKBO6QvWGrXeq+tbmJkucaiUqI6Wt4aA20X6GUySNHC9SCiutN3FEYXwk1iDVEwXW8ECBrK1+RxKJOjWjsPlYJHRrBaFS4jRhgW3hJh6iS1CilbXQo2SCXj11TxS+vxRoqAf7SFdkFDZtj//hphtvOFLVP0XD/5bgeIJ+A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=arm.com; dmarc=pass action=none header.from=arm.com; dkim=pass header.d=arm.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arm.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=14+c+GAFBEeax4OFoKirH2vJUORZLtnKCfXXmX7xGY0=; b=Eofy/W6cVhJm3VGjx+kwH01HETymkyg2kj+wP+HKLrOIsaekreOSjQwITfXWocQvZ8Sl7KkZHqe7CHW6OXoO5Y1odQtv9I+R6M4abYtolLP5/dbvjBR1h2OSqHd05qr/Kgy4DF1JX4DN6mx5DPPXOM32gOJUTkNq2EvCuqkhCXk= Authentication-Results-Original: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com; Received: from DU0PR08MB8639.eurprd08.prod.outlook.com (2603:10a6:10:401::11) by FRZPR08MB11046.eurprd08.prod.outlook.com (2603:10a6:d10:135::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7719.32; Tue, 2 Jul 2024 12:50:16 +0000 Received: from DU0PR08MB8639.eurprd08.prod.outlook.com ([fe80::253c:bc53:f4b8:3c7a]) by DU0PR08MB8639.eurprd08.prod.outlook.com ([fe80::253c:bc53:f4b8:3c7a%5]) with mapi id 15.20.7741.017; Tue, 2 Jul 2024 12:50:16 +0000 Message-ID: Date: Tue, 2 Jul 2024 13:50:15 +0100 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 2/3] Track basic SCMI statistics Content-Language: en-GB To: Cristian Marussi Cc: "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "arm-scmi@vger.kernel.org" , Sudeep Holla References: <20240701142851.1448515-1-luke.parkin@arm.com> <20240701142851.1448515-3-luke.parkin@arm.com> <490bb053-f2dd-4c6a-a976-c8d21d66eb4c@arm.com> From: Luke Parkin In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: LO2P265CA0260.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:8a::32) To DU0PR08MB8639.eurprd08.prod.outlook.com (2603:10a6:10:401::11) MIME-Version: 1.0 X-MS-TrafficTypeDiagnostic: DU0PR08MB8639:EE_|FRZPR08MB11046:EE_|DB3PEPF0000885C:EE_|PAVPR08MB9186:EE_ X-MS-Office365-Filtering-Correlation-Id: f06c3428-06ae-4c23-15e3-08dc9a958b76 x-checkrecipientrouted: true NoDisclaimer: true X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam-Untrusted: BCL:0;ARA:13230040|366016|376014|1800799024; X-Microsoft-Antispam-Message-Info-Original: =?utf-8?B?TkQ2UllhOVVHTUVyb3MxWkljV3Y0aVhFV0F6S0pxK2YzbEd2V0tnSkxGdGJ0?= =?utf-8?B?ZlNtNnNmTTNiaDY3RUtWWk9JWE1Td3hYeXNIaU8zWUk1Y21meXI5RGJDRGZo?= =?utf-8?B?V09IVmt2SXBIRmk3QmVzTUhoQ2xtSnJwNWxXYUdtU0k0Y2ZLeWh3cWNRa3BH?= =?utf-8?B?NU1SMnk2WUFuL0d5SE93L0ZkRmlORldIcVM4RHFJTzlsT0k0WDdrc0ZhR3gy?= =?utf-8?B?ajVzS1ZCdUIvdjVzTytjeitOQys5WnZTcmJoZ2VZbXRtaUZrSkxDOXM2ZmV6?= =?utf-8?B?UDhaZkwwNHBVdlBBbnVDTG81UmJ5Z2cya2wra1pGRVNuVnE0bHdJbU5zMi9P?= =?utf-8?B?NW5RTUlhKzVJbDAvZ0NmV1pqREVZZDVvMWxBOGdOQjlKYVFrb1FwbWk3OWp4?= =?utf-8?B?YzN1RkR2M2dteC93MS85YW4wTGJLTm0yM2lTY2MzMnNqRmVOS2x5QktBMEpQ?= =?utf-8?B?czY5Zi9wQ0FDUWVvWTVTVGpKb2N4NzR6ZEVVUHg4OUx6dFBPTDRITU84WUtl?= =?utf-8?B?R3RMdER5UFhZR05Ma2RnVGZQMzAxUElLV2xaZ1N6VVppZjV1b1hFdE04enZF?= =?utf-8?B?VlFsQWNqR2VmZlRBbU1aYmY5b1IxazR2c0ZscW5tY0IrYkVKcjc1NUVnNS9M?= =?utf-8?B?OVNKbENYQy9odjBWMHJxQVpnUHBqVnNkemt0SENLL09ROExIdGkrSVBod2tR?= =?utf-8?B?S2JEcUpBUGQwVlY1RVVzdXBvY0xnMzFmWFdINStFamZ2ajFjUkxhRUtLcFdh?= =?utf-8?B?TWpLUUNlc2dvOS8zRVVSaFVwY0xpcWxiMXdGZUpDM0dmQ1dGZVBwSHVFaHNK?= =?utf-8?B?ZzhHblN4YloyQ1ZlcmpZd1JDVEhtdVBFZE91OFZRdDdmSHkrMFZGOERMbU1Y?= =?utf-8?B?QW9OVGpUbU5pU2J4MGlkWEpXQ0huUG12WFVFMHRITDZXKzRIUmN5VmNPQzls?= =?utf-8?B?V1MvUmtmaUtac0hJM0lVYmdINEozQk9mK0FMcDFrbXk2L01QUjFZVkNBdXZP?= =?utf-8?B?bkFodTJNNUxBUEFiN1RXL0p2Nm1aRVB4N3lrQnZHLzdJSTZBNHJud2JxcEp2?= =?utf-8?B?cXZBT0hMY2pzY3JPVUIzRStLTE5OMjFUMkRHeHVKeGZLQmlnR2c1cGh3REVM?= =?utf-8?B?U3BwTVFLdU5OSXIwSjFFWGJZTTZTbG1LZms1REkvKzY1WHYyMW9ENDFENzN6?= =?utf-8?B?NFhWS0prR2F2WGxmL1JBSW9lRmRPVVlyV045UW4yWnVid0laWDNRRFM4b0RT?= =?utf-8?B?NUl5N1dMWTY5Wis3YnRnUk0wVDI2WWFkQlIxUGcwZFVyTlM1UmM2NE5JSDNF?= =?utf-8?B?TDQvS3JuQkVKV3hqaUxrZ1VSdHVhRXF3QXIzN3Q1QkR2N1o5Q2thekFHaXVH?= =?utf-8?B?M05yNDJFSGZiRlNWUDJUOWtLWHRPVzhSb0xKd3puOUJBLyt6cHhQdDZ5czhT?= =?utf-8?B?THoyV2pXRm0xekpnZVgwOWwyald1YmdWdjdGbThrZ1NsM3NuRzRXaUczMTVw?= =?utf-8?B?ckFPY2FWbFFzOW1qMkVGU0Q2eW1tRGtwRUp0SmE3VXpPZHJHeUROZGZZczhl?= =?utf-8?B?M0ZISndXZEZGN0lXVVhPZXEzUlVSM1FOUUF5L1ViSXViWDVCeEZTZmdObDdv?= =?utf-8?B?eTlaYWdmcFU4QmdyL1VTQlFWa0grb1ZIRUw0TXVZZGhHUXVYQkFoRzc2RUJi?= =?utf-8?B?aWM2UlBhazZaVGNUVXdKbW5iTE9sMUp0cHl6VW1RQng1dFZkZmhsNDhlaXhZ?= =?utf-8?B?Sm5nUjZGMTdYUzRPRTBKbU5nK3FucHp3SktqdWFYRnp3TmlXWkdzT2VDM2px?= =?utf-8?B?RlFwUndkdlBqMVhaamxsQT09?= X-Forefront-Antispam-Report-Untrusted: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DU0PR08MB8639.eurprd08.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(366016)(376014)(1800799024);DIR:OUT;SFP:1101; X-MS-Exchange-Transport-CrossTenantHeadersStamped: FRZPR08MB11046 Original-Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=arm.com; X-EOPAttributedMessage: 0 X-MS-Exchange-Transport-CrossTenantHeadersStripped: DB3PEPF0000885C.eurprd02.prod.outlook.com X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id-Prvs: d890e67f-390f-427e-87ff-08dc9a9585ea X-Microsoft-Antispam: BCL:0;ARA:13230040|35042699022|36860700013|376014|1800799024|82310400026; X-Microsoft-Antispam-Message-Info: =?utf-8?B?MmJVeGp1VXRGYmowY3NiQ1cyTUhtSXl1bzMrYzB3ZnQ4VTZzQXJnc05ZNlBi?= =?utf-8?B?TjBkc1NVSWZYT0toSE1IKzAySkUzS21XNzhla1JZTktJNjRkUkRVSTg1ZEwz?= =?utf-8?B?a3BORnNSWGFUbGxIRXlacTlUcnVLMUl3K2xpcmVGQnI3Y2NWRU1XRlhYMjBw?= =?utf-8?B?WEliNFFTYXZTTmdRMHBRbnNtakloeWdVaXVTa0xETzROVTVPakZNejYvRUZr?= =?utf-8?B?L1Mrck9RR24zNFlDSnc3WVJScjlmNkUzendlem9Ic1JKelV2b2JsaFNRZDJh?= =?utf-8?B?SW9QNDFxaWxRZHN4aVdNalQ1R21qblNiVGZYeHhNQkV1T0J6cE05Ynh1VzdM?= =?utf-8?B?TndnMkZzQW8zVElFcmVCeHZVV1lSZlp0S3hvNzlRanE5a3J3TkdiZ0RxdFFI?= =?utf-8?B?eDZNazA3T2FQT3B2YmF3cVUxeXFRM1V5TEYrZUhUMHg0VU8rR3l6dHd0WWFp?= =?utf-8?B?UWJRdnV2RFNEelE1TzY4ajdSK3VTdmJrWElveW9qUVUrcWkrKzdnTDZHcW51?= =?utf-8?B?YXcrSEF0TTM1SkdUdXl6NG1sSTVvRStQS3FlemUxNmdGdEkzQ0IxMTB1dHo3?= =?utf-8?B?cHZYSHVOMjVaREU3SGpxRTJpTldveS9weUZ3S0Mva0pIdDdGazRMd1hsWUh0?= =?utf-8?B?UWpQL0ZZb3hqNUJYYklRNGJ3dnBuSkVnOGpyZXVRT3VITm0xWVV2TFdGT3Nj?= =?utf-8?B?OVRUczlRak15UUh0ek5pREh3aDNXd0xlQkpoeVJKaE1RUmxWV0EwR1phT0c4?= =?utf-8?B?SFBnWldNZWNtQWo4UjZGYW9WV2NUbE9MN0MzdVAxN01jSXNxelZib2FFaDNL?= =?utf-8?B?MVp1bXlwdGNIa29nZTgrcnFSL3Jld1BMT0tMenlaYWp3ODYvbDAvUGx0Zlgv?= =?utf-8?B?UDNmM3Rkemw3NFBpU3d6cXFvbEh2V0dPQTN3dkJwb1Fza0xNdWRXcTl4aDQw?= =?utf-8?B?dnZhbmNDUW5SVjV5Y2pVRW96eHRjWTdVZ3EwUENoeGExTFp1bjZnUkZNSnZ5?= =?utf-8?B?djBPSEY2ZWw2U0IrbEpRMCtNY29oVVl4OUtqcnZYSUlmaEVpcXpWMERsdm5E?= =?utf-8?B?UitJSlViR1pnMnRxZHRIVWkwMXNyYWJGRkd1Q1c5Nm5qZ1JTWm1tTkloRkI3?= =?utf-8?B?RCsrUkk4N1plaVlTdU1nOXA2V3lZam43djRYdWJwWHR4N0RFNEJCMHorcVdH?= =?utf-8?B?bTNheGpuUlE1ZnhqQ2d2VjlpREpaOE5aNUYxQU81cWgyL0lOOWhkZDBTWDVH?= =?utf-8?B?Q0RiczlscE1xejZTVEtLYkd6MnRvek5jZEFSa1NKajNwdEp1ZHdsOENxaVpF?= =?utf-8?B?RkJidXVFSEpUeDM5ZzFMN0dhUmxJU1loZGl3TFMyaXYwWFlZZXBYK1E1aWk0?= =?utf-8?B?cmdRN3ZjM3JEdTdsdmZSNkw5clRNRUUxSW5URXlUODJVVDFLS01KZmRwNGhQ?= =?utf-8?B?Q2lVaUxaZUhmRWpxUWc4UFhtRzY5MFhsK1B6cGt1SmQybDRWVVUvemNKYnl4?= =?utf-8?B?TWczVmpMZFhYSGFXeW9XOFY5Wjc2VWcyVWFkb1dZenltaWEyQU5VMGozUGpm?= =?utf-8?B?VnVzSEo4SmtBUHY0ZFdwOStHdlV6QkhHTTZRTVV1RWpNcTdXNmVUNFlaRG53?= =?utf-8?B?cE9JZXlyaUttWldFZ1JOSzd5alRKR1JhOW5yc1VzLzlMOXFTM0RxUkE3U085?= =?utf-8?B?NEQ4SzBrYTVqWUQzK0dTZW4yb2xjcHdvaWFUMG5PY0JPWENONk5OemVUd1o5?= =?utf-8?B?WURBcklMTjlxVUMrMDJ0bUhBd2w2WHppekdSM2xSQldTOVNNV1dKbEp2b2Fl?= =?utf-8?B?QWRPSUhjYW5LaFBrUTF5b2EzNzdRWmZlSDQyZk1CY1RUTHBGRGdqNFlJT0Nk?= =?utf-8?B?VWMyLzFWdmw1czhSUVVwcjhzS29NNXpEdG9YZENEeTZQQU12SzNLRmVWOGtZ?= =?utf-8?Q?FRHIeQNmPBlx83CuAF+TKnlJGZiZOAD7?= X-Forefront-Antispam-Report: CIP:63.35.35.123;CTRY:IE;LANG:en;SCL:1;SRV:;IPV:CAL;SFV:NSPM;H:64aa7808-outbound-1.mta.getcheckrecipient.com;PTR:ec2-63-35-35-123.eu-west-1.compute.amazonaws.com;CAT:NONE;SFS:(13230040)(35042699022)(36860700013)(376014)(1800799024)(82310400026);DIR:OUT;SFP:1101; X-OriginatorOrg: arm.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 02 Jul 2024 12:50:25.9406 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: f06c3428-06ae-4c23-15e3-08dc9a958b76 X-MS-Exchange-CrossTenant-Id: f34e5979-57d9-4aaa-ad4d-b122a662184d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=f34e5979-57d9-4aaa-ad4d-b122a662184d;Ip=[63.35.35.123];Helo=[64aa7808-outbound-1.mta.getcheckrecipient.com] X-MS-Exchange-CrossTenant-AuthSource: DB3PEPF0000885C.eurprd02.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAVPR08MB9186 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240702_055031_668778_7769EBA9 X-CRM114-Status: GOOD ( 16.00 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org > #ifdef CONFIG_ARM_SCMI_DEBUG_STATISTICS > static inline void scmi_log_stats(atomic_t *cnt) > { > atomic_inc(cnt); > } > #else > static inline void scmi_log_stats(atomic_t *cnt) { } > #endif > but those calls can be split and placed alone where that some condition is > already check normally as in as an example in scmi_handle_response(): > > if (xfer->hdr.type == MSG_TYPE_DELAYED_RESP) { > scmi_clear_channel(info, cinfo); > complete(xfer->async_done); > + scmi_log_stats(&info->stats.dlyd_response_ok); > } else { > complete(&xfer->done); > + scmi_log_stats(&info->stats.response_ok); > } > > ...what do you think, am I missing something else ? Ah yeah, that looks better to me. I'll use that. Thanks! Luke On 02/07/2024 12:10, Cristian Marussi wrote: > On Tue, Jul 02, 2024 at 10:57:23AM +0100, Luke Parkin wrote: >>> Ok, so IMO, this is the main core thing to rework in this series: the >>> "counting" operation/block should be defined as a macro so that it can >>> be fully compiled out when STATS=n, because these are counters >>> incremented on the hot path for each message, not just once in a while, >>> so the above if(IS_ENABELD()) now will be there and evaluated even when >>> STATS=n. >>> >>> Something like: >>> >>> #ifdef CONFIG_ARM_SCMI_DEBUG_STATISTICS >>> #define SCMI_LOG_STATS(counter) \ >>> \ >>> #else >>> #define SCMI_LOG_STATS(counter) >>> #endif >>> >>> .... I have not thought it through eh...so it could be radically >>> different...the point is ... the counting machinery should just >>> disappear at compile time when STATS=n >> >> Hey Cristian, Unless I've missed something, It looks like IS_ENABLED() does do >> what you ask for. >> In Documentation/process/coding-style.rst:1168 it reccomends using IS_ENABLED >> for conditional compilation over #if and #ifdef, saying that the compiler will >> constant-fold the conditional away. > > Yes indeed, it will be compiled out anyway, forgot that despite having > it used myself a few lines below :D .... but from the readability standpoint, > given that we will sprinkle this all over the code, wont be much clearer to > conditionally define once for all an inline function (like mentioned at the > start of that coding-style.rst paragraph) or a macro in a header (like common.h) > to wrap the atomic_inc > > #ifdef CONFIG_ARM_SCMI_DEBUG_STATISTICS > static inline void scmi_log_stats(atomic_t *cnt) > { > atomic_inc(cnt); > } > #else > static inline void scmi_log_stats(atomic_t *cnt) { } > #endif > > and then just call it plainly wherever it needs, knowing that the compiler > will equally compile it out all-over when STATS=n. > > ifdeffery is discouraged in the code flow but it is acceptable to define > alternative nop fucntions in a header. > > Also because in some of the callsite you handle 2 stats with some ifcond > (conditional on the IS_ENABLED that is good) and that could be a problem, > but those calls can be split and placed alone where that some condition is > already check normally as in as an example in scmi_handle_response(): > > if (xfer->hdr.type == MSG_TYPE_DELAYED_RESP) { > scmi_clear_channel(info, cinfo); > complete(xfer->async_done); > + scmi_log_stats(&info->stats.dlyd_response_ok); > } else { > complete(&xfer->done); > + scmi_log_stats(&info->stats.response_ok); > } > > ...what do you think, am I missing something else ? > > Thanks, > Cristian