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>
