Re: [Qemu-devel] [PATCH v2] multiboot: Fix bss segment support

2012-01-23 Thread Kevin Wolf
Am 20.12.2011 19:49, schrieb Göran Weinholt: > Alexander Graf writes: > >> Yes, this patch makes things work again :). Thanks a lot! >> >> The only thing I could nitpick on would be the coding style - checkpatch.pl >> complains :). Could you please resend with braces? >> Justin, Please also queu

Re: [Qemu-devel] [PATCH v2] multiboot: Fix bss segment support

2011-12-20 Thread Alexander Graf
On 20.12.2011, at 19:49, Göran Weinholt wrote: > Alexander Graf writes: > >> Yes, this patch makes things work again :). Thanks a lot! >> >> The only thing I could nitpick on would be the coding style - checkpatch.pl >> complains :). Could you please resend with braces? >> Justin, Please al

Re: [Qemu-devel] [PATCH v2] multiboot: Fix bss segment support

2011-12-20 Thread Göran Weinholt
Alexander Graf writes: > Yes, this patch makes things work again :). Thanks a lot! > > The only thing I could nitpick on would be the coding style - checkpatch.pl > complains :). Could you please resend with braces? > Justin, Please also queue this for 1.0-stable when it comes in its final form.

Re: [Qemu-devel] [PATCH v2] multiboot: Fix bss segment support

2011-12-20 Thread Alexander Graf
On 20.12.2011, at 12:53, Göran Weinholt wrote: > Alexander Graf writes: > >> On 19.12.2011, at 23:01, Anthony Liguori wrote: >> >>On 12/19/2011 11:35 AM, Alexander Graf wrote: >> >>On 24.07.2011, at 17:55, Göran Weinholt wrote: >> >>Multiboot images can specify a bss

Re: [Qemu-devel] [PATCH v2] multiboot: Fix bss segment support

2011-12-20 Thread Göran Weinholt
Alexander Graf writes: > On 19.12.2011, at 23:01, Anthony Liguori wrote: > > On 12/19/2011 11:35 AM, Alexander Graf wrote: > > On 24.07.2011, at 17:55, Göran Weinholt wrote: > > Multiboot images can specify a bss segment. The boot loader must > clear >

Re: [Qemu-devel] [PATCH v2] multiboot: Fix bss segment support

2011-12-19 Thread Alexander Graf
On 19.12.2011, at 23:01, Anthony Liguori wrote: > On 12/19/2011 11:35 AM, Alexander Graf wrote: >> >> On 24.07.2011, at 17:55, Göran Weinholt wrote: >> >>> Multiboot images can specify a bss segment. The boot loader must clear >>> the memory of the bss and ensure that no modules or structures a

Re: [Qemu-devel] [PATCH v2] multiboot: Fix bss segment support

2011-12-19 Thread Anthony Liguori
On 12/19/2011 11:35 AM, Alexander Graf wrote: On 24.07.2011, at 17:55, Göran Weinholt wrote: Multiboot images can specify a bss segment. The boot loader must clear the memory of the bss and ensure that no modules or structures are allocated inside it. Several fields are provided in the Multibo

Re: [Qemu-devel] [PATCH v2] multiboot: Fix bss segment support

2011-12-19 Thread Alexander Graf
On 24.07.2011, at 17:55, Göran Weinholt wrote: > Multiboot images can specify a bss segment. The boot loader must clear > the memory of the bss and ensure that no modules or structures are > allocated inside it. Several fields are provided in the Multiboot > header that were previously not used p

Re: [Qemu-devel] [PATCH v2] multiboot: Fix bss segment support

2011-07-29 Thread Anthony Liguori
On 07/24/2011 10:55 AM, Göran Weinholt wrote: Multiboot images can specify a bss segment. The boot loader must clear the memory of the bss and ensure that no modules or structures are allocated inside it. Several fields are provided in the Multiboot header that were previously not used properly.

Re: [Qemu-devel] [PATCH v2] multiboot: Fix bss segment support

2011-07-24 Thread malc
On Sun, 24 Jul 2011, G?ran Weinholt wrote: > Multiboot images can specify a bss segment. The boot loader must clear > the memory of the bss and ensure that no modules or structures are > allocated inside it. Several fields are provided in the Multiboot > header that were previously not used proper

[Qemu-devel] [PATCH v2] multiboot: Fix bss segment support

2011-07-24 Thread Göran Weinholt
Multiboot images can specify a bss segment. The boot loader must clear the memory of the bss and ensure that no modules or structures are allocated inside it. Several fields are provided in the Multiboot header that were previously not used properly. The header is now used to determine how much dat