From patchwork Fri Jun 21 16:50:58 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Yuan, Perry" X-Patchwork-Id: 806880 Received: from NAM12-MW2-obe.outbound.protection.outlook.com (mail-mw2nam12on2049.outbound.protection.outlook.com [40.107.244.49]) (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 3D27316ABF3; Fri, 21 Jun 2024 16:51:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.244.49 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718988693; cv=fail; b=Jso9792aDKtiXIdgOCzmUNhGPFdQ89gwVuousjSHEsgzyRZzPtrHwKvSoz9JzG1Uta/hUjDcm4Ify8n4WrGJtR3AY2yhRkafCF09aePOzjS3rfUiXzbV2CQ5VLNnN2Y6Q/Q/yNTFbmJPHoHaWr/z8z/eq91BJQR7OxSIgYvIIZE= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1718988693; c=relaxed/simple; bh=VPnkaK6QQZKyd0qyGfloYPwag1g/W8qHeGJbJPcnjN4=; h=From:To:CC:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=mDWziIdRISbY2uplpPdrIORFMgdSlNVY4wCa0UYeDbjsiLehxgVsLOEKvYtJipTNd6YmFhrZ20G62NVV9OtDImN48ne9VanKcp3Tcq5ArWzOGfitFBgBSsmUfX0vN2Y8gTmv6XUDwzN8bfWssgmroBVr93M73jk90JN+ODMoCoY= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com; spf=fail smtp.mailfrom=amd.com; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b=oPGrUMVa; arc=fail smtp.client-ip=40.107.244.49 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=amd.com Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=amd.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=amd.com header.i=@amd.com header.b="oPGrUMVa" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=bcGCZ/7FTSHsnCapcKnPN7N28FZjPTKsTlO/W4zc6sCPhVfNEWIMCrDYJYXeJzKwgqOM112WGznhM6pj0yf92KTx9PBKEgUmZW5chgVs9AMBpUrzU+b+tqRvrPMLpciBYlLgyxZALD/gpB23AiiHdwew8mvcERBGlbko852oKoKlngh9rMcpu7RkpIc556ksexCQ6ZCZyfvnNYkotUcFkS/tuVaBq8THLRtU85eFV3HUbZ2Vtpt0qNBO4P/gruW2nxt9Dy7IYF6jUWtPwQG/Gf1tdMZBP3OpV24cjKdeRfteA8+92jtGBxUTZNvn5CTeQyRKH0mA2k34h4+vG0qf8A== 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=UkBgW1FoEz+y4K0wWUhOkxzqKaui0JxO5sx0rMNUGyg=; b=LLH3RoGlDT9yOwzhskE4w+7Z3RRh2/Kss9bmRSO3tArYBwlSkeXoDKYqJxcPKfgNXW0cFTHSuGrsLaJVjmD49P7WRgN6udzE9ESRyWg9VAwbQfFnxr2xx4ydYNWemZBIlYRiNL15ySL2bsjWhCJ+7aKnB/rk4mCi+ikuOyuorgQcg4GArb1aLoq53yYpB6o+QoKpo7rmTlfTMKQnaRuME930vdTgp6TAEWPaAy/cHpq2RCJ94bW4qf0YDjIo7NqjxE3no5wf1TWF9xeqQhrZkXBvPRuF1dRs3xzASGThoi4HhzrwjFjhtJ1W6gG9XHy15ZX4oWIrp1bhXig1KjwDAw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=intel.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=UkBgW1FoEz+y4K0wWUhOkxzqKaui0JxO5sx0rMNUGyg=; b=oPGrUMVaOzbPDiWZqkKBFApLkl+jLnd7Dm+Z5A/Bjqh4hIe5YtnRF39Ig6Fl/FtfAPekIBni/TJL4fbxlyfGM47cRPFxs+m30gYbDb0TFZvsTT9taR+IqEiH8wz0x/b8aokqHtINJbPYFCtYZ8UTHHe+B37xVsTSrVtj5l98pvk= Received: from SJ0PR05CA0191.namprd05.prod.outlook.com (2603:10b6:a03:330::16) by CY5PR12MB6647.namprd12.prod.outlook.com (2603:10b6:930:40::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7698.21; Fri, 21 Jun 2024 16:51:28 +0000 Received: from SJ5PEPF000001F0.namprd05.prod.outlook.com (2603:10b6:a03:330:cafe::37) by SJ0PR05CA0191.outlook.office365.com (2603:10b6:a03:330::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7677.32 via Frontend Transport; Fri, 21 Jun 2024 16:51:28 +0000 X-MS-Exchange-Authentication-Results: spf=pass (sender IP is 165.204.84.17) smtp.mailfrom=amd.com; dkim=none (message not signed) header.d=none;dmarc=pass action=none header.from=amd.com; Received-SPF: Pass (protection.outlook.com: domain of amd.com designates 165.204.84.17 as permitted sender) receiver=protection.outlook.com; client-ip=165.204.84.17; helo=SATLEXMB04.amd.com; pr=C Received: from SATLEXMB04.amd.com (165.204.84.17) by SJ5PEPF000001F0.mail.protection.outlook.com (10.167.242.68) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.20.7677.15 via Frontend Transport; Fri, 21 Jun 2024 16:51:28 +0000 Received: from pyuan-Chachani-VN.amd.com (10.180.168.240) by SATLEXMB04.amd.com (10.181.40.145) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Fri, 21 Jun 2024 11:51:24 -0500 From: Perry Yuan To: , , , CC: , , , , , Subject: [PATCH v13 3/9] cpufreq: introduce init_boost callback to initialize boost state for pstate drivers Date: Sat, 22 Jun 2024 00:50:58 +0800 Message-ID: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: SATLEXMB03.amd.com (10.181.40.144) To SATLEXMB04.amd.com (10.181.40.145) X-EOPAttributedMessage: 0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ5PEPF000001F0:EE_|CY5PR12MB6647:EE_ X-MS-Office365-Filtering-Correlation-Id: 01172b1b-ef7e-42aa-97cf-08dc9212652f X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; ARA:13230037|36860700010|1800799021|376011|82310400023; X-Microsoft-Antispam-Message-Info: Ysg90GrcWHcvU4nzuULA+osMXUyXhDo07bJ/Y5YmFfD0OXH7z8eIRAfLYisEBAMHkV0We/4QO/n81WY6L7kHkRI2bvUQ+/JCvQUdkL1WVS5QonlwqEvqBupGbRj44VPCVAnEyAAz5Hdx7C5LB0g1Ry4kN1+YnuWxY068aJEQTkm0yaZmGpcGVQo75awWcc0ndnwloDbwiV5DMzOYoci7ok64gh7KsC6QexXLRSnwNfjcTdECkpFGG4HgDN8xjr+xL2iAyT17WuBBByi8Aluvdm/icxklRqdZKOYOUYnFQdClfPyh3i8K4JDHeZw4AWoVoajAKbyNC/2c1cgjhjS6J0n/pq4oD3vaEa1PsAAqpv2lNRBH9pBlhbXeewcs6AwSqhoDLFLGJxlANoAgHFO4P19xqWuLdGueHpbNbLu7OkCfMEpFnwTE0gRQTwb+Yw7AeaQ+ai3mxM77vpmlnRXlyAcdr1eUt2teF4JftWeVrE5RrRjuikZjeNfc1EgUWZQXDszneHR1zmWdXQK/WmBO5QWtrb+WBdsVMRkjNYE6aEWC9lt6Y4bl3ZkqgijLey07+bQ36wCyeziuk2zVGZ94O5iM+80G0FzVK9saKujsqOlE/IIniDOsRbqVeXMamb3sSAdDszAWXCUWxfQtYUUtjkSvd8TKets9XbvnN8NUju9t3pmmD+ckkUCWRVNSRFI7Lr8QkXTbzo9E2Buvagko0haEernNQiy8VTK4D9K81W8c1wmp0LIL4rduX8o1t9khn/+Lbo6EhUZsQNmPeYMSlufX/IkMBxAUwKh+onNKSAfNeU0WYCqzaaGHeeRRuGJQCUWfUyH1vuyBCj6G82KsOJjSzx8NlOTD7ixwDu+CXMBsw4YS6Iehm+mzIXYgM/3f9+gcLPi7xDYAgzCkCEWZxFfSSS7SQgpj8TshCZRldRoFjphstiHor5+wTgSbcjlDOCAhru/4JSpXfmSu71KzErz1qRqXLDfCFuer2SR11+u9/okK8SJi1nXf2tIC1tSS+I5aQ756w1fVUnjiCpxOYOjyp1xNrWBbXsUktoS6NWTQEZfi1tKIdLFLv4dKSCOft3V7apCE/CEiv+Tqb/HBn55olJGh8Ez5ilwQNovc4zXFCPvqItp+0WFEAf462Rs2USnepnR6FWrq5tL5nIlqID42IzZQ79uMMsN+YCZ9WsLM5OPsCQpDVS+y7VhjX76FhcvIwwGr/T4QGfegbgsk4JLyGz0rs2UZLYaZ8tsswgylasGIx6flL2ygpeVqXtDo/LjNCTfX/gxAPxKkEWjeUFqZmuZvOt8BVNVxrThY+GDM3shaDRAU2O7C27YZY9P+QJ8gWZ/Le2X9tP/vWEBEExTkR3sVIBbCsNki6nH7jasr5rhg/qMdg+MCm/JKFkkXXzjUjlZIF1UhFpaLlPxbUw== X-Forefront-Antispam-Report: CIP:165.204.84.17; CTRY:US; LANG:en; SCL:1; SRV:; IPV:CAL; SFV:NSPM; H:SATLEXMB04.amd.com; PTR:InfoDomainNonexistent; CAT:NONE; SFS:(13230037)(36860700010)(1800799021)(376011)(82310400023); DIR:OUT; SFP:1101; X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Jun 2024 16:51:28.3046 (UTC) X-MS-Exchange-CrossTenant-Network-Message-Id: 01172b1b-ef7e-42aa-97cf-08dc9212652f X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=3dd8961f-e488-4e60-8e11-a82d994e183d; Ip=[165.204.84.17]; Helo=[SATLEXMB04.amd.com] X-MS-Exchange-CrossTenant-AuthSource: SJ5PEPF000001F0.namprd05.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Anonymous X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY5PR12MB6647 Introduce a new init_boost callback in cpufreq to initialize the boost state for specific pstate drivers. This initialization is required before calling the set_boost interface for each CPU. The init_boost callback will set up and synchronize each CPU's current boost state before invoking the set_boost function. Without this step, the boost state may be inconsistent across CPUs. Signed-off-by: Perry Yuan --- drivers/cpufreq/cpufreq.c | 14 ++++++++++++-- include/linux/cpufreq.h | 2 ++ 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c index 1fdabb660231..0c99d2dfdb53 100644 --- a/drivers/cpufreq/cpufreq.c +++ b/drivers/cpufreq/cpufreq.c @@ -1429,8 +1429,18 @@ static int cpufreq_online(unsigned int cpu) goto out_free_policy; } - /* Let the per-policy boost flag mirror the cpufreq_driver boost during init */ - policy->boost_enabled = cpufreq_boost_enabled() && policy_has_boost_freq(policy); + /* init boost state to prepare set_boost callback for each CPU */ + if (cpufreq_driver->init_boost) { + ret = cpufreq_driver->init_boost(policy); + if (ret) { + pr_debug("%s: %d: boost initialization failed\n", __func__, + __LINE__); + goto out_offline_policy; + } + } else { + /* Let the per-policy boost flag mirror the cpufreq_driver boost during init */ + policy->boost_enabled = cpufreq_boost_enabled() && policy_has_boost_freq(policy); + } /* * The initialization has succeeded and the policy is online. diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h index 20f7e98ee8af..0698c0292d8f 100644 --- a/include/linux/cpufreq.h +++ b/include/linux/cpufreq.h @@ -409,6 +409,8 @@ struct cpufreq_driver { bool boost_enabled; int (*set_boost)(struct cpufreq_policy *policy, int state); + /* initialize boost state to be consistent before calling set_boost */ + int (*init_boost)(struct cpufreq_policy *policy); /* * Set by drivers that want to register with the energy model after the * policy is properly initialized, but before the governor is started.