]> git.baikalelectronics.ru Git - kernel.git/commit
Revert "libceph: clear r_req_lru_item in __unregister_linger_request()"
authorIlya Dryomov <idryomov@gmail.com>
Mon, 11 May 2015 14:53:34 +0000 (17:53 +0300)
committerIlya Dryomov <idryomov@gmail.com>
Wed, 20 May 2015 18:02:46 +0000 (21:02 +0300)
commit64ee5cbc120cd8af5289bcbb5914e6004fe22e2b
tree93a6adc6c25c566458e593cad779776a6b620a1b
parentcd0d521a17d98ee2a8e7041cee2d22490226d249
Revert "libceph: clear r_req_lru_item in __unregister_linger_request()"

This reverts commit 0282d81d13a724677208cebef84e0cd4a9d768c4.

.. which introduced a regression that prevented all lingering requests
requeued in kick_requests() from ever being sent to the OSDs, resulting
in a lot of missed notifies.  In retrospect it's pretty obvious that
r_req_lru_item item in the case of lingering requests can be used not
only for notarget, but also for unsent linkage due to how tightly
actual map and enqueue operations are coupled in __map_request().

The assertion that was being silenced is taken care of in the previous
("libceph: request a new osdmap if lingering request maps to no osd")
commit: by always kicking homeless lingering requests we ensure that
none of them ends up on the notarget list outside of the critical
section guarded by request_mutex.

Cc: stable@vger.kernel.org # 3.18+, needs cd0d521a17d9 "libceph: request a new osdmap if lingering request maps to no osd"
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Reviewed-by: Sage Weil <sage@redhat.com>
net/ceph/osd_client.c