From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 6FE0B1DA53; Thu, 1 May 2025 12:30:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1746102638; cv=none; b=REC2bBwZHNlcHhYW91osgKpIP04eCZAUhBV0ayYdg2bNGAwpBG7wEY1MkJGVZvYYPOvj2agcat36Yp0NnUXW9rHPRlK3DwnPBHK6hcp0vA6kzEQcGxTlF2u5bc9rTE8c3YYOoasRkeFY3QcTjBId300prCwSUdPmBn9jvK8vXJU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1746102638; c=relaxed/simple; bh=KnQE95qVoJVNX8ZCH72+8W7tHrtRe/I3nmQ2e0u34lQ=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=cz0ZbfEtHjapFZ9/S03SUzQ8ggz+rNVNvpdNSSt47Gm3ktM+gkaMdAxVBDBgN/kbRWK0A3R5oDvGyF1tYinaxjpH/Y3Yakm16z7wN8eapwjOl0bGVmfOyGkmxK4JJhvrrxf2OgAZwhklTqp2GrE40i9Zp4sIvS4lBWPUP+6Wvfw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 3BC3A1D14; Thu, 1 May 2025 05:30:29 -0700 (PDT) Received: from [192.168.178.25] (usa-sjc-mx-foss1.foss.arm.com [172.31.20.19]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 2C10A3F673; Thu, 1 May 2025 05:30:34 -0700 (PDT) Message-ID: Date: Thu, 1 May 2025 14:30:32 +0200 Precedence: bulk X-Mailing-List: linux-pm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [RFT][PATCH v1 7/8] cpufreq: intel_pstate: Align perf domains with L2 cache To: "Rafael J. Wysocki" Cc: "Rafael J. Wysocki" , Linux PM , LKML , Lukasz Luba , Peter Zijlstra , Srinivas Pandruvada , Morten Rasmussen , Vincent Guittot , Ricardo Neri , Pierre Gondois , Christian Loehle , Tim Chen References: <3344336.aeNJFYEL58@rjwysocki.net> <1964444.taCxCBeP46@rjwysocki.net> From: Dietmar Eggemann Content-Language: en-US In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 30/04/2025 21:29, Rafael J. Wysocki wrote: > On Sun, Apr 27, 2025 at 6:23 PM Dietmar Eggemann > wrote: >> >> On 16/04/2025 20:10, Rafael J. Wysocki wrote: >>> From: Rafael J. Wysocki [...] >> There seems to be an issue with late CPU-hotplug-in and this alignment >> on L2 cache boundaries. [...] > I see. > > What happens is that cpu_cacheinfo hides information on offline CPUs, > so when CPU20 goes online, it doesn't see any other CPUs sharing the > L2 with it. Accordingly, a PD is created just for itself. > > When CPU21 goes online, it sees that CPU20 shares the L2 with it, so > the code attempts to create a PD for them both which fails. > > This could be addressed, but the code would need to be a bit more > complex and the current hardware seems to do better with a PD per CPU, > so I'll drop the $subject patch for now. Ah OK, thanks!