Messages in this thread | | | From | Michael Kelley <> | Subject | RE: [PATCH v3] x86/hyper-v: Mark TLFS structures packed | Date | Mon, 3 Dec 2018 16:26:02 +0000 |
| |
From: Vitaly Kuznetsov <vkuznets@redhat.com> Sent: Monday, December 3, 2018 6:07 AM > > The TLFS structures are used for hypervisor-guest communication and must > exactly meet the specification. > > Compilers can add alignment padding to structures or reorder struct members > for randomization and optimization, which would break the hypervisor ABI. > > Mark the structures as packed to prevent this. > > Suggested-by: Nadav Amit <nadav.amit@gmail.com> > Signed-off-by: Vitaly Kuznetsov <vkuznets@redhat.com> > Acked-by: Thomas Gleixner <tglx@linutronix.de> > Acked-by: Nadav Amit <nadav.amit@gmail.com> > --- > - Changes since v2: > - add __packed to struct 'u' defined within union 'hv_port_id' > [Michael Kelley] > > - This is a follow-up to my "[PATCH v2 0/4] x86/kvm/hyper-v: Implement > Direct Mode for synthetic timers" series, as suggested by Thomas I'm > routing it to KVM tree to avoid merge conflicts. > ---
Reviewed-by: Michael Kelley <mikelley@microsoft.com>
| |