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