Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Jon Kohler <jon@nutanix.com>
To:     Jacob Pan <jacob.jun.pan@linux.intel.com>,
        Len Brown <lenb@kernel.org>,
        "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>,
        Artem Bityutskiy <artem.bityutskiy@linux.intel.com>,
        linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org
Cc:     Jon Kohler <jon@nutanix.com>
Subject: [PATCH] intel_idle: add CPUIDLE_FLAG_IRQ_ENABLE to SPR C1 and C1E
Date:   Thu, 30 Jun 2022 15:43:09 -0400
Message-Id: <20220630194309.40465-1-jon@nutanix.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?SzEh/Ugoj/j5QGhWQXN6xCBql+UgE7+UPSOvrqcrvJr+lMwrdj9OyLOaMkN1?=
 =?us-ascii?Q?W9jlTgvK1RFfXrzcGwDfHQ0AM4CuF4IO/o4oWy9qijkESat8e87yv/grZYnd?=
 =?us-ascii?Q?+dsCXoN67+pZ/GPN/IaEMArlKYfCghvX9fSbI/JuYNthZ/3YwyMeBICNvKGg?=
 =?us-ascii?Q?xxVdl8T+A1Gs1bOnUCuURqyFl9qJwto6iXLR0/M2HXGi847USd0xy2P8DL3s?=
 =?us-ascii?Q?2vQJRQOWKnpuSXeFeXIeQZrzyShoFdSZnLU/UfJIpavJcnwyGWhn8UeYTXEA?=
 =?us-ascii?Q?oaJu6vDYkkF0nUz6UAJRPAdQMh+IlGQ92ro1ADybl2vChT7qIBWUBprBebsT?=
 =?us-ascii?Q?j5axxzqJ+ANUwLlcakTLADn72poDaNJfPOXsc4WZwD5wS1X0GifXyjMX6o0a?=
 =?us-ascii?Q?4qauOVng05teAX/BzK6DilAAzQxG5hct/YysAvUz/QEsFEXDHc+6mW7pk4wM?=
 =?us-ascii?Q?qln379aGCWLFrhG9CHKeI9j9jHvoDE2rpFpXTsYsAKbLBPnFdIRoedbllJio?=
 =?us-ascii?Q?2Ki0RDRQAKr5e5qEJri+HDifQN0RH++9WcyUGV0X9jajtEoWNausezysNcg5?=
 =?us-ascii?Q?HOyD7atyr9mK0AupfF2CCShNHuwD2/kJ9DnK13jIPwhGl0BO+o/n0+MDVc9O?=
 =?us-ascii?Q?k51XO0n8fpVCN+tVSj521B8EHTGDxK4iVHHwYyN0tg9Ugx+BS6c/gPA+BJLv?=
 =?us-ascii?Q?3qUQ39vjvetSuicpTv+7jPmlupQcE8t9p1BwA2iZnHdk4l38NJIhsFVRaAqg?=
 =?us-ascii?Q?//xS9S0XBN+imz4Js4mlQ4JjUOkmd0o5lWHv1WgEl9ZDnTitHpE3uN703jgk?=
 =?us-ascii?Q?KyI9UoWr6gjnGsCHv/P2vieQzqjFK9gv2Y9DySF0Y1i9kEkvNT6mC29m9bN/?=
 =?us-ascii?Q?+GzY2XGuKIUGp6KkcoCL5EUc+GzPYcdWP5uSYgrGo39yOJHxqWp7VkmGwMf3?=
 =?us-ascii?Q?6+Bz/ElgzEaQaapWdB7JDNkxmAPrfhH9H556779tRkZIV1jLCyogXGX+DdJV?=
 =?us-ascii?Q?U/GQsPxqYjwc1TDnGjLn2sZEWeqGuPdqTNDH0VRPJfmNXf4Js1cBXoC+jtAz?=
 =?us-ascii?Q?kBbmDasnqY/1NmIjxrdPMbAxhAe3YNy0ecaia9AbCI+6auws7EkKX34X/Cw9?=
 =?us-ascii?Q?9fzr/SDxNSKsSuOhO8xMtvw2Je3YsgV6hgzsLcGwmgSp5MtO/vyQJCZB/em4?=
 =?us-ascii?Q?0JsTTnKUMP6aGw+UhMZ6F89EIDYsqptTsaF3mgKUo8JWQnALjTmno1AY0xrO?=
 =?us-ascii?Q?3IJ/5M/ytimKquJcaiSTc5naEtUs9KddkABso9cg5iskUn/Ba06NVEaSM5G3?=
 =?us-ascii?Q?QV9v9bKL+wb7/2zR6HKo+BMzSw8M8NNZ7AKRnH4ja5KHxszXaZPoCSbIaMCK?=
 =?us-ascii?Q?oS8ZNu6bQxSy6aOcg2jx8oTQ3RgaDU3fMZ1nz5hl1njhBHfJ3pIsMfRjUDoo?=
 =?us-ascii?Q?qASE1MXtdAcPPXIuaPW65WwGgxS8TSmHitvuCr6EvkvfJJtmcl4NpifPfrSk?=
 =?us-ascii?Q?ezmP2IwEg/DJoZaAQ4pyRXYO3i8J8Nzm9XngjRo1VhYkXTIdLEVqPq4H2Gtg?=
 =?us-ascii?Q?toacwbZ+RsvHQk0EUIQ1l9RYkq74dFE149Vc8dwK1VtZRaeGId5bgczq/PQ4?=
 =?us-ascii?Q?lPluV3MJTDc9zgxZhAbuyUTlh7ZVfKZn9f1OQcpe5sjEsWNPpOJ7/eOp9DWM?=
 =?us-ascii?Q?MiN9nQ=3D=3D?=
X-OriginatorOrg: nutanix.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 604a03ef-b4e9-466d-4847-08da5ad0d495
X-MS-Exchange-CrossTenant-AuthSource: BL0PR02MB4579.namprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 30 Jun 2022 19:43:39.3378
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: bb047546-786f-4de1-bd75-24e5b6f79043
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: INKcNpSVp5cKQsN5jDRg0NVfNv3Yl51UP6rVJoVATfsJ1dMKIRcXKjnwWkXDcTnMj74erfczuPABQXqIzus0ZhoLO9cebCdjahUS/4upm38=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR02MB7567
X-Proofpoint-GUID: CnQnfn8WiD_YDYk2HHqoQWLpVwPISiMP
X-Proofpoint-ORIG-GUID: CnQnfn8WiD_YDYk2HHqoQWLpVwPISiMP
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.883,Hydra:6.0.517,FMLib:17.11.122.1
 definitions=2022-06-30_13,2022-06-28_01,2022-06-22_01
X-Proofpoint-Spam-Reason: safe
X-Spam-Status: No, score=-2.5 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,
        SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Add CPUIDLE_FLAG_IRQ_ENABLE to spr_cstates C1 and C1E, which will
allow local IRQs to be enabled during fast idle transitions on SPR.

Note: Enabling this for both C1 and C1E is slightly different than
the approach for SKX/ICX, where CPUIDLE_FLAG_IRQ_ENABLE is only
enabled on C1; however, given that SPR target/exit latency is 1/1
for c1 and 2/4 for C1E, respectively, which is slower than C1
for SKX, it seems prudent to now enable it on both states.

This is also important as on SPR it is possible for only C1 or
only C1E to be enabled (i.e. one of them would be disabled), so
only enabling C1 would short change C1E-only configurations.

Fixes: 9edf3c0ffef0 ("intel_idle: add SPR support")
Signed-off-by: Jon Kohler <jon@nutanix.com>
---
 drivers/idle/intel_idle.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/drivers/idle/intel_idle.c b/drivers/idle/intel_idle.c
index 424ef470223d..f51857cddf2b 100644
--- a/drivers/idle/intel_idle.c
+++ b/drivers/idle/intel_idle.c
@@ -893,7 +893,7 @@ static struct cpuidle_state spr_cstates[] __initdata = {
 	{
 		.name = "C1",
 		.desc = "MWAIT 0x00",
-		.flags = MWAIT2flg(0x00),
+		.flags = MWAIT2flg(0x00) | CPUIDLE_FLAG_IRQ_ENABLE,
 		.exit_latency = 1,
 		.target_residency = 1,
 		.enter = &intel_idle,
@@ -902,7 +902,8 @@ static struct cpuidle_state spr_cstates[] __initdata = {
 		.name = "C1E",
 		.desc = "MWAIT 0x01",
 		.flags = MWAIT2flg(0x01) | CPUIDLE_FLAG_ALWAYS_ENABLE |
-					   CPUIDLE_FLAG_UNUSABLE,
+					   CPUIDLE_FLAG_UNUSABLE  |
+					   CPUIDLE_FLAG_IRQ_ENABLE,
 		.exit_latency = 2,
 		.target_residency = 4,
 		.enter = &intel_idle,
-- 
2.30.1 (Apple Git-130)