[Qemu-devel] Re: AHCI in SeaBIOS
On Tue, Feb 08, 2011 at 12:57:41AM +0100, Alexander Graf wrote: > Hi Kevin, > > Do you remember why you put AHCI in with default=n? I'd like to see > it enabled in Qemu 0.14 and IIUC we use the default configuration > for that. Hi Alex, Sorry - I've gotten behind on emails. The reason for not enabling it by default was for two reasons - it had not been tested on real hardware, and it was unclear if doing the jump into 32bit mode would have adverse impact. > Speaking of which - it would be awesome of we had a companion > SeaBIOS release with Qemu 0.14, so we don't have to fetch a random > git snapshot but potentially even could maintain a stable SeaBIOS > for during the lifetime of 0.14 :). It's about time to make a new release of seabios, so I'll see if that can be done in a week or so. -Kevin
[Qemu-devel] Re: AHCI in SeaBIOS
Kevin O'Connor wrote: > On Tue, Feb 08, 2011 at 12:57:41AM +0100, Alexander Graf wrote: > >> Hi Kevin, >> >> Do you remember why you put AHCI in with default=n? I'd like to see >> it enabled in Qemu 0.14 and IIUC we use the default configuration >> for that. >> > > Hi Alex, > > Sorry - I've gotten behind on emails. > > The reason for not enabling it by default was for two reasons - it had > not been tested on real hardware, and it was unclear if doing the jump > into 32bit mode would have adverse impact. > Do you have to do the jump even when AHCI is unused? If no, it shouldn't hurt, right? Anthony, can we enable it only for the Qemu build? >> Speaking of which - it would be awesome of we had a companion >> SeaBIOS release with Qemu 0.14, so we don't have to fetch a random >> git snapshot but potentially even could maintain a stable SeaBIOS >> for during the lifetime of 0.14 :). >> > > It's about time to make a new release of seabios, so I'll see if that > can be done in a week or so. > Very nice, thank you! It would be great if we could sync that up with the 0.14 release. Anthony? Alex
[Qemu-devel] Re: AHCI in SeaBIOS
Kevin O'Connor wrote: > On Thu, Feb 10, 2011 at 04:25:11PM +0100, Alexander Graf wrote: > >> Kevin O'Connor wrote: >> >>> On Tue, Feb 08, 2011 at 12:57:41AM +0100, Alexander Graf wrote: >>> Do you remember why you put AHCI in with default=n? I'd like to see it enabled in Qemu 0.14 and IIUC we use the default configuration for that. >>> The reason for not enabling it by default was for two reasons - it had >>> not been tested on real hardware, and it was unclear if doing the jump >>> into 32bit mode would have adverse impact. >>> >> Do you have to do the jump even when AHCI is unused? If no, it shouldn't >> hurt, right? >> > > The 32bit jump is only done if an AHCI drive is found and one tries to > read/write to it. > Very good, so it really shouldn't hurt. We could add another option that only kicks off the detection when Qemu is found, no? That way we're sure to not break real hardware, but have the functionality in Qemu :) Alex
[Qemu-devel] Re: AHCI in SeaBIOS
On 02/10/2011 04:37 PM, Anthony Liguori wrote: We're right around the corner from -rc2 so let's wait until after 0.14 is released so we have a full release cycle to test it. And have unusable or only partly usable features (already mentioned multiple times: boot order, AHCI) in 0.14? Paolo
[Qemu-devel] Re: AHCI in SeaBIOS
On Thu, Feb 10, 2011 at 04:25:11PM +0100, Alexander Graf wrote: > Kevin O'Connor wrote: > > On Tue, Feb 08, 2011 at 12:57:41AM +0100, Alexander Graf wrote: > >> Do you remember why you put AHCI in with default=n? I'd like to see > >> it enabled in Qemu 0.14 and IIUC we use the default configuration > >> for that. > > The reason for not enabling it by default was for two reasons - it had > > not been tested on real hardware, and it was unclear if doing the jump > > into 32bit mode would have adverse impact. > > Do you have to do the jump even when AHCI is unused? If no, it shouldn't > hurt, right? The 32bit jump is only done if an AHCI drive is found and one tries to read/write to it. -Kevin
Re: [Qemu-devel] Re: AHCI in SeaBIOS
On 02/10/2011 04:25 PM, Alexander Graf wrote: Kevin O'Connor wrote: On Tue, Feb 08, 2011 at 12:57:41AM +0100, Alexander Graf wrote: Hi Kevin, Do you remember why you put AHCI in with default=n? I'd like to see it enabled in Qemu 0.14 and IIUC we use the default configuration for that. Hi Alex, Sorry - I've gotten behind on emails. The reason for not enabling it by default was for two reasons - it had not been tested on real hardware, and it was unclear if doing the jump into 32bit mode would have adverse impact. Do you have to do the jump even when AHCI is unused? If no, it shouldn't hurt, right? Anthony, can we enable it only for the Qemu build? Speaking of which - it would be awesome of we had a companion SeaBIOS release with Qemu 0.14, so we don't have to fetch a random git snapshot but potentially even could maintain a stable SeaBIOS for during the lifetime of 0.14 :). It's about time to make a new release of seabios, so I'll see if that can be done in a week or so. Very nice, thank you! It would be great if we could sync that up with the 0.14 release. Anthony? We're right around the corner from -rc2 so let's wait until after 0.14 is released so we have a full release cycle to test it. Regards, Anthony Liguori Alex