From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1MsdAv-0008Cy-Uv
	for qemu-devel@nongnu.org; Tue, 29 Sep 2009 09:55:29 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1MsdAr-0008Ca-6o
	for qemu-devel@nongnu.org; Tue, 29 Sep 2009 09:55:29 -0400
Received: from [199.232.76.173] (port=57199 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1MsdAr-0008CT-46
	for qemu-devel@nongnu.org; Tue, 29 Sep 2009 09:55:25 -0400
Received: from mail-qy0-f173.google.com ([209.85.221.173]:49078)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <anthony@codemonkey.ws>) id 1MsdAq-0004j0-Kx
	for qemu-devel@nongnu.org; Tue, 29 Sep 2009 09:55:24 -0400
Received: by qyk3 with SMTP id 3so3951952qyk.4
	for <qemu-devel@nongnu.org>; Tue, 29 Sep 2009 06:55:24 -0700 (PDT)
Message-ID: <4AC211C8.1060202@codemonkey.ws>
Date: Tue, 29 Sep 2009 08:55:20 -0500
From: Anthony Liguori <anthony@codemonkey.ws>
MIME-Version: 1.0
References: <4AB7A01A.3000206@redhat.com> <4AB8992B.7070709@redhat.com>
	<4AB8DD53.7070806@redhat.com> <4AB8DECA.3090908@redhat.com>
	<4AB8E88C.4040103@redhat.com> <4AB980E6.2070203@codemonkey.ws>
	<4AB9AA8E.7060800@third-harmonic.com> <4AC1A49A.1010308@redhat.com>
	<20090929065856.GC25389@redhat.com> <4AC1B5CD.7050702@redhat.com>
	<20090929085450.GE25389@redhat.com>
In-Reply-To: <20090929085450.GE25389@redhat.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Subject: [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config
	breakage
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.gnu.org/pipermail/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: "Michael S. Tsirkin" <mst@redhat.com>
Cc: john cooper <john.cooper@third-harmonic.com>, john cooper <john.cooper@redhat.com>, Rusty Russell <rusty@rustcorp.com.au>, qemu-devel@nongnu.org, Avi Kivity <avi@redhat.com>, jens.axboe@oracle.com, Vadim Rozenfeld <vrozenfe@redhat.com>

Michael S. Tsirkin wrote:
> On Tue, Sep 29, 2009 at 09:22:53AM +0200, Avi Kivity wrote:
>   
>> On 09/29/2009 08:58 AM, Michael S. Tsirkin wrote:
>>     
>>> Using bar per feature we'll quickly run out of BARs.
>>> We already use a BAR for MSI-X - let's add
>>> ATA identity there?
>>>    
>>>       
>> Mixing unrelated features will quickly cause confusion.
>>     
>
> Note if we ever switch to 64 bit BARs, we'll only have 3 of these
> available, so the ID would be using the last free one.
> We can just as well plan ahead for when we'll add another feature?
>
> I agree fixed offsets are messy though.  I previously proposed storing
> the offset to the identity data in an i/o register. This will let each
> implementation lay out this data in an optimal manner, without
> confusion.
>   

1) There's no need to make the identity page separate from the config 
space.  The real problem is the config space is too small.  We can fix 
that without making any changes to virtio-blk by putting the config 
space in a separate BAR.

2) Passing an ATA identity page is goofy.  We should just pass the 
serial number and let Linux generate the identity page.  Just because 
Linux requires this as it's user space interface, that doesn't mean that 
other guests will (like Windows).  Instead of exposing an opaque blob, 
we should expose the information we need in a structured way.

Regards,

Anthony Liguori