]> git.ipfire.org Git - thirdparty/kernel/stable.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)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 6 Jun 2015 15:20:51 +0000 (08:20 -0700)
commitf0034a3a16b3c04b88a87ca2a49839f74f668215
tree65c15e88ec1665d2beec82054c3a2a0c0713c4f0
parent5b4715d14861470804d44ee916ed2ce2e3e6fd62
Revert "libceph: clear r_req_lru_item in __unregister_linger_request()"

commit 521a04d06a729e5971cdee7f84080387ed320527 upstream.

This reverts commit ba9d114ec5578e6e99a4dfa37ff8ae688040fd64.

.. 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.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Reviewed-by: Sage Weil <sage@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
net/ceph/osd_client.c