Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:2 as permitted sender) client-ip=2620:137:e000::3:2;
Message-ID: <c46b6bbf-d308-4ad3-9e70-dda6894a40f0@oracle.com>
Date:   Tue, 28 Nov 2023 16:09:03 +0200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 1/7] x86: Move ARCH_HAS_CPU_RELAX to arch
Content-Language: ro
To:     Petr Mladek <pmladek@suse.com>
Cc:     linux-arm-kernel@lists.infradead.org, kvm@vger.kernel.org,
        linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org,
        catalin.marinas@arm.com, will@kernel.org, tglx@linutronix.de,
        mingo@redhat.com, bp@alien8.de, x86@kernel.org, hpa@zytor.com,
        pbonzini@redhat.com, wanpengli@tencent.com, vkuznets@redhat.com,
        rafael@kernel.org, daniel.lezcano@linaro.org,
        akpm@linux-foundation.org, peterz@infradead.org,
        dianders@chromium.org, npiggin@gmail.com,
        rick.p.edgecombe@intel.com, joao.m.martins@oracle.com,
        juerg.haefliger@canonical.com, mic@digikod.net, arnd@arndb.de,
        ankur.a.arora@oracle.com
References: <1700488898-12431-1-git-send-email-mihai.carabas@oracle.com>
 <1700488898-12431-2-git-send-email-mihai.carabas@oracle.com>
 <ZWSrMzHEbdynTA8A@alley>
From:   Mihai Carabas <mihai.carabas@oracle.com>
In-Reply-To: <ZWSrMzHEbdynTA8A@alley>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?dlVyVENyOGZyUlp3Nzhva2pORWNNaTVrOWJJeStxZ2Q3S3RyNWFQaWlCaDB4?=
 =?utf-8?B?RkVVRmJGZm5yYlVBMXhZdy9QNWFMWjQ2eWxOazRZTG0rWGtuOWdrYytLKzln?=
 =?utf-8?B?U3Nnc1VHdnEyeTFvekNrTDhwUmZjOVBnc0t2NmhiSWI3WFlhdkJWaTgxRHBj?=
 =?utf-8?B?SlV1OGhKT2x5SnRtZXNPbUdJQXVobkYyT050QVhXcVpyR1RpRmhXbXV4M1dW?=
 =?utf-8?B?T1VwcloxWGlFQ0RRVlI2K0VqNjhKTitJT1lwNnVpV2ZMbm0yemlLUnBYRGFF?=
 =?utf-8?B?SVZ0MGpqYXNjODA2K2Q0bjZ6NU44dnN4WXdJcE9WQXBEaFBTc002eEhHUEVs?=
 =?utf-8?B?dzlMZE5IYWk5WGY2NEhEMnBzU0hUWDdFaXZlZXBqd3BSREx2TnlxMFg0aG1z?=
 =?utf-8?B?dGJBZVhsUjFpbkdabk1xdXVtOEhMc2R2VGFKUkhtNXdKR0ZDQWFzd05GQTJY?=
 =?utf-8?B?RWJndzl5d2xKSmZWVkpKY3pnRm5aS0JoK3JmN1FtRVA0YzhMcEUxTHNVNjc1?=
 =?utf-8?B?cHdpVFBLcUZ2aHlJSnh2T2prTlAydEM3SEIrT2tFV2xUZEZGcU8zdzJ3dGpP?=
 =?utf-8?B?c1lVcU5ubElmZVJhSjZxdTNTVjlLMlhEWmNZNytqaWk4MHlQZG84UTdaaTha?=
 =?utf-8?B?Ni8zTVRUanBEd2JxL3dmUytLOSsxNHpiM1FJakZja1ZxbHNPdE1SQ2Nlb01q?=
 =?utf-8?B?QVRjUnVESmdiUG5OV0QxRDdiM3p6QTFRcXhsQkZRakJReERXUHdwMEdRMVBL?=
 =?utf-8?B?ZVlmTUJ1YUNZRStGdnpNWFNtUzM2ckUwVmlLc09yNjBHUHZqdFVvckcxUTRn?=
 =?utf-8?B?eHFxYzhpNGlWdEM3MW5YczNCQW9PTTU3dUIrVXBDenh6NWRsREttMyt3clF0?=
 =?utf-8?B?dlMxR2hUR3NZMkp1YVk4RHg3Q3JWS1l2QVByc2puTjF1VmJhaU5VTS9JVUg1?=
 =?utf-8?B?UkY4SWpBc2RzUmxEOHVXQXFiaFhGTWxZczU3eVpubHBCL2VmMEhGczhodmpI?=
 =?utf-8?B?K3ppN25rbWRNVGFPNU5YNk1yVkdkUGNqTCtYc2FNY1p2M1paMmcrT3JqNk83?=
 =?utf-8?B?bHRHWVZqNVMzZ29pRG1UNGlxNmsvangrWDFqUHNRZkFtM0pZYzZTYUdkbFNo?=
 =?utf-8?B?WHZPazBLSDhTby9HOXVabnMrUmQwMm50emZIV25Yb0Y3L3BRb0FYOWxSSlVK?=
 =?utf-8?B?SENaK1lJam85ZEZEZ1djK2hMWGhneTUzSllqdDBJeUo3OTVVNHhpS1lhK0k4?=
 =?utf-8?B?VWxJZUxXakR4WEk4dEEwOE43WEsxL1VKWncvRnVlVXFxNlNuMUtxTk8vWnpO?=
 =?utf-8?B?cElINUJ2YnEwazYwR0NLTWJDM3ZPbnNTTFZoWjhKOHhaejZ5VGptSWVmNTAy?=
 =?utf-8?B?VzR3amxxdWhkNW5oaFQ1LzBBQTdIQ3hLT2xIeld1aHJFYmx3Qmdad3R5WWVy?=
 =?utf-8?B?dTlsdytWeVNkYVZOdmZxMC8raGVXaWdqMi9pcEN3bndna1MzeEZ5NUt3Ly9p?=
 =?utf-8?B?SnRUZURkcWF3cVRDNUoyWHFJYUxYVGFXOWx1YTkrcUlTUFNjbUNsem5VVXg4?=
 =?utf-8?B?M29pOTBtYXRiOGRKalE5K0VEa1pIejd3dEpIVXd5QXNpZlRJVHNVM2FNbzgy?=
 =?utf-8?B?MVAxM3plN3hSMTU0djYwQkxVV08rRnRiU0ZlNkx3SlJSd095aDlLTVBzQmpu?=
 =?utf-8?B?UjlXL1ZEWG9JaWd0bDBNMGNXWmcrTlJlM1hhcmdhUnY3RlVnNnNPVnlUY3Rn?=
 =?utf-8?B?eHhyWk1qNzQwMGMxQ0RRNDF0aWVZRnVOVFpJc045MDgzSk9PaWttUzNnS1lt?=
 =?utf-8?B?bjhoY0hVdjZlWGJKZnhibmtkbjVmR3oxMU12V0xQTkUzaUMrQ1NMMTdoVk0z?=
 =?utf-8?B?eTA3SldxVTNQbEh6MXVWUXlIYlp5Q2lTOUgyWnlpRmhFOHVUSlUzaUVqM2FK?=
 =?utf-8?B?eWJ4V3d0dXhpOTFMaU9BbU10L2RZY0lSS2Zod3FLZDlZaTNuSnE0Z3dWQTlp?=
 =?utf-8?B?NlhoUHpQd2wvaDlnR1UzZEdZN1dRa1kwY3JlZTkwSFFxc091emk5VXJXZkg4?=
 =?utf-8?B?bWRBNjFIM0ZGNkpkYmdjTEx2bFVWVUpjVThzZ25aOWpOU3IrS0I5dDNhcWV2?=
 =?utf-8?B?TmhZSnRsS3BaNHFQdzdQejVmWkVUcFBMdFJzRGJKZ2hQSWlXR2dRaDJ3M1pv?=
 =?utf-8?B?RGc9PQ==?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?utf-8?B?aGpSVUpmcHVLWjYvMi9HQjJrZTVjN2ZrcjVjazhmamYxd3ZVS2F4eDhXaHlr?=
 =?utf-8?B?MmNGcThnam5ZTXZFTUpvN2pJRWhsa2hndGoyNzdCV2ZoSFdDc3dHWGdoaEUx?=
 =?utf-8?B?a0xKQ2xFU2ZLTmFPbFhzV1YzN2JyaTVPRE4xelFrelgxU0dBU1lWT2hWMzRB?=
 =?utf-8?B?TE1QckUvUGRuWmpESEszRFl6YTluTCtpblFKSU5CODdaK09ITUdyajN4L3JB?=
 =?utf-8?B?L0t1eU1PYTQ2VUVrTGRDejFxU2VGamNDK2xPTzZhZFo1Mko0TEtmR0lEU1lq?=
 =?utf-8?B?ejBnUXFiQlJURWlSK3dXS2t2WXFWQVVqYVUxL0YrcExBZzFaQlU0SnFGNktP?=
 =?utf-8?B?MWNCS0JMekRpaWFlY1pVeW5rWjdBc3dmTGdRQmJpaDlpUkxQZUdCYUw5Y0ht?=
 =?utf-8?B?eTNjMlZPNTBLZlZ5dWtFb09kRm9IeVlYWXRVR01Nc0o2YzcrU3p5UVpDUnpR?=
 =?utf-8?B?NzQyUjU3VTVuejdVckw0UXB6R3FWWjNJV3Q1cy96OTRxYnN6VmNnLzg1UWJz?=
 =?utf-8?B?bEF5Umc3NmtWWWhJYjJuVFZSY0RLOXpuQVNqVzN5R0dDQUZJcEFoWFcyM0Ix?=
 =?utf-8?B?NE5nblVMMExPUEg1VkllV0tBQUlhRlo4MEdlU1FWMVBVZjFkWEc1akFiZ29v?=
 =?utf-8?B?UjZUMlQ2b0dnM1kwTG5wSEdNQlJHdFoydVFob2cxTGtKQUZPRStscmNFbS9V?=
 =?utf-8?B?bENYZWNFa0RSUUNqYklVZEpEYmNTYW42SEFkbWdxMW5mbUlicjQwZkttTTRr?=
 =?utf-8?B?UUhralFpN1pNenEvQUdCRUZyOHpRWFdmS25OYkt0UXdwaFRDMGRON0ZnOG5u?=
 =?utf-8?B?TkNMd0daQjlqa1VVT05Kdm9CL3ZvcE4wOUVNb215MnRNUnFtaVBLZm1qR1cr?=
 =?utf-8?B?bSt3RC9yZXFqZmVIcjJsZ21nS1RIaE5rbDRMb1lDaVozekFHNE9JYjlGbFls?=
 =?utf-8?B?bU5ZSWl6emlmUDdTZDlUM0hMMGplNzQ5UmplMzZhMkJzWHF3QXpHMjVKcWF4?=
 =?utf-8?B?dnhxQ1hpUHNaa25LYjBmT3U4Ykh5YWp2OWdNbDZJdDVrb1J2ZDlyS2liKzVK?=
 =?utf-8?B?K1p3NktYWW9XRWx4L3dGNkJRR2FGeS9uaW1FbW45Rjh5WFRQTXkxT0NRMzNB?=
 =?utf-8?B?YWF1NmZEKzZVcHFNR0RrblcxWXJjOWQzbmsrL0sxWDh2Q05lTWx6WGFJdmFi?=
 =?utf-8?B?eDEvRlJxL2xreEdORXJPV1kvanhzL2FjcHgxQzg4bVl6Q0tOaU5Jenp4OEJX?=
 =?utf-8?B?RkFJb2g4MFNwcHJucW1mL2NVTFYvSHpwRzh0UXdxWkd4dHpqQmY3bXdWaUFJ?=
 =?utf-8?B?OFhGTEVCMVRvVklxc2VQbG9sQXFJWWt5M0txc1BGVVF2Z0ovY1pVSnlXTGFt?=
 =?utf-8?B?dFZ5YkNVRnBSUFFUODRIRDJhWkptOWM3aVFBSHF6RHc3TnloZ2N4UTJ0Rkwr?=
 =?utf-8?B?YUdXVEw2VDdyVllXUnkzR3J1Y1dmMlg3UERDL0NyS0h1b1VEK2x1TkxxNktG?=
 =?utf-8?B?ck5pWXhBNXNLOURtNEM5MGFwTE9mYTY2dDljUGU4bS9rM1cxT1owNHlZR1Bw?=
 =?utf-8?B?bUs0UVA2Q0hBelVXYy9wNmYrT1FZQk9FcUFaVisvYWRYQ1ppVDRmUytJMENQ?=
 =?utf-8?B?TklhMHRXdzdlQ1FGVFRGMnBmYWsvUjRrL2ZTclUrYTVsZjdiaUxYdk9kSU1i?=
 =?utf-8?B?Qkx2aEdZNHpwN0VKOEZWdzhhc1VZM0hoYk0yWkNTZi9OeTRDc1BDSUpsRWVZ?=
 =?utf-8?B?QnJuZXNGNWFrOUt0T2NkQm5CVHlVcW5KaWxleHoybEZYUDc5UVBoNVN5d3BZ?=
 =?utf-8?B?MDZVUmpncjZCR3lWSHptQT09?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 142680db-99e1-4c50-eeb2-08dbf01b9881
X-MS-Exchange-CrossTenant-AuthSource: PH0PR10MB5894.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 Nov 2023 14:09:11.8833
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: DAHK9JATh48zpqgAqr8YzQVLBROo4xbnwHAaMjJn82+frHlCcR8LquOOU4OAu4R3+lgT5hpb2bfveGPBqCDQuP2t6odvRzsFTZAMEolJekU=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR10MB5672
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.997,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2023-11-28_14,2023-11-27_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 mlxscore=0 spamscore=0 mlxlogscore=999
 bulkscore=0 adultscore=0 phishscore=0 malwarescore=0 suspectscore=0
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2311060000
 definitions=main-2311280113
X-Proofpoint-ORIG-GUID: u947oXHHrF4nPgtav02XhhPMZOqr-dIY
X-Proofpoint-GUID: u947oXHHrF4nPgtav02XhhPMZOqr-dIY
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable
	autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Tue, 28 Nov 2023 06:12:06 -0800 (PST)

>> From: Joao Martins <joao.m.martins@oracle.com>
>>
>> ARM64 is going to use it for haltpoll support (for poll-state)
>> so move the definition to be arch-agnostic and allow architectures
>> to override it.
> This says that the definition is moved.
>
>> diff --git a/arch/Kconfig b/arch/Kconfig
>> index 4a85a10b12fd..92af0e9bc35e 100644
>> --- a/arch/Kconfig
>> +++ b/arch/Kconfig
>> @@ -1371,6 +1371,9 @@ config RELR
>>   config ARCH_HAS_MEM_ENCRYPT
>>   	bool
>>   
>> +config ARCH_HAS_CPU_RELAX
>> +	bool
>> +
>>   config ARCH_HAS_CC_PLATFORM
>>   	bool
>>   
>> diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
>> index d1c362f479d9..0c77670d020e 100644
>> --- a/arch/x86/Kconfig
>> +++ b/arch/x86/Kconfig
>> @@ -73,6 +73,7 @@ config X86
>>   	select ARCH_HAS_CACHE_LINE_SIZE
>>   	select ARCH_HAS_CPU_CACHE_INVALIDATE_MEMREGION
>>   	select ARCH_HAS_CPU_FINALIZE_INIT
>> +	select ARCH_HAS_CPU_RELAX
>>   	select ARCH_HAS_CURRENT_STACK_POINTER
>>   	select ARCH_HAS_DEBUG_VIRTUAL
>>   	select ARCH_HAS_DEBUG_VM_PGTABLE	if !X86_PAE
> But the definion is only added here.
>
> I would expect that the patch also removes the original definion.

Thanks for catching this. I updated the patch:

diff --git a/arch/Kconfig b/arch/Kconfig
index 12d51495caec..626ddd9ba7e0 100644
--- a/arch/Kconfig
+++ b/arch/Kconfig
@@ -1371,6 +1371,9 @@ config RELR
  config ARCH_HAS_MEM_ENCRYPT
         bool

+config ARCH_HAS_CPU_RELAX
+       bool
+
  config ARCH_HAS_CC_PLATFORM
         bool

diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
index 66bfabae8814..aaca90ba791a 100644
--- a/arch/x86/Kconfig
+++ b/arch/x86/Kconfig
@@ -72,6 +72,7 @@ config X86
         select ARCH_HAS_CACHE_LINE_SIZE
         select ARCH_HAS_CPU_CACHE_INVALIDATE_MEMREGION
         select ARCH_HAS_CPU_FINALIZE_INIT
+       select ARCH_HAS_CPU_RELAX
         select ARCH_HAS_CURRENT_STACK_POINTER
         select ARCH_HAS_DEBUG_VIRTUAL
         select ARCH_HAS_DEBUG_VM_PGTABLE        if !X86_PAE
@@ -363,9 +364,6 @@ config ARCH_MAY_HAVE_PC_FDC
  config GENERIC_CALIBRATE_DELAY
         def_bool y

-config ARCH_HAS_CPU_RELAX
-       def_bool y
-
  config ARCH_HIBERNATION_POSSIBLE
         def_bool y