diff mbox series

x86/mem_sharing: fix wrong field name used in 2c5119d

Message ID fcc458c391c6c43cce9b393cf7eb197a5428273e.1610504682.git.tamas.lengyel@intel.com (mailing list archive)
State New
Headers show
Series x86/mem_sharing: fix wrong field name used in 2c5119d | expand

Commit Message

Lengyel, Tamas Jan. 13, 2021, 2:28 a.m. UTC
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(-)

Comments

Andrew Cooper Jan. 13, 2021, 7:11 p.m. UTC | #1
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 mbox series

Patch

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;
     }