[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] vhost-scsi: add missing virtqueue_size para
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] vhost-scsi: add missing virtqueue_size parameter |
Date: |
Tue, 5 Dec 2017 02:59:09 +0200 |
On Mon, Dec 04, 2017 at 03:00:45PM -0500, Eric Farman wrote:
>
>
> On 12/04/2017 01:05 PM, Peter Maydell wrote:
> > On 4 December 2017 at 17:52, Eric Farman <address@hidden> wrote:
> > > On 12/01/2017 10:16 AM, Paolo Bonzini wrote:
> > > >
> > > > On 01/12/2017 16:15, Eric Farman wrote:
> > > > > diff --git a/hw/scsi/vhost-scsi.c b/hw/scsi/vhost-scsi.c
> > > > > index cd4ab05233..9c1bea8ff3 100644
> > > > > --- a/hw/scsi/vhost-scsi.c
> > > > > +++ b/hw/scsi/vhost-scsi.c
> > > > > @@ -233,6 +233,8 @@ static Property vhost_scsi_properties[] = {
> > > > > DEFINE_PROP_STRING("wwpn", VirtIOSCSICommon, conf.wwpn),
> > > > > DEFINE_PROP_UINT32("boot_tpgt", VirtIOSCSICommon,
> > > > > conf.boot_tpgt,
> > > > > 0),
> > > > > DEFINE_PROP_UINT32("num_queues", VirtIOSCSICommon,
> > > > > conf.num_queues,
> > > > > 1),
> > > > > + DEFINE_PROP_UINT32("virtqueue_size", VirtIOSCSICommon,
> > > > > conf.virtqueue_size,
> > > > > + 128),
> > > > > DEFINE_PROP_UINT32("max_sectors", VirtIOSCSICommon,
> > > > > conf.max_sectors,
> > > > > 0xFFFF),
> > > > > DEFINE_PROP_UINT32("cmd_per_lun", VirtIOSCSICommon,
> > > > > conf.cmd_per_lun, 128),
> >
> > > Wow, super quick, thank you! Seeing conversations today, is this queued
> > > for
> > > -rc4, or 2.12 and backport to qemu-stable?
> >
> > If people can come to a consensus on whether this is -rc4 material
> > by Tuesday 12:00 GMT that would be helpful.
>
> I'd vote for -rc4, since it's a new guest crash since 2.10.
I think so too.
> But I also
> won't complain if it waits for 2.11.1 since I was out of the office during
> the whole freeze period and no one else noticed it.
>
> - Eric
- [Qemu-devel] [PATCH for 2.11 0/1] Fix guest boot with vhost-scsi, Eric Farman, 2017/12/01
- [Qemu-devel] [PATCH 1/1] vhost-scsi: add missing virtqueue_size parameter, Eric Farman, 2017/12/01
- Re: [Qemu-devel] [PATCH 1/1] vhost-scsi: add missing virtqueue_size parameter, Paolo Bonzini, 2017/12/01
- Re: [Qemu-devel] [PATCH 1/1] vhost-scsi: add missing virtqueue_size parameter, Eric Farman, 2017/12/04
- Re: [Qemu-devel] [PATCH 1/1] vhost-scsi: add missing virtqueue_size parameter, Peter Maydell, 2017/12/04
- Re: [Qemu-devel] [PATCH 1/1] vhost-scsi: add missing virtqueue_size parameter, Eric Farman, 2017/12/04
- Re: [Qemu-devel] [PATCH 1/1] vhost-scsi: add missing virtqueue_size parameter,
Michael S. Tsirkin <=
- Re: [Qemu-devel] [PATCH 1/1] vhost-scsi: add missing virtqueue_size parameter, Peter Maydell, 2017/12/05
- Re: [Qemu-devel] [PATCH 1/1] vhost-scsi: add missing virtqueue_size parameter, Eric Farman, 2017/12/05
Re: [Qemu-devel] [PATCH 1/1] vhost-scsi: add missing virtqueue_size parameter, Richard W.M. Jones, 2017/12/02