Commit 4403f806 authored by Jan Beulich's avatar Jan Beulich Committed by Juergen Gross
Browse files

xen/x86: drop redundant zeroing from cpu_initialize_context() 



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: default avatarJan Beulich <jbeulich@suse.com>
Reviewed-by: default avatarBoris Ostrovsky <boris.ostrvsky@oracle.com>

Link: https://lore.kernel.org/r/14881835-a48e-29fa-0870-e177b10fcf65@suse.com


Signed-off-by: default avatarJuergen Gross <jgross@suse.com>
parent d859ed25

arch/x86/xen/smp_pv.c

+0 −4
Original line number Diff line number Diff line
@@ -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);