diff options
author | Huang Ying <ying.huang@intel.com> | 2010-06-22 02:23:11 -0400 |
---|---|---|
committer | Avi Kivity <avi@redhat.com> | 2010-08-01 03:47:11 -0400 |
commit | bbeb34062fbad287c949a945a516a0c15b179993 (patch) | |
tree | cf29b50e076ba1ddc07d5b1292f243653a2f67a9 /mm/memory-failure.c | |
parent | 6c3f6041172b78d5532c6bf3680d304e92ec2e66 (diff) |
KVM: Fix a race condition for usage of is_hwpoison_address()
is_hwpoison_address accesses the page table, so the caller must hold
current->mm->mmap_sem in read mode. So fix its usage in hva_to_pfn of
kvm accordingly.
Comment is_hwpoison_address to remind other users.
Reported-by: Avi Kivity <avi@redhat.com>
Signed-off-by: Huang Ying <ying.huang@intel.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
Diffstat (limited to 'mm/memory-failure.c')
-rw-r--r-- | mm/memory-failure.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/mm/memory-failure.c b/mm/memory-failure.c index 378b0f61fd3c..6b44e52cacaa 100644 --- a/mm/memory-failure.c +++ b/mm/memory-failure.c | |||
@@ -1298,6 +1298,9 @@ done: | |||
1298 | return ret; | 1298 | return ret; |
1299 | } | 1299 | } |
1300 | 1300 | ||
1301 | /* | ||
1302 | * The caller must hold current->mm->mmap_sem in read mode. | ||
1303 | */ | ||
1301 | int is_hwpoison_address(unsigned long addr) | 1304 | int is_hwpoison_address(unsigned long addr) |
1302 | { | 1305 | { |
1303 | pgd_t *pgdp; | 1306 | pgd_t *pgdp; |