Re: [Xen-devel] [PATCH] x86/mem_access: move p2m_mem_access_sanity_check() from header

2018-11-21 Thread Razvan Cojocaru
On 11/21/18 12:00 PM, Jan Beulich wrote: On 20.11.18 at 19:05, wrote: >> --- a/xen/arch/x86/mm/mem_access.c >> +++ b/xen/arch/x86/mm/mem_access.c >> @@ -541,6 +541,11 @@ void arch_p2m_set_access_required(struct domain *d, >> bool access_required) >> #endif >> } >> >> +bool p2m_mem_access

Re: [Xen-devel] [PATCH] x86/mem_access: move p2m_mem_access_sanity_check() from header

2018-11-21 Thread Jan Beulich
>>> On 20.11.18 at 19:05, wrote: > --- a/xen/arch/x86/mm/mem_access.c > +++ b/xen/arch/x86/mm/mem_access.c > @@ -541,6 +541,11 @@ void arch_p2m_set_access_required(struct domain *d, bool > access_required) > #endif > } > > +bool p2m_mem_access_sanity_check(struct domain *d) > +{ > +return

Re: [Xen-devel] [PATCH] x86/mem_access: move p2m_mem_access_sanity_check() from header

2018-11-20 Thread Tamas K Lengyel
On Tue, Nov 20, 2018 at 11:05 AM Razvan Cojocaru wrote: > > Move p2m_mem_access_sanity_check() from the asm-x86/mem_access.h > header, where it currently is declared inline, to > arch/x86/mm/mem_access.c. This allows source code that includes it > directly, or indirectly (such as xen/mem_access.h)

[Xen-devel] [PATCH] x86/mem_access: move p2m_mem_access_sanity_check() from header

2018-11-20 Thread Razvan Cojocaru
Move p2m_mem_access_sanity_check() from the asm-x86/mem_access.h header, where it currently is declared inline, to arch/x86/mm/mem_access.c. This allows source code that includes it directly, or indirectly (such as xen/mem_access.h), to not worry about also including sched.h for is_hvm_domain(). In