New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Detect FACS even for reduced hardware platforms #933
Conversation
Nit: |
The FACS is optional even on hardware reduced platforms, and may exist for the purpose of communicating the hardware_signature field to provoke a clean reboot instead of a resume from hibernation. Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
ffd58ed
to
44fc328
Compare
Hi David, I will need to check and confirm if this PR can possibly go against the ACPI (6.5) Specification or not since usually there is a good reason for prior design decisions made by previous (or even current) ACPICA maintainers! Also please confirm in the meantime that whatever you tried works fine on reduced hardware or if it still is under test, thanks a lot! |
Hi Saket, Yes, it's tested on Arm64 in conjunction with https://lore.kernel.org/lkml/20240312134148.727454-2-dwmw2@infradead.org/#t (and tianocore/edk2@ba96acd to make UEFI actually expose the FACS). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes total sense to me.
Thanks a lot for this contribution, LGTM now! |
ACPICA PR acpica/acpica#933 The FACS is optional even on hardware reduced platforms, and may exist for the purpose of communicating the hardware_signature field to provoke a clean reboot instead of a resume from hibernation. Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
ACPICA PR acpica/acpica#933 The FACS is optional even on hardware reduced platforms, and may exist for the purpose of communicating the hardware_signature field to provoke a clean reboot instead of a resume from hibernation. Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
ACPICA PR acpica/acpica#933 The FACS is optional even on hardware reduced platforms, and may exist for the purpose of communicating the hardware_signature field to provoke a clean reboot instead of a resume from hibernation. Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
ACPICA PR acpica/acpica#933 The FACS is optional even on hardware reduced platforms, and may exist for the purpose of communicating the hardware_signature field to provoke a clean reboot instead of a resume from hibernation. Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
ACPICA PR acpica/acpica#933 The FACS is optional even on hardware reduced platforms, and may exist for the purpose of communicating the hardware_signature field to provoke a clean reboot instead of a resume from hibernation. Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
ACPICA PR acpica/acpica#933 The FACS is optional even on hardware reduced platforms, and may exist for the purpose of communicating the hardware_signature field to provoke a clean reboot instead of a resume from hibernation. Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
ACPICA PR acpica/acpica#933 The FACS is optional even on hardware reduced platforms, and may exist for the purpose of communicating the hardware_signature field to provoke a clean reboot instead of a resume from hibernation. Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
The FACS is optional even on hardware reduced platforms, and may exist for the purpose of communicating the hardware_signature field to provke a clean reboot instead of a resume from hibernation.