diff options
author | Sage Weil <sage@newdream.net> | 2010-11-02 16:41:47 -0400 |
---|---|---|
committer | Sage Weil <sage@newdream.net> | 2011-01-12 18:15:13 -0500 |
commit | 4af25fdda6943f311a63034f80933e4d6d6e3a19 (patch) | |
tree | 8d0be03aedc4d3ea09ba0b8d3f1b63df4ef302e1 /fs/ceph/mds_client.c | |
parent | 14303d20f3ae3e6ab626c77a4aac202b3bafd377 (diff) |
ceph: drop redundant r_mds field
The r_mds field is redundant, since we can find the same information at
r_session->s_mds, and when r_session is NULL then r_mds is meaningless.
Signed-off-by: Sage Weil <sage@newdream.net>
Diffstat (limited to 'fs/ceph/mds_client.c')
-rw-r--r-- | fs/ceph/mds_client.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/fs/ceph/mds_client.c b/fs/ceph/mds_client.c index 9be29b06a2d9..e22e8b41d572 100644 --- a/fs/ceph/mds_client.c +++ b/fs/ceph/mds_client.c | |||
@@ -1705,7 +1705,6 @@ static int __prepare_send_request(struct ceph_mds_client *mdsc, | |||
1705 | struct ceph_msg *msg; | 1705 | struct ceph_msg *msg; |
1706 | int flags = 0; | 1706 | int flags = 0; |
1707 | 1707 | ||
1708 | req->r_mds = mds; | ||
1709 | req->r_attempts++; | 1708 | req->r_attempts++; |
1710 | if (req->r_inode) { | 1709 | if (req->r_inode) { |
1711 | struct ceph_cap *cap = | 1710 | struct ceph_cap *cap = |
@@ -2068,8 +2067,11 @@ static void handle_reply(struct ceph_mds_session *session, struct ceph_msg *msg) | |||
2068 | goto out; | 2067 | goto out; |
2069 | } else { | 2068 | } else { |
2070 | struct ceph_inode_info *ci = ceph_inode(req->r_inode); | 2069 | struct ceph_inode_info *ci = ceph_inode(req->r_inode); |
2071 | struct ceph_cap *cap = | 2070 | struct ceph_cap *cap = NULL; |
2072 | ceph_get_cap_for_mds(ci, req->r_mds);; | 2071 | |
2072 | if (req->r_session) | ||
2073 | cap = ceph_get_cap_for_mds(ci, | ||
2074 | req->r_session->s_mds); | ||
2073 | 2075 | ||
2074 | dout("already using auth"); | 2076 | dout("already using auth"); |
2075 | if ((!cap || cap != ci->i_auth_cap) || | 2077 | if ((!cap || cap != ci->i_auth_cap) || |