summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2012-11-05 07:42:17 +0100
committerKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>2013-05-15 16:23:40 +0200
commit9eff37a8713939f218ab8bf0dc93f1d67af7b8b4 (patch)
treee795f75eb46dc0ecbef17d8b2d13e355343a39b5
parentxen/pci: Used cached MSI-X capability offset (diff)
downloadlinux-9eff37a8713939f218ab8bf0dc93f1d67af7b8b4.tar.xz
linux-9eff37a8713939f218ab8bf0dc93f1d67af7b8b4.zip
xen/privcmd: fix condition in privcmd_close()
The parenthesis are in the wrong place so the original code is equivalent to: if (!xen_feature(XENFEAT_writable_descriptor_tables)) { ... Which obviously was not intended. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
-rw-r--r--drivers/xen/privcmd.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/xen/privcmd.c b/drivers/xen/privcmd.c
index ca2b00e9d558..2cfc24d76fc5 100644
--- a/drivers/xen/privcmd.c
+++ b/drivers/xen/privcmd.c
@@ -504,7 +504,7 @@ static void privcmd_close(struct vm_area_struct *vma)
struct page **pages = vma->vm_private_data;
int numpgs = (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
- if (!xen_feature(XENFEAT_auto_translated_physmap || !numpgs || !pages))
+ if (!xen_feature(XENFEAT_auto_translated_physmap) || !numpgs || !pages)
return;
xen_unmap_domain_mfn_range(vma, numpgs, pages);