[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v9 5/7] memory: add check_get_page_from_gfn() as a wrapper...



On Wed, Sep 19, 2018 at 10:59:34AM +0200, Roger Pau Monné wrote:
> On Wed, Sep 19, 2018 at 02:31:47AM -0600, Jan Beulich wrote:
> > >>> On 19.09.18 at 10:01, <Paul.Durrant@xxxxxxxxxx> wrote:
> > >> From: Xen-devel [mailto:xen-devel-bounces@xxxxxxxxxxxxxxxxxxxx] On Behalf
> > >> Of Paul Durrant
> > >> Sent: 19 September 2018 08:56
> > >> 
> > >> > From: Jan Beulich [mailto:JBeulich@xxxxxxxx]
> > >> > Sent: 19 September 2018 07:03
> > >> >
> > >> > >>> On 18.09.18 at 18:37, <Paul.Durrant@xxxxxxxxxx> wrote:
> > >> > > Moving to p2m-common won't work. The function declaration involves a
> > >> > > p2m_type_t argument and that enum is defined in the arm and x86
> > >> specific
> > >> > > headers. I propose therefore to leave this patch as-is.
> > >> >
> > >> > Leaving the duplication in place is just the last resort imo. Does
> > >> > xen/mm.h
> > >> > not work either?
> > >> 
> > >> No, it won't. It has to be something *after* the definition of the
> > >> p2m_type_t enum. I could, as Julien suggested, move the inclusion of p2m-
> > >> common after that point, but it would mean the header guards would cease
> > >> to DTRT of course. I'm not sure whether C will allow me to forward 
> > >> declare
> > >> the enum (not something I've tried) but I could give that a go. Any other
> > >> suggestions?
> > >> 
> > > 
> > > Forward declaration of the enum does indeed appear to work, so I'll go 
> > > with 
> > > that.
> > 
> > That's an extension I'm not even sure all gcc versions support (I've checked
> > 4.3 just now, where it works). Roger, any chance you know whether clang
> > supports this?
> 
> I've just tested the following with clang 6 (which is the version used
> by osstest):
> 
> enum foo;
> enum foo { a, b, c };
> 
> And it works fine. I think travis or gitlab tested older versions of
> clang (the ones on the Linux distros).

They have clang 3.4 IIRC.

P.S. you can easily get the same environment locally by using the
in-tree containerization script.

Wei.

> 
> Thanks, Roger.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.