diff options
author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2013-06-25 10:50:04 -0400 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2013-06-28 10:04:36 -0400 |
commit | 6110d70bdf99f9d0448f1f61798542e3b123b42a (patch) | |
tree | 46701e054133568085ba4d8b2518dbac4a0b9247 | |
parent | 26ea8f9239a062a47cfef38ea9c63409906366ff (diff) |
drbd: fix error return code in drbd_init()
Fix to return a negative error code from the error handling
case instead of 0, as returned elsewhere in this function.
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Lars Ellenberg <lars.ellenberg@linbit.com>
Signed-off-by: Andreas Gruenbacher <agruen@linbit.com>
Signed-off-by: Philipp Reisner <philipp.reisner@linbit.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
-rw-r--r-- | drivers/block/drbd/drbd_main.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/block/drbd/drbd_main.c b/drivers/block/drbd/drbd_main.c index a5dca6affcbb..49040a336949 100644 --- a/drivers/block/drbd/drbd_main.c +++ b/drivers/block/drbd/drbd_main.c | |||
@@ -2762,8 +2762,6 @@ int __init drbd_init(void) | |||
2762 | /* | 2762 | /* |
2763 | * allocate all necessary structs | 2763 | * allocate all necessary structs |
2764 | */ | 2764 | */ |
2765 | err = -ENOMEM; | ||
2766 | |||
2767 | init_waitqueue_head(&drbd_pp_wait); | 2765 | init_waitqueue_head(&drbd_pp_wait); |
2768 | 2766 | ||
2769 | drbd_proc = NULL; /* play safe for drbd_cleanup */ | 2767 | drbd_proc = NULL; /* play safe for drbd_cleanup */ |
@@ -2773,6 +2771,7 @@ int __init drbd_init(void) | |||
2773 | if (err) | 2771 | if (err) |
2774 | goto fail; | 2772 | goto fail; |
2775 | 2773 | ||
2774 | err = -ENOMEM; | ||
2776 | drbd_proc = proc_create_data("drbd", S_IFREG | S_IRUGO , NULL, &drbd_proc_fops, NULL); | 2775 | drbd_proc = proc_create_data("drbd", S_IFREG | S_IRUGO , NULL, &drbd_proc_fops, NULL); |
2777 | if (!drbd_proc) { | 2776 | if (!drbd_proc) { |
2778 | printk(KERN_ERR "drbd: unable to register proc file\n"); | 2777 | printk(KERN_ERR "drbd: unable to register proc file\n"); |
@@ -2803,7 +2802,6 @@ int __init drbd_init(void) | |||
2803 | fail: | 2802 | fail: |
2804 | drbd_cleanup(); | 2803 | drbd_cleanup(); |
2805 | if (err == -ENOMEM) | 2804 | if (err == -ENOMEM) |
2806 | /* currently always the case */ | ||
2807 | printk(KERN_ERR "drbd: ran out of memory\n"); | 2805 | printk(KERN_ERR "drbd: ran out of memory\n"); |
2808 | else | 2806 | else |
2809 | printk(KERN_ERR "drbd: initialization failure\n"); | 2807 | printk(KERN_ERR "drbd: initialization failure\n"); |