| Message ID | 20220720131204.591104-1-lulu@redhat.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | vhost-vdpa: add support for configure interrupt | expand |
Hi Micheal. Would you help review these patches, not sure if this is ok to merge? Thanks Cindy On Wed, Jul 20, 2022 at 9:12 PM Cindy Lu <lulu@redhat.com> wrote: > > Add support for virtio-mmio bus > active the notifier while the backend support configure interrupt > misc fixes from v1 > > Change in v3 > fix the coding style problems > > Change in v4 > misc fixes from v3 > merge the set_config_notifier to set_guest_notifier > when vdpa start, check the feature by VIRTIO_NET_F_STATUS > > Change in v5 > misc fixes from v4 > split the code to introduce configure interrupt type and the callback function > will init the configure interrupt in all virtio-pci and virtio-mmio bus, but will > only active while using vhost-vdpa driver > > Change in v6 > misc fixes from v5 > decouple vq from interrupt setting and misc process > fix the bug in virtio_net_handle_rx > > Change in v7 > misc fixes from v6 > decouple vq from interrupt setting and misc process > decouple vq from vector use/release process > decouple vq from set notifier fd handler process > move config_notifier and masked_config_notifier to VirtIODevice > fix the bug in virtio_net_handle_rx, add more information > add VIRTIO_CONFIG_IRQ_IDX as the queue number for configure interrupt > > Change in v8 > misc fixes from v7 > decouple vq from interrupt setting and misc process > decouple vq from vector use/release process > decouple vq from set notifier fd handler process > move the vhost configure interrupt to vhost_net > > Change in v9 > misc fixes from v8 > address the comments from v8 > > Change in v10 > fix the hang issue in qtest > address the comments from v9 > > Change in v11 > fix the crash in aarch64 plateform. > fix the crash upstream reported > > Changes in v12 > fix the typo and the comments > > changes in v13 > re-send the patches by git-publish > > Cindy Lu (10): > virtio: introduce macro VIRTIO_CONFIG_IRQ_IDX > virtio-pci: decouple notifier from interrupt process > virtio-pci: decouple the single vector from the interrupt process > vhost: introduce new VhostOps vhost_set_config_call > vhost-vdpa: add support for config interrupt > virtio: add support for configure interrupt > vhost: add support for configure interrupt > virtio-net: add support for configure interrupt > virtio-mmio: add support for configure interrupt > virtio-pci: add support for configure interrupt > > hw/display/vhost-user-gpu.c | 13 ++ > hw/net/vhost_net.c | 9 + > hw/net/virtio-net.c | 20 ++- > hw/virtio/trace-events | 1 + > hw/virtio/vhost-user-fs.c | 14 ++ > hw/virtio/vhost-vdpa.c | 8 + > hw/virtio/vhost-vsock-common.c | 14 ++ > hw/virtio/vhost.c | 78 +++++++- > hw/virtio/virtio-crypto.c | 14 ++ > hw/virtio/virtio-mmio.c | 27 +++ > hw/virtio/virtio-pci.c | 283 +++++++++++++++++++++--------- > hw/virtio/virtio-pci.h | 4 +- > hw/virtio/virtio.c | 29 +++ > include/hw/virtio/vhost-backend.h | 3 + > include/hw/virtio/vhost.h | 4 + > include/hw/virtio/virtio.h | 7 + > include/net/vhost_net.h | 2 + > 17 files changed, 443 insertions(+), 87 deletions(-) > > -- > 2.34.3 > >
On Fri, Jul 22, 2022 at 03:46:52PM +0800, Cindy Lu wrote: > Hi Micheal. > Would you help review these patches, not sure if this is ok to merge? > Thanks > Cindy Pls note threading is broken in the patchset. Care to repost?
On Fri, 22 Jul 2022 at 20:21, Michael S. Tsirkin <mst@redhat.com> wrote: > > On Fri, Jul 22, 2022 at 03:46:52PM +0800, Cindy Lu wrote: > > Hi Micheal. > > Would you help review these patches, not sure if this is ok to merge? > > Thanks > > Cindy > > Pls note threading is broken in the patchset. Care to repost? > Thanks Micheal, I have reposted a new version, would you help review it? Thanks > -- > MST >