[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2 1/2] x86/mm: fix a potential race condition in map_pages_to_xen().



Hi,

On 11/13/2017 11:06 AM, Jan Beulich wrote:
On 13.11.17 at 11:34, <yu.c.zhang@xxxxxxxxxxxxxxx> wrote:
Our debug showed the concerned page->count_info was already(and
unexpectedly)
cleared in free_xenheap_pages(), and the call trace should be like this:

free_xenheap_pages()
      ^
      |
free_xen_pagetable()
      ^
      |
map_pages_to_xen()
      ^
      |
update_xen_mappings()
      ^
      |
get_page_from_l1e()
      ^
      |
mod_l1_entry()
      ^
      |
do_mmu_update()

This ...

Is above description convincing enough? :-)

... is indeed enough for me to suggest to Julien that we take both
patches (once they're ready). But it's his decision in the end.

I will wait the series to be ready before giving my release-ack.

Cheers,


Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.