aboutsummaryrefslogtreecommitdiffstats
path: root/tools/firmware/hvmloader
diff options
context:
space:
mode:
authorKeir Fraser <keir.fraser@citrix.com>2010-08-02 17:11:33 +0100
committerKeir Fraser <keir.fraser@citrix.com>2010-08-02 17:11:33 +0100
commitcef33294d4bf6912138687780c7949fedc97140e (patch)
tree090e1525f1884226ad59671fa10b8cdd27da196d /tools/firmware/hvmloader
parent5dd77158ed278410a91608a835fdf313a524c5cc (diff)
downloadxen-cef33294d4bf6912138687780c7949fedc97140e.tar.gz
xen-cef33294d4bf6912138687780c7949fedc97140e.tar.bz2
xen-cef33294d4bf6912138687780c7949fedc97140e.zip
Walking the page lists needs the page_alloc lock
There are a few places in Xen where we walk a domain's page lists without holding the page_alloc lock. They race with updates to the page lists, which are normally rare but can be quite common under PoD when the domain is close to its memory limit and the PoD reclaimer is busy. This patch protects those places by taking the page_alloc lock. I think this is OK for the two debug-key printouts - they don't run from irq context and look deadlock-free. The tboot change seems safe too unless tboot shutdown functions are called from irq context or with the page_alloc lock held. The p2m one is the scariest but there are already code paths in PoD that take the page_alloc lock with the p2m lock held so it's no worse than existing code. Signed-off-by: Tim Deegan <Tim.Deegan@citrix.com> xen-unstable changeset: 21881:57de3a3118bb xen-unstable date: Wed Jul 28 07:54:12 2010 +0100
Diffstat (limited to 'tools/firmware/hvmloader')
0 files changed, 0 insertions, 0 deletions