[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 0/21] Please pull xen-20170421-tag for 2.10
From: |
Greg Kurz |
Subject: |
Re: [Qemu-devel] [PULL 0/21] Please pull xen-20170421-tag for 2.10 |
Date: |
Tue, 25 Apr 2017 08:44:08 +0200 |
On Mon, 24 Apr 2017 16:44:53 -0700 (PDT)
Stefano Stabellini <address@hidden> wrote:
> On Mon, 24 Apr 2017, Peter Maydell wrote:
> > On 24 April 2017 at 22:25, Stefano Stabellini <address@hidden> wrote:
> > > diff --git a/hw/9pfs/xen-9pfs.h b/hw/9pfs/xen-9pfs.h
> > > new file mode 100644
> > > index 0000000..18f0ec0
> > > --- /dev/null
> > > +++ b/hw/9pfs/xen-9pfs.h
> > > @@ -0,0 +1,14 @@
> > > +/*
> > > + * Xen 9p backend
> > > + *
> > > + * Copyright Aporeto 2017
> > > + *
> > > + * Authors:
> > > + * Stefano Stabellini <address@hidden>
> > > + *
> > > + */
> >
> > Trivial file, but I prefer it if we have a brief license
> > statement in every file, just to be clear (it might
> > accumulate more code later).
>
> Sure
>
> > > +
> > > +#include <xen/io/protocols.h>
> > > +#include "hw/xen/io/ring.h"
> > > +
> > > +DEFINE_XEN_FLEX_RING_AND_INTF(xen_9pfs);
> >
> > Is it worth a comment to dissuade people from thinking they can
> > inline the file back into xen-9p-backend.c ?
>
> Thanks for the quick review! Here you go:
>
>
> diff --git a/hw/9pfs/xen-9p-backend.c b/hw/9pfs/xen-9p-backend.c
> index 7e962aa..9c7f41a 100644
> --- a/hw/9pfs/xen-9p-backend.c
> +++ b/hw/9pfs/xen-9p-backend.c
> @@ -13,12 +13,9 @@
> #include "hw/hw.h"
> #include "hw/9pfs/9p.h"
> #include "hw/xen/xen_backend.h"
> -#include "hw/xen/io/ring.h"
> +#include "hw/9pfs/xen-9pfs.h"
> #include "qemu/config-file.h"
> #include "fsdev/qemu-fsdev.h"
> -#include <xen/io/protocols.h>
> -
> -DEFINE_XEN_FLEX_RING_AND_INTF(xen_9pfs);
>
> #define VERSIONS "1"
> #define MAX_RINGS 8
> diff --git a/hw/9pfs/xen-9pfs.h b/hw/9pfs/xen-9pfs.h
> new file mode 100644
> index 0000000..6e33d77
> --- /dev/null
> +++ b/hw/9pfs/xen-9pfs.h
> @@ -0,0 +1,21 @@
> +/*
> + * Xen 9p backend
> + *
> + * Copyright Aporeto 2017
> + *
> + * Authors:
> + * Stefano Stabellini <address@hidden>
> + *
> + * This work is licensed under the terms of the GNU GPL version 2.
"version 2 or later" maybe ?
> + * See the COPYING file in the top-level directory.
> + *
> + */
> +
> +#include <xen/io/protocols.h>
> +#include "hw/xen/io/ring.h"
> +
> +/*
> + * Do not merge into xen-9p-backend.c: clang doesn't allow unused static
> + * inline functions in c files.
> + */
> +DEFINE_XEN_FLEX_RING_AND_INTF(xen_9pfs);
pgpvxfVvTZbjM.pgp
Description: OpenPGP digital signature
- [Qemu-devel] [PULL 15/21] xen/9pfs: implement in/out_iov_from_pdu and vmarshal/vunmarshal, (continued)
- [Qemu-devel] [PULL 15/21] xen/9pfs: implement in/out_iov_from_pdu and vmarshal/vunmarshal, Stefano Stabellini, 2017/04/21
- [Qemu-devel] [PULL 17/21] xen/9pfs: build and register Xen 9pfs backend, Stefano Stabellini, 2017/04/21
- [Qemu-devel] [PULL 18/21] add xen-9p-backend to MAINTAINERS under Xen, Stefano Stabellini, 2017/04/21
- [Qemu-devel] [PULL 19/21] move xen-common.c to hw/xen/, Stefano Stabellini, 2017/04/21
- [Qemu-devel] [PULL 21/21] move xen-mapcache.c to hw/i386/xen/, Stefano Stabellini, 2017/04/21
- [Qemu-devel] [PULL 20/21] move xen-hvm.c to hw/i386/xen/, Stefano Stabellini, 2017/04/21
- Re: [Qemu-devel] [PULL 0/21] Please pull xen-20170421-tag for 2.10, Peter Maydell, 2017/04/24
- Re: [Qemu-devel] [PULL 0/21] Please pull xen-20170421-tag for 2.10, Stefano Stabellini, 2017/04/24
- Re: [Qemu-devel] [PULL 0/21] Please pull xen-20170421-tag for 2.10, Peter Maydell, 2017/04/24
- Re: [Qemu-devel] [PULL 0/21] Please pull xen-20170421-tag for 2.10, Stefano Stabellini, 2017/04/24
- Re: [Qemu-devel] [PULL 0/21] Please pull xen-20170421-tag for 2.10,
Greg Kurz <=
- Re: [Qemu-devel] [PULL 0/21] Please pull xen-20170421-tag for 2.10, Markus Armbruster, 2017/04/25
- Re: [Qemu-devel] [PULL 0/21] Please pull xen-20170421-tag for 2.10, Stefano Stabellini, 2017/04/25
- Re: [Qemu-devel] [PULL 0/21] Please pull xen-20170421-tag for 2.10, Markus Armbruster, 2017/04/26