Re: [Xen-devel] [PATCH] x86/mm: Alter is_xen_fixed_mfn() to use mfn_t

2017-02-14 Thread Julien Grall
Hi Andrew, On 02/07/2017 06:57 PM, Andrew Cooper wrote: The predicate is common between x86 and ARM, and is unlikley to be different/ NIT: s/unlikley/unlikely/ on other architectures. Drop the arch declarations and introduce a common static inline in xen/mm.h instead. Signed-off-by:

Re: [Xen-devel] [PATCH] x86/mm: Alter is_xen_fixed_mfn() to use mfn_t

2017-02-08 Thread George Dunlap
On 08/02/17 13:07, Jan Beulich wrote: On 07.02.17 at 19:57, wrote: >> --- a/xen/include/xen/mm.h >> +++ b/xen/include/xen/mm.h >> @@ -599,4 +599,10 @@ static inline void filtered_flush_tlb_mask(uint32_t >> tlbflush_timestamp) >> } >> } >> >> +static inline

Re: [Xen-devel] [PATCH] x86/mm: Alter is_xen_fixed_mfn() to use mfn_t

2017-02-08 Thread Jan Beulich
>>> On 07.02.17 at 19:57, wrote: > --- a/xen/include/xen/mm.h > +++ b/xen/include/xen/mm.h > @@ -599,4 +599,10 @@ static inline void filtered_flush_tlb_mask(uint32_t > tlbflush_timestamp) > } > } > > +static inline bool is_xen_fixed_mfn(mfn_t mfn) > +{ > +

[Xen-devel] [PATCH] x86/mm: Alter is_xen_fixed_mfn() to use mfn_t

2017-02-07 Thread Andrew Cooper
The predicate is common between x86 and ARM, and is unlikley to be different on other architectures. Drop the arch declarations and introduce a common static inline in xen/mm.h instead. Signed-off-by: Andrew Cooper --- CC: Jan Beulich CC: Stefano