diff options
author | Sage Weil <sage@newdream.net> | 2010-03-01 16:34:17 -0800 |
---|---|---|
committer | Sage Weil <sage@newdream.net> | 2010-03-01 16:34:17 -0800 |
commit | d0974d1b97b70b181cffa76c46a363955b45e467 (patch) | |
tree | 950fd1c5e0c00d9083ec9a7730fb232a2bd5a734 /src/TODO | |
parent | 73f49ad1d43d7ae920e5814fca38f9300e236e97 (diff) | |
download | ceph-d0974d1b97b70b181cffa76c46a363955b45e467.tar.gz |
mds: put forced open sessions in OPENING then OPEN
We use OPENING state to indicate sessions that are being
imported. Fix get_or_add_open_session() to NOT set the session
state (except to STATE_NEW if new) so that the caller can do
the right thing. Otherwise, the prepare_force_open_sessions()
can't tell if it just forced open a session (and needs it to be
OPENING) or if it was already open. Subsequently cap migrations
weren't working if the client didn't already have a session
open.
There is still a bug: if the import aborts, we have an OPENING
session with no actual open client_session message queued. Maybe
we should have a different state instead of OPENING... IMPORTING?
Diffstat (limited to 'src/TODO')
-rw-r--r-- | src/TODO | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -60,6 +60,8 @@ filestore - need an osdmap cache layer? bugs +- mds prepare_force_open_sessions, then import aborts.. session is still OPENING but no client_session is sent... +- multimds: untar kernel, control-z, sync - wget mismatch with multiple mds? - rm -r failure (on kernel tree) - dbench 1, restart mds (may take a few times), dbench will error out. |