Commit dc2bf000 authored by Markus Elfring's avatar Markus Elfring Committed by Linus Torvalds

lib/test: delete five error messages for failed memory allocations

Omit extra messages for a memory allocation failure in these functions.

This issue was detected by using the Coccinelle software.

Link: http://lkml.kernel.org/r/410a4c5a-4ee0-6fcc-969c-103d8e496b78@users.sourceforge.netSigned-off-by: default avatarMarkus Elfring <elfring@users.sourceforge.net>
Acked-by: default avatarMichal Hocko <mhocko@suse.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent d6b28e09
...@@ -353,10 +353,9 @@ static noinline void __init memcg_accounted_kmem_cache(void) ...@@ -353,10 +353,9 @@ static noinline void __init memcg_accounted_kmem_cache(void)
*/ */
for (i = 0; i < 5; i++) { for (i = 0; i < 5; i++) {
p = kmem_cache_alloc(cache, GFP_KERNEL); p = kmem_cache_alloc(cache, GFP_KERNEL);
if (!p) { if (!p)
pr_err("Allocation failed\n");
goto free_cache; goto free_cache;
}
kmem_cache_free(cache, p); kmem_cache_free(cache, p);
msleep(100); msleep(100);
} }
......
...@@ -783,10 +783,8 @@ static int kmod_config_sync_info(struct kmod_test_device *test_dev) ...@@ -783,10 +783,8 @@ static int kmod_config_sync_info(struct kmod_test_device *test_dev)
free_test_dev_info(test_dev); free_test_dev_info(test_dev);
test_dev->info = vzalloc(config->num_threads * test_dev->info = vzalloc(config->num_threads *
sizeof(struct kmod_test_device_info)); sizeof(struct kmod_test_device_info));
if (!test_dev->info) { if (!test_dev->info)
dev_err(test_dev->dev, "Cannot alloc test_dev info\n");
return -ENOMEM; return -ENOMEM;
}
return 0; return 0;
} }
...@@ -1089,10 +1087,8 @@ static struct kmod_test_device *alloc_test_dev_kmod(int idx) ...@@ -1089,10 +1087,8 @@ static struct kmod_test_device *alloc_test_dev_kmod(int idx)
struct miscdevice *misc_dev; struct miscdevice *misc_dev;
test_dev = vzalloc(sizeof(struct kmod_test_device)); test_dev = vzalloc(sizeof(struct kmod_test_device));
if (!test_dev) { if (!test_dev)
pr_err("Cannot alloc test_dev\n");
goto err_out; goto err_out;
}
mutex_init(&test_dev->config_mutex); mutex_init(&test_dev->config_mutex);
mutex_init(&test_dev->trigger_mutex); mutex_init(&test_dev->trigger_mutex);
......
...@@ -76,17 +76,14 @@ static int __init list_sort_test(void) ...@@ -76,17 +76,14 @@ static int __init list_sort_test(void)
pr_debug("start testing list_sort()\n"); pr_debug("start testing list_sort()\n");
elts = kcalloc(TEST_LIST_LEN, sizeof(*elts), GFP_KERNEL); elts = kcalloc(TEST_LIST_LEN, sizeof(*elts), GFP_KERNEL);
if (!elts) { if (!elts)
pr_err("error: cannot allocate memory\n");
return err; return err;
}
for (i = 0; i < TEST_LIST_LEN; i++) { for (i = 0; i < TEST_LIST_LEN; i++) {
el = kmalloc(sizeof(*el), GFP_KERNEL); el = kmalloc(sizeof(*el), GFP_KERNEL);
if (!el) { if (!el)
pr_err("error: cannot allocate memory\n");
goto exit; goto exit;
}
/* force some equivalencies */ /* force some equivalencies */
el->value = prandom_u32() % (TEST_LIST_LEN / 3); el->value = prandom_u32() % (TEST_LIST_LEN / 3);
el->serial = i; el->serial = i;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment