dm ioctl: drop use of __GFP_REPEAT in copy_params()'s __vmalloc() call
authorMichal Hocko <mhocko@suse.com>
Thu, 28 Apr 2016 13:24:03 +0000 (15:24 +0200)
committerMike Snitzer <snitzer@redhat.com>
Thu, 5 May 2016 19:25:55 +0000 (15:25 -0400)
copy_params()'s use of __GFP_REPEAT for the __vmalloc() call doesn't make much
sense because vmalloc doesn't rely on costly high order allocations.

Signed-off-by: Michal Hocko <mhocko@suse.com>
Signed-off-by: Mike Snitzer <snitzer@redhat.com>
drivers/md/dm-ioctl.c

index 2adf81d81fca4237f6c911211a0d5b0dc21546d1..2c7ca258c4e47ac3ab005752983925413c56cd61 100644 (file)
@@ -1723,7 +1723,7 @@ static int copy_params(struct dm_ioctl __user *user, struct dm_ioctl *param_kern
        if (!dmi) {
                unsigned noio_flag;
                noio_flag = memalloc_noio_save();
-               dmi = __vmalloc(param_kernel->data_size, GFP_NOIO | __GFP_REPEAT | __GFP_HIGH | __GFP_HIGHMEM, PAGE_KERNEL);
+               dmi = __vmalloc(param_kernel->data_size, GFP_NOIO | __GFP_HIGH | __GFP_HIGHMEM, PAGE_KERNEL);
                memalloc_noio_restore(noio_flag);
                if (dmi)
                        *param_flags |= DM_PARAMS_VMALLOC;