Re: [Xen-devel] [PATCH] xen/x86: Zero out .bss for PV guests

2016-02-24 Thread Boris Ostrovsky
On 02/24/2016 11:22 AM, Luis R. Rodriguez wrote: On Wed, Feb 24, 2016 at 6:58 AM, David Vrabel wrote: Yes. Can you respin with a commit message explaining? (Or just provide the message here and I'll fix it up). Is there no way to re-use somehow the clear_bss() from bare metal? xen_start_in

Re: [Xen-devel] [PATCH] xen/x86: Zero out .bss for PV guests

2016-02-24 Thread Luis R. Rodriguez
On Wed, Feb 24, 2016 at 6:58 AM, David Vrabel wrote: > Yes. Can you respin with a commit message explaining? (Or just provide > the message here and I'll fix it up). Is there no way to re-use somehow the clear_bss() from bare metal? This uses a section range: /* Don't add a printk in there. pr

Re: [Xen-devel] [PATCH] xen/x86: Zero out .bss for PV guests

2016-02-24 Thread David Vrabel
On 24/02/16 14:52, Boris Ostrovsky wrote: > On 02/24/2016 09:15 AM, Andrew Cooper wrote: >> On 24/02/16 14:12, David Vrabel wrote: >>> On 22/02/16 22:06, Boris Ostrovsky wrote: Baremetal kernels clear .bss early in the boot. Since Xen PV guests don't excecute that early code they sho

Re: [Xen-devel] [PATCH] xen/x86: Zero out .bss for PV guests

2016-02-24 Thread Boris Ostrovsky
On 02/24/2016 09:15 AM, Andrew Cooper wrote: On 24/02/16 14:12, David Vrabel wrote: On 22/02/16 22:06, Boris Ostrovsky wrote: Baremetal kernels clear .bss early in the boot. Since Xen PV guests don't excecute that early code they should do it too. (Since we introduce macros for specifying 32-

Re: [Xen-devel] [PATCH] xen/x86: Zero out .bss for PV guests

2016-02-24 Thread Andrew Cooper
On 24/02/16 14:12, David Vrabel wrote: > On 22/02/16 22:06, Boris Ostrovsky wrote: >> Baremetal kernels clear .bss early in the boot. Since Xen PV guests don't >> excecute that early code they should do it too. >> >> (Since we introduce macros for specifying 32- and 64-bit registers we >> can get r

Re: [Xen-devel] [PATCH] xen/x86: Zero out .bss for PV guests

2016-02-24 Thread David Vrabel
On 22/02/16 22:06, Boris Ostrovsky wrote: > Baremetal kernels clear .bss early in the boot. Since Xen PV guests don't > excecute that early code they should do it too. > > (Since we introduce macros for specifying 32- and 64-bit registers we > can get rid of ifdefs in startup_xen()) .bss must hav

[Xen-devel] [PATCH] xen/x86: Zero out .bss for PV guests

2016-02-22 Thread Boris Ostrovsky
Baremetal kernels clear .bss early in the boot. Since Xen PV guests don't excecute that early code they should do it too. (Since we introduce macros for specifying 32- and 64-bit registers we can get rid of ifdefs in startup_xen()) Signed-off-by: Boris Ostrovsky --- arch/x86/xen/xen-head.S | 29