diff options
| author |   Ilya Dryomov <idryomov@redhat.com> | 2014-11-05 19:33:44 +0300 |
|---|---|---|
| committer | Ilya Dryomov <idryomov@redhat.com> | 2014-11-13 22:26:34 +0300 |
| commit | cc9f1f518cec079289d11d732efa490306b1ddad (patch) | |
| tree | 2794e80d4e637c30b764eb560f14a17ca0df7f27 /net/ceph/osd_client.c | |
| parent | libceph: clear r_req_lru_item in __unregister_linger_request() (diff) | |
| download | linux-dev-cc9f1f518cec079289d11d732efa490306b1ddad.tar.xz linux-dev-cc9f1f518cec079289d11d732efa490306b1ddad.zip | |
libceph: change from BUG to WARN for __remove_osd() asserts
No reason to use BUG_ON for osd request list assertions.
Signed-off-by: Ilya Dryomov <idryomov@redhat.com>
Reviewed-by: Alex Elder <elder@linaro.org>
Diffstat (limited to '')
| -rw-r--r-- | net/ceph/osd_client.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/ceph/osd_client.c b/net/ceph/osd_client.c index decc3b74e65f..6f164289bde8 100644 --- a/net/ceph/osd_client.c +++ b/net/ceph/osd_client.c @@ -1007,8 +1007,8 @@ static void put_osd(struct ceph_osd *osd) static void __remove_osd(struct ceph_osd_client *osdc, struct ceph_osd *osd) { dout("__remove_osd %p\n", osd); - BUG_ON(!list_empty(&osd->o_requests)); - BUG_ON(!list_empty(&osd->o_linger_requests)); + WARN_ON(!list_empty(&osd->o_requests)); + WARN_ON(!list_empty(&osd->o_linger_requests)); rb_erase(&osd->o_node, &osdc->osds); list_del_init(&osd->o_osd_lru); |