[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC 1/1] virtio: move host_features
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH RFC 1/1] virtio: move host_features |
Date: |
Tue, 26 May 2015 09:45:27 +0200 |
On Tue, 26 May 2015 14:58:02 +0800
Shannon Zhao <address@hidden> wrote:
>
>
> On 2015/5/22 20:11, Cornelia Huck wrote:
> > Move host_features from the individual transport proxies into
> > the virtio device. Transports may continue to add feature bits
> > during device plugging.
> >
> > This should it make easier to offer different sets of host features
> > for virtio-1/transitional support.
> >
> > Signed-off-by: Cornelia Huck <address@hidden>
>
> Test virtio-mmio and virtio-pci. The host_features are same with/without
> this patch. There is a small comment on this below.
>
> Tested-by: Shannon Zhao <address@hidden>
>
> > ---
> > hw/s390x/s390-virtio-bus.c | 18 ++----------------
> > hw/s390x/virtio-ccw.c | 29 ++++++-----------------------
> > hw/s390x/virtio-ccw.h | 4 ----
> > hw/virtio/virtio-bus.c | 18 +++++-------------
> > hw/virtio/virtio-mmio.c | 22 +++-------------------
> > hw/virtio/virtio-pci.c | 17 ++++-------------
> > hw/virtio/virtio.c | 17 +++++++++--------
> > include/hw/virtio/virtio-bus.h | 1 -
> > include/hw/virtio/virtio.h | 1 +
> > 9 files changed, 30 insertions(+), 97 deletions(-)
> > diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
> > index 10123f3..1817a07 100644
> > --- a/hw/virtio/virtio-mmio.c
> > +++ b/hw/virtio/virtio-mmio.c
> > @@ -80,7 +80,6 @@ typedef struct {
> > SysBusDevice parent_obj;
> > MemoryRegion iomem;
> > qemu_irq irq;
> > - uint32_t host_features;
>
> Maybe we should delete host_features in PCI and s390 as well.
I thought I did, but probably missed them.
>
> > /* Guest accessible state needing migration and reset */
> > uint32_t host_features_sel;
> > uint32_t guest_features_sel;
Thanks for looking at this patch.