On 2025-06-19 07:37, Juergen Gross wrote:
Suspend for PVH is doing some crazy stuff with the Xenstore and console
PFNs:
- In arch_pre_suspend() the PFNs are "translated" to MFNs, even if that
translation is a nop.
- The result is stored via start_info_ptr, which points to a struct
start_info only in case of PV mode.
- After resuming or a cancelled suspend, the PFNs are retrieved again
and "translated".
- The "MFNs" are stored again via start_info_ptr, which is still not
pointing at a struct start_info, and from where the data is never
read again after that.
Drop all of that completely.
Signed-off-by: Juergen Gross <jgr...@suse.com>
start_info_ptr points at an hvm_start_info. Assuming I counted
properly, accessing ->console/store is writing past the end of the struct.
Reviewed-by: Jason Andryuk <jason.andr...@amd.com>