Hi Ilya, On 02/13/2017 12:11 PM, Ilya Dryomov wrote:
Hi Artur, How about the attached patch? handle_timeout() is going to iterate over all but the homeless OSD anyway; all it costs us is a couple of tests, so I don't think a separate work is needed. abort_request() is a simple wrapper around complete_request(), making it safe to call at any time -- replace it with complete_request() for now if you want to try this out.
Using one job for sending keepalive requests and completing stuck requests brings us to the need to check that osd_keepalive_timeout is not larger then osd_request_timeout. So we should not forget to say about it in the documentation. Is it worth creating a correlation between osd_keepalive_timeout and osd_request_timeout?
-- Artur -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
