Browse Source

ceph: move xattr initialzation before the encoding past the ceph_mds_caps

Just for clarity. This part is inside the header, so it makes sense to
group it with the rest of the stuff in the header.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
Reviewed-by: Yan, Zheng <zyan@redhat.com>
Jeff Layton 8 years ago
parent
commit
9670079f5f
1 changed files with 7 additions and 7 deletions
  1. 7 7
      fs/ceph/caps.c

+ 7 - 7
fs/ceph/caps.c

@@ -1057,6 +1057,13 @@ static int send_cap_msg(struct ceph_mds_session *session,
 	fc->gid = cpu_to_le32(from_kgid(&init_user_ns, gid));
 	fc->gid = cpu_to_le32(from_kgid(&init_user_ns, gid));
 	fc->mode = cpu_to_le32(mode);
 	fc->mode = cpu_to_le32(mode);
 
 
+	fc->xattr_version = cpu_to_le64(xattr_version);
+	if (xattrs_buf) {
+		msg->middle = ceph_buffer_get(xattrs_buf);
+		fc->xattr_len = cpu_to_le32(xattrs_buf->vec.iov_len);
+		msg->hdr.middle_len = cpu_to_le32(xattrs_buf->vec.iov_len);
+	}
+
 	p = fc + 1;
 	p = fc + 1;
 	/* flock buffer size */
 	/* flock buffer size */
 	ceph_encode_32(&p, 0);
 	ceph_encode_32(&p, 0);
@@ -1069,13 +1076,6 @@ static int send_cap_msg(struct ceph_mds_session *session,
 	/* oldest_flush_tid */
 	/* oldest_flush_tid */
 	ceph_encode_64(&p, oldest_flush_tid);
 	ceph_encode_64(&p, oldest_flush_tid);
 
 
-	fc->xattr_version = cpu_to_le64(xattr_version);
-	if (xattrs_buf) {
-		msg->middle = ceph_buffer_get(xattrs_buf);
-		fc->xattr_len = cpu_to_le32(xattrs_buf->vec.iov_len);
-		msg->hdr.middle_len = cpu_to_le32(xattrs_buf->vec.iov_len);
-	}
-
 	ceph_con_send(&session->s_con, msg);
 	ceph_con_send(&session->s_con, msg);
 	return 0;
 	return 0;
 }
 }