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>

Reply via email to