]> git.apps.os.sepia.ceph.com Git - ceph-client.git/commitdiff
xen/x86: drop redundant zeroing from cpu_initialize_context()
authorJan Beulich <jbeulich@suse.com>
Thu, 16 Sep 2021 15:05:29 +0000 (17:05 +0200)
committerJuergen Gross <jgross@suse.com>
Mon, 20 Sep 2021 14:55:54 +0000 (16:55 +0200)
Just after having obtained the pointer from kzalloc() there's no reason
at all to set part of the area to all zero yet another time. Similarly
there's no point explicitly clearing "ldt_ents".

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Boris Ostrovsky <boris.ostrvsky@oracle.com>
Link: https://lore.kernel.org/r/14881835-a48e-29fa-0870-e177b10fcf65@suse.com
Signed-off-by: Juergen Gross <jgross@suse.com>
arch/x86/xen/smp_pv.c

index 96afadf9878ef14ca00625445b362d4a4c07996a..7ed56c6075b0cc8d7e1fd97288fac2761983e2bc 100644 (file)
@@ -290,8 +290,6 @@ cpu_initialize_context(unsigned int cpu, struct task_struct *idle)
 
        gdt = get_cpu_gdt_rw(cpu);
 
-       memset(&ctxt->fpu_ctxt, 0, sizeof(ctxt->fpu_ctxt));
-
        /*
         * Bring up the CPU in cpu_bringup_and_idle() with the stack
         * pointing just below where pt_regs would be if it were a normal
@@ -308,8 +306,6 @@ cpu_initialize_context(unsigned int cpu, struct task_struct *idle)
 
        xen_copy_trap_info(ctxt->trap_ctxt);
 
-       ctxt->ldt_ents = 0;
-
        BUG_ON((unsigned long)gdt & ~PAGE_MASK);
 
        gdt_mfn = arbitrary_virt_to_mfn(gdt);