[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] libacpi: Remove CPU hotplug and GPE handling from PVH DSDTs
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Alejandro Vallejo <alejandro.garciavallejo@xxxxxxx>
- Date: Wed, 10 Sep 2025 17:16:54 +0200
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=suse.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)
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=wlCm9wSUY/e2X1TE/PRNyvakaoCfl+WJ+Dh7fT51WK8=; b=St7OphjDZeww2kd2+nc4+kn4nk+PZ9yI5IILBSTrn+RLBtAsSQyzMxnRHC67vcRHLDkQpn+PvYofar/7wTy5Pcb4xeq/64cmfib2AQhN7TQL+yjCJOrjtQXu0nxRRoyaqzH6Huux0Ts5uxF0GyewPePnsnOfJ8dBtxqqZOwuyZsU3dfnRgjxeDiqkgSDW9wTy+kMs8WLodEiMFkwupx/c4BdUA4CxUaG9Hr3gCYiRvXndEpOvPSXUd7IBQMQv32sbe/0gIq6HTGz07jaghn+RF6G60KRTW4bjWTzmMQ4yGlU5G22xdz52RBt2LlXiibl2c/LAcKL3yVgsHSU1oAlHg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=n93j8innBKuJuVUUwFqUcFCMNoCSuv+XN0q9R5+CjpNNF+wIa9GmDWmsayir2xcDaoLIKoiHrElq5JdRJ40KIvkP/26oTM9rBX4pHKI+1vDUESzCLEi9qFpliLY838tSLS71qMUUnNmN/FZlQu1cO9XJWZcSx9+NZZuAkPwGr3ks0kxYQz2RtHczN1L2UaMU/46MfpvxgeoQ3e6T50FFoWf0ec9d0gFOR4aPr8JNhA9XaScEEEyowWkzzWcrJCzRe3PABncEKz+ZsixdBrS7iDGjhJIUYqpz8UVXIgeHMLVbVwEWpqgwomDOBeKeG8E949WcofRequTo6+LrlKLhQQ==
- Cc: Anthony PERARD <anthony.perard@xxxxxxxxxx>, Grygorii Strashko <grygorii_strashko@xxxxxxxx>, Roger Pau Monne <roger.pau@xxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Wed, 10 Sep 2025 15:17:06 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On Wed Sep 10, 2025 at 5:02 PM CEST, Jan Beulich wrote:
> On 10.09.2025 16:49, Alejandro Vallejo wrote:
>> CPU hotplug relies on the guest having access to the legacy online CPU
>> bitmap that QEMU provides at PIO 0xAF00. But PVH guests have no DM, so
>> this causes the MADT to get corrupted due to spurious modifications of
>> the "online" flag in MADT entries and the table checksum during the
>> initial acpica passes.
>
> I don't understand this MADT corruption aspect, which - aiui - is why
> there's a Fixes: tag here. The code change itself looks plausible.
>
> Jan
When there's no DM to provide a real and honest online CPU bitmap on PIO 0xAF00
then we get all 1s (because there's no IOREQ server). Which confuses the GPE
handler.
Somehow, the GPE handler is being triggered. Whether this is due to a real SCI
or just it being spuriously executed as part of the initial acpica pass, I don't
know.
Both statements combined means the checksum and online flags in the MADT get
changed after initial parsing making it appear as-if all 128 CPUs were plugged.
This patch makes the checksums be correct after acpica init.
Grygorii noticed the checksum mismatch while validating an ACPI dump on a PVH
Linux system.
Cheers,
Alejandro
|
