xen/balloon: flush persistent kmaps in correct position
authorWei Liu <wei.liu2@citrix.com>
Sat, 15 Mar 2014 16:11:47 +0000 (16:11 +0000)
committerDavid Vrabel <david.vrabel@citrix.com>
Tue, 25 Mar 2014 10:38:30 +0000 (10:38 +0000)
commit09ed3d5ba06137913960f9c9385f71fc384193ab
tree582150ce743efb10051e0bc13747e3e1a24005b9
parentb098d6726bbfb94c06d6e1097466187afddae61f
xen/balloon: flush persistent kmaps in correct position

Xen balloon driver will update ballooned out pages' P2M entries to point
to scratch page for PV guests. In 24f69373e2 ("xen/balloon: don't alloc
page while non-preemptible", kmap_flush_unused was moved after updating
P2M table. In that case for 32 bit PV guest we might end up with

  P2M    X -----> S  (S is mfn of balloon scratch page)
  M2P    Y -----> X  (Y is mfn in persistent kmap entry)

kmap_flush_unused() iterates through all the PTEs in the kmap address
space, using pte_to_page() to obtain the page. If the p2m and the m2p
are inconsistent the incorrect page is returned.  This will clear
page->address on the wrong page which may cause subsequent oopses if
that page is currently kmap'ed.

Move the flush back between get_page and __set_phys_to_machine to fix
this.

Signed-off-by: Wei Liu <wei.liu2@citrix.com>
Signed-off-by: David Vrabel <david.vrabel@citrix.com>
Cc: stable@vger.kernel.org # 3.12+
drivers/xen/balloon.c