From: Zhu Qiyu qiyuzhu2@amd.com
[ Upstream commit 3db5648c4d608b5483470efc1da9780b081242dd ]
Commit 088984c8d54c ("ACPI: PRM: Find EFI_MEMORY_RUNTIME block for PRM handler and context") introduced non-essential printing "Failed to find VA for GUID: xxxx, PA: 0x0" which may confuse users to think that something wrong is going on while it is not the case.
According to the PRM Spec Section 4.1.2 [1], both static data buffer address and ACPI parameter buffer address may be NULL if they are not needed, so there is no need to print out the "Failed to find VA ... " in those cases.
Link: https://uefi.org/sites/default/files/resources/Platform%20Runtime%20Mechanis... # [1] Signed-off-by: Zhu Qiyu qiyuzhu2@amd.com Link: https://patch.msgid.link/20250704014104.82524-1-qiyuzhu2@amd.com [ rjw: Edits in new comments, subject and changelog ] Signed-off-by: Rafael J. Wysocki rafael.j.wysocki@intel.com Signed-off-by: Sasha Levin sashal@kernel.org ---
LLM Generated explanations, may be completely bogus:
**YES**
This commit should be backported to stable kernel trees for the following reasons:
## Bug Fix for User-Visible Issue The commit fixes false warning messages that were confusing users. The original commit 088984c8d54c (which was already backported to stable with "Cc: All applicable stable@vger.kernel.org") introduced an unconditional warning in `efi_pa_va_lookup()`:
```c pr_warn("Failed to find VA for GUID: %pUL, PA: 0x%llx", guid, pa); ```
This warning was printed even when the physical address (PA) was legitimately 0/NULL, which according to the PRM specification is valid for both `static_data_buffer_address` and `acpi_param_buffer_address` when they are not needed.
## Small and Contained Fix The fix is minimal and surgical: 1. Removes the unconditional warning from `efi_pa_va_lookup()` 2. Adds conditional warnings only when addresses are non-zero but lookup fails: - For handler_addr: warns if lookup fails (this should never be zero) - For static_data_buffer_addr: only warns if `handler_info->static_data_buffer_address` is non-zero but lookup fails - For acpi_param_buffer_addr: only warns if `handler_info->acpi_param_buffer_address` is non-zero but lookup fails
## Regression Fix Since the problematic commit 088984c8d54c has been backported to v6.12 stable kernels, this fix addresses a regression in stable trees where users are seeing confusing warning messages for legitimate NULL addresses.
## Low Risk The change: - Does not modify any functional behavior - Only adjusts when warning messages are printed - Has no architectural changes or side effects - Is confined to the ACPI PRM subsystem
## Clear Benefit The fix eliminates user confusion from false warnings while maintaining proper warnings for actual error conditions (when a non-zero address fails lookup).
Given that the original problematic commit is in stable kernels and this is a targeted fix for that regression with minimal risk, it clearly meets the stable backporting criteria.
drivers/acpi/prmt.c | 26 ++++++++++++++++++++++++-- 1 file changed, 24 insertions(+), 2 deletions(-)
diff --git a/drivers/acpi/prmt.c b/drivers/acpi/prmt.c index 7747ca4168ab..215ca8d60616 100644 --- a/drivers/acpi/prmt.c +++ b/drivers/acpi/prmt.c @@ -85,8 +85,6 @@ static u64 efi_pa_va_lookup(efi_guid_t *guid, u64 pa) } }
- pr_warn("Failed to find VA for GUID: %pUL, PA: 0x%llx", guid, pa); - return 0; }
@@ -154,13 +152,37 @@ acpi_parse_prmt(union acpi_subtable_headers *header, const unsigned long end) guid_copy(&th->guid, (guid_t *)handler_info->handler_guid); th->handler_addr = (void *)efi_pa_va_lookup(&th->guid, handler_info->handler_address); + /* + * Print a warning message if handler_addr is zero which is not expected to + * ever happen. + */ + if (unlikely(!th->handler_addr)) + pr_warn("Failed to find VA of handler for GUID: %pUL, PA: 0x%llx", + &th->guid, handler_info->handler_address);
th->static_data_buffer_addr = efi_pa_va_lookup(&th->guid, handler_info->static_data_buffer_address); + /* + * According to the PRM specification, static_data_buffer_address can be zero, + * so avoid printing a warning message in that case. Otherwise, if the + * return value of efi_pa_va_lookup() is zero, print the message. + */ + if (unlikely(!th->static_data_buffer_addr && handler_info->static_data_buffer_address)) + pr_warn("Failed to find VA of static data buffer for GUID: %pUL, PA: 0x%llx", + &th->guid, handler_info->static_data_buffer_address);
th->acpi_param_buffer_addr = efi_pa_va_lookup(&th->guid, handler_info->acpi_param_buffer_address);
+ /* + * According to the PRM specification, acpi_param_buffer_address can be zero, + * so avoid printing a warning message in that case. Otherwise, if the + * return value of efi_pa_va_lookup() is zero, print the message. + */ + if (unlikely(!th->acpi_param_buffer_addr && handler_info->acpi_param_buffer_address)) + pr_warn("Failed to find VA of acpi param buffer for GUID: %pUL, PA: 0x%llx", + &th->guid, handler_info->acpi_param_buffer_address); + } while (++cur_handler < tm->handler_count && (handler_info = get_next_handler(handler_info)));
return 0;