Whamcloud - gitweb
merge b_md onto HEAD. as best as I can remember:
[fs/lustre-release.git] / lustre / lib / mds_updates.c
index 645679c..8da9f39 100644 (file)
@@ -1,11 +1,14 @@
-/*
- * Lustre Light Update Records
- *
- * This code is issued under the GNU General Public License.
- * See the file COPYING in this distribution
+/* -*- mode: c; c-basic-offset: 8; indent-tabs-mode: nil; -*-
+ * vim:expandtab:shiftwidth=8:tabstop=8:
  *
  * Copryright (C) 2002 Cluster File Systems, Inc.
  *
+ *   This file is part of Lustre, http://www.sf.net/projects/lustre/
+ *
+ *   This code is issued under the GNU General Public License.
+ *   See the file COPYING in this distribution
+ *
+ * Lustre Lite Update Records
  */
 
 #include <linux/config.h>
 #include <linux/string.h>
 #include <linux/stat.h>
 #include <linux/errno.h>
-#include <linux/locks.h>
+#if (LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0))
+#include <linux/locks.h>   // for wait_on_buffer
+#else 
+#include <linux/buffer_head.h>   // for wait_on_buffer
+#endif
 #include <linux/unistd.h>
 
 #include <asm/system.h>
 #include <linux/obd_support.h>
 #include <linux/lustre_lib.h>
 #include <linux/lustre_mds.h>
+#include <linux/lustre_lite.h>
 
-/* packing of MDS records */
-void mds_create_pack(struct mds_rec_create *rec, struct inode *inode, const char *name, int namelen, __u32 mode, __u64 id, __u32 uid, __u32 gid, __u64 time, const char *tgt, int tgtlen)
+void mds_pack_inode2fid(struct ll_fid *fid, struct inode *inode)
 {
-       char *tmp = (char *)rec + sizeof(*rec); 
-       /* XXX do something about time, uid, gid */
-       rec->cr_reclen = 
-               HTON__u32(sizeof(*rec) + size_round0(namelen) + 
-                         size_round0(tgtlen));
-       rec->cr_opcode = HTON__u32(REINT_CREATE);
-
-       ll_inode2fid(&rec->cr_fid, inode); 
-       rec->cr_mode = HTON__u32(mode);
-       rec->cr_id = HTON__u64(id);
-       rec->cr_uid = HTON__u32(uid);
-       rec->cr_gid = HTON__u32(gid);
-       rec->cr_time = HTON__u64(time);
-       rec->cr_namelen = HTON__u32(namelen + 1); /* for terminating \0 */ 
-       LOGL0(name, namelen, tmp); 
-       if (tgt) { 
-               rec->cr_tgtlen = HTON__u32(tgtlen + 1); 
-               LOGL0(tgt, tgtlen, tmp); 
-       }
+        fid->id = HTON__u64(inode->i_ino);
+        fid->generation = HTON__u32(inode->i_generation);
+        fid->f_type = HTON__u32(S_IFMT & inode->i_mode);
 }
 
 
-void mds_setattr_pack(struct mds_rec_setattr *rec, struct inode *inode, struct iattr *iattr)
+void mds_pack_inode2body(struct mds_body *b, struct inode *inode)
 {
-       rec->sa_reclen = HTON__u32(sizeof(*rec));
-       rec->sa_opcode = HTON__u32(REINT_SETATTR);
-
-       ll_inode2fid(&rec->sa_fid, inode); 
-       rec->sa_valid = HTON__u32(iattr->ia_valid);
-       rec->sa_mode = HTON__u32(iattr->ia_mode);
-       rec->sa_uid = HTON__u32(iattr->ia_uid);
-       rec->sa_gid = HTON__u32(iattr->ia_gid);
-       rec->sa_size = HTON__u64(iattr->ia_size);
-       rec->sa_atime = HTON__u64(iattr->ia_atime);
-       rec->sa_mtime = HTON__u64(iattr->ia_mtime);
-       rec->sa_ctime = HTON__u64(iattr->ia_ctime);
-       rec->sa_attr_flags = HTON__u32(iattr->ia_attr_flags);
+        b->valid = OBD_MD_FLID | OBD_MD_FLATIME | OBD_MD_FLMTIME |
+                OBD_MD_FLCTIME | OBD_MD_FLSIZE | OBD_MD_FLUID | OBD_MD_FLGID |
+                OBD_MD_FLTYPE | OBD_MD_FLMODE | OBD_MD_FLNLINK | OBD_MD_FLGENER;
+        b->ino = HTON__u32(inode->i_ino);
+        b->atime = HTON__u32(inode->i_atime);
+        b->mtime = HTON__u32(inode->i_mtime);
+        b->ctime = HTON__u32(inode->i_ctime);
+        b->mode = HTON__u32(inode->i_mode);
+        b->size = HTON__u64(inode->i_size);
+        b->uid = HTON__u32(inode->i_uid);
+        b->gid = HTON__u32(inode->i_gid);
+        b->flags = HTON__u32(inode->i_flags);
+        b->rdev = HTON__u32(b->rdev);
+        b->nlink = HTON__u32(inode->i_nlink);
+        b->generation = HTON__u32(inode->i_generation);
 }
 
-void mds_unlink_pack(struct mds_rec_unlink *rec, 
-                    struct inode *inode, const char *name, int namelen)
+
+void mds_pack_fid(struct ll_fid *fid)
 {
-       char *tmp = (char *)rec + sizeof(*rec); 
+        fid->id = HTON__u64(fid->id);
+        fid->generation = HTON__u32(fid->generation);
+        fid->f_type = HTON__u32(fid->f_type);
+}
 
-       rec->ul_reclen = HTON__u32(sizeof(*rec)) + size_round0(namelen);
-       rec->ul_opcode = HTON__u32(REINT_UNLINK);
+static void mds_pack_body(struct mds_body *b)
+{
+        if (b == NULL)
+                LBUG();
+
+        b->fsuid = HTON__u32(current->fsuid);
+        b->fsgid = HTON__u32(current->fsgid);
+        b->capability = HTON__u32(current->cap_effective);
+
+        mds_pack_fid(&b->fid1);
+        mds_pack_fid(&b->fid2);
+        b->size = HTON__u64(b->size);
+        b->ino = HTON__u32(b->ino);
+        b->valid = HTON__u32(b->valid);
+        b->mode = HTON__u32(b->mode);
+        b->uid = HTON__u32(b->uid);
+        b->gid = HTON__u32(b->gid);
+        b->mtime = HTON__u32(b->mtime);
+        b->ctime = HTON__u32(b->ctime);
+        b->atime = HTON__u32(b->atime);
+        b->flags = HTON__u32(b->flags);
+        b->rdev = HTON__u32(b->rdev);
+        b->nlink = HTON__u32(b->nlink);
+        b->generation = HTON__u32(b->generation);
+}
 
-       ll_inode2fid(&rec->ul_fid1, inode); 
-       rec->ul_namelen = HTON__u32(namelen + 1); /* for terminating \0 */ 
-       LOGL0(name, namelen, tmp); 
+void mds_getattr_pack(struct ptlrpc_request *req, int offset,
+                      struct inode *inode,
+                      const char *name, int namelen)
+{
+        struct mds_body *b;
+        b = lustre_msg_buf(req->rq_reqmsg, offset);
+
+        b->fsuid = HTON__u32(current->fsuid);
+        b->fsgid = HTON__u32(current->fsgid);
+        b->capability = HTON__u32(current->cap_effective);
+
+        ll_inode2fid(&b->fid1, inode);
+        if (name) {
+                char *tmp;
+                tmp = lustre_msg_buf(req->rq_reqmsg, offset + 1);
+                LOGL0(name, namelen, tmp);
+        }
 }
 
-void mds_link_pack(struct mds_rec_link *rec, 
-                    struct inode *inode, struct inode *dir,
-                    const char *name, int namelen)
+void mds_readdir_pack(struct ptlrpc_request *req, int offset,
+                      obd_id ino, int type)
 {
-       char *tmp = (char *)rec + sizeof(*rec); 
-       rec->lk_reclen = HTON__u32(sizeof(*rec)) + size_round0(namelen);
-       rec->lk_opcode = HTON__u32(REINT_LINK);
-
-       ll_inode2fid(&rec->lk_fid1, inode); 
-       ll_inode2fid(&rec->lk_fid2, dir); 
-       rec->lk_namelen = HTON__u32(namelen + 1); /* for terminating \0 */ 
-       LOGL0(name, namelen, tmp); 
+        struct mds_body *b;
+
+        b = lustre_msg_buf(req->rq_reqmsg, offset);
+        b->fsuid = HTON__u32(current->fsuid);
+        b->fsgid = HTON__u32(current->fsgid);
+        b->capability = HTON__u32(current->cap_effective);
+        b->fid1.id = HTON__u64(ino);
+        b->fid1.f_type = HTON__u32(type);
+        b->size = HTON__u64(offset);
 }
 
-void mds_rename_pack(struct mds_rec_rename *rec, struct inode *srcdir, struct inode *tgtdir, const char *name, int namelen, const char *tgt, int tgtlen)
+
+void mds_pack_req_body(struct ptlrpc_request *req)
 {
-       char *tmp = (char *)rec + sizeof(*rec); 
-       /* XXX do something about time, uid, gid */
-       rec->rn_reclen = 
-               HTON__u32(sizeof(*rec) + size_round0(namelen) + 
-                         size_round0(tgtlen));
-       rec->rn_opcode = HTON__u32(REINT_RENAME);
-
-       ll_inode2fid(&rec->rn_fid1, srcdir); 
-       ll_inode2fid(&rec->rn_fid2, tgtdir); 
-       rec->rn_namelen = HTON__u32(namelen + 1); /* for terminating \0 */ 
-       LOGL0(name, namelen, tmp); 
-       if (tgt) { 
-               rec->rn_tgtlen = HTON__u32(tgtlen + 1); 
-               LOGL0(tgt, tgtlen, tmp); 
-       }
+        struct mds_body *b = lustre_msg_buf(req->rq_reqmsg, 0);
+        mds_pack_body(b);
+}
+
+void mds_pack_rep_body(struct ptlrpc_request *req)
+{
+        struct mds_body *b = lustre_msg_buf(req->rq_repmsg, 0);
+        mds_pack_body(b);
 }
 
-/* unpacking */
 
-static int mds_update_hdr_unpack(char *buf, int len,
-                                 struct mds_update_record *r)
+/* packing of MDS records */
+void mds_create_pack(struct ptlrpc_request *req, int offset, struct inode *dir,
+                     __u32 mode, __u64 rdev, __u32 uid, __u32 gid, __u64 time,
+                     const char *name, int namelen,
+                     const void *data, int datalen)
 {
-       struct mds_update_record_hdr *hdr = (struct mds_update_record_hdr *)buf;
-       
-       r->ur_reclen = NTOH__u32(hdr->ur_reclen);
-       if (len < sizeof(*hdr) || len != r->ur_reclen) { 
-               CERROR("invalid buffer length\n"); 
-               return -EFAULT;
-       }
-       r->ur_opcode = NTOH__u32(hdr->ur_opcode); 
-       return 0;
+        struct mds_rec_create *rec;
+        char *tmp;
+        rec = lustre_msg_buf(req->rq_reqmsg, offset);
+
+        /* XXX do something about time, uid, gid */
+        rec->cr_opcode = HTON__u32(REINT_CREATE);
+        rec->cr_fsuid = HTON__u32(current->fsuid);
+        rec->cr_fsgid = HTON__u32(current->fsgid);
+        rec->cr_cap = HTON__u32(current->cap_effective);
+        ll_inode2fid(&rec->cr_fid, dir);
+        memset(&rec->cr_replayfid, 0, sizeof(rec->cr_replayfid));
+        rec->cr_mode = HTON__u32(mode);
+        rec->cr_rdev = HTON__u64(rdev);
+        rec->cr_uid = HTON__u32(uid);
+        rec->cr_gid = HTON__u32(gid);
+        rec->cr_time = HTON__u64(time);
+
+        tmp = lustre_msg_buf(req->rq_reqmsg, offset + 1);
+        LOGL0(name, namelen, tmp);
+
+        if (data) {
+                tmp = lustre_msg_buf(req->rq_reqmsg, offset + 2);
+                LOGL0(data, datalen, tmp);
+        }
 }
 
-static int mds_setattr_unpack(char *buf, int len, struct mds_update_record *r)
+void mds_setattr_pack(struct ptlrpc_request *req, int offset,
+                      struct inode *inode, struct iattr *iattr,
+                      const char *name, int namelen)
 {
+        struct mds_rec_setattr *rec;
+        rec = lustre_msg_buf(req->rq_reqmsg, offset);
+
+        rec->sa_opcode = HTON__u32(REINT_SETATTR);
+        rec->sa_fsuid = HTON__u32(current->fsuid);
+        rec->sa_fsgid = HTON__u32(current->fsgid);
+        rec->sa_cap = HTON__u32(current->cap_effective);
+        ll_inode2fid(&rec->sa_fid, inode);
+        rec->sa_valid = HTON__u32(iattr->ia_valid);
+        rec->sa_mode = HTON__u32(iattr->ia_mode);
+        rec->sa_uid = HTON__u32(iattr->ia_uid);
+        rec->sa_gid = HTON__u32(iattr->ia_gid);
+        rec->sa_size = HTON__u64(iattr->ia_size);
+        rec->sa_atime = HTON__u64(iattr->ia_atime);
+        rec->sa_mtime = HTON__u64(iattr->ia_mtime);
+        rec->sa_ctime = HTON__u64(iattr->ia_ctime);
+        rec->sa_attr_flags = HTON__u32(iattr->ia_attr_flags);
+
+        if (namelen) {
+                char *tmp;
+                tmp = lustre_msg_buf(req->rq_reqmsg, offset + 1);
+                LOGL0(name, namelen, tmp);
+        }
+}
 
-       struct iattr *attr = &r->ur_iattr;
-       struct mds_rec_setattr *rec = (struct mds_rec_setattr *)buf; 
-
-       if (len < sizeof(*rec)) { 
-               CERROR("invalid buffer length\n"); 
-               return -EFAULT;
-       }
-
-       r->ur_fid1 = &rec->sa_fid; 
-       attr->ia_valid = NTOH__u32(rec->sa_valid);
-       attr->ia_mode = NTOH__u32(rec->sa_mode);
-       attr->ia_uid = NTOH__u32(rec->sa_uid);
-       attr->ia_gid = NTOH__u32(rec->sa_gid);
-       attr->ia_size = NTOH__u64(rec->sa_size);
-       attr->ia_atime = NTOH__u64(rec->sa_atime);
-       attr->ia_mtime = NTOH__u64(rec->sa_mtime);
-       attr->ia_ctime = NTOH__u64(rec->sa_ctime);
-       attr->ia_attr_flags = NTOH__u32(rec->sa_attr_flags);
-       return 0; 
+void mds_unlink_pack(struct ptlrpc_request *req, int offset,
+                     struct inode *inode, struct inode *child, __u32 mode,
+                     const char *name, int namelen)
+{
+        struct mds_rec_unlink *rec;
+        char *tmp;
+
+        rec = lustre_msg_buf(req->rq_reqmsg, offset);
+
+        rec->ul_opcode = HTON__u32(REINT_UNLINK);
+        rec->ul_fsuid = HTON__u32(current->fsuid);
+        rec->ul_fsgid = HTON__u32(current->fsgid);
+        rec->ul_cap = HTON__u32(current->cap_effective);
+        rec->ul_mode = HTON__u32(mode);
+        ll_inode2fid(&rec->ul_fid1, inode);
+        if (child)
+                ll_inode2fid(&rec->ul_fid2, child);
+
+        tmp = lustre_msg_buf(req->rq_reqmsg, offset + 1);
+        LOGL0(name, namelen, tmp);
 }
 
-static int mds_create_unpack(char *buf, int len, struct mds_update_record *r)
+void mds_link_pack(struct ptlrpc_request *req, int offset,
+                   struct inode *inode, struct inode *dir,
+                   const char *name, int namelen)
 {
-       struct mds_rec_create *rec = (struct mds_rec_create *)buf; 
-       char *ptr, *end;
-
-       if (len < sizeof(*rec)) { 
-               CERROR("invalid buffer length\n"); 
-               return -EFAULT;
-       }
-       
-       ptr = (char *)rec + sizeof(*rec); 
-       end = ptr + len - sizeof(*rec); 
-       
-       r->ur_fid1 = &rec->cr_fid;
-       r->ur_mode = NTOH__u32(rec->cr_mode);
-       r->ur_id = NTOH__u64(rec->cr_id);
-       r->ur_uid = NTOH__u32(rec->cr_uid);
-       r->ur_gid = NTOH__u32(rec->cr_gid);
-       r->ur_time = NTOH__u64(rec->cr_time);
-       r->ur_namelen = NTOH__u32(rec->cr_namelen);
-       r->ur_tgtlen = NTOH__u32(rec->cr_tgtlen);
-
-       UNLOGL0(r->ur_name, char, r->ur_namelen, ptr, end); 
-       UNLOGL0(r->ur_tgt, char, r->ur_tgtlen, ptr, end);
-       return 0;
+        struct mds_rec_link *rec;
+        char *tmp;
+
+        rec = lustre_msg_buf(req->rq_reqmsg, offset);
+
+        rec->lk_opcode = HTON__u32(REINT_LINK);
+        rec->lk_fsuid = HTON__u32(current->fsuid);
+        rec->lk_fsgid = HTON__u32(current->fsgid);
+        rec->lk_cap = HTON__u32(current->cap_effective);
+        ll_inode2fid(&rec->lk_fid1, inode);
+        ll_inode2fid(&rec->lk_fid2, dir);
+
+        tmp = lustre_msg_buf(req->rq_reqmsg, offset + 1);
+        LOGL0(name, namelen, tmp);
 }
 
-static int mds_link_unpack(char *buf, int len, struct mds_update_record *r)
+void mds_rename_pack(struct ptlrpc_request *req, int offset,
+                     struct inode *srcdir, struct inode *tgtdir,
+                     const char *old, int oldlen, const char *new, int newlen)
 {
-       struct mds_rec_link *rec = (struct mds_rec_link *)buf; 
-       char *ptr, *end;
-
-       if (len < sizeof(*rec)) { 
-               CERROR("invalid buffer length\n"); 
-               return -EFAULT;
-       }
-       
-       ptr = (char *)rec + sizeof(*rec); 
-       end = ptr + len - sizeof(*rec); 
-       
-       r->ur_fid1 = &rec->lk_fid1;
-       r->ur_fid2 = &rec->lk_fid2;
-       r->ur_namelen = NTOH__u32(rec->lk_namelen);
-       UNLOGL0(r->ur_name, char, r->ur_namelen, ptr, end); 
-       return 0;
+        struct mds_rec_rename *rec;
+        char *tmp;
+
+        rec = lustre_msg_buf(req->rq_reqmsg, offset);
+
+        /* XXX do something about time, uid, gid */
+        rec->rn_opcode = HTON__u32(REINT_RENAME);
+        rec->rn_fsuid = HTON__u32(current->fsuid);
+        rec->rn_fsgid = HTON__u32(current->fsgid);
+        rec->rn_cap = HTON__u32(current->cap_effective);
+        ll_inode2fid(&rec->rn_fid1, srcdir);
+        ll_inode2fid(&rec->rn_fid2, tgtdir);
+
+        tmp = lustre_msg_buf(req->rq_reqmsg, offset + 1);
+        LOGL0(old, oldlen, tmp);
+
+        if (new) {
+                tmp = lustre_msg_buf(req->rq_reqmsg, offset + 2);
+                LOGL0(new, newlen, tmp);
+        }
 }
 
+/* unpacking */
+void mds_unpack_fid(struct ll_fid *fid)
+{
+        fid->id = NTOH__u64(fid->id);
+        fid->generation = NTOH__u32(fid->generation);
+        fid->f_type = NTOH__u32(fid->f_type);
+}
+
+void mds_unpack_body(struct mds_body *b)
+{
+        if (b == NULL)
+                LBUG();
+
+        mds_unpack_fid(&b->fid1);
+        mds_unpack_fid(&b->fid2);
+        b->size = NTOH__u64(b->size);
+        b->valid = NTOH__u32(b->valid);
+        b->fsuid = NTOH__u32(b->fsuid);
+        b->fsgid = NTOH__u32(b->fsgid);
+        b->capability = NTOH__u32(b->capability);
+        b->ino = NTOH__u32(b->ino);
+        b->mode = NTOH__u32(b->mode);
+        b->uid = NTOH__u32(b->uid);
+        b->gid = NTOH__u32(b->gid);
+        b->mtime = NTOH__u32(b->mtime);
+        b->ctime = NTOH__u32(b->ctime);
+        b->atime = NTOH__u32(b->atime);
+        b->flags = NTOH__u32(b->flags);
+        b->rdev = NTOH__u32(b->rdev);
+        b->nlink = NTOH__u32(b->nlink);
+        b->generation = NTOH__u32(b->generation);
+}
 
-static int mds_unlink_unpack(char *buf, int len, struct mds_update_record *r)
+static int mds_setattr_unpack(struct ptlrpc_request *req, int offset,
+                              struct mds_update_record *r)
 {
-       struct mds_rec_unlink *rec = (struct mds_rec_unlink *)buf; 
-       char *ptr, *end;
-       ENTRY;
-
-       if (len < sizeof(*rec)) { 
-               CERROR("invalid buffer length\n"); 
-               return -EFAULT;
-       }
-       
-       ptr = (char *)rec + sizeof(*rec); 
-       end = ptr + len - sizeof(*rec); 
-       
-       r->ur_fid1 = &rec->ul_fid1;
-       r->ur_namelen = NTOH__u32(rec->ul_namelen);
-       UNLOGL0(r->ur_name, char, r->ur_namelen, ptr, end); 
-       EXIT;
-       return 0;
+        struct iattr *attr = &r->ur_iattr;
+        struct mds_rec_setattr *rec = lustre_msg_buf(req->rq_reqmsg, offset);
+        ENTRY;
+
+        if (req->rq_reqmsg->bufcount < offset + 1 ||
+            req->rq_reqmsg->buflens[offset] != sizeof(*rec))
+                RETURN(-EFAULT);
+
+        r->ur_fsuid = NTOH__u32(rec->sa_fsuid);
+        r->ur_fsgid = NTOH__u32(rec->sa_fsgid);
+        r->ur_cap = NTOH__u32(rec->sa_cap);
+        r->ur_fid1 = &rec->sa_fid;
+        attr->ia_valid = NTOH__u32(rec->sa_valid);
+        attr->ia_mode = NTOH__u32(rec->sa_mode);
+        attr->ia_uid = NTOH__u32(rec->sa_uid);
+        attr->ia_gid = NTOH__u32(rec->sa_gid);
+        attr->ia_size = NTOH__u64(rec->sa_size);
+        attr->ia_atime = NTOH__u64(rec->sa_atime);
+        attr->ia_mtime = NTOH__u64(rec->sa_mtime);
+        attr->ia_ctime = NTOH__u64(rec->sa_ctime);
+        attr->ia_attr_flags = NTOH__u32(rec->sa_attr_flags);
+
+        if (req->rq_reqmsg->bufcount == offset + 2) {
+                r->ur_namelen = req->rq_reqmsg->buflens[offset + 1];
+                r->ur_name = lustre_msg_buf(req->rq_reqmsg, offset + 1);
+        } else
+                r->ur_namelen = 0;
+
+        RETURN(0);
 }
 
-static int mds_rename_unpack(char *buf, int len, struct mds_update_record *r)
+static int mds_create_unpack(struct ptlrpc_request *req, int offset,
+                             struct mds_update_record *r)
 {
-       struct mds_rec_rename *rec = (struct mds_rec_rename *)buf; 
-       char *ptr, *end;
-
-       if (len < sizeof(*rec)) { 
-               CERROR("invalid buffer length\n"); 
-               return -EFAULT;
-       }
-       
-       ptr = (char *)rec + sizeof(*rec); 
-       end = ptr + len - sizeof(*rec); 
-       
-       r->ur_fid1 = &rec->rn_fid1;
-       r->ur_fid2 = &rec->rn_fid2;
-       r->ur_namelen = NTOH__u32(rec->rn_namelen);
-       r->ur_tgtlen = NTOH__u32(rec->rn_tgtlen);
-
-       UNLOGL0(r->ur_name, char, r->ur_namelen, ptr, end); 
-       UNLOGL0(r->ur_tgt, char, r->ur_tgtlen, ptr, end);
-       return 0;
+        struct mds_rec_create *rec = lustre_msg_buf(req->rq_reqmsg, offset);
+        ENTRY;
+
+        if (req->rq_reqmsg->bufcount < offset + 2 ||
+            req->rq_reqmsg->buflens[offset] != sizeof(*rec))
+                RETURN(-EFAULT);
+
+        r->ur_fsuid = NTOH__u32(rec->cr_fsuid);
+        r->ur_fsgid = NTOH__u32(rec->cr_fsgid);
+        r->ur_cap = NTOH__u32(rec->cr_cap);
+        r->ur_fid1 = &rec->cr_fid;
+        r->ur_fid2 = &rec->cr_replayfid;
+        r->ur_mode = NTOH__u32(rec->cr_mode);
+        r->ur_rdev = NTOH__u64(rec->cr_rdev);
+        r->ur_uid = NTOH__u32(rec->cr_uid);
+        r->ur_gid = NTOH__u32(rec->cr_gid);
+        r->ur_time = NTOH__u64(rec->cr_time);
+
+        r->ur_name = lustre_msg_buf(req->rq_reqmsg, offset + 1);
+        r->ur_namelen = req->rq_reqmsg->buflens[offset + 1];
+
+        if (req->rq_reqmsg->bufcount == offset + 3) {
+                r->ur_tgt = lustre_msg_buf(req->rq_reqmsg, offset + 2);
+                r->ur_tgtlen = req->rq_reqmsg->buflens[offset + 2];
+        } else {
+                r->ur_tgt = NULL;
+                r->ur_tgtlen = 0;
+        }
+        RETURN(0);
 }
 
-typedef int (*update_unpacker)(char *, int , struct mds_update_record *); 
+static int mds_link_unpack(struct ptlrpc_request *req, int offset,
+                           struct mds_update_record *r)
+{
+        struct mds_rec_link *rec = lustre_msg_buf(req->rq_reqmsg, offset);
+        ENTRY;
+
+        if (req->rq_reqmsg->bufcount != offset + 2 ||
+            req->rq_reqmsg->buflens[offset] != sizeof(*rec))
+                RETURN(-EFAULT);
+
+        r->ur_fsuid = NTOH__u32(rec->lk_fsuid);
+        r->ur_fsgid = NTOH__u32(rec->lk_fsgid);
+        r->ur_cap = NTOH__u32(rec->lk_cap);
+        r->ur_fid1 = &rec->lk_fid1;
+        r->ur_fid2 = &rec->lk_fid2;
+
+        r->ur_name = lustre_msg_buf(req->rq_reqmsg, offset + 1);
+        r->ur_namelen = req->rq_reqmsg->buflens[offset + 1];
+        RETURN(0);
+}
+
+static int mds_unlink_unpack(struct ptlrpc_request *req, int offset,
+                             struct mds_update_record *r)
+{
+        struct mds_rec_unlink *rec = lustre_msg_buf(req->rq_reqmsg, offset);
+        ENTRY;
+
+        if (req->rq_reqmsg->bufcount != offset + 2 ||
+            req->rq_reqmsg->buflens[offset] != sizeof(*rec))
+                RETURN(-EFAULT);
+
+        r->ur_fsuid = NTOH__u32(rec->ul_fsuid);
+        r->ur_fsgid = NTOH__u32(rec->ul_fsgid);
+        r->ur_cap = NTOH__u32(rec->ul_cap);
+        r->ur_mode = NTOH__u32(rec->ul_mode);
+        r->ur_fid1 = &rec->ul_fid1;
+        r->ur_fid2 = &rec->ul_fid2;
+
+        r->ur_name = lustre_msg_buf(req->rq_reqmsg, offset + 1);
+        r->ur_namelen = req->rq_reqmsg->buflens[offset + 1];
+        RETURN(0);
+}
+
+static int mds_rename_unpack(struct ptlrpc_request *req, int offset,
+                             struct mds_update_record *r)
+{
+        struct mds_rec_rename *rec = lustre_msg_buf(req->rq_reqmsg, offset);
+        ENTRY;
+
+        if (req->rq_reqmsg->bufcount != offset + 3 ||
+            req->rq_reqmsg->buflens[offset] != sizeof(*rec))
+                RETURN(-EFAULT);
+
+        r->ur_fsuid = NTOH__u32(rec->rn_fsuid);
+        r->ur_fsgid = NTOH__u32(rec->rn_fsgid);
+        r->ur_cap = NTOH__u32(rec->rn_cap);
+        r->ur_fid1 = &rec->rn_fid1;
+        r->ur_fid2 = &rec->rn_fid2;
+
+        r->ur_name = lustre_msg_buf(req->rq_reqmsg, offset + 1);
+        r->ur_namelen = req->rq_reqmsg->buflens[offset + 1];
+
+        r->ur_tgt = lustre_msg_buf(req->rq_reqmsg, offset + 2);
+        r->ur_tgtlen = req->rq_reqmsg->buflens[offset + 2];
+        RETURN(0);
+}
+
+typedef int (*update_unpacker)(struct ptlrpc_request *req, int offset,
+                               struct mds_update_record *r);
 
 static update_unpacker mds_unpackers[REINT_MAX + 1] = {
-       [REINT_SETATTR] mds_setattr_unpack,     
+        [REINT_SETATTR] mds_setattr_unpack,
         [REINT_CREATE] mds_create_unpack,
         [REINT_LINK] mds_link_unpack,
         [REINT_UNLINK] mds_unlink_unpack,
         [REINT_RENAME] mds_rename_unpack,
 };
 
-int mds_update_unpack(char *buf, int len, struct mds_update_record *r)
+int mds_update_unpack(struct ptlrpc_request *req, int offset,
+                      struct mds_update_record *rec)
 {
-       int rc; 
-       ENTRY;
-
-       rc = mds_update_hdr_unpack(buf, len, r);
-
-       if (rc) { 
-               EXIT;
-               return -EFAULT;
-       }
-
-       if ( r->ur_opcode<0 || r->ur_opcode > REINT_MAX) { 
-               EXIT;
-               return EFAULT; 
-       }
-       
-       rc = mds_unpackers[r->ur_opcode](buf, len, r);
-       EXIT;
-       return rc;
+        __u32 *opcode = lustre_msg_buf(req->rq_reqmsg, offset);
+        int rc, realop;
+        ENTRY;
+
+        if (!opcode || req->rq_reqmsg->buflens[offset] < sizeof(*opcode))
+                RETURN(-EFAULT);
+
+        realop = rec->ur_opcode = NTOH__u32(*opcode);
+        realop &= REINT_OPCODE_MASK;
+
+        if (realop < 0 || realop > REINT_MAX)
+                RETURN(-EFAULT);
+
+        rc = mds_unpackers[realop](req, offset, rec);
+        RETURN(rc);
 }