Re: [Xen-devel] [MINIOS PATCH v2] Include string.h before console.h

[Wei Liu <wei.liu2@xxxxxxxxxx>]

CC Stefano and Julien

On Thu, Oct 05, 2017 at 10:28:00AM +0100, Wei Liu wrote:
> On Thu, Oct 05, 2017 at 03:22:33AM -0600, Jan Beulich wrote:
> > >>> On 04.10.17 at 15:51, <wei.liu2@xxxxxxxxxx> wrote:
> > > Starting from Xen commit af8d93564, it is required to include string.h
> > > before console.h.
> > 
> > But that's broken then - there shouldn't be such an effect for
> > anyone updating their public headers. IOW I think this needs
> > to be dealt with by further adjusting console.h, e.g. by
> > making the additions depend on a manifest symbol to be
> > defined.
> 
> This should work.
> 
> This isn't just console.h alone. Any header that uses
> DEFINE_XEN_FLEX_RING will potentially need a manifest symbol. We get
> away so far because the two other headers 9pfs.h and pvcalls.h are
> relatively new.

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