diff options
author | NeilBrown <neilb@suse.de> | 2013-07-16 06:44:29 (GMT) |
---|---|---|
committer | NeilBrown <neilb@suse.de> | 2013-08-27 06:05:32 (GMT) |
commit | 60559da4d8c3259ea41a14ca4cfcd83022bff6bd (patch) | |
tree | 4eb200925cc4067a59d0709bde0cf8b19d22e266 /drivers/md | |
parent | d8dfad3876e4386666b759da3c833d62fb8b2267 (diff) | |
download | linux-60559da4d8c3259ea41a14ca4cfcd83022bff6bd.tar.xz |
md: don't call md_allow_write in get_bitmap_file.
There is no really need as GFP_NOIO is very likely sufficient,
and failure is not catastrophic.
Calling md_allow_write here will convert a read-auto array to
read/write which could be confusing when you are just performing
a read operation.
Signed-off-by: NeilBrown <neilb@suse.de>
Diffstat (limited to 'drivers/md')
-rw-r--r-- | drivers/md/md.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/md/md.c b/drivers/md/md.c index 9f13e13..bba8732 100644 --- a/drivers/md/md.c +++ b/drivers/md/md.c @@ -5628,10 +5628,7 @@ static int get_bitmap_file(struct mddev * mddev, void __user * arg) char *ptr, *buf = NULL; int err = -ENOMEM; - if (md_allow_write(mddev)) - file = kmalloc(sizeof(*file), GFP_NOIO); - else - file = kmalloc(sizeof(*file), GFP_KERNEL); + file = kmalloc(sizeof(*file), GFP_NOIO); if (!file) goto out; |