On 08/02/17 13:07, Jan Beulich wrote:
>>>> On 07.02.17 at 19:57, <andrew.coop...@citrix.com> 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)
>> +{
>> + return (pfn_to_paddr(mfn_x(mfn)) >= __pa(&_start) &&
>> + pfn_to_paddr(mfn_x(mfn)) <= __pa(&_end));
>> +}
>
> Even if you only morph what's been there before, I do think the
> latter wants to be < instead of <= . With that adjusted
> Reviewed-by: Jan Beulich <jbeul...@suse.com>
> but I'm also unconvinced it wouldn't be better to compare PFNs
> instead of physical addresses here.
Not positive this falls under my remit but just in case:
Acked-by: George Dunlap <george.dun...@citrix.com>
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
