diff options
author | Ilya Dryomov <idryomov@redhat.com> | 2014-11-05 19:33:44 +0300 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-03-06 14:40:54 -0800 |
commit | 54ff4c89a5445fa8f313a338c1cf5478317df154 (patch) | |
tree | 31fa563e9cefcfae7f96c23fdf9df367f02bdb5f | |
parent | 5d3c6d27f48ce3b501c988bd0ab2232a0d4612c6 (diff) |
libceph: change from BUG to WARN for __remove_osd() asserts
commit cc9f1f518cec079289d11d732efa490306b1ddad upstream.
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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-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 d2e44eeb83ec..3f34304efd8a 100644 --- a/net/ceph/osd_client.c +++ b/net/ceph/osd_client.c @@ -968,8 +968,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); |