[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC v2 1/3] virtio: add vhost-user-fs-ccw device
From: |
Cornelia Huck |
Subject: |
Re: [RFC v2 1/3] virtio: add vhost-user-fs-ccw device |
Date: |
Tue, 28 Jul 2020 13:33:18 +0200 |
On Tue, 28 Jul 2020 13:25:57 +0200
Halil Pasic <pasic@linux.ibm.com> wrote:
> On Tue, 28 Jul 2020 12:31:51 +0200
> Cornelia Huck <cohuck@redhat.com> wrote:
>
> > On Fri, 17 Jul 2020 11:29:27 +0200
> > Marc Hartmayer <mhartmay@linux.ibm.com> wrote:
> >
> > > From: Halil Pasic <pasic@linux.ibm.com>
> > >
> > > Wire up the CCW device for vhost-user-fs.
> > >
> > > Signed-off-by: Halil Pasic <pasic@linux.ibm.com>
> > > ---
> > > hw/s390x/Makefile.objs | 1 +
> > > hw/s390x/vhost-user-fs-ccw.c | 73 ++++++++++++++++++++++++++++++++++++
> > > 2 files changed, 74 insertions(+)
> > > create mode 100644 hw/s390x/vhost-user-fs-ccw.c
(...)
> > > +static void vhost_user_fs_ccw_instance_init(Object *obj)
> > > +{
> > > + VHostUserFSCcw *dev = VHOST_USER_FS_CCW(obj);
> > > +
> >
> > This needs
> >
> > ccw_dev->force_revision_1 = true;
> >
>
> I'm OK with that as well. Just out of curiosity, why do we need it? Is
> there a virtio-ccw revision 1 feature this device inherently needs?
Yes, the VERSION_1 feature :)
(All newer virtio devices are virtio-1 only, and the code has recently
started enforcing this.)
- [RFC v2 0/3] Enable virtio-fs on s390x, Marc Hartmayer, 2020/07/17
- [RFC v2 2/3] libvhost-user: handle endianness as mandated by the spec, Marc Hartmayer, 2020/07/17
- Re: [RFC v2 2/3] libvhost-user: handle endianness as mandated by the spec, Stefan Hajnoczi, 2020/07/21
- Re: [RFC v2 2/3] libvhost-user: handle endianness as mandated by the spec, Michael S. Tsirkin, 2020/07/21
- Re: [RFC v2 2/3] libvhost-user: handle endianness as mandated by the spec, Marc Hartmayer, 2020/07/28
- Re: [RFC v2 2/3] libvhost-user: handle endianness as mandated by the spec, Michael S. Tsirkin, 2020/07/29
- Re: [RFC v2 2/3] libvhost-user: handle endianness as mandated by the spec, Marc Hartmayer, 2020/07/29