[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 03/17] virtio: use standard virtio_ring.h
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v3 03/17] virtio: use standard virtio_ring.h |
Date: |
Wed, 18 Feb 2015 15:59:35 +0100 |
On Tue, Feb 17, 2015 at 01:44:26PM +0100, Thomas Huth wrote:
> On Mon, 16 Feb 2015 22:35:40 +0100
> "Michael S. Tsirkin" <address@hidden> wrote:
>
> > Switch to virtio_ring.h from standard headers.
> >
> > Signed-off-by: Michael S. Tsirkin <address@hidden>
> > ---
> > include/hw/virtio/dataplane/vring.h | 2 +-
> > include/hw/virtio/virtio_ring.h | 167
> > ------------------------------------
> > hw/net/vhost_net.c | 2 +-
> > 3 files changed, 2 insertions(+), 169 deletions(-)
> > delete mode 100644 include/hw/virtio/virtio_ring.h
> >
> > diff --git a/include/hw/virtio/dataplane/vring.h
> > b/include/hw/virtio/dataplane/vring.h
> > index d3e086a..836d26a 100644
> > --- a/include/hw/virtio/dataplane/vring.h
> > +++ b/include/hw/virtio/dataplane/vring.h
> > @@ -18,7 +18,7 @@
> > #define VRING_H
> >
> > #include "qemu-common.h"
> > -#include "hw/virtio/virtio_ring.h"
> > +#include "standard-headers/linux/virtio_ring.h"
> > #include "hw/virtio/virtio.h"
> >
> > typedef struct {
> > diff --git a/include/hw/virtio/virtio_ring.h
> > b/include/hw/virtio/virtio_ring.h
> > deleted file mode 100644
> > index 0b42e6e..0000000
> > --- a/include/hw/virtio/virtio_ring.h
> > +++ /dev/null
> > @@ -1,167 +0,0 @@
> > -#ifndef _LINUX_VIRTIO_RING_H
> > -#define _LINUX_VIRTIO_RING_H
> > -/*
> > - * This file is copied from /usr/include/linux while converting __uNN types
> > - * to uXX_t, __inline__ to inline, and tab to spaces.
> > - * */
>
> Hmmm, in the old header, the "__inline__" was replaced with
> "inline" ... in the new header from patch 2, there's now "__inline__"
> again. Was there a good reason for this replacement? If yes, your
> update-linux-headers.sh patch should maybe replace the "__inline__"
> into "inline", too?
>
> Apart from that, the patch looks good to me.
>
> Thomas
Probably a good idea, will do, though I wonder why does
qemu use always_inline so aggressively.
OK to do as patch on top? This series is too big already.
--
MST
- [Qemu-devel] [PATCH v3 00/17] virtio: pull headers from linux, Michael S. Tsirkin, 2015/02/16
- [Qemu-devel] [PATCH v3 01/17] scripts/update-linux-headers.sh: pull virtio hdrs, Michael S. Tsirkin, 2015/02/16
- [Qemu-devel] [PATCH v3 03/17] virtio: use standard virtio_ring.h, Michael S. Tsirkin, 2015/02/16
- [Qemu-devel] [PATCH v3 02/17] include: import virtio headers from linux 4.0, Michael S. Tsirkin, 2015/02/16
- [Qemu-devel] [PATCH v3 05/17] virtio-balloon: use standard headers, Michael S. Tsirkin, 2015/02/16
- [Qemu-devel] [PATCH v3 04/17] virtio: use standard-headers, Michael S. Tsirkin, 2015/02/16
- [Qemu-devel] [PATCH v3 06/17] virtio-9p: use standard headers, Michael S. Tsirkin, 2015/02/16
- [Qemu-devel] [PATCH v3 07/17] virtio-blk: switch to standard-headers, Michael S. Tsirkin, 2015/02/16