Re: [Qemu-devel] [PATCH] target-i386: allow any alignment for SMBASE

2015-10-23 Thread Laszlo Ersek
On 10/23/15 20:19, Eduardo Habkost wrote: > On Fri, Oct 23, 2015 at 04:40:01PM +0200, Laszlo Ersek wrote: >> Eduardo, >> >> On 10/13/15 21:45, Eduardo Habkost wrote: >>> On Tue, Oct 13, 2015 at 09:42:50PM +0200, Paolo Bonzini wrote: On 13/10/2015 21:30, Eduardo Habkost wrote: > Y

Re: [Qemu-devel] [PATCH] target-i386: allow any alignment for SMBASE

2015-10-23 Thread Eduardo Habkost
On Fri, Oct 23, 2015 at 04:40:01PM +0200, Laszlo Ersek wrote: > Eduardo, > > On 10/13/15 21:45, Eduardo Habkost wrote: > > On Tue, Oct 13, 2015 at 09:42:50PM +0200, Paolo Bonzini wrote: > >> > >> > >> On 13/10/2015 21:30, Eduardo Habkost wrote: > >>> Yeah, the shutdown behavior was never implement

Re: [Qemu-devel] [PATCH] target-i386: allow any alignment for SMBASE

2015-10-23 Thread Laszlo Ersek
Eduardo, On 10/13/15 21:45, Eduardo Habkost wrote: > On Tue, Oct 13, 2015 at 09:42:50PM +0200, Paolo Bonzini wrote: >> >> >> On 13/10/2015 21:30, Eduardo Habkost wrote: >>> Yeah, the shutdown behavior was never implemented before, so now we are >>> just deviating from the documented Pentium & 486

Re: [Qemu-devel] [PATCH] target-i386: allow any alignment for SMBASE

2015-10-13 Thread Eduardo Habkost
On Tue, Oct 13, 2015 at 09:42:50PM +0200, Paolo Bonzini wrote: > > > On 13/10/2015 21:30, Eduardo Habkost wrote: > > Yeah, the shutdown behavior was never implemented before, so now we are > > just deviating from the documented Pentium & 486 behavior in a different > > (and less surprising?) way.

Re: [Qemu-devel] [PATCH] target-i386: allow any alignment for SMBASE

2015-10-13 Thread Paolo Bonzini
On 13/10/2015 21:30, Eduardo Habkost wrote: > Yeah, the shutdown behavior was never implemented before, so now we are > just deviating from the documented Pentium & 486 behavior in a different > (and less surprising?) way. > >> > >> > With the typo fix in the Cc tag: >> > >> > Reviewed-by: Las

Re: [Qemu-devel] [PATCH] target-i386: allow any alignment for SMBASE

2015-10-13 Thread Eduardo Habkost
On Mon, Oct 12, 2015 at 07:32:17PM +0200, Laszlo Ersek wrote: > On 10/12/15 18:25, Paolo Bonzini wrote: > > Processors up to the Pentium (says Bochs---I do not have old enough > > manuals) require a 32KiB alignment for the SMBASE, but newer processors > > do not need that, and Tiano Core will use n

Re: [Qemu-devel] [PATCH] target-i386: allow any alignment for SMBASE

2015-10-12 Thread Laszlo Ersek
On 10/12/15 18:25, Paolo Bonzini wrote: > Processors up to the Pentium (says Bochs---I do not have old enough > manuals) require a 32KiB alignment for the SMBASE, but newer processors > do not need that, and Tiano Core will use non-aligned SMBASE values. The Quark package has the following comment

[Qemu-devel] [PATCH] target-i386: allow any alignment for SMBASE

2015-10-12 Thread Paolo Bonzini
Processors up to the Pentium (says Bochs---I do not have old enough manuals) require a 32KiB alignment for the SMBASE, but newer processors do not need that, and Tiano Core will use non-aligned SMBASE values. Reported-by: Michael D Kinney Cc: Laszlo Ersek Cc: Jordan Justen Cc: Eduardo Habkost