[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 19.09.18 at 09:56, <Paul.Durrant@xxxxxxxxxx> wrote:
>> 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?

I'm afraid I don't understand your concern wrt include guards. Each header
has its own. I'm actually surprised the current inclusion point is at the top
of the file, rather than after at least the basic type definitions each arch
has to supply.

Jan



_______________________________________________
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®.