From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from [140.186.70.92] (port=53553 helo=eggs.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1PnYXd-0002kb-U1
	for qemu-devel@nongnu.org; Thu, 10 Feb 2011 10:34:46 -0500
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <agraf@suse.de>) id 1PnYXW-0003Ms-5e
	for qemu-devel@nongnu.org; Thu, 10 Feb 2011 10:34:39 -0500
Received: from cantor2.suse.de ([195.135.220.15]:46372 helo=mx2.suse.de)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <agraf@suse.de>) id 1PnYXV-0003MO-Vi
	for qemu-devel@nongnu.org; Thu, 10 Feb 2011 10:34:38 -0500
Message-ID: <4D54058B.9000708@suse.de>
Date: Thu, 10 Feb 2011 16:34:35 +0100
From: Alexander Graf <agraf@suse.de>
MIME-Version: 1.0
References: <F91D9E37-8250-4111-8B3C-CB8E5B16514E@suse.de>
	<20110210151534.GA7881@morn.localdomain> <4D540357.70504@suse.de>
	<20110210153339.GA9074@morn.localdomain>
In-Reply-To: <20110210153339.GA9074@morn.localdomain>
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit
Subject: [Qemu-devel] Re: AHCI in SeaBIOS
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Kevin O'Connor <kevin@koconnor.net>
Cc: Gerd Hoffmann <kraxel@redhat.com>, qemu-devel List <qemu-devel@nongnu.org>

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