Whamcloud - gitweb
LU-5577 mdc: fix comparison between signed and unsigned 79/11379/17
authorDmitry Eremin <dmitry.eremin@intel.com>
Wed, 29 Oct 2014 11:37:08 +0000 (14:37 +0300)
committerOleg Drokin <oleg.drokin@intel.com>
Tue, 4 Nov 2014 17:59:41 +0000 (17:59 +0000)
Change type of client_obd->*_mds_*size from int to __u32 and
argumanets of related create/rename/setattr functions.
Change type of op_data->op_namelen to size_t.
Change type of argument size for all mdc_*_pack() to size_t.

Signed-off-by: Dmitry Eremin <dmitry.eremin@intel.com>
Change-Id: I97f4fa6928c24fa416c334206c75f9885266b1ae
Reviewed-on: http://review.whamcloud.com/11379
Tested-by: Jenkins
Reviewed-by: John L. Hammond <john.hammond@intel.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/include/obd.h
lustre/include/obd_class.h
lustre/llite/llite_internal.h
lustre/llite/llite_lib.c
lustre/lmv/lmv_intent.c
lustre/lmv/lmv_obd.c
lustre/mdc/mdc_internal.h
lustre/mdc/mdc_lib.c
lustre/mdc/mdc_locks.c
lustre/mdc/mdc_reint.c
lustre/mdc/mdc_request.c

index 61cb703..a4bc0cf 100644 (file)
@@ -228,10 +228,10 @@ struct client_obd {
         int                      cl_conn_count;
        /* max_mds_easize is purely a performance thing so we don't have to
         * call obd_size_diskmd() all the time. */
-       int                      cl_default_mds_easize;
-       int                      cl_max_mds_easize;
-       int                      cl_default_mds_cookiesize;
-       int                      cl_max_mds_cookiesize;
+       __u32                    cl_default_mds_easize;
+       __u32                    cl_max_mds_easize;
+       __u32                    cl_default_mds_cookiesize;
+       __u32                    cl_max_mds_cookiesize;
 
         enum lustre_sec_part     cl_sp_me;
         enum lustre_sec_part     cl_sp_to;
@@ -811,7 +811,7 @@ struct md_op_data {
         struct lustre_handle    op_handle;
         obd_time                op_mod_time;
         const char             *op_name;
-        int                     op_namelen;
+       size_t                  op_namelen;
         __u32                   op_mode;
         struct lmv_stripe_md   *op_mea1;
         struct lmv_stripe_md   *op_mea2;
@@ -1049,8 +1049,8 @@ struct md_ops {
                       struct md_open_data *, struct ptlrpc_request **);
 
        int (*m_create)(struct obd_export *, struct md_op_data *,
-                       const void *, int, int, __u32, __u32, cfs_cap_t,
-                       __u64, struct ptlrpc_request **);
+                       const void *, size_t, umode_t, uid_t, gid_t,
+                       cfs_cap_t, __u64, struct ptlrpc_request **);
 
        int (*m_enqueue)(struct obd_export *, struct ldlm_enqueue_info *,
                         const union ldlm_policy_data *,
@@ -1069,11 +1069,11 @@ struct md_ops {
                      struct ptlrpc_request **);
 
        int (*m_rename)(struct obd_export *, struct md_op_data *,
-                       const char *, int, const char *, int,
+                       const char *, size_t, const char *, size_t,
                        struct ptlrpc_request **);
 
        int (*m_setattr)(struct obd_export *, struct md_op_data *, void *,
-                        int , void *, int, struct ptlrpc_request **,
+                       size_t , void *, size_t, struct ptlrpc_request **,
                         struct md_open_data **mod);
 
        int (*m_fsync)(struct obd_export *, const struct lu_fid *,
@@ -1119,7 +1119,7 @@ struct md_ops {
        int (*m_getattr_name)(struct obd_export *, struct md_op_data *,
                              struct ptlrpc_request **);
 
-       int (*m_init_ea_size)(struct obd_export *, int, int, int, int);
+       int (*m_init_ea_size)(struct obd_export *, __u32, __u32, __u32, __u32);
 
        int (*m_get_lustre_md)(struct obd_export *, struct ptlrpc_request *,
                               struct obd_export *, struct obd_export *,
@@ -1239,7 +1239,8 @@ static inline const char *lu_dev_name(const struct lu_device *lu_dev)
         return lu_dev->ld_obd->obd_name;
 }
 
-static inline bool filename_is_volatile(const char *name, int namelen, int *idx)
+static inline bool filename_is_volatile(const char *name, size_t namelen,
+                                       int *idx)
 {
        const char      *start;
        char            *end;
index 0d3a943..cf29104 100644 (file)
@@ -1503,9 +1503,9 @@ static inline int md_close(struct obd_export *exp, struct md_op_data *op_data,
 }
 
 static inline int md_create(struct obd_export *exp, struct md_op_data *op_data,
-                            const void *data, int datalen, int mode, __u32 uid,
-                            __u32 gid, cfs_cap_t cap_effective, __u64 rdev,
-                            struct ptlrpc_request **request)
+                           const void *data, size_t datalen, umode_t mode,
+                           uid_t uid, gid_t gid, cfs_cap_t cap_effective,
+                           __u64 rdev, struct ptlrpc_request **request)
 {
         int rc;
         ENTRY;
@@ -1585,8 +1585,8 @@ static inline int md_link(struct obd_export *exp, struct md_op_data *op_data,
 }
 
 static inline int md_rename(struct obd_export *exp, struct md_op_data *op_data,
-                            const char *old, int oldlen, const char *new,
-                            int newlen, struct ptlrpc_request **request)
+                           const char *old, size_t oldlen, const char *new,
+                           size_t newlen, struct ptlrpc_request **request)
 {
         int rc;
         ENTRY;
@@ -1598,9 +1598,9 @@ static inline int md_rename(struct obd_export *exp, struct md_op_data *op_data,
 }
 
 static inline int md_setattr(struct obd_export *exp, struct md_op_data *op_data,
-                             void *ea, int ealen, void *ea2, int ea2len,
-                             struct ptlrpc_request **request,
-                             struct md_open_data **mod)
+                            void *ea, size_t ealen, void *ea2, size_t ea2len,
+                            struct ptlrpc_request **request,
+                            struct md_open_data **mod)
 {
         int rc;
         ENTRY;
index 204c42e..1a5623d 100644 (file)
@@ -959,9 +959,9 @@ int ll_get_max_cookiesize(struct ll_sb_info *sbi, int *max_cookiesize);
 int ll_get_default_cookiesize(struct ll_sb_info *sbi, int *default_cookiesize);
 int ll_process_config(struct lustre_cfg *lcfg);
 struct md_op_data *ll_prep_md_op_data(struct md_op_data *op_data,
-                                      struct inode *i1, struct inode *i2,
-                                      const char *name, int namelen,
-                                      int mode, __u32 opc, void *data);
+                                     struct inode *i1, struct inode *i2,
+                                     const char *name, size_t namelen,
+                                     __u32 mode, __u32 opc, void *data);
 void ll_finish_md_op_data(struct md_op_data *op_data);
 int ll_get_obd_name(struct inode *inode, unsigned int cmd, unsigned long arg);
 char *ll_get_fsname(struct super_block *sb, char *buf, int buflen);
index bfc14a9..7274ff6 100644 (file)
@@ -2527,9 +2527,9 @@ int ll_process_config(struct lustre_cfg *lcfg)
 
 /* this function prepares md_op_data hint for passing ot down to MD stack. */
 struct md_op_data * ll_prep_md_op_data(struct md_op_data *op_data,
-                                       struct inode *i1, struct inode *i2,
-                                       const char *name, int namelen,
-                                       int mode, __u32 opc, void *data)
+                                      struct inode *i1, struct inode *i2,
+                                      const char *name, size_t namelen,
+                                      __u32 mode, __u32 opc, void *data)
 {
         LASSERT(i1 != NULL);
 
index 41f36df..dc1298d 100644 (file)
@@ -523,8 +523,9 @@ int lmv_intent_lock(struct obd_export *exp, struct md_op_data *op_data,
        LASSERT(fid_is_sane(&op_data->op_fid1));
 
        CDEBUG(D_INODE, "INTENT LOCK '%s' for "DFID" '%.*s' on "DFID"\n",
-               LL_IT2STR(it), PFID(&op_data->op_fid2), op_data->op_namelen,
-               op_data->op_name, PFID(&op_data->op_fid1));
+               LL_IT2STR(it), PFID(&op_data->op_fid2),
+               (int)op_data->op_namelen, op_data->op_name,
+               PFID(&op_data->op_fid1));
 
        rc = lmv_check_connect(obd);
        if (rc)
index b6a40ae..ccaecc4 100644 (file)
@@ -261,8 +261,9 @@ static int lmv_connect(const struct lu_env *env,
        RETURN(rc);
 }
 
-static int lmv_init_ea_size(struct obd_export *exp, int easize,
-                           int def_easize, int cookiesize, int def_cookiesize)
+static int lmv_init_ea_size(struct obd_export *exp,
+                           __u32 easize, __u32 def_easize,
+                           __u32 cookiesize, __u32 def_cookiesize)
 {
        struct obd_device       *obd = exp->exp_obd;
        struct lmv_obd          *lmv = &obd->u.lmv;
@@ -1789,9 +1790,9 @@ lmv_locate_mds(struct lmv_obd *lmv, struct md_op_data *op_data,
 }
 
 int lmv_create(struct obd_export *exp, struct md_op_data *op_data,
-               const void *data, int datalen, int mode, __u32 uid,
-               __u32 gid, cfs_cap_t cap_effective, __u64 rdev,
-               struct ptlrpc_request **request)
+               const void *data, size_t datalen, umode_t mode, uid_t uid,
+               gid_t gid, cfs_cap_t cap_effective, __u64 rdev,
+               struct ptlrpc_request **request)
 {
        struct obd_device       *obd = exp->exp_obd;
        struct lmv_obd          *lmv = &obd->u.lmv;
@@ -1811,8 +1812,8 @@ int lmv_create(struct obd_export *exp, struct md_op_data *op_data,
                RETURN(PTR_ERR(tgt));
 
        CDEBUG(D_INODE, "CREATE name '%.*s' on "DFID" -> mds #%x\n",
-              op_data->op_namelen, op_data->op_name, PFID(&op_data->op_fid1),
-              op_data->op_mds);
+               (int)op_data->op_namelen, op_data->op_name,
+               PFID(&op_data->op_fid1), op_data->op_mds);
 
        rc = lmv_fid_alloc(NULL, exp, &op_data->op_fid2, op_data);
        if (rc)
@@ -1914,9 +1915,9 @@ lmv_getattr_name(struct obd_export *exp,struct md_op_data *op_data,
        if (IS_ERR(tgt))
                RETURN(PTR_ERR(tgt));
 
-       CDEBUG(D_INODE, "GETATTR_NAME for %*s on "DFID" -> mds #%u\n",
-              op_data->op_namelen, op_data->op_name, PFID(&op_data->op_fid1),
-              tgt->ltd_idx);
+       CDEBUG(D_INODE, "GETATTR_NAME for %*s on "DFID" -> mds #%d\n",
+               (int)op_data->op_namelen, op_data->op_name,
+               PFID(&op_data->op_fid1), tgt->ltd_idx);
 
        rc = md_getattr_name(tgt->ltd_exp, op_data, preq);
        if (rc != 0)
@@ -2012,7 +2013,7 @@ static int lmv_link(struct obd_export *exp, struct md_op_data *op_data,
        LASSERT(op_data->op_namelen != 0);
 
        CDEBUG(D_INODE, "LINK "DFID":%*s to "DFID"\n",
-              PFID(&op_data->op_fid2), op_data->op_namelen,
+              PFID(&op_data->op_fid2), (int)op_data->op_namelen,
               op_data->op_name, PFID(&op_data->op_fid1));
 
        op_data->op_fsuid = from_kuid(&init_user_ns, current_fsuid());
@@ -2049,8 +2050,9 @@ static int lmv_link(struct obd_export *exp, struct md_op_data *op_data,
 }
 
 static int lmv_rename(struct obd_export *exp, struct md_op_data *op_data,
-                      const char *old, int oldlen, const char *new, int newlen,
-                      struct ptlrpc_request **request)
+                     const char *old, size_t oldlen,
+                     const char *new, size_t newlen,
+                     struct ptlrpc_request **request)
 {
        struct obd_device       *obd = exp->exp_obd;
        struct lmv_obd          *lmv = &obd->u.lmv;
@@ -2061,9 +2063,9 @@ static int lmv_rename(struct obd_export *exp, struct md_op_data *op_data,
        LASSERT(oldlen != 0);
 
        CDEBUG(D_INODE, "RENAME %.*s in "DFID":%d to %.*s in "DFID":%d\n",
-              oldlen, old, PFID(&op_data->op_fid1),
+              (int)oldlen, old, PFID(&op_data->op_fid1),
               op_data->op_mea1 ? op_data->op_mea1->lsm_md_stripe_count : 0,
-              newlen, new, PFID(&op_data->op_fid2),
+              (int)newlen, new, PFID(&op_data->op_fid2),
               op_data->op_mea2 ? op_data->op_mea2->lsm_md_stripe_count : 0);
 
        rc = lmv_check_connect(obd);
@@ -2170,9 +2172,9 @@ static int lmv_rename(struct obd_export *exp, struct md_op_data *op_data,
 }
 
 static int lmv_setattr(struct obd_export *exp, struct md_op_data *op_data,
-                       void *ea, int ealen, void *ea2, int ea2len,
-                       struct ptlrpc_request **request,
-                       struct md_open_data **mod)
+                       void *ea, size_t ealen, void *ea2, size_t ea2len,
+                       struct ptlrpc_request **request,
+                       struct md_open_data **mod)
 {
        struct obd_device       *obd = exp->exp_obd;
        struct lmv_obd          *lmv = &obd->u.lmv;
index 43e9944..56798e1 100644 (file)
@@ -44,30 +44,30 @@ extern struct lprocfs_seq_vars lprocfs_mdc_obd_vars[];
 #endif
 
 void mdc_pack_body(struct ptlrpc_request *req, const struct lu_fid *fid,
-                   struct obd_capa *oc, __u64 valid, int ea_size,
-                   __u32 suppgid, int flags);
+                  struct obd_capa *oc, __u64 valid, size_t ea_size,
+                  __u32 suppgid, __u32 flags);
 void mdc_pack_capa(struct ptlrpc_request *req,
                   const struct req_msg_field *field, struct obd_capa *oc);
-int mdc_pack_req(struct ptlrpc_request *req, int version, int opc);
 void mdc_swap_layouts_pack(struct ptlrpc_request *req,
                           struct md_op_data *op_data);
-void mdc_readdir_pack(struct ptlrpc_request *req, __u64 pgoff, __u32 size,
+void mdc_readdir_pack(struct ptlrpc_request *req, __u64 pgoff, size_t size,
                       const struct lu_fid *fid, struct obd_capa *oc);
-void mdc_getattr_pack(struct ptlrpc_request *req, __u64 valid, int flags,
-                      struct md_op_data *data, int ea_size);
+void mdc_getattr_pack(struct ptlrpc_request *req, __u64 valid, __u32 flags,
+                     struct md_op_data *data, size_t ea_size);
 void mdc_setattr_pack(struct ptlrpc_request *req, struct md_op_data *op_data,
-                     void *ea, int ealen, void *ea2, int ea2len);
+                     void *ea, size_t ealen, void *ea2, size_t ea2len);
 void mdc_create_pack(struct ptlrpc_request *req, struct md_op_data *op_data,
-                     const void *data, int datalen, __u32 mode, __u32 uid,
-                     __u32 gid, cfs_cap_t capability, __u64 rdev);
+                    const void *data, size_t datalen, umode_t mode,
+                    uid_t uid, gid_t gid, cfs_cap_t capability, __u64 rdev);
 void mdc_open_pack(struct ptlrpc_request *req, struct md_op_data *op_data,
-                  __u32 mode, __u64 rdev, __u64 flags, const void *data,
-                  int datalen);
+                  umode_t mode, __u64 rdev, __u64 flags,
+                  const void *data, size_t datalen);
 void mdc_unlink_pack(struct ptlrpc_request *req, struct md_op_data *op_data);
 void mdc_getxattr_pack(struct ptlrpc_request *req, struct md_op_data *op_data);
 void mdc_link_pack(struct ptlrpc_request *req, struct md_op_data *op_data);
 void mdc_rename_pack(struct ptlrpc_request *req, struct md_op_data *op_data,
-                     const char *old, int oldlen, const char *new, int newlen);
+                    const char *old, size_t oldlen,
+                    const char *new, size_t newlen);
 void mdc_close_pack(struct ptlrpc_request *req, struct md_op_data *op_data);
 
 /* mdc/mdc_locks.c */
@@ -98,10 +98,6 @@ int mdc_resource_get_unused(struct obd_export *exp, const struct lu_fid *fid,
 int mdc_fid_alloc(const struct lu_env *env, struct obd_export *exp,
                  struct lu_fid *fid, struct md_op_data *op_data);
 
-int mdc_open(struct obd_export *exp, obd_id ino, int type, int flags,
-             struct lov_mds_md *lmm, int lmm_size, struct lustre_handle *fh,
-             struct ptlrpc_request **);
-
 struct obd_client_handle;
 
 int mdc_get_lustre_md(struct obd_export *md_exp, struct ptlrpc_request *req,
@@ -120,17 +116,18 @@ void mdc_commit_open(struct ptlrpc_request *req);
 void mdc_replay_open(struct ptlrpc_request *req);
 
 int mdc_create(struct obd_export *exp, struct md_op_data *op_data,
-               const void *data, int datalen, int mode, __u32 uid, __u32 gid,
-               cfs_cap_t capability, __u64 rdev,
-               struct ptlrpc_request **request);
+               const void *data, size_t datalen,
+               umode_t mode, uid_t uid, gid_t gid,
+               cfs_cap_t capability, __u64 rdev,
+               struct ptlrpc_request **request);
 int mdc_link(struct obd_export *exp, struct md_op_data *op_data,
              struct ptlrpc_request **request);
 int mdc_rename(struct obd_export *exp, struct md_op_data *op_data,
-               const char *old, int oldlen, const char *new, int newlen,
-               struct ptlrpc_request **request);
+               const char *old, size_t oldlen, const char *new, size_t newlen,
+               struct ptlrpc_request **request);
 int mdc_setattr(struct obd_export *exp, struct md_op_data *op_data,
-                void *ea, int ealen, void *ea2, int ea2len,
-                struct ptlrpc_request **request, struct md_open_data **mod);
+               void *ea, size_t ealen, void *ea2, size_t ea2len,
+               struct ptlrpc_request **request, struct md_open_data **mod);
 int mdc_unlink(struct obd_export *exp, struct md_op_data *op_data,
                struct ptlrpc_request **request);
 int mdc_cancel_unused(struct obd_export *exp, const struct lu_fid *fid,
index 092ba2b..279152d 100644 (file)
@@ -90,7 +90,7 @@ void mdc_swap_layouts_pack(struct ptlrpc_request *req,
 
 void mdc_pack_body(struct ptlrpc_request *req,
                   const struct lu_fid *fid, struct obd_capa *oc,
-                  __u64 valid, int ea_size, __u32 suppgid, int flags)
+                  __u64 valid, size_t ea_size, __u32 suppgid, __u32 flags)
 {
        struct mdt_body *b = req_capsule_client_get(&req->rq_pill,
                                                    &RMF_MDT_BODY);
@@ -138,8 +138,8 @@ static void mdc_pack_name(struct ptlrpc_request *req,
        LASSERT(cpy_len == name_len && lu_name_is_valid_2(buf, cpy_len));
 }
 
-void mdc_readdir_pack(struct ptlrpc_request *req, __u64 pgoff,
-                      __u32 size, const struct lu_fid *fid, struct obd_capa *oc)
+void mdc_readdir_pack(struct ptlrpc_request *req, __u64 pgoff, size_t size,
+                     const struct lu_fid *fid, struct obd_capa *oc)
 {
         struct mdt_body *b = req_capsule_client_get(&req->rq_pill,
                                                     &RMF_MDT_BODY);
@@ -155,8 +155,8 @@ void mdc_readdir_pack(struct ptlrpc_request *req, __u64 pgoff,
 
 /* packing of MDS records */
 void mdc_create_pack(struct ptlrpc_request *req, struct md_op_data *op_data,
-                     const void *data, int datalen, __u32 mode,
-                     __u32 uid, __u32 gid, cfs_cap_t cap_effective, __u64 rdev)
+                    const void *data, size_t datalen, umode_t mode,
+                    uid_t uid, gid_t gid, cfs_cap_t cap_effective, __u64 rdev)
 {
        struct mdt_rec_create   *rec;
        char                    *tmp;
@@ -193,7 +193,7 @@ void mdc_create_pack(struct ptlrpc_request *req, struct md_op_data *op_data,
        }
 }
 
-static __u64 mds_pack_open_flags(__u64 flags, __u32 mode)
+static inline __u64 mds_pack_open_flags(__u64 flags)
 {
        __u64 cr_flags = (flags & (FMODE_READ | FMODE_WRITE |
                                   MDS_OPEN_FL_INTERNAL));
@@ -225,8 +225,8 @@ static __u64 mds_pack_open_flags(__u64 flags, __u32 mode)
 
 /* packing of MDS records */
 void mdc_open_pack(struct ptlrpc_request *req, struct md_op_data *op_data,
-                  __u32 mode, __u64 rdev, __u64 flags, const void *lmm,
-                  int lmmlen)
+                  umode_t mode, __u64 rdev, __u64 flags, const void *lmm,
+                  size_t lmmlen)
 {
        struct mdt_rec_create *rec;
        char *tmp;
@@ -241,7 +241,7 @@ void mdc_open_pack(struct ptlrpc_request *req, struct md_op_data *op_data,
        rec->cr_fsgid   = from_kgid(&init_user_ns, current_fsgid());
        rec->cr_cap    = cfs_curproc_cap_pack();
        rec->cr_mode   = mode;
-       cr_flags = mds_pack_open_flags(flags, mode);
+       cr_flags        = mds_pack_open_flags(flags);
        rec->cr_rdev   = rdev;
        rec->cr_umask  = current_umask();
        if (op_data != NULL) {
@@ -355,7 +355,7 @@ static void mdc_ioepoch_pack(struct mdt_ioepoch *epoch,
 }
 
 void mdc_setattr_pack(struct ptlrpc_request *req, struct md_op_data *op_data,
-                      void *ea, int ealen, void *ea2, int ea2len)
+                     void *ea, size_t ealen, void *ea2, size_t ea2len)
 {
         struct mdt_rec_setattr *rec;
         struct mdt_ioepoch *epoch;
@@ -444,7 +444,8 @@ void mdc_link_pack(struct ptlrpc_request *req, struct md_op_data *op_data)
 }
 
 void mdc_rename_pack(struct ptlrpc_request *req, struct md_op_data *op_data,
-                     const char *old, int oldlen, const char *new, int newlen)
+                    const char *old, size_t oldlen,
+                    const char *new, size_t newlen)
 {
         struct mdt_rec_rename *rec;
 
@@ -474,8 +475,8 @@ void mdc_rename_pack(struct ptlrpc_request *req, struct md_op_data *op_data,
                mdc_pack_name(req, &RMF_SYMTGT, new, newlen);
 }
 
-void mdc_getattr_pack(struct ptlrpc_request *req, __u64 valid, int flags,
-                      struct md_op_data *op_data, int ea_size)
+void mdc_getattr_pack(struct ptlrpc_request *req, __u64 valid, __u32 flags,
+                     struct md_op_data *op_data, size_t ea_size)
 {
         struct mdt_body *b = req_capsule_client_get(&req->rq_pill,
                                                     &RMF_MDT_BODY);
index 0cddc30..0f943b1 100644 (file)
@@ -264,7 +264,7 @@ mdc_intent_open_pack(struct obd_export *exp, struct lookup_intent *it,
        struct obd_device       *obddev = class_exp2obd(exp);
        struct ldlm_intent      *lit;
        const void              *lmm = op_data->op_data;
-       int                      lmmsize = op_data->op_data_size;
+       __u32                    lmmsize = op_data->op_data_size;
        struct list_head         cancels = LIST_HEAD_INIT(cancels);
        int                      count = 0;
        int                      mode;
@@ -359,7 +359,8 @@ mdc_intent_getxattr_pack(struct obd_export *exp,
 {
        struct ptlrpc_request   *req;
        struct ldlm_intent      *lit;
-       int                     rc, count = 0, maxdata;
+       int                     rc, count = 0;
+       __u32                   maxdata;
        struct list_head        cancels = LIST_HEAD_INIT(cancels);
 
        ENTRY;
@@ -454,7 +455,7 @@ static struct ptlrpc_request *mdc_intent_getattr_pack(struct obd_export *exp,
                                               OBD_MD_FLRMTPERM : OBD_MD_FLACL);
        struct ldlm_intent    *lit;
        int                    rc;
-       int                     easize;
+       __u32                   easize;
        ENTRY;
 
         req = ptlrpc_request_alloc(class_exp2cliimp(exp),
@@ -566,7 +567,7 @@ static int mdc_finish_enqueue(struct obd_export *exp,
        struct lustre_intent_data *intent = &it->d.lustre;
        struct ldlm_lock    *lock;
        void                *lvb_data = NULL;
-       int                  lvb_len = 0;
+       __u32                lvb_len = 0;
         ENTRY;
 
         LASSERT(rc >= 0);
@@ -1028,10 +1029,11 @@ static int mdc_finish_intent_lock(struct obd_export *exp,
                         it->d.lustre.it_lock_handle = lockh->cookie;
                 }
         }
-        CDEBUG(D_DENTRY,"D_IT dentry %.*s intent: %s status %d disp %x rc %d\n",
-               op_data->op_namelen, op_data->op_name, ldlm_it2str(it->it_op),
-               it->d.lustre.it_status, it->d.lustre.it_disposition, rc);
-        RETURN(rc);
+       CDEBUG(D_DENTRY,"D_IT dentry %.*s intent: %s status %d disp %x rc %d\n",
+               (int)op_data->op_namelen, op_data->op_name,
+               ldlm_it2str(it->it_op), it->d.lustre.it_status,
+               it->d.lustre.it_disposition, rc);
+       RETURN(rc);
 }
 
 int mdc_revalidate_lock(struct obd_export *exp, struct lookup_intent *it,
@@ -1142,7 +1144,7 @@ int mdc_intent_lock(struct obd_export *exp, struct md_op_data *op_data,
        LASSERT(it);
 
        CDEBUG(D_DLMTRACE, "(name: %.*s,"DFID") in obj "DFID
-               ", intent: %s flags %#"LPF64"o\n", op_data->op_namelen,
+               ", intent: %s flags %#"LPF64"o\n", (int)op_data->op_namelen,
                op_data->op_name, PFID(&op_data->op_fid2),
                PFID(&op_data->op_fid1), ldlm_it2str(it->it_op),
                it->it_flags);
@@ -1232,30 +1234,30 @@ out:
 }
 
 int mdc_intent_getattr_async(struct obd_export *exp,
-                             struct md_enqueue_info *minfo,
-                             struct ldlm_enqueue_info *einfo)
+                            struct md_enqueue_info *minfo,
+                            struct ldlm_enqueue_info *einfo)
 {
-        struct md_op_data       *op_data = &minfo->mi_data;
-        struct lookup_intent    *it = &minfo->mi_it;
-        struct ptlrpc_request   *req;
-        struct mdc_getattr_args *ga;
-        struct obd_device       *obddev = class_exp2obd(exp);
-        struct ldlm_res_id       res_id;
-        /*XXX: Both MDS_INODELOCK_LOOKUP and MDS_INODELOCK_UPDATE are needed
-         *     for statahead currently. Consider CMD in future, such two bits
-         *     maybe managed by different MDS, should be adjusted then. */
-        ldlm_policy_data_t       policy = {
-                                        .l_inodebits = { MDS_INODELOCK_LOOKUP | 
-                                                         MDS_INODELOCK_UPDATE }
-                                 };
-        int                      rc = 0;
-       __u64                    flags = LDLM_FL_HAS_INTENT;
+       struct md_op_data       *op_data = &minfo->mi_data;
+       struct lookup_intent    *it = &minfo->mi_it;
+       struct ptlrpc_request   *req;
+       struct mdc_getattr_args *ga;
+       struct obd_device       *obddev = class_exp2obd(exp);
+       struct ldlm_res_id       res_id;
+       /*XXX: Both MDS_INODELOCK_LOOKUP and MDS_INODELOCK_UPDATE are needed
+        *     for statahead currently. Consider CMD in future, such two bits
+        *     maybe managed by different MDS, should be adjusted then. */
+       ldlm_policy_data_t       policy = {
+                                       .l_inodebits = { MDS_INODELOCK_LOOKUP |
+                                                        MDS_INODELOCK_UPDATE }
+                                };
+       int                      rc = 0;
+       __u64                    flags = LDLM_FL_HAS_INTENT;
        ENTRY;
 
        CDEBUG(D_DLMTRACE, "name: %.*s in inode "DFID", intent: %s flags %#"
                LPF64"o\n",
-               op_data->op_namelen, op_data->op_name, PFID(&op_data->op_fid1),
-               ldlm_it2str(it->it_op), it->it_flags);
+               (int)op_data->op_namelen, op_data->op_name,
+               PFID(&op_data->op_fid1), ldlm_it2str(it->it_op), it->it_flags);
 
        fid_build_reg_res_name(&op_data->op_fid1, &res_id);
        req = mdc_intent_getattr_pack(exp, it, op_data);
index 7c68524..6142510 100644 (file)
@@ -102,8 +102,8 @@ int mdc_resource_get_unused(struct obd_export *exp, const struct lu_fid *fid,
 }
 
 int mdc_setattr(struct obd_export *exp, struct md_op_data *op_data,
-                void *ea, int ealen, void *ea2, int ea2len,
-                struct ptlrpc_request **request, struct md_open_data **mod)
+               void *ea, size_t ealen, void *ea2, size_t ea2len,
+               struct ptlrpc_request **request, struct md_open_data **mod)
 {
        struct list_head cancels = LIST_HEAD_INIT(cancels);
         struct ptlrpc_request *req;
@@ -208,9 +208,10 @@ int mdc_setattr(struct obd_export *exp, struct md_op_data *op_data,
 }
 
 int mdc_create(struct obd_export *exp, struct md_op_data *op_data,
-               const void *data, int datalen, int mode, __u32 uid, __u32 gid,
-               cfs_cap_t cap_effective, __u64 rdev,
-               struct ptlrpc_request **request)
+               const void *data, size_t datalen,
+               umode_t mode, uid_t uid, gid_t gid,
+               cfs_cap_t cap_effective, __u64 rdev,
+               struct ptlrpc_request **request)
 {
         struct ptlrpc_request *req;
         int level, rc;
@@ -418,8 +419,8 @@ int mdc_link(struct obd_export *exp, struct md_op_data *op_data,
 }
 
 int mdc_rename(struct obd_export *exp, struct md_op_data *op_data,
-               const char *old, int oldlen, const char *new, int newlen,
-               struct ptlrpc_request **request)
+               const char *old, size_t oldlen, const char *new, size_t newlen,
+               struct ptlrpc_request **request)
 {
        struct list_head cancels = LIST_HEAD_INIT(cancels);
         struct obd_device *obd = exp->exp_obd;
index ca463cd..f43caa8 100644 (file)
@@ -520,7 +520,7 @@ int mdc_get_lustre_md(struct obd_export *exp, struct ptlrpc_request *req,
                 if (rc < 0)
                         GOTO(out, rc);
 
-                if (rc < sizeof(*md->lsm)) {
+               if (rc < (typeof(rc))sizeof(*md->lsm)) {
                         CDEBUG(D_INFO, "lsm size too small: "
                                "rc < sizeof (*md->lsm) (%d < %d)\n",
                                rc, (int)sizeof(*md->lsm));
@@ -555,7 +555,7 @@ int mdc_get_lustre_md(struct obd_export *exp, struct ptlrpc_request *req,
                        if (rc < 0)
                                GOTO(out, rc);
 
-                       if (rc < sizeof(*md->lmv)) {
+                       if (rc < (typeof(rc))sizeof(*md->lmv)) {
                                CDEBUG(D_INFO, "size too small:  "
                                       "rc < sizeof(*md->lmv) (%d < %d)\n",
                                        rc, (int)sizeof(*md->lmv));
@@ -1981,7 +1981,7 @@ out:
        return rc;
 }
 
-static struct kuc_hdr *changelog_kuc_hdr(char *buf, int len, int flags)
+static struct kuc_hdr *changelog_kuc_hdr(char *buf, size_t len, __u32 flags)
 {
        struct kuc_hdr *lh = (struct kuc_hdr *)buf;
 
@@ -2014,7 +2014,7 @@ static int changelog_kkuc_cb(const struct lu_env *env, struct llog_handle *llh,
        struct changelog_show           *cs = data;
        struct llog_changelog_rec       *rec = (struct llog_changelog_rec *)hdr;
        struct kuc_hdr                  *lh;
-       int                              len;
+       size_t                           len;
        int                              rc;
        ENTRY;
 
@@ -2047,7 +2047,7 @@ static int changelog_kkuc_cb(const struct lu_env *env, struct llog_handle *llh,
         memcpy(lh + 1, &rec->cr, len - sizeof(*lh));
 
         rc = libcfs_kkuc_msg_put(cs->cs_fp, lh);
-       CDEBUG(D_HSM, "kucmsg fp %p len %d rc %d\n", cs->cs_fp, len, rc);
+       CDEBUG(D_HSM, "kucmsg fp %p len %zu rc %d\n", cs->cs_fp, len, rc);
 
         RETURN(rc);
 }
@@ -2494,7 +2494,7 @@ static void lustre_swab_hai(struct hsm_action_item *h)
 static void lustre_swab_hal(struct hsm_action_list *h)
 {
        struct hsm_action_item  *hai;
-       int                      i;
+       __u32                    i;
 
        __swab32s(&h->hal_version);
        __swab32s(&h->hal_count);
@@ -2543,7 +2543,7 @@ static int mdc_ioc_hsm_ct_start(struct obd_export *exp,
  * @param val KUC message (kuc_hdr + hsm_action_list)
  * @param len total length of message
  */
-static int mdc_hsm_copytool_send(int len, void *val)
+static int mdc_hsm_copytool_send(size_t len, void *val)
 {
        struct kuc_hdr          *lh = (struct kuc_hdr *)val;
        struct hsm_action_list  *hal = (struct hsm_action_list *)(lh + 1);
@@ -2551,8 +2551,8 @@ static int mdc_hsm_copytool_send(int len, void *val)
        ENTRY;
 
        if (len < sizeof(*lh) + sizeof(*hal)) {
-               CERROR("Short HSM message %d < %d\n", len,
-                      (int) (sizeof(*lh) + sizeof(*hal)));
+               CERROR("Short HSM message %zu < %zu\n", len,
+                      sizeof(*lh) + sizeof(*hal));
                RETURN(-EPROTO);
        }
        if (lh->kuc_magic == __swab16(KUC_MAGIC)) {
@@ -2670,18 +2670,18 @@ int mdc_get_info(const struct lu_env *env, struct obd_export *exp,
        int rc = -EINVAL;
 
        if (KEY_IS(KEY_MAX_EASIZE)) {
-               int mdsize, *max_easize;
+               __u32 mdsize, *max_easize;
 
                if (*vallen != sizeof(int))
                        RETURN(-EINVAL);
-               mdsize = *(int *)val;
+               mdsize = *(__u32 *)val;
                if (mdsize > exp->exp_obd->u.cli.cl_max_mds_easize)
                        exp->exp_obd->u.cli.cl_max_mds_easize = mdsize;
                max_easize = val;
                *max_easize = exp->exp_obd->u.cli.cl_max_mds_easize;
                RETURN(0);
        } else if (KEY_IS(KEY_DEFAULT_EASIZE)) {
-               int *default_easize;
+               __u32 *default_easize;
 
                if (*vallen != sizeof(int))
                        RETURN(-EINVAL);
@@ -2689,7 +2689,7 @@ int mdc_get_info(const struct lu_env *env, struct obd_export *exp,
                *default_easize = exp->exp_obd->u.cli.cl_default_mds_easize;
                RETURN(0);
        } else if (KEY_IS(KEY_MAX_COOKIESIZE)) {
-               int mdsize, *max_cookiesize;
+               __u32 mdsize, *max_cookiesize;
 
                if (*vallen != sizeof(int))
                        RETURN(-EINVAL);
@@ -2700,7 +2700,7 @@ int mdc_get_info(const struct lu_env *env, struct obd_export *exp,
                *max_cookiesize = exp->exp_obd->u.cli.cl_max_mds_cookiesize;
                RETURN(0);
        } else if (KEY_IS(KEY_DEFAULT_COOKIESIZE)) {
-               int *default_cookiesize;
+               __u32 *default_cookiesize;
 
                if (*vallen != sizeof(int))
                        RETURN(-EINVAL);
@@ -2718,7 +2718,7 @@ int mdc_get_info(const struct lu_env *env, struct obd_export *exp,
                 *data = imp->imp_connect_data;
                 RETURN(0);
         } else if (KEY_IS(KEY_TGT_COUNT)) {
-                *((int *)val) = 1;
+               *((__u32 *)val) = 1;
                 RETURN(0);
         }
 
@@ -2950,8 +2950,9 @@ err_rpc_lock:
  * a large number of stripes is possible.  If a larger reply buffer is
  * required it will be reallocated in the ptlrpc layer due to overflow.
  */
-static int mdc_init_ea_size(struct obd_export *exp, int easize,
-                           int def_easize, int cookiesize, int def_cookiesize)
+static int mdc_init_ea_size(struct obd_export *exp, __u32 easize,
+                           __u32 def_easize, __u32 cookiesize,
+                           __u32 def_cookiesize)
 {
        struct obd_device *obd = exp->exp_obd;
        struct client_obd *cli = &obd->u.cli;