On Sun, 27 Dec 2020 22:31:56 -0800 Nadav Amit <nadav.amit@xxxxxxxxx> wrote: [...] > >> Thanks for doing that. Before I test it, did you also fix the issue > >> of x86’s setup_vm() [1]? > >> > >> [1] https://www.spinics.net/lists/kvm/msg230470.html > > > > unfortunately no, because I could not reproduce it. > > > > In theory setup_vm should just work, since it is called twice, but > > on different address ranges. > > > > The only issue I can think of is if the second call overlaps > > multiple areas. > > > > can you send me the memory map of that machine you're running the > > tests on? (/proc/iomem) > > Sorry for the late response. > > I see two calls to _page_alloc_init_area() with AREA_LOW_NUMBER, one > with (base_pfn=621, top_pfn=bfdd0) and one with (base_pfn=100000 > top_pfn=240000). ok, I could reproduce the issue now. to put it simply, the old code is broken. I could not reproduce the issue with the new code, so you should be able to test it.
