Re: [Xen-devel] [PATCH] p2m: move p2m-common.h inclusion point

2018-11-06 Thread George Dunlap
On 10/05/2018 11:14 AM, Jan Beulich wrote: > The header is (hence its name) supposed to be a helper for the per-arch > p2m.h files. It was never supposed to be included directly, and for the > purpose of putting common function declarations into the common header > it is more helpful if things

Re: [Xen-devel] [PATCH] p2m: move p2m-common.h inclusion point

2018-11-06 Thread Jan Beulich
>>> On 05.10.18 at 12:21, wrote: >> -Original Message- >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: 05 October 2018 11:14 >> To: xen-devel >> Cc: Julien Grall ; Andrew Cooper >> ; Paul Durrant ; Wei >> Liu ; George Dunlap ; Ian >> Jackson ; Stefano Stabellini >> ; Konrad

Re: [Xen-devel] [PATCH] p2m: move p2m-common.h inclusion point

2018-10-05 Thread Julien Grall
Hi, On 10/05/2018 11:14 AM, Jan Beulich wrote: The header is (hence its name) supposed to be a helper for the per-arch p2m.h files. It was never supposed to be included directly, and for the purpose of putting common function declarations into the common header it is more helpful if things like

Re: [Xen-devel] [PATCH] p2m: move p2m-common.h inclusion point

2018-10-05 Thread Jan Beulich
>>> On 05.10.18 at 12:21, wrote: >> -Original Message- >> From: Jan Beulich [mailto:jbeul...@suse.com] >> Sent: 05 October 2018 11:14 >> To: xen-devel >> Cc: Julien Grall ; Andrew Cooper >> ; Paul Durrant ; Wei >> Liu ; George Dunlap ; Ian >> Jackson ; Stefano Stabellini >> ; Konrad

Re: [Xen-devel] [PATCH] p2m: move p2m-common.h inclusion point

2018-10-05 Thread Paul Durrant
> -Original Message- > From: Jan Beulich [mailto:jbeul...@suse.com] > Sent: 05 October 2018 11:14 > To: xen-devel > Cc: Julien Grall ; Andrew Cooper > ; Paul Durrant ; Wei > Liu ; George Dunlap ; Ian > Jackson ; Stefano Stabellini > ; Konrad Rzeszutek Wilk ; > Tim (Xen.org) > Subject:

[Xen-devel] [PATCH] p2m: move p2m-common.h inclusion point

2018-10-05 Thread Jan Beulich
The header is (hence its name) supposed to be a helper for the per-arch p2m.h files. It was never supposed to be included directly, and for the purpose of putting common function declarations into the common header it is more helpful if things like p2m_t are already available at the inclusion