| Message ID | fcc458c391c6c43cce9b393cf7eb197a5428273e.1610504682.git.tamas.lengyel@intel.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | x86/mem_sharing: fix wrong field name used in 2c5119d | expand |
On 13/01/2021 02:28, Tamas K Lengyel wrote: > The arch_domain struct has "msr", not "msrs". > > Reported-by: Andrew Cooper <andrew.cooper3@citrix.com> > Signed-off-by: Tamas K Lengyel <tamas.lengyel@intel.com> > --- > I've build tested originally with "msr" but typod "msrs" when creating the > patch that was sent in. Sorry for the noise. Acked-by: Andrew Cooper <andrew.cooper3@citrix.com> It was actually a Travis Randconfig run which happened to turn CONFIG_MEM_SHARING on. I'll add words to that effect, but probably not a URL as its about to cease to exist.
diff --git a/xen/arch/x86/mm/mem_sharing.c b/xen/arch/x86/mm/mem_sharing.c index a98a1709c2..c8a6d11b90 100644 --- a/xen/arch/x86/mm/mem_sharing.c +++ b/xen/arch/x86/mm/mem_sharing.c @@ -1781,7 +1781,7 @@ static int fork(struct domain *cd, struct domain *d) domain_pause(d); cd->max_pages = d->max_pages; *cd->arch.cpuid = *d->arch.cpuid; - *cd->arch.msrs = *d->arch.msrs; + *cd->arch.msr = *d->arch.msr; cd->parent = d; }
The arch_domain struct has "msr", not "msrs". Reported-by: Andrew Cooper <andrew.cooper3@citrix.com> Signed-off-by: Tamas K Lengyel <tamas.lengyel@intel.com> --- I've build tested originally with "msr" but typod "msrs" when creating the patch that was sent in. Sorry for the noise. --- xen/arch/x86/mm/mem_sharing.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)