qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 3/6] virtio-pci: fix 1.0 virtqueue migration


From: Cornelia Huck
Subject: Re: [Qemu-devel] [PATCH 3/6] virtio-pci: fix 1.0 virtqueue migration
Date: Mon, 24 Aug 2015 16:14:06 +0200

On Mon, 24 Aug 2015 13:37:06 +0800
Jason Wang <address@hidden> wrote:

> On 08/21/2015 05:43 PM, Cornelia Huck wrote:
> > On Fri, 21 Aug 2015 17:05:47 +0800
> > Jason Wang <address@hidden> wrote:

> >> +        k->has_modern_state &&
> >> +        k->has_modern_state(qbus->parent);
> >> +}
> > I don't really like this "modern_state" stuff (which is pci specific)
> > creeping into core.
> >
> > How about introducing "extra_state" and/or "extra_queue_state" (or
> > something like that) instead?
> >
> 
> Ok, if you don't like pci specific name, maybe something like
> "virtio_1_state" is better?

I was thinking more along the lines of "transport wants to save/restore
additional state for the device" - which is not neccessarily depending
on virtio-1. It would be good if a transport can extend the state
without needlessly introducing incompatibilities.

pci can handle its modern state via this then and encapsulate it.




reply via email to

[Prev in Thread] Current Thread [Next in Thread]