Hi,
On 09/22/2017 11:44 PM, Stefano Stabellini wrote:
Adding Jan
On Fri, 22 Sep 2017, Bhupinder Thakur wrote:
DEFINE_XEN_FLEX_RING(xencons) defines common helper functions such as
xencons_queued() to tell the current size of the ring buffer,
xencons_mask() to mask off the index, which are useful helper functions.
pl011 emulation code will use these helper functions.
io/console.h includes io/ring.h which defines DEFINE_XEN_FLEX_RING.
In console/daemon/io.c, string.h had to be included before io/console.h
because ring.h uses string functions.
Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org>
Reviewed-by: Stefano Stabellini <sstabell...@kernel.org>
Acked-by: Wei Liu <wei.l...@citrix.com>
Acked-by: Konrad Rzeszutek Wilk <konrad.w...@oracle.com>
Unfortunately this patch breaks the build on x86.
I am a bit surprised that this breaks only build on x86. I was expecting
the header checks to be done on all the architecture ... hmmm... looking
at the Makefile, the checks is only done when building natively. I guess
you are cross-compiling?
It might be interesting to look at getting headers check in
cross-compile environment given that this is the main way to build the
hypervisor today.
The reason is that
DEFINE_XEN_FLEX_RING requires C99, and the current header checks in
xen/include/Makefile use ANSI C.
I was not able to spot why DEFINE_XEN_FLEX_RING would require C99. Can
you detail it?
Cheers,
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel