diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-08-15 01:58:59 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-01-09 15:24:25 -0500 |
commit | 3834fb30d260b8fa6401fff271c65cd29ba94424 (patch) | |
tree | ce82151ec76058d997b90c611ea82464e217642c /net | |
parent | 53341d7de3f013cc39a3692bdfd02032ce722dff (diff) |
libceph: create_singlethread_workqueue() doesn't return ERR_PTRs
commit dbcae088fa660086bde6e10d63bb3c9264832d85 upstream.
create_singlethread_workqueue() returns NULL on error, and it doesn't
return ERR_PTRs.
I tweaked the error handling a little to be consistent with earlier in
the function.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Sage Weil <sage@inktank.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'net')
-rw-r--r-- | net/ceph/osd_client.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c index 311161eac5c4..ac34cb5120ed 100644 --- a/net/ceph/osd_client.c +++ b/net/ceph/osd_client.c | |||
@@ -2257,12 +2257,10 @@ int ceph_osdc_init(struct ceph_osd_client *osdc, struct ceph_client *client) | |||
2257 | if (err < 0) | 2257 | if (err < 0) |
2258 | goto out_msgpool; | 2258 | goto out_msgpool; |
2259 | 2259 | ||
2260 | err = -ENOMEM; | ||
2260 | osdc->notify_wq = create_singlethread_workqueue("ceph-watch-notify"); | 2261 | osdc->notify_wq = create_singlethread_workqueue("ceph-watch-notify"); |
2261 | if (IS_ERR(osdc->notify_wq)) { | 2262 | if (!osdc->notify_wq) |
2262 | err = PTR_ERR(osdc->notify_wq); | ||
2263 | osdc->notify_wq = NULL; | ||
2264 | goto out_msgpool; | 2263 | goto out_msgpool; |
2265 | } | ||
2266 | return 0; | 2264 | return 0; |
2267 | 2265 | ||
2268 | out_msgpool: | 2266 | out_msgpool: |