A few more Lustre styel cleanups
cover A few more Lustre styel cleanups
From: Oleg Drokin <[email protected]>
More of the checkpatch warning fixes.
Oleg Drokin (3):
staging/lustre: Fix braces {} style staging/lustre: Fix blank line after/before {/} style staging/lustre: Get rid of ldlm_policy_res_t typedef
drivers/staging/lustre/lustre/fld/fld_cache.c | 3 +- drivers/staging/lustre/lustre/fld/fld_request.c | 1 - drivers/staging/lustre/lustre/include/lu_object.h | 1 - .../lustre/lustre/include/lustre/lustre_idl.h | 3 +- drivers/staging/lustre/lustre/include/obd_class.h | 3 +- drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 2 - drivers/staging/lustre/lustre/ldlm/ldlm_lib.c | 1 - drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 1 - drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 50 +++++++++++----------- drivers/staging/lustre/lustre/llite/dir.c | 22 +++++----- drivers/staging/lustre/lustre/llite/llite_close.c | 3 +- drivers/staging/lustre/lustre/llite/llite_lib.c | 9 ++-- drivers/staging/lustre/lustre/llite/lloop.c | 3 +- drivers/staging/lustre/lustre/llite/lproc_llite.c | 1 - drivers/staging/lustre/lustre/llite/namei.c | 6 ++- drivers/staging/lustre/lustre/llite/rw.c | 6 ++- drivers/staging/lustre/lustre/llite/rw26.c | 3 +- drivers/staging/lustre/lustre/llite/vvp_dev.c | 12 ++++-- drivers/staging/lustre/lustre/llite/vvp_io.c | 3 +- drivers/staging/lustre/lustre/llite/xattr.c | 4 +- drivers/staging/lustre/lustre/llite/xattr_cache.c | 1 - drivers/staging/lustre/lustre/lmv/lmv_obd.c | 3 +- drivers/staging/lustre/lustre/lov/lov_dev.c | 10 +++-- drivers/staging/lustre/lustre/lov/lov_io.c | 10 +++-- drivers/staging/lustre/lustre/lov/lov_obd.c | 1 - drivers/staging/lustre/lustre/lov/lov_object.c | 6 ++- drivers/staging/lustre/lustre/lov/lov_pack.c | 5 +-- drivers/staging/lustre/lustre/lov/lov_request.c | 1 - drivers/staging/lustre/lustre/lov/lovsub_dev.c | 9 ++-- drivers/staging/lustre/lustre/lov/lovsub_lock.c | 3 +- drivers/staging/lustre/lustre/lov/lovsub_object.c | 7 +-- drivers/staging/lustre/lustre/mdc/mdc_lib.c | 1 - drivers/staging/lustre/lustre/mdc/mdc_locks.c | 1 - drivers/staging/lustre/lustre/mgc/mgc_request.c | 1 - drivers/staging/lustre/lustre/obdclass/cl_io.c | 13 +++--- drivers/staging/lustre/lustre/obdclass/cl_object.c | 9 ++-- drivers/staging/lustre/lustre/obdclass/class_obd.c | 1 - drivers/staging/lustre/lustre/obdclass/genops.c | 1 - drivers/staging/lustre/lustre/obdclass/llog.c | 1 - .../lustre/lustre/obdclass/lprocfs_status.c | 3 +- drivers/staging/lustre/lustre/obdclass/lu_object.c | 5 +-- .../staging/lustre/lustre/obdclass/lustre_peer.c | 3 +- .../staging/lustre/lustre/obdclass/obd_config.c | 1 - .../staging/lustre/lustre/obdecho/echo_client.c | 2 - drivers/staging/lustre/lustre/osc/osc_cache.c | 1 - drivers/staging/lustre/lustre/osc/osc_io.c | 4 +- drivers/staging/lustre/lustre/osc/osc_lock.c | 3 +- drivers/staging/lustre/lustre/osc/osc_object.c | 3 +- drivers/staging/lustre/lustre/osc/osc_page.c | 4 +- drivers/staging/lustre/lustre/osc/osc_request.c | 4 +- drivers/staging/lustre/lustre/ptlrpc/client.c | 5 +-- drivers/staging/lustre/lustre/ptlrpc/events.c | 1 - drivers/staging/lustre/lustre/ptlrpc/import.c | 3 -- .../staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c | 10 ++--- drivers/staging/lustre/lustre/ptlrpc/nrs.c | 1 - .../staging/lustre/lustre/ptlrpc/pack_generic.c | 3 -- 56 files changed, 141 insertions(+), 136 deletions(-)
Signature · 1 line
2.1.0
PATCH 1/3 staging/lustre: Fix braces {} style
From: Oleg Drokin <[email protected]>
This fixes all checkpatch form of this from the Lustre tree:
CHECK: braces {} should be used on all arms of this statement
--- drivers/staging/lustre/lustre/fld/fld_cache.c | 3 ++- .../lustre/lustre/include/lustre/lustre_idl.h | 3 ++- drivers/staging/lustre/lustre/include/obd_class.h | 3 ++- drivers/staging/lustre/lustre/llite/dir.c | 20 +++++++++++--------- drivers/staging/lustre/lustre/llite/llite_close.c | 3 ++- drivers/staging/lustre/lustre/llite/llite_lib.c | 9 +++++---- drivers/staging/lustre/lustre/llite/lloop.c | 3 ++- drivers/staging/lustre/lustre/llite/namei.c | 6 ++++-- drivers/staging/lustre/lustre/llite/rw.c | 6 ++++-- drivers/staging/lustre/lustre/llite/rw26.c | 3 ++- drivers/staging/lustre/lustre/llite/vvp_dev.c | 12 ++++++++---- drivers/staging/lustre/lustre/llite/vvp_io.c | 3 ++- drivers/staging/lustre/lustre/llite/xattr.c | 3 ++- drivers/staging/lustre/lustre/lmv/lmv_obd.c | 3 ++- drivers/staging/lustre/lustre/lov/lov_dev.c | 10 ++++++---- drivers/staging/lustre/lustre/lov/lov_io.c | 9 ++++++--- drivers/staging/lustre/lustre/lov/lov_object.c | 6 ++++-- drivers/staging/lustre/lustre/lov/lov_pack.c | 4 ++-- drivers/staging/lustre/lustre/lov/lovsub_dev.c | 9 ++++++--- drivers/staging/lustre/lustre/lov/lovsub_lock.c | 3 ++- drivers/staging/lustre/lustre/lov/lovsub_object.c | 6 ++++-- drivers/staging/lustre/lustre/obdclass/cl_io.c | 13 ++++++++----- drivers/staging/lustre/lustre/obdclass/cl_object.c | 9 ++++++--- .../staging/lustre/lustre/obdclass/lprocfs_status.c | 3 ++- drivers/staging/lustre/lustre/obdclass/lu_object.c | 3 ++- drivers/staging/lustre/lustre/obdclass/lustre_peer.c | 3 ++- drivers/staging/lustre/lustre/osc/osc_io.c | 3 ++- drivers/staging/lustre/lustre/osc/osc_lock.c | 3 ++- drivers/staging/lustre/lustre/osc/osc_object.c | 3 ++- drivers/staging/lustre/lustre/osc/osc_page.c | 3 ++- drivers/staging/lustre/lustre/osc/osc_request.c | 3 ++- drivers/staging/lustre/lustre/ptlrpc/client.c | 4 ++-- drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c | 9 +++++---- 33 files changed, 117 insertions(+), 69 deletions(-)
drivers/staging/lustre/lustre/fld/fld_cache.c +2 -1
@@ -178,8 +178,9 @@ restart_fixup:if (n_range->lsr_end <= c_range->lsr_end) { *n_range = *c_range;fld_cache_entry_delete (cache, f_curr); - }else + }else { n_range->lsr_start = c_range->lsr_end; + } }/* we could have overlap over next
drivers/staging/lustre/lustre/include/lustre/lustre_idl.h +2 -1
@@ -1001,8 +1001,9 @@ static inline int lu_dirent_calc_size(int namelen, __u16 attr) size = (sizeof (struct lu_dirent ) + namelen + align) & ~align; size +=sizeof (struct luda_type ); - }else + }else { size =sizeof (struct lu_dirent ) + namelen; + }return (size +7 ) & ~7 ; }
drivers/staging/lustre/lustre/include/obd_class.h +2 -1
@@ -490,8 +490,9 @@ static inline int obd_setup(struct obd_device *obd, struct lustre_cfg *cfg) obd->obd_lu_dev = d; d->ld_obd = obd; rc =0 ; - }else + }else { rc =PTR_ERR (d); + } }lu_context_exit (&session_ctx);lu_context_fini (&session_ctx);
drivers/staging/lustre/lustre/llite/dir.c +11 -9
@@ -1497,8 +1497,9 @@ free_lmv: cmd == LL_IOC_MDC_GETINFO)) { rc = 0; goto skip_lmm; - } else + } else { goto out_req; + } } if (cmd == IOC_MDC_GETFILESTRIPE ||
@@ -1711,15 +1712,16 @@ out_quotactl:return ll_flush_ctx (inode);#ifdef CONFIG_FS_POSIX_ACL case LL_IOC_RMTACL : { -if (sbi->ll_flags &LL_SBI_RMT_CLIENT &&is_root_inode (inode)) { -struct ll_file_data *fd =LUSTRE_FPRIVATE (file); +if (sbi->ll_flags &LL_SBI_RMT_CLIENT &&is_root_inode (inode)) { +struct ll_file_data *fd =LUSTRE_FPRIVATE (file); - rc =rct_add (&sbi->ll_rct,current_pid (), arg); -if (!rc) - fd->fd_flags |=LL_FILE_RMTACL ; -return rc; - }else -return 0 ; + rc =rct_add (&sbi->ll_rct,current_pid (), arg); +if (!rc) + fd->fd_flags |=LL_FILE_RMTACL ; +return rc; + }else { +return 0 ; + } }#endif case LL_IOC_GETOBDCOUNT : {
drivers/staging/lustre/lustre/llite/llite_close.c +2 -1
@@ -323,8 +323,9 @@ static struct ll_inode_info *ll_close_next_lli(struct ll_close_queue *lcq) lli = list_entry(lcq->lcq_head.next, struct ll_inode_info, lli_close_list);list_del_init (&lli->lli_close_list); - }else if (atomic_read (&lcq->lcq_stop)) + }else if (atomic_read (&lcq->lcq_stop)) { lli =ERR_PTR (-EALREADY ); + }spin_unlock (&lcq->lcq_lock);return lli;
drivers/staging/lustre/lustre/llite/llite_lib.c +5 -4
@@ -1596,8 +1596,9 @@ void ll_update_inode(struct inode *inode, struct lustre_md *md)" to the " DFID ", inode %lu/%u(%p)\n" ,PFID (&lli->lli_fid),PFID (&body->fid1), inode->i_ino, inode->i_generation, inode); - }else + }else { lli->lli_fid = body->fid1; + } }LASSERT (fid_seq (&lli->lli_fid) !=0 );
@@ -2065,11 +2066,11 @@ int ll_obd_statfs(struct inode *inode, void __user *arg) }memcpy (&type, data->ioc_inlbuf1,sizeof (__u32)); -if (type &LL_STATFS_LMV ) +if (type &LL_STATFS_LMV ) { exp = sbi->ll_md_exp; -else if (type &LL_STATFS_LOV ) + }else if (type &LL_STATFS_LOV ) { exp = sbi->ll_dt_exp; -else { + }else { rc = -ENODEV ; goto out_statfs; }
drivers/staging/lustre/lustre/llite/lloop.c +2 -1
@@ -274,8 +274,9 @@ static void loop_add_bio(struct lloop_device *lo, struct bio *bio)if (lo->lo_biotail) { lo->lo_biotail->bi_next = bio; lo->lo_biotail = bio; - }else + }else { lo->lo_bio = lo->lo_biotail = bio; + }spin_unlock_irqrestore (&lo->lo_lock, flags);atomic_inc (&lo->lo_pending);
drivers/staging/lustre/lustre/llite/namei.c +4 -2
@@ -128,10 +128,12 @@ struct inode *ll_iget(struct super_block *sb, ino_t hash,if (rc !=0 ) {iget_failed (inode); inode =NULL ; - }else + }else {unlock_new_inode (inode); - }else if (!(inode->i_state & (I_FREEING |I_CLEAR ))) + } + }else if (!(inode->i_state & (I_FREEING |I_CLEAR ))) {ll_update_inode (inode, md); + }CDEBUG (D_VFSTRACE ,"got inode: %p for " DFID "\n" , inode,PFID (&md->body->fid1)); }
drivers/staging/lustre/lustre/llite/rw.c +4 -2
@@ -132,8 +132,9 @@ struct ll_cl_context *ll_cl_init(struct file *file, struct page *vmpage) lcc->lcc_page = page;lu_ref_add (&page->cp_reference,"cl_io" , io); result =0 ; - }else + }else { result =PTR_ERR (page); + } }if (result) {ll_cl_fini (lcc);
@@ -488,8 +489,9 @@ static int ll_read_ahead_pages(const struct lu_env *env,if (rc ==1 ) { (*reserved_pages)--; count++; - }else if (rc == -ENOLCK ) + }else if (rc == -ENOLCK ) {break ; + } }else if (stride_ria) {/* If it is not in the read-ahead window, and it is * read-ahead mode, then check whether it should skip
drivers/staging/lustre/lustre/llite/rw26.c +2 -1
@@ -98,8 +98,9 @@ static void ll_invalidatepage(struct page *vmpage, unsigned int offset,cl_page_delete (env, page);cl_page_put (env, page); } - }else + }else {LASSERT (vmpage->private ==0 ); + }cl_env_put (env, &refcheck); } }
drivers/staging/lustre/lustre/llite/vvp_dev.c +8 -4
@@ -342,8 +342,9 @@ int cl_sb_init(struct super_block *sb) sbi->ll_site =cl2lu_dev (cl)->ld_site; }cl_env_put (env, &refcheck); - }else + }else { rc =PTR_ERR (env); + }return rc; }
@@ -589,16 +590,19 @@ static int vvp_pgcache_show(struct seq_file *f, void *v)if (page) {vvp_pgcache_page_show (env, f, page);cl_page_put (env, page); - }else + }else {seq_puts (f,"missing\n" ); + }lu_object_ref_del (&clob->co_lu,"dump" , current);cl_object_put (env, clob); - }else + }else {seq_printf (f,"%llx missing\n" , pos); + }cl_env_put (env, &refcheck); result =0 ; - }else + }else { result =PTR_ERR (env); + }return result; }
drivers/staging/lustre/lustre/llite/vvp_io.c +2 -1
@@ -1179,8 +1179,9 @@ static int vvp_io_fault_start(const struct lu_env *env,if (result == -EDQUOT ) result = -ENOSPC ; goto out; - }else + }else {cl_page_disown (env, io, page); + } } }
drivers/staging/lustre/lustre/llite/xattr.c +2 -1
@@ -181,8 +181,9 @@ int ll_setxattr_common(struct inode *inode, const char *name, size = rc; pv = (const char *)new_value; - }else + }else {return -EOPNOTSUPP ; + } valid |=rce_ops2valid (rce->rce_ops); }
drivers/staging/lustre/lustre/lmv/lmv_obd.c +2 -1
@@ -1122,8 +1122,9 @@ static int lmv_iocontrol(unsigned int cmd, struct obd_export *exp,if (!rc) rc = err; } - }else + }else { set =1 ; + } }if (!set && !rc) rc = -EIO ;
drivers/staging/lustre/lustre/lov/lov_dev.c +6 -4
@@ -262,8 +262,9 @@ static int lov_req_init(const struct lu_env *env, struct cl_device *dev,if (lr) {cl_req_slice_add (req, &lr->lr_cl, dev, &lov_req_ops); result =0 ; - }else + }else { result = -ENOMEM ; + }return result; }
@@ -332,14 +333,15 @@ static struct lov_device_emerg **lov_emerg_alloc(int nr)cl_page_list_init (&em->emrg_page_list); em->emrg_env =cl_env_alloc (&em->emrg_refcheck,LCT_REMEMBER |LCT_NOREF ); -if (!IS_ERR (em->emrg_env)) +if (!IS_ERR (em->emrg_env)) { em->emrg_env->le_ctx.lc_cookie =0x2 ; -else { + }else { result =PTR_ERR (em->emrg_env); em->emrg_env =NULL ; } - }else + }else { result = -ENOMEM ; + } }if (result !=0 ) {lov_emerg_free (emerg, nr);
drivers/staging/lustre/lustre/lov/lov_io.c +6 -3
@@ -225,8 +225,9 @@ struct lov_io_sub *lov_sub_get(const struct lu_env *env,if (!sub->sub_io_initialized) { sub->sub_stripe = stripe; rc =lov_io_sub_init (env, lio, sub); - }else + }else { rc =0 ; + }if (rc ==0 )lov_sub_enter (sub);else
@@ -294,8 +295,9 @@ static int lov_io_subio_init(const struct lu_env *env, struct lov_io *lio, lio->lis_single_subio_index =-1 ; lio->lis_active_subios =0 ; result =0 ; - }else + }else { result = -ENOMEM ; + }return result; }
@@ -413,8 +415,9 @@ static int lov_io_iter_init(const struct lu_env *env,lov_sub_put (sub);CDEBUG (D_VFSTRACE ,"shrink: %d [%llu, %llu)\n" , stripe, start, end); - }else + }else { rc =PTR_ERR (sub); + }if (!rc)list_add_tail (&sub->sub_linkage, &lio->lis_active);
drivers/staging/lustre/lustre/lov/lov_object.c +4 -2
@@ -283,8 +283,9 @@ static int lov_init_raid0(const struct lu_env *env, }if (result ==0 )cl_object_header (&lov->lo_cl)->coh_page_bufsize += psz; - }else + }else { result = -ENOMEM ; + } out:return result; }
@@ -935,8 +936,9 @@ struct lu_object *lov_object_alloc(const struct lu_env *env,* for object with different layouts. */ obj->lo_ops = &lov_lu_obj_ops; - }else + }else { obj =NULL ; + }return obj; }
drivers/staging/lustre/lustre/lov/lov_pack.c +2 -2
@@ -457,9 +457,9 @@ int lov_getstripe(struct obd_export *exp, struct lov_stripe_md *lsm, }/* User wasn't expecting this many OST entries */ -if (lum.lmm_stripe_count ==0 ) +if (lum.lmm_stripe_count ==0 ) { lmm_size = lum_size; -else if (lum.lmm_stripe_count < lmmk->lmm_stripe_count) { + }else if (lum.lmm_stripe_count < lmmk->lmm_stripe_count) { rc = -EOVERFLOW ; goto out_set; }
drivers/staging/lustre/lustre/lov/lovsub_dev.c +6 -3
@@ -151,8 +151,9 @@ static int lovsub_req_init(const struct lu_env *env, struct cl_device *dev,if (lsr) {cl_req_slice_add (req, &lsr->lsrq_cl, dev, &lovsub_req_ops); result =0 ; - }else + }else { result = -ENOMEM ; + }return result; }
@@ -182,10 +183,12 @@ static struct lu_device *lovsub_device_alloc(const struct lu_env *env, d =lovsub2lu_dev (lsd); d->ld_ops = &lovsub_lu_ops; lsd->acid_cl.cd_ops = &lovsub_cl_ops; - }else + }else { d =ERR_PTR (result); - }else + } + }else { d =ERR_PTR (-ENOMEM ); + }return d; }
drivers/staging/lustre/lustre/lov/lovsub_lock.c +2 -1
@@ -77,8 +77,9 @@ int lovsub_lock_init(const struct lu_env *env, struct cl_object *obj,INIT_LIST_HEAD (&lsk->lss_parents);cl_lock_slice_add (lock, &lsk->lss_cl, obj, &lovsub_lock_ops); result =0 ; - }else + }else { result = -ENOMEM ; + }return result; }
drivers/staging/lustre/lustre/lov/lovsub_object.c +4 -2
@@ -67,8 +67,9 @@ int lovsub_object_init(const struct lu_env *env, struct lu_object *obj,lu_object_add (obj, below);cl_object_page_init (lu2cl (obj),sizeof (struct lovsub_page )); result =0 ; - }else + }else { result = -ENOMEM ; + }return result; }
@@ -154,8 +155,9 @@ struct lu_object *lovsub_object_alloc(const struct lu_env *env,lu_object_add_top (&hdr->coh_lu, obj); los->lso_cl.co_ops = &lovsub_ops; obj->lo_ops = &lovsub_lu_obj_ops; - }else + }else { obj =NULL ; + }return obj; }
drivers/staging/lustre/lustre/obdclass/cl_io.c +8 -5
@@ -504,9 +504,9 @@ int cl_io_lock_add(const struct lu_env *env, struct cl_io *io, {int result; -if (cl_lockset_merge (&io->ci_lockset, &link->cill_descr)) +if (cl_lockset_merge (&io->ci_lockset, &link->cill_descr)) { result =1 ; -else { + }else {list_add (&link->cill_linkage, &io->ci_lockset.cls_todo); result =0 ; }
@@ -536,8 +536,9 @@ int cl_io_lock_alloc_add(const struct lu_env *env, struct cl_io *io, result =cl_io_lock_add (env, io, link);if (result)/* lock match */ link->cill_fini (env, link); - }else + }else { result = -ENOMEM ; + }return result; }
@@ -1202,14 +1203,16 @@ struct cl_req *cl_req_alloc(const struct lu_env *env, struct cl_page *page,if (req->crq_o) { req->crq_nrobjs = nr_objects; result =cl_req_init (env, req, page); - }else + }else { result = -ENOMEM ; + }if (result !=0 ) {cl_req_completion (env, req, result); req =ERR_PTR (result); } - }else + }else { req =ERR_PTR (-ENOMEM ); + }return req; }EXPORT_SYMBOL (cl_req_alloc);
drivers/staging/lustre/lustre/obdclass/cl_object.c +6 -3
@@ -654,8 +654,9 @@ static struct lu_env *cl_env_new(__u32 ctx_tags, __u32 ses_tags, void *debug)lu_context_enter (&cle->ce_ses); env->le_ses = &cle->ce_ses;cl_env_init0 (cle, debug); - }else + }else {lu_env_fini (env); + } }if (rc !=0 ) {kmem_cache_free (cl_env_kmem, cle);
@@ -664,8 +665,9 @@ static struct lu_env *cl_env_new(__u32 ctx_tags, __u32 ses_tags, void *debug)CL_ENV_INC (create);CL_ENV_INC (total); } - }else + }else { env =ERR_PTR (-ENOMEM ); + }return env; }
@@ -1095,8 +1097,9 @@ struct cl_device *cl_type_setup(const struct lu_env *env, struct lu_site *site,CERROR ("can't init device '%s', %d\n" , typename, rc); d =ERR_PTR (rc); } - }else + }else {CERROR ("Cannot allocate device: '%s'\n" , typename); + }return lu2cl_dev (d); }EXPORT_SYMBOL (cl_type_setup);
drivers/staging/lustre/lustre/obdclass/lprocfs_status.c +2 -1
@@ -694,8 +694,9 @@ int lprocfs_rd_import(struct seq_file *m, void *data)do_div (sum, ret.lc_count); ret.lc_sum = sum; - }else + }else { ret.lc_sum =0 ; + }seq_printf (m," rpcs:\n" " inflight: %u\n"
drivers/staging/lustre/lustre/obdclass/lu_object.c +2 -1
@@ -716,8 +716,9 @@ struct lu_object *lu_object_find_slice(const struct lu_env *env, obj =lu_object_locate (top->lo_header, dev->ld_type);if (!obj)lu_object_put (env, top); - }else + }else { obj = top; + }return obj; }EXPORT_SYMBOL (lu_object_find_slice);
drivers/staging/lustre/lustre/obdclass/lustre_peer.c +2 -1
@@ -163,8 +163,9 @@ int class_del_uuid(const char *uuid)break ; } } - }else + }else {list_splice_init (&g_uuid_list, &deathrow); + }spin_unlock (&g_uuid_lock);if (uuid &&list_empty (&deathrow)) {
drivers/staging/lustre/lustre/osc/osc_io.c +2 -1
@@ -842,8 +842,9 @@ int osc_req_init(const struct lu_env *env, struct cl_device *dev,if (or) {cl_req_slice_add (req, &or->or_cl, dev, &osc_req_ops); result =0 ; - }else + }else { result = -ENOMEM ; + }return result; }
drivers/staging/lustre/lustre/osc/osc_lock.c +2 -1
@@ -624,8 +624,9 @@ static int osc_ldlm_glimpse_ast(struct ldlm_lock *dlmlock, void *data) result = -ELDLM_NO_LOCK_DATA ; }cl_env_nested_put (&nest, env); - }else + }else { result =PTR_ERR (env); + } req->rq_status = result;return result; }
drivers/staging/lustre/lustre/osc/osc_object.c +2 -1
@@ -292,8 +292,9 @@ struct lu_object *osc_object_alloc(const struct lu_env *env,lu_object_init (obj,NULL , dev); osc->oo_cl.co_ops = &osc_ops; obj->lo_ops = &osc_lu_obj_ops; - }else + }else { obj =NULL ; + }return obj; }
drivers/staging/lustre/lustre/osc/osc_page.c +2 -1
@@ -421,8 +421,9 @@ static int osc_cache_too_much(struct client_obd *cli)return lru_shrink_max; else if (pages >= budget / 2) return lru_shrink_min; - }else if (pages >= budget *2 ) + }else if (pages >= budget *2 ) {return lru_shrink_min; + }return 0 ; }
drivers/staging/lustre/lustre/osc/osc_request.c +2 -1
@@ -3065,8 +3065,9 @@ static int osc_import_event(struct obd_device *obd,ldlm_namespace_cleanup (ns,LDLM_FL_LOCAL_ONLY );cl_env_put (env, &refcheck); - }else + }else { rc =PTR_ERR (env); + }break ; }case IMP_EVENT_ACTIVE : {
drivers/staging/lustre/lustre/ptlrpc/client.c +2 -2
@@ -595,9 +595,9 @@ static int __ptlrpc_request_bufs_pack(struct ptlrpc_request *request,struct obd_import *imp = request->rq_import;int rc; -if (unlikely (ctx)) +if (unlikely (ctx)) { request->rq_cli_ctx =sptlrpc_cli_ctx_get (ctx); -else { + }else { rc =sptlrpc_req_get_ctx (request);if (rc) goto out_free;
drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c +5 -4
@@ -679,11 +679,11 @@ static ssize_t ptlrpc_lprocfs_nrs_seq_write(struct file *file,/** * The second token is either NULL, or an optional [reg|hp] string */ -if (strcmp (cmd,"reg" ) ==0 ) +if (strcmp (cmd,"reg" ) ==0 ) { queue =PTLRPC_NRS_QUEUE_REG ; -else if (strcmp (cmd,"hp" ) ==0 ) + }else if (strcmp (cmd,"hp" ) ==0 ) { queue =PTLRPC_NRS_QUEUE_HP ; -else { + }else { rc = -EINVAL ; goto out; }
@@ -693,8 +693,9 @@ default_queue:if (queue ==PTLRPC_NRS_QUEUE_HP && !nrs_svc_has_hp (svc)) { rc = -ENODEV ; goto out; - }else if (queue ==PTLRPC_NRS_QUEUE_BOTH && !nrs_svc_has_hp (svc)) + }else if (queue ==PTLRPC_NRS_QUEUE_BOTH && !nrs_svc_has_hp (svc)) { queue =PTLRPC_NRS_QUEUE_REG ; + }/** * Serialize NRS core lprocfs operations with policy registration/
Signature · 1 line
2.1.0
PATCH 2/3 staging/lustre: Fix blank line after/before {/} style
In reply to [email protected]From: Oleg Drokin <[email protected]>
This patch fixes all checkpatch occurences of
CHECK: Blank lines aren't necessary after an open brace '{'
CHECK: Blank lines aren't necessary before a close brace '}' in Lustre code.
--- drivers/staging/lustre/lustre/fld/fld_request.c | 1 - drivers/staging/lustre/lustre/include/lu_object.h | 1 - drivers/staging/lustre/lustre/ldlm/ldlm_lib.c | 1 - drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 1 - drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 1 - drivers/staging/lustre/lustre/llite/dir.c | 2 -- drivers/staging/lustre/lustre/llite/lproc_llite.c | 1 - drivers/staging/lustre/lustre/llite/xattr.c | 1 - drivers/staging/lustre/lustre/llite/xattr_cache.c | 1 - drivers/staging/lustre/lustre/lov/lov_io.c | 1 - drivers/staging/lustre/lustre/lov/lov_obd.c | 1 - drivers/staging/lustre/lustre/lov/lov_pack.c | 1 - drivers/staging/lustre/lustre/lov/lov_request.c | 1 - drivers/staging/lustre/lustre/lov/lovsub_object.c | 1 - drivers/staging/lustre/lustre/mdc/mdc_lib.c | 1 - drivers/staging/lustre/lustre/mdc/mdc_locks.c | 1 - drivers/staging/lustre/lustre/mgc/mgc_request.c | 1 - drivers/staging/lustre/lustre/obdclass/class_obd.c | 1 - drivers/staging/lustre/lustre/obdclass/genops.c | 1 - drivers/staging/lustre/lustre/obdclass/llog.c | 1 - drivers/staging/lustre/lustre/obdclass/lu_object.c | 2 -- drivers/staging/lustre/lustre/obdclass/obd_config.c | 1 - drivers/staging/lustre/lustre/obdecho/echo_client.c | 2 -- drivers/staging/lustre/lustre/osc/osc_cache.c | 1 - drivers/staging/lustre/lustre/osc/osc_io.c | 1 - drivers/staging/lustre/lustre/osc/osc_page.c | 1 - drivers/staging/lustre/lustre/osc/osc_request.c | 1 - drivers/staging/lustre/lustre/ptlrpc/client.c | 1 - drivers/staging/lustre/lustre/ptlrpc/events.c | 1 - drivers/staging/lustre/lustre/ptlrpc/import.c | 3 --- drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c | 1 - drivers/staging/lustre/lustre/ptlrpc/nrs.c | 1 - drivers/staging/lustre/lustre/ptlrpc/pack_generic.c | 3 --- 33 files changed, 40 deletions(-)
drivers/staging/lustre/lustre/fld/fld_request.c +0 -1
@@ -95,7 +95,6 @@ static void fld_exit_request(struct client_obd *cli)spin_lock (&cli->cl_loi_list_lock); cli->cl_r_in_flight--;list_for_each_safe (l, tmp, &cli->cl_cache_waiters) { -if (cli->cl_r_in_flight >= cli->cl_max_rpcs_in_flight) {/* No free request slots anymore */ break ;
drivers/staging/lustre/lustre/include/lu_object.h +0 -1
@@ -198,7 +198,6 @@ typedef int (*lu_printer_t)(const struct lu_env *env,* Operations specific for particular lu_object. */ struct lu_object_operations { -/** * Allocate lower-layer parts of the object by calling * lu_device_operations::ldo_object_alloc() of the corresponding
drivers/staging/lustre/lustre/ldlm/ldlm_lib.c +0 -1
@@ -429,7 +429,6 @@ err_ldlm:ldlm_put_ref (); err:return rc; - }EXPORT_SYMBOL (client_obd_setup);
drivers/staging/lustre/lustre/ldlm/ldlm_lock.c +0 -1
@@ -810,7 +810,6 @@ void ldlm_lock_decref_internal(struct ldlm_lock *lock, __u32 mode) }else if (!lock->l_readers && !lock->l_writers && !(lock->l_flags &LDLM_FL_NO_LRU ) && !(lock->l_flags &LDLM_FL_BL_AST )) { -LDLM_DEBUG (lock,"add lock into lru list" );/* If this is a client-side namespace and this was the last
drivers/staging/lustre/lustre/ldlm/ldlm_request.c +0 -1
@@ -1804,7 +1804,6 @@ static void ldlm_namespace_foreach(struct ldlm_namespace *ns,cfs_hash_for_each_nolock (ns->ns_rs_hash, ldlm_res_iter_helper, &helper); - }/* non-blocking function to manipulate a lock whose cb_data is being put away.
drivers/staging/lustre/lustre/llite/dir.c +0 -2
@@ -1056,7 +1056,6 @@ static int ll_ioc_copy_end(struct super_block *sb, struct hsm_copy *copy)/* hpk_errval must be >= 0 */ hpk.hpk_errval =EBUSY ; } - } progress:
@@ -1385,7 +1384,6 @@ out_free: lmv_out_free:obd_ioctl_freedata (buf, len);return rc; - }case LL_IOC_LOV_SETSTRIPE : {struct lov_user_md_v3 lumv3;
drivers/staging/lustre/lustre/llite/lproc_llite.c +0 -1
@@ -254,7 +254,6 @@ static ssize_t max_read_ahead_mb_store(struct kobject *kobj, pages_number *=1 << (20 -PAGE_CACHE_SHIFT );/* MB -> pages */ if (pages_number > totalram_pages /2 ) { -CERROR ("can't set file readahead more than %lu MB\n" , totalram_pages >> (20 -PAGE_CACHE_SHIFT +1 ));/*1/2 of RAM*/ return -ERANGE ;
drivers/staging/lustre/lustre/llite/xattr.c +0 -1
@@ -339,7 +339,6 @@ int ll_getxattr_common(struct inode *inode, const char *name,*/ if (xattr_type ==XATTR_ACL_ACCESS_T && !(sbi->ll_flags &LL_SBI_RMT_CLIENT )) { -struct posix_acl *acl;spin_lock (&lli->lli_lock);
drivers/staging/lustre/lustre/llite/xattr_cache.c +0 -1
@@ -229,7 +229,6 @@ static int ll_xattr_cache_valid(struct ll_inode_info *lli)*/ static int ll_xattr_cache_destroy_locked (struct ll_inode_info *lli) { -if (!ll_xattr_cache_valid (lli))return 0 ;
drivers/staging/lustre/lustre/lov/lov_io.c +0 -1
@@ -441,7 +441,6 @@ static int lov_io_rw_iter_init(const struct lu_env *env,/* fast path for common case. */ if (lio->lis_nr_subios !=1 && !cl_io_is_append (io)) { -lov_do_div64 (start, ssize); next = (start +1 ) * ssize;if (next <= start * ssize)
drivers/staging/lustre/lustre/lov/lov_obd.c +0 -1
@@ -964,7 +964,6 @@ int lov_process_config_base(struct obd_device *obd, struct lustre_cfg *lcfg,CERROR ("Unknown command: %d\n" , lcfg->lcfg_command); rc = -EINVAL ; goto out; - } } out:
drivers/staging/lustre/lustre/lov/lov_pack.c +0 -1
@@ -136,7 +136,6 @@ int lov_packmd(struct obd_export *exp, struct lov_mds_md **lmmp,CERROR ("bad mem LOV MAGIC: 0x%08X != 0x%08X nor 0x%08X\n" , lmm_magic,LOV_MAGIC_V1 ,LOV_MAGIC_V3 );return -EINVAL ; - }if (lsm) {
drivers/staging/lustre/lustre/lov/lov_request.c +0 -1
@@ -235,7 +235,6 @@ out:if (tmp_oa)kmem_cache_free (obdo_cachep, tmp_oa);return rc; - }int lov_fini_getattr_set (struct lov_request_set *set)
drivers/staging/lustre/lustre/lov/lovsub_object.c +0 -1
@@ -71,7 +71,6 @@ int lovsub_object_init(const struct lu_env *env, struct lu_object *obj, result = -ENOMEM ; }return result; - }static void lovsub_object_free (const struct lu_env *env,struct lu_object *obj)
drivers/staging/lustre/lustre/mdc/mdc_lib.c +0 -1
@@ -438,7 +438,6 @@ void mdc_getattr_pack(struct ptlrpc_request *req, __u64 valid, int flags,char *tmp =req_capsule_client_get (&req->rq_pill, &RMF_NAME );LOGL0 (op_data->op_name, op_data->op_namelen, tmp); - } }
drivers/staging/lustre/lustre/mdc/mdc_locks.c +0 -1
@@ -963,7 +963,6 @@ static int mdc_finish_intent_lock(struct obd_export *exp,if (fid_is_sane (&op_data->op_fid2) && it->it_create_mode &M_CHECK_STALE && it->it_op !=IT_GETATTR ) { -/* Also: did we find the same inode? */ /* sever can return one of two fids: * op_fid2 - new allocated fid - if file is created.
drivers/staging/lustre/lustre/mgc/mgc_request.c +0 -1
@@ -1720,7 +1720,6 @@ static int mgc_process_config(struct obd_device *obd, u32 len, void *buf)CERROR ("Unknown command: %d\n" , lcfg->lcfg_command); rc = -EINVAL ; goto out; - } } out:
drivers/staging/lustre/lustre/obdclass/class_obd.c +0 -1
@@ -335,7 +335,6 @@ int class_handle_ioctl(unsigned int cmd, unsigned long arg) err =0 ; goto out; } - }if (data->ioc_dev ==OBD_DEV_BY_DEVNAME ) {
drivers/staging/lustre/lustre/obdclass/genops.c +0 -1
@@ -604,7 +604,6 @@ int obd_init_caches(void) out:obd_cleanup_caches ();return -ENOMEM ; - }/* map connection to client */
drivers/staging/lustre/lustre/obdclass/llog.c +0 -1
@@ -265,7 +265,6 @@ repeat:for (rec = (struct llog_rec_hdr *)buf; (char *)rec < buf +LLOG_CHUNK_SIZE ; rec = (struct llog_rec_hdr *)((char *)rec + rec->lrh_len)) { -CDEBUG (D_OTHER ,"processing rec 0x%p type %#x\n" , rec, rec->lrh_type);
drivers/staging/lustre/lustre/obdclass/lu_object.c +0 -2
@@ -104,7 +104,6 @@ void lu_object_put(const struct lu_env *env, struct lu_object *o)if (!cfs_hash_bd_dec_and_lock (site->ls_obj_hash, &bd, &top->loh_ref)) {if (lu_object_is_dying (top)) { -/* * somebody may be waiting for this, currently only * used for cl_object, see cl_object_put_last().
@@ -358,7 +357,6 @@ int lu_site_purge(const struct lu_env *env, struct lu_site *s, int nr)if (count >0 && --count ==0 )break ; - }cfs_hash_bd_unlock (s->ls_obj_hash, &bd,1 );cond_resched ();
drivers/staging/lustre/lustre/obdclass/obd_config.c +0 -1
@@ -961,7 +961,6 @@ int class_process_config(struct lustre_cfg *lcfg)default : { err =obd_process_config (obd,sizeof (*lcfg), lcfg); goto out; - } } out:
drivers/staging/lustre/lustre/obdecho/echo_client.c +0 -2
@@ -1368,7 +1368,6 @@ static int echo_client_kbrw(struct echo_device *ed, int rw, struct obdo *oa,for (i =0 , pgp = pga, off = offset; i < npages; i++, pgp++, off +=PAGE_CACHE_SIZE ) { -LASSERT (!pgp->pg);/* for cleanup */ rc = -ENOMEM ;
@@ -1845,7 +1844,6 @@ static int __init obdecho_init(void)static void /*__exit*/ obdecho_exit (void ) {echo_client_exit (); - }MODULE_AUTHOR ("OpenSFS, Inc. <http://www.lustre.org/>" );
drivers/staging/lustre/lustre/osc/osc_cache.c +0 -1
@@ -1820,7 +1820,6 @@ static void osc_process_ar(struct osc_async_rc *ar, __u64 xid, ar->ar_force_sync =1 ; ar->ar_min_xid =ptlrpc_sample_next_xid ();return ; - }if (ar->ar_force_sync && (xid >= ar->ar_min_xid))
drivers/staging/lustre/lustre/osc/osc_io.c +0 -1
@@ -782,7 +782,6 @@ static void osc_req_attr_set(const struct lu_env *env, oa->o_valid |=OBD_MD_FLID ; }if (flags &OBD_MD_FLHANDLE ) { - clerq = slice->crs_req;LASSERT (!list_empty (&clerq->crq_pages)); apage = container_of(clerq->crq_pages.next,
drivers/staging/lustre/lustre/osc/osc_page.c +0 -1
@@ -759,7 +759,6 @@ static int osc_lru_reserve(const struct lu_env *env, struct osc_object *obj,LASSERT (atomic_read (cli->cl_lru_left) >=0 );while (!atomic_add_unless (cli->cl_lru_left,-1 ,0 )) { -/* run out of LRU spaces, try to drop some by itself */ rc =osc_lru_reclaim (cli);if (rc <0 )
drivers/staging/lustre/lustre/osc/osc_request.c +0 -1
@@ -837,7 +837,6 @@ static void osc_announce_cached(struct client_obd *cli, struct obdo *oa,spin_unlock (&cli->cl_loi_list_lock);CDEBUG (D_CACHE ,"dirty: %llu undirty: %u dropped %u grant: %llu\n" , oa->o_dirty, oa->o_undirty, oa->o_dropped, oa->o_grant); - }void osc_update_next_shrink (struct client_obd *cli)
drivers/staging/lustre/lustre/ptlrpc/client.c +0 -1
@@ -1082,7 +1082,6 @@ static int ptlrpc_console_allow(struct ptlrpc_request *req)*/ if ((lustre_handle_is_used (&req->rq_import->imp_remote_handle)) && (opc ==OST_CONNECT || opc ==MDS_CONNECT || opc ==MGS_CONNECT )) { -/* Suppress timed out reconnect requests */ if (req->rq_timedout)return 0 ;
drivers/staging/lustre/lustre/ptlrpc/events.c +0 -1
@@ -69,7 +69,6 @@ void request_out_callback(lnet_event_t *ev) req->rq_req_unlink =0 ;if (ev->type ==LNET_EVENT_UNLINK || ev->status !=0 ) { -/* Failed send: make it seem like the reply timed out, just * like failing sends in client.c does currently... */
drivers/staging/lustre/lustre/ptlrpc/import.c +0 -3
@@ -1001,7 +1001,6 @@ finish:return 0 ; } }else { -spin_lock (&imp->imp_lock);list_del (&imp->imp_conn_current->oic_item);list_add (&imp->imp_conn_current->oic_item, &imp->imp_conn_list);
@@ -1370,7 +1369,6 @@ int ptlrpc_import_recovery_state_machine(struct obd_import *imp)if (rc) goto out; } - }if (imp->imp_state ==LUSTRE_IMP_REPLAY_WAIT ) {
@@ -1453,7 +1451,6 @@ int ptlrpc_disconnect_import(struct obd_import *imp, int noclose) back_to_sleep, LWI_ON_SIGNAL_NOOP, NULL); rc =l_wait_event (imp->imp_recovery_waitq, !ptlrpc_import_in_recovery (imp), &lwi); - }spin_lock (&imp->imp_lock);
drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c +0 -1
@@ -1321,6 +1321,5 @@ int lprocfs_wr_pinger_recov(struct file *file, const char __user *buffer,up_read (&obd->u.cli.cl_sem);return count; - }EXPORT_SYMBOL (lprocfs_wr_pinger_recov);
drivers/staging/lustre/lustre/ptlrpc/nrs.c +0 -1
@@ -1038,7 +1038,6 @@ static int nrs_policy_unregister_locked(struct ptlrpc_nrs_pol_desc *desc)LASSERT (mutex_is_locked (&ptlrpc_all_services_mutex));list_for_each_entry (svc, &ptlrpc_all_services, srv_list) { -if (!nrs_policy_compatible (svc, desc) ||unlikely (svc->srv_is_stopping))continue ;
drivers/staging/lustre/lustre/ptlrpc/pack_generic.c +0 -3
@@ -1160,7 +1160,6 @@ __u32 lustre_msg_get_timeout(struct lustre_msg *msg)if (!pb) {CERROR ("invalid msg %p: no ptlrpc body!\n" , msg);return 0 ; - }return pb->pb_timeout; }
@@ -1179,7 +1178,6 @@ __u32 lustre_msg_get_service_time(struct lustre_msg *msg)if (!pb) {CERROR ("invalid msg %p: no ptlrpc body!\n" , msg);return 0 ; - }return pb->pb_service_time; }
@@ -1572,7 +1570,6 @@ static void lustre_swab_obdo(struct obdo *o) CLASSERT(offsetof(typeof(*o), o_padding_4) != 0); CLASSERT(offsetof(typeof(*o), o_padding_5) != 0); CLASSERT(offsetof(typeof(*o), o_padding_6) != 0); - }void lustre_swab_obd_statfs (struct obd_statfs *os)
Signature · 1 line
2.1.0
Greg Kroah-Hartman
maintainerRe: [PATCH 2/3] staging/lustre: Fix blank line after/before {/} style
Quoted 8 lines · green at linuxhacker.ru
From: Oleg Drokin <[email protected]>
This patch fixes all checkpatch occurences of
CHECK: Blank lines aren't necessary after an open brace '{'
CHECK: Blank lines aren't necessary before a close brace '}' in Lustre code.Signed-off-by Oleg Drokin <[email protected]>
Doesn't apply anymore after merging with 4.6-rc3, can you rebase and resend?
thanks,
greg k-h
PATCH 3/3 staging/lustre: Get rid of ldlm_policy_res_t typedef
In reply to [email protected]From: Oleg Drokin <[email protected]>
Directly use enum ldlm_policy_res everywhere.
--- drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 2 - drivers/staging/lustre/lustre/ldlm/ldlm_request.c | 49 +++++++++++----------- 2 files changed, 24 insertions(+), 27 deletions(-)
drivers/staging/lustre/lustre/ldlm/ldlm_internal.h +0 -2
@@ -218,8 +218,6 @@ enum ldlm_policy_res { LDLM_POLICY_SKIP_LOCK }; -typedef enum ldlm_policy_res ldlm_policy_res_t ; -#define LDLM_POOL_SYSFS_PRINT_int (v) sprintf(buf, "%d\n", v)#define LDLM_POOL_SYSFS_SET_int (a, b) { a = b; }#define LDLM_POOL_SYSFS_PRINT_u64 (v) sprintf(buf, "%lld\n", v)
drivers/staging/lustre/lustre/ldlm/ldlm_request.c +24 -25
103 lines not on this page · raw message
Signature · 1 line
2.1.0