Hi Jan,

On 02/03/18 14:45, Jan Beulich wrote:
On 21.02.18 at 15:02, <julien.gr...@arm.com> wrote:
No functional change intended.

Signed-off Julien Grall <julien.gr...@arm.com>

Acked-by: Jan Beulich <jbeul...@suse.com>
with one remark:

@@ -5545,8 +5547,7 @@ static void __memguard_change_range(void *p, unsigned 
long l, int guard)
      if ( guard )
          flags &= ~_PAGE_PRESENT;
- map_pages_to_xen(
-        _p, virt_to_maddr(p) >> PAGE_SHIFT, _l >> PAGE_SHIFT, flags);
+    map_pages_to_xen(_p, mfn_x(virt_to_mfn(p)), _l >> PAGE_SHIFT, flags);

Would have been nice if this was switched to PFN_DOWN() instead
of the open coded shift right away.

I can do that.


Julien Grall

Xen-devel mailing list

Reply via email to