summaryrefslogtreecommitdiffstats
path: root/arch/ia64
diff options
context:
space:
mode:
authorDavidlohr Bueso <dave@stgolabs.net>2018-01-22 09:21:37 -0800
committerTony Luck <tony.luck@intel.com>2018-03-05 14:43:45 -0800
commit69c907022a7d9325cdc5c9dd064571e445df9a47 (patch)
tree459e3c517dfe66d0058a41106092dbfd957d3abe /arch/ia64
parent339d541a01f70b247f7e6266c1110aa3cef56b9f (diff)
downloadop-kernel-dev-69c907022a7d9325cdc5c9dd064571e445df9a47.zip
op-kernel-dev-69c907022a7d9325cdc5c9dd064571e445df9a47.tar.gz
ia64/err-inject: Use get_user_pages_fast()
At the point of sysfs callback, the call to gup is done without mmap_sem (or any lock for that matter). This is racy. As such, use the get_user_pages_fast() alternative and safely avoid taking the lock, if possible. Signed-off-by: Davidlohr Bueso <dbueso@suse.de> Signed-off-by: Tony Luck <tony.luck@intel.com>
Diffstat (limited to 'arch/ia64')
-rw-r--r--arch/ia64/kernel/err_inject.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/ia64/kernel/err_inject.c b/arch/ia64/kernel/err_inject.c
index 85bba43..658a8e0 100644
--- a/arch/ia64/kernel/err_inject.c
+++ b/arch/ia64/kernel/err_inject.c
@@ -142,7 +142,7 @@ store_virtual_to_phys(struct device *dev, struct device_attribute *attr,
u64 virt_addr=simple_strtoull(buf, NULL, 16);
int ret;
- ret = get_user_pages(virt_addr, 1, FOLL_WRITE, NULL, NULL);
+ ret = get_user_pages_fast(virt_addr, 1, FOLL_WRITE, NULL);
if (ret<=0) {
#ifdef ERR_INJ_DEBUG
printk("Virtual address %lx is not existing.\n",virt_addr);
OpenPOWER on IntegriCloud