[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20141126154843.GC10568@redhat.com>
Date: Wed, 26 Nov 2014 17:48:43 +0200
From: "Michael S. Tsirkin" <mst@...hat.com>
To: David Hildenbrand <dahi@...ux.vnet.ibm.com>
Cc: Cornelia Huck <cornelia.huck@...ibm.com>,
linux-kernel@...r.kernel.org, David Miller <davem@...emloft.net>,
rusty@....ibm.com, nab@...ux-iscsi.org, pbonzini@...hat.com,
Thomas Huth <thuth@...ux.vnet.ibm.com>,
Rusty Russell <rusty@...tcorp.com.au>,
virtualization@...ts.linux-foundation.org,
linux-api@...r.kernel.org
Subject: Re: [PATCH v4 13/42] virtio_blk: v1.0 support
On Wed, Nov 26, 2014 at 04:40:45PM +0100, David Hildenbrand wrote:
> > On Tue, Nov 25, 2014 at 06:55:16PM +0100, Cornelia Huck wrote:
> > > On Tue, 25 Nov 2014 18:42:18 +0200
> > > "Michael S. Tsirkin" <mst@...hat.com> wrote:
> > >
> > > > Based on patch by Cornelia Huck.
> > > >
> > > > Note: for consistency, and to avoid sparse errors,
> > > > convert all fields, even those no longer in use
> > > > for virtio v1.0.
> > > >
> > > > Reviewed-by: Thomas Huth <thuth@...ux.vnet.ibm.com>
> > > > Reviewed-by: David Hildenbrand <dahi@...ux.vnet.ibm.com>
> > > > Signed-off-by: Cornelia Huck <cornelia.huck@...ibm.com>
> > > > Signed-off-by: Michael S. Tsirkin <mst@...hat.com>
> > > > ---
> > > > include/uapi/linux/virtio_blk.h | 15 ++++-----
> > > > drivers/block/virtio_blk.c | 70 ++++++++++++++++++++++++-----------------
> > > > 2 files changed, 49 insertions(+), 36 deletions(-)
> > > >
> > >
> > > Is this sufficiently unchanged so that the original r-bs still hold?
> >
> > You are right, I'm afraid I'll have to drop these.
> > Thomas, David, care to comment?
>
> I've got this on my list, will try to find some spare time tomorrow!
>
> > Even better - have the cycles to review the latest version?
>
> Yes, will try.
>
> >
Thanks!
If/when you do, pls make sure to pick the latest version.
You can always get it from vhost-net branch in my tree.
I will likely post another version tomorrow: Cornelia
found a bug in virtio-scsi.
--
MST
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists