diff options
author | Josh Durgin <josh.durgin@inktank.com> | 2013-12-02 19:11:48 -0800 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-03-31 09:58:12 -0700 |
commit | 4892ed8deb6989cd9c831156920bae490d6ad4d1 (patch) | |
tree | 2006deb862e0c92a441c58ce57bf2c413dc75e52 /include/dt-bindings | |
parent | 601cfbf0c6702f2fdc33f0dedf9632961318f8ff (diff) | |
download | linux-3.10-4892ed8deb6989cd9c831156920bae490d6ad4d1.tar.gz linux-3.10-4892ed8deb6989cd9c831156920bae490d6ad4d1.tar.bz2 linux-3.10-4892ed8deb6989cd9c831156920bae490d6ad4d1.zip |
libceph: block I/O when PAUSE or FULL osd map flags are set
commit d29adb34a94715174c88ca93e8aba955850c9bde upstream.
The PAUSEWR and PAUSERD flags are meant to stop the cluster from
processing writes and reads, respectively. The FULL flag is set when
the cluster determines that it is out of space, and will no longer
process writes. PAUSEWR and PAUSERD are purely client-side settings
already implemented in userspace clients. The osd does nothing special
with these flags.
When the FULL flag is set, however, the osd responds to all writes
with -ENOSPC. For cephfs, this makes sense, but for rbd the block
layer translates this into EIO. If a cluster goes from full to
non-full quickly, a filesystem on top of rbd will not behave well,
since some writes succeed while others get EIO.
Fix this by blocking any writes when the FULL flag is set in the osd
client. This is the same strategy used by userspace, so apply it by
default. A follow-on patch makes this configurable.
__map_request() is called to re-target osd requests in case the
available osds changed. Add a paused field to a ceph_osd_request, and
set it whenever an appropriate osd map flag is set. Avoid queueing
paused requests in __map_request(), but force them to be resent if
they become unpaused.
Also subscribe to the next osd map from the monitor if any of these
flags are set, so paused requests can be unblocked as soon as
possible.
Fixes: http://tracker.ceph.com/issues/6079
Reviewed-by: Sage Weil <sage@inktank.com>
Signed-off-by: Josh Durgin <josh.durgin@inktank.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'include/dt-bindings')
0 files changed, 0 insertions, 0 deletions