Re: [Xen-devel] [PULL 0/21] Please pull xen-20170421-tag for 2.10

[Peter Maydell <peter.maydell@xxxxxxxxxx>]

On 21 April 2017 at 21:14, Stefano Stabellini <sstabellini@xxxxxxxxxx> wrote:
> The following changes since commit 55a19ad8b2d0797e3a8fe90ab99a9bb713824059:
>
>   Update version for v2.9.0-rc1 release (2017-03-21 17:13:29 +0000)
>
> are available in the git repository at:
>
>   git://xenbits.xen.org/people/sstabellini/qemu-dm.git tags/xen-20170421-tag
>
> for you to fetch changes up to b0d48550a2d10f0fa8c30a7cc3ff5a1cbda8d4c4:
>
>   move xen-mapcache.c to hw/i386/xen/ (2017-04-21 12:41:29 -0700)
>
> ----------------------------------------------------------------
> Xen 2017/04/21
>
> ----------------------------------------------------------------

Hi; I'm afraid this doesn't build with clang:


  CC      hw/9pfs/xen-9p-backend.o
/home/petmay01/linaro/qemu-for-merges/hw/9pfs/xen-9p-backend.c:21:1:
error: unused fun
ction 'xen_9pfs_get_ring_ptr' [-Werror,-Wunused-function]
DEFINE_XEN_FLEX_RING_AND_INTF(xen_9pfs);
^
/home/petmay01/linaro/qemu-for-merges/include/hw/xen/io/ring.h:469:79:
note: expanded
from macro 'DEFINE_XEN_FLEX_RING_AND_INTF'
};                                                                            \
                                                                              ^
/home/petmay01/linaro/qemu-for-merges/include/hw/xen/io/ring.h:387:30:
note: expanded from macro '\
DEFINE_XEN_FLEX_RING'
static inline unsigned char *name##_get_ring_ptr(unsigned char *buf,          \
                             ^
<scratch space>:151:1: note: expanded from here
xen_9pfs_get_ring_ptr
^
/home/petmay01/linaro/qemu-for-merges/hw/9pfs/xen-9p-backend.c:21:1:
error: unused function 'xen_9pfs_write_packet'
[-Werror,-Wunused-function]
/home/petmay01/linaro/qemu-for-merges/include/hw/xen/io/ring.h:469:79:
note: expanded from macro 'DEFINE_XEN_FLEX_RING_AND_INTF'
};                                                                            \
                                                                              ^
/home/petmay01/linaro/qemu-for-merges/include/hw/xen/io/ring.h:412:20:
note: expanded from macro '\
DEFINE_XEN_FLEX_RING'
static inline void name##_write_packet(unsigned char *buf,                    \
                   ^
<scratch space>:155:1: note: expanded from here
xen_9pfs_write_packet
^
2 errors generated.
/home/petmay01/linaro/qemu-for-merges/rules.mak:69: recipe for target
'hw/9pfs/xen-9p-backend.o' failed


Clang requires that functions, even inline functions, defined in .c files
must be used.

thanks
-- PMM

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel