]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
osdc/Objecter: only post_rx_buffer if no op timeout 2598/head
authorSage Weil <sage@redhat.com>
Thu, 25 Sep 2014 19:34:11 +0000 (12:34 -0700)
committerSage Weil <sage@redhat.com>
Thu, 25 Sep 2014 19:34:11 +0000 (12:34 -0700)
commit126d0b30e990519b8f845f99ba893fdcd56de447
tree68fe1c6558f840b7591c3252e29ca49452ed7a22
parent0115a55aa3a4e46e5c734b952000d509107e261e
osdc/Objecter: only post_rx_buffer if no op timeout

If we post an rx buffer and there is a timeout, the revocation can happen
while the reader has consumed the buffers but before it has decoded and
constructed the message.  In particular, we calculate a crc32c over the
data portion of the message after we've taken the buffers and dropped the
lock.

Instead of fixing this race (for example, by reverifying rx_buffers under
the lock while calculating the crc.. bleh), just skip the rx buffer
optimization entirely when a timeout is present.

Note that this doesn't cover the op_cancel() paths, but none of those users
provide static buffers to read into.

Fixes: #9582
Backport: firefly, dumpling
Signed-off-by: Sage Weil <sage@redhat.com>
src/osdc/Objecter.cc