| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20150822141628.GA5818@zed>
Date: Sat, 22 Aug 2015 17:16:28 +0300
From: Mike Rapoport <mike.rapoport@...il.com>
To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: Oleg Drokin <oleg.drokin@...el.com>,
Andreas Dilger <andreas.dilger@...el.com>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] staging: lustre: fix whitespace errors reported by
checkpatch.pl
Please ignore this one, I'm going to resend it along with some others in
a while,
Thanks.
On Wed, Aug 19, 2015 at 09:47:22AM +0300, Mike Rapoport wrote:
> Added/removed spaces and replaced '+1' with '1' in several places to eliminate
> SPACING and POINTER_LOCATION errors reported by checkpatch.pl
>
> Signed-off-by: Mike Rapoport <mike.rapoport@...il.com>
> ---
>
> This time with more elaborate changelog...
>
> .../staging/lustre/lnet/klnds/socklnd/socklnd_cb.c | 4 +-
> drivers/staging/lustre/lnet/selftest/conrpc.h | 4 +-
> drivers/staging/lustre/lnet/selftest/console.c | 16 +--
> drivers/staging/lustre/lnet/selftest/selftest.h | 2 +-
> drivers/staging/lustre/lustre/include/cl_object.h | 14 +-
> drivers/staging/lustre/lustre/include/lu_object.h | 16 +--
> .../lustre/lustre/include/lustre/lustre_idl.h | 12 +-
> .../lustre/lustre/include/lustre/lustre_user.h | 4 +-
> .../staging/lustre/lustre/include/lustre_capa.h | 2 +-
> .../staging/lustre/lustre/include/lustre_disk.h | 2 +-
> drivers/staging/lustre/lustre/include/lustre_dlm.h | 6 +-
> .../lustre/lustre/include/lustre_dlm_flags.h | 152 ++++++++++-----------
> drivers/staging/lustre/lustre/include/lustre_lib.h | 6 +-
> drivers/staging/lustre/lustre/include/lustre_net.h | 2 +-
> drivers/staging/lustre/lustre/include/obd.h | 2 +-
> .../staging/lustre/lustre/include/obd_support.h | 2 +-
> drivers/staging/lustre/lustre/libcfs/tracefile.c | 4 +-
> drivers/staging/lustre/lustre/llite/file.c | 4 +-
> .../staging/lustre/lustre/llite/llite_internal.h | 2 +-
> drivers/staging/lustre/lustre/llite/lproc_llite.c | 2 +-
> drivers/staging/lustre/lustre/llite/rw.c | 6 +-
> drivers/staging/lustre/lustre/llite/statahead.c | 4 +-
> drivers/staging/lustre/lustre/llite/vvp_io.c | 2 +-
> drivers/staging/lustre/lustre/lmv/lmv_obd.c | 2 +-
> drivers/staging/lustre/lustre/lov/lov_obd.c | 4 +-
> drivers/staging/lustre/lustre/obdclass/cl_io.c | 10 +-
> drivers/staging/lustre/lustre/obdclass/cl_lock.c | 6 +-
> drivers/staging/lustre/lustre/obdclass/cl_object.c | 2 +-
> drivers/staging/lustre/lustre/obdclass/class_obd.c | 4 +-
> drivers/staging/lustre/lustre/obdclass/genops.c | 6 +-
> drivers/staging/lustre/lustre/obdclass/llog.c | 4 +-
> .../staging/lustre/lustre/obdclass/obd_config.c | 2 +-
> drivers/staging/lustre/lustre/osc/osc_request.c | 6 +-
> .../staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c | 2 +-
> 34 files changed, 159 insertions(+), 159 deletions(-)
>
> diff --git a/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_cb.c b/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_cb.c
> index 0d5aac6..4f9c5c8 100644
> --- a/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_cb.c
> +++ b/drivers/staging/lustre/lnet/klnds/socklnd/socklnd_cb.c
> @@ -714,7 +714,7 @@ ksocknal_queue_tx_locked (ksock_tx_t *tx, ksock_conn_t *conn)
> LASSERT(tx->tx_resid == tx->tx_nob);
>
> CDEBUG (D_NET, "Packet %p type %d, nob %d niov %d nkiov %d\n",
> - tx, (tx->tx_lnetmsg != NULL) ? tx->tx_lnetmsg->msg_hdr.type:
> + tx, (tx->tx_lnetmsg != NULL) ? tx->tx_lnetmsg->msg_hdr.type :
> KSOCK_MSG_NOOP,
> tx->tx_nob, tx->tx_niov, tx->tx_nkiov);
>
> @@ -1092,7 +1092,7 @@ ksocknal_new_packet (ksock_conn_t *conn, int nob_to_skip)
> conn->ksnc_rx_iov[niov].iov_len = nob;
> niov++;
> skipped += nob;
> - nob_to_skip -=nob;
> + nob_to_skip -= nob;
>
> } while (nob_to_skip != 0 && /* mustn't overflow conn's rx iov */
> niov < sizeof(conn->ksnc_rx_iov_space) / sizeof (struct iovec));
> diff --git a/drivers/staging/lustre/lnet/selftest/conrpc.h b/drivers/staging/lustre/lnet/selftest/conrpc.h
> index 7d33cf9..ae7ed75 100644
> --- a/drivers/staging/lustre/lnet/selftest/conrpc.h
> +++ b/drivers/staging/lustre/lnet/selftest/conrpc.h
> @@ -105,8 +105,8 @@ typedef struct lstcon_rpc_trans {
>
> #define LST_TRANS_STATQRY 0x21
>
> -typedef int (* lstcon_rpc_cond_func_t)(int, struct lstcon_node *, void *);
> -typedef int (* lstcon_rpc_readent_func_t)(int, srpc_msg_t *, lstcon_rpc_ent_t *);
> +typedef int (*lstcon_rpc_cond_func_t)(int, struct lstcon_node *, void *);
> +typedef int (*lstcon_rpc_readent_func_t)(int, srpc_msg_t *, lstcon_rpc_ent_t *);
>
> int lstcon_sesrpc_prep(struct lstcon_node *nd, int transop,
> unsigned version, lstcon_rpc_t **crpc);
> diff --git a/drivers/staging/lustre/lnet/selftest/console.c b/drivers/staging/lustre/lnet/selftest/console.c
> index f47c8f2..5c624e1 100644
> --- a/drivers/staging/lustre/lnet/selftest/console.c
> +++ b/drivers/staging/lustre/lnet/selftest/console.c
> @@ -308,7 +308,7 @@ lstcon_group_ndlink_release(lstcon_group_t *grp, lstcon_ndlink_t *ndl)
> {
> list_del_init(&ndl->ndl_link);
> lstcon_ndlink_release(ndl);
> - grp->grp_nnode --;
> + grp->grp_nnode--;
> }
>
> static void
> @@ -320,7 +320,7 @@ lstcon_group_ndlink_move(lstcon_group_t *old,
>
> list_del(&ndl->ndl_hlink);
> list_del(&ndl->ndl_link);
> - old->grp_nnode --;
> + old->grp_nnode--;
>
> list_add_tail(&ndl->ndl_hlink, &new->grp_ndl_hash[idx]);
> list_add_tail(&ndl->ndl_link, &new->grp_ndl_list);
> @@ -527,7 +527,7 @@ lstcon_group_add(char *name)
> lstcon_group_t *grp;
> int rc;
>
> - rc = (lstcon_group_find(name, &grp) == 0)? -EEXIST: 0;
> + rc = (lstcon_group_find(name, &grp) == 0) ? -EEXIST : 0;
> if (rc != 0) {
> /* find a group with same name */
> lstcon_group_put(grp);
> @@ -816,7 +816,7 @@ lstcon_group_info(char *name, lstcon_ndlist_ent_t *gents_p,
> LST_NODE_STATE_COUNTER(ndl->ndl_node, gentp);
>
> rc = copy_to_user(gents_p, gentp,
> - sizeof(lstcon_ndlist_ent_t)) ? -EFAULT: 0;
> + sizeof(lstcon_ndlist_ent_t)) ? -EFAULT : 0;
>
> LIBCFS_FREE(gentp, sizeof(lstcon_ndlist_ent_t));
>
> @@ -847,7 +847,7 @@ lstcon_batch_add(char *name)
> int i;
> int rc;
>
> - rc = (lstcon_batch_find(name, &bat) == 0)? -EEXIST: 0;
> + rc = (lstcon_batch_find(name, &bat) == 0) ? -EEXIST : 0;
> if (rc != 0) {
> CDEBUG(D_NET, "Batch %s already exists\n", name);
> return rc;
> @@ -911,7 +911,7 @@ lstcon_batch_list(int index, int len, char *name_up)
> list_for_each_entry(bat, &console_session.ses_bat_list, bat_link) {
> if (index-- == 0) {
> return copy_to_user(name_up, bat->bat_name, len) ?
> - -EFAULT: 0;
> + -EFAULT : 0;
> }
> }
>
> @@ -956,7 +956,7 @@ lstcon_batch_info(char *name, lstcon_test_batch_ent_t *ent_up, int server,
> &test->tes_dst_grp->grp_ndl_list;
>
> if (dents_up != NULL) {
> - rc = lstcon_nodes_getent((server ? srvlst: clilst),
> + rc = lstcon_nodes_getent((server ? srvlst : clilst),
> index_p, ndent_p, dents_up);
> return rc;
> }
> @@ -1683,7 +1683,7 @@ int
> lstcon_session_match(lst_sid_t sid)
> {
> return (console_session.ses_id.ses_nid == sid.ses_nid &&
> - console_session.ses_id.ses_stamp == sid.ses_stamp) ? 1: 0;
> + console_session.ses_id.ses_stamp == sid.ses_stamp) ? 1 : 0;
> }
>
> static void
> diff --git a/drivers/staging/lustre/lnet/selftest/selftest.h b/drivers/staging/lustre/lnet/selftest/selftest.h
> index 7939e4e..dfe3342 100644
> --- a/drivers/staging/lustre/lnet/selftest/selftest.h
> +++ b/drivers/staging/lustre/lnet/selftest/selftest.h
> @@ -562,7 +562,7 @@ static inline const char *
> swi_state2str (int state)
> {
> #define STATE2STR(x) case x: return #x
> - switch(state) {
> + switch (state) {
> default:
> LBUG();
> STATE2STR(SWI_STATE_NEWBORN);
> diff --git a/drivers/staging/lustre/lustre/include/cl_object.h b/drivers/staging/lustre/lustre/include/cl_object.h
> index d56c8be..04dcb51 100644
> --- a/drivers/staging/lustre/lustre/include/cl_object.h
> +++ b/drivers/staging/lustre/lustre/include/cl_object.h
> @@ -1095,7 +1095,7 @@ do { \
> \
> if (cfs_cdebug_show(mask, DEBUG_SUBSYSTEM)) { \
> cl_page_print(env, &msgdata, lu_cdebug_printer, page); \
> - CDEBUG(mask, format , ## __VA_ARGS__); \
> + CDEBUG(mask, format, ## __VA_ARGS__); \
> } \
> } while (0)
>
> @@ -1108,7 +1108,7 @@ do { \
> \
> if (cfs_cdebug_show(mask, DEBUG_SUBSYSTEM)) { \
> cl_page_header_print(env, &msgdata, lu_cdebug_printer, page); \
> - CDEBUG(mask, format , ## __VA_ARGS__); \
> + CDEBUG(mask, format, ## __VA_ARGS__); \
> } \
> } while (0)
>
> @@ -1475,13 +1475,13 @@ enum cl_lock_flags {
> * lock has been cancelled. This flag is never cleared once set (by
> * cl_lock_cancel0()).
> */
> - CLF_CANCELLED = 1 << 0,
> + CLF_CANCELLED = 1 << 0,
> /** cancellation is pending for this lock. */
> - CLF_CANCELPEND = 1 << 1,
> + CLF_CANCELPEND = 1 << 1,
> /** destruction is pending for this lock. */
> - CLF_DOOMED = 1 << 2,
> + CLF_DOOMED = 1 << 2,
> /** from enqueue RPC reply upcall. */
> - CLF_FROM_UPCALL= 1 << 3,
> + CLF_FROM_UPCALL = 1 << 3,
> };
>
> /**
> @@ -1833,7 +1833,7 @@ do { \
> \
> if (cfs_cdebug_show(mask, DEBUG_SUBSYSTEM)) { \
> cl_lock_print(env, &msgdata, lu_cdebug_printer, lock); \
> - CDEBUG(mask, format , ## __VA_ARGS__); \
> + CDEBUG(mask, format, ## __VA_ARGS__); \
> } \
> } while (0)
>
> diff --git a/drivers/staging/lustre/lustre/include/lu_object.h b/drivers/staging/lustre/lustre/include/lu_object.h
> index e1d72a7..a16c9ea 100644
> --- a/drivers/staging/lustre/lustre/include/lu_object.h
> +++ b/drivers/staging/lustre/lustre/include/lu_object.h
> @@ -790,7 +790,7 @@ do { \
> \
> if (cfs_cdebug_show(mask, DEBUG_SUBSYSTEM)) { \
> lu_object_print(env, &msgdata, lu_cdebug_printer, object);\
> - CDEBUG(mask, format , ## __VA_ARGS__); \
> + CDEBUG(mask, format, ## __VA_ARGS__); \
> } \
> } while (0)
>
> @@ -805,7 +805,7 @@ do { \
> lu_object_header_print(env, &msgdata, lu_cdebug_printer,\
> (object)->lo_header); \
> lu_cdebug_printer(env, &msgdata, "\n"); \
> - CDEBUG(mask, format , ## __VA_ARGS__); \
> + CDEBUG(mask, format, ## __VA_ARGS__); \
> } \
> } while (0)
>
> @@ -1131,7 +1131,7 @@ struct lu_context_key {
> \
> return value; \
> } \
> - struct __##mod##__dummy_init {;} /* semicolon catcher */
> + struct __##mod##__dummy_init {; } /* semicolon catcher */
>
> #define LU_KEY_FINI(mod, type) \
> static void mod##_key_fini(const struct lu_context *ctx, \
> @@ -1141,7 +1141,7 @@ struct lu_context_key {
> \
> OBD_FREE_PTR(info); \
> } \
> - struct __##mod##__dummy_fini {;} /* semicolon catcher */
> + struct __##mod##__dummy_fini {; } /* semicolon catcher */
>
> #define LU_KEY_INIT_FINI(mod, type) \
> LU_KEY_INIT(mod, type); \
> @@ -1193,28 +1193,28 @@ void lu_context_key_revive (struct lu_context_key *key);
> mod##_key_init_generic(__VA_ARGS__, NULL); \
> return lu_context_key_register_many(__VA_ARGS__, NULL); \
> } \
> - struct __##mod##_dummy_type_init {;}
> + struct __##mod##_dummy_type_init {; }
>
> #define LU_TYPE_FINI(mod, ...) \
> static void mod##_type_fini(struct lu_device_type *t) \
> { \
> lu_context_key_degister_many(__VA_ARGS__, NULL); \
> } \
> - struct __##mod##_dummy_type_fini {;}
> + struct __##mod##_dummy_type_fini {; }
>
> #define LU_TYPE_START(mod, ...) \
> static void mod##_type_start(struct lu_device_type *t) \
> { \
> lu_context_key_revive_many(__VA_ARGS__, NULL); \
> } \
> - struct __##mod##_dummy_type_start {;}
> + struct __##mod##_dummy_type_start {; }
>
> #define LU_TYPE_STOP(mod, ...) \
> static void mod##_type_stop(struct lu_device_type *t) \
> { \
> lu_context_key_quiesce_many(__VA_ARGS__, NULL); \
> } \
> - struct __##mod##_dummy_type_stop {;}
> + struct __##mod##_dummy_type_stop {; }
>
>
>
> diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h
> index ac78dbc..e79af19 100644
> --- a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h
> +++ b/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h
> @@ -827,7 +827,7 @@ static inline int lu_fid_eq(const struct lu_fid *f0, const struct lu_fid *f1)
> typeof(val0) __val0 = (val0); \
> typeof(val1) __val1 = (val1); \
> \
> - (__val0 == __val1 ? 0 : __val0 > __val1 ? +1 : -1); \
> + (__val0 == __val1 ? 0 : __val0 > __val1 ? 1 : -1); \
> })
>
> static inline int lu_fid_cmp(const struct lu_fid *f0,
> @@ -1404,9 +1404,9 @@ void lustre_swab_connect(struct obd_connect_data *ocd);
> * algorithm and also the OBD_FL_CKSUM* flags.
> */
> typedef enum {
> - OBD_CKSUM_CRC32 = 0x00000001,
> - OBD_CKSUM_ADLER = 0x00000002,
> - OBD_CKSUM_CRC32C= 0x00000004,
> + OBD_CKSUM_CRC32 = 0x00000001,
> + OBD_CKSUM_ADLER = 0x00000002,
> + OBD_CKSUM_CRC32C = 0x00000004,
> } cksum_type_t;
>
> /*
> @@ -1444,7 +1444,7 @@ enum obdo_flags {
> OBD_FL_DELORPHAN = 0x00000004, /* if set in o_flags delete orphans */
> OBD_FL_NORPC = 0x00000008, /* set in o_flags do in OSC not OST */
> OBD_FL_IDONLY = 0x00000010, /* set in o_flags only adjust obj id*/
> - OBD_FL_RECREATE_OBJS= 0x00000020, /* recreate missing obj */
> + OBD_FL_RECREATE_OBJS = 0x00000020, /* recreate missing obj */
> OBD_FL_DEBUG_CHECK = 0x00000040, /* echo client/server debug check */
> OBD_FL_NO_USRQUOTA = 0x00000100, /* the object's owner is over quota */
> OBD_FL_NO_GRPQUOTA = 0x00000200, /* the object's group is over quota */
> @@ -3574,7 +3574,7 @@ struct link_ea_entry {
> unsigned char lee_reclen[2];
> unsigned char lee_parent_fid[sizeof(struct lu_fid)];
> char lee_name[0];
> -}__attribute__((packed));
> +} __attribute__((packed));
>
> /** fid2path request/reply structure */
> struct getinfo_fid2path {
> diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_user.h b/drivers/staging/lustre/lustre/include/lustre/lustre_user.h
> index 9b1bb23..d2836a8 100644
> --- a/drivers/staging/lustre/lustre/include/lustre/lustre_user.h
> +++ b/drivers/staging/lustre/lustre/include/lustre/lustre_user.h
> @@ -798,14 +798,14 @@ struct changelog_ext_rec {
>
> static inline int changelog_rec_size(struct changelog_rec *rec)
> {
> - return CHANGELOG_REC_EXTENDED(rec) ? sizeof(struct changelog_ext_rec):
> + return CHANGELOG_REC_EXTENDED(rec) ? sizeof(struct changelog_ext_rec) :
> sizeof(*rec);
> }
>
> static inline char *changelog_rec_name(struct changelog_rec *rec)
> {
> return CHANGELOG_REC_EXTENDED(rec) ?
> - ((struct changelog_ext_rec *)rec)->cr_name: rec->cr_name;
> + ((struct changelog_ext_rec *)rec)->cr_name : rec->cr_name;
> }
>
> static inline int changelog_rec_snamelen(struct changelog_ext_rec *rec)
> diff --git a/drivers/staging/lustre/lustre/include/lustre_capa.h b/drivers/staging/lustre/lustre/include/lustre_capa.h
> index fe19534..11bcdb9 100644
> --- a/drivers/staging/lustre/lustre/include/lustre_capa.h
> +++ b/drivers/staging/lustre/lustre/include/lustre_capa.h
> @@ -171,7 +171,7 @@ CDEBUG(level, fmt " capability key@%p seq %llu keyid %u\n", \
> ##args, k, capa_key_seq(k), capa_key_keyid(k)); \
> } while (0)
>
> -typedef int (* renew_capa_cb_t)(struct obd_capa *, struct lustre_capa *);
> +typedef int (*renew_capa_cb_t)(struct obd_capa *, struct lustre_capa *);
>
> /* obdclass/capa.c */
> extern struct list_head capa_list[];
> diff --git a/drivers/staging/lustre/lustre/include/lustre_disk.h b/drivers/staging/lustre/lustre/include/lustre_disk.h
> index 9b28331..131985d 100644
> --- a/drivers/staging/lustre/lustre/include/lustre_disk.h
> +++ b/drivers/staging/lustre/lustre/include/lustre_disk.h
> @@ -365,7 +365,7 @@ static inline void check_lcd(char *obd_name, int index,
> struct lsd_client_data *lcd)
> {
> int length = sizeof(lcd->lcd_uuid);
> - if (strnlen((char*)lcd->lcd_uuid, length) == length) {
> + if (strnlen((char *)lcd->lcd_uuid, length) == length) {
> lcd->lcd_uuid[length - 1] = '\0';
>
> LCONSOLE_ERROR("the client UUID (%s) on %s for exports stored in last_rcvd(index = %d) is bad!\n",
> diff --git a/drivers/staging/lustre/lustre/include/lustre_dlm.h b/drivers/staging/lustre/lustre/include/lustre_dlm.h
> index 3552546..92b24be 100644
> --- a/drivers/staging/lustre/lustre/include/lustre_dlm.h
> +++ b/drivers/staging/lustre/lustre/include/lustre_dlm.h
> @@ -1025,7 +1025,7 @@ char *ldlm_it2str(int it);
> * with a debugging message that is ldlm-related
> */
> #define LDLM_DEBUG_NOLOCK(format, a...) \
> - CDEBUG(D_DLMTRACE, "### " format "\n" , ##a)
> + CDEBUG(D_DLMTRACE, "### " format "\n", ##a)
>
> /**
> * Support function for lock information printing into debug logs.
> @@ -1051,7 +1051,7 @@ void _ldlm_lock_debug(struct ldlm_lock *lock,
> #define LDLM_DEBUG_LIMIT(mask, lock, fmt, a...) do { \
> static struct cfs_debug_limit_state _ldlm_cdls; \
> LIBCFS_DEBUG_MSG_DATA_DECL(msgdata, mask, &_ldlm_cdls); \
> - ldlm_lock_debug(&msgdata, mask, &_ldlm_cdls, lock, "### " fmt , ##a);\
> + ldlm_lock_debug(&msgdata, mask, &_ldlm_cdls, lock, "### " fmt, ##a);\
> } while (0)
>
> #define LDLM_ERROR(lock, fmt, a...) LDLM_DEBUG_LIMIT(D_ERROR, lock, fmt, ## a)
> @@ -1062,7 +1062,7 @@ void _ldlm_lock_debug(struct ldlm_lock *lock,
> if (likely(lock != NULL)) { \
> LIBCFS_DEBUG_MSG_DATA_DECL(msgdata, D_DLMTRACE, NULL); \
> ldlm_lock_debug(&msgdata, D_DLMTRACE, NULL, lock, \
> - "### " fmt , ##a); \
> + "### " fmt, ##a); \
> } else { \
> LDLM_DEBUG_NOLOCK("no dlm lock: " fmt, ##a); \
> } \
> diff --git a/drivers/staging/lustre/lustre/include/lustre_dlm_flags.h b/drivers/staging/lustre/lustre/include/lustre_dlm_flags.h
> index d4cc096..d27bdae0 100644
> --- a/drivers/staging/lustre/lustre/include/lustre_dlm_flags.h
> +++ b/drivers/staging/lustre/lustre/include/lustre_dlm_flags.h
> @@ -51,58 +51,58 @@
>
> /** extent, mode, or resource changed */
> #define LDLM_FL_LOCK_CHANGED 0x0000000000000001ULL // bit 0
> -#define ldlm_is_lock_changed(_l) LDLM_TEST_FLAG(( _l), 1ULL << 0)
> -#define ldlm_set_lock_changed(_l) LDLM_SET_FLAG(( _l), 1ULL << 0)
> +#define ldlm_is_lock_changed(_l) LDLM_TEST_FLAG((_l), 1ULL << 0)
> +#define ldlm_set_lock_changed(_l) LDLM_SET_FLAG((_l), 1ULL << 0)
> #define ldlm_clear_lock_changed(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 0)
>
> /**
> * Server placed lock on granted list, or a recovering client wants the
> * lock added to the granted list, no questions asked. */
> #define LDLM_FL_BLOCK_GRANTED 0x0000000000000002ULL // bit 1
> -#define ldlm_is_block_granted(_l) LDLM_TEST_FLAG(( _l), 1ULL << 1)
> -#define ldlm_set_block_granted(_l) LDLM_SET_FLAG(( _l), 1ULL << 1)
> +#define ldlm_is_block_granted(_l) LDLM_TEST_FLAG((_l), 1ULL << 1)
> +#define ldlm_set_block_granted(_l) LDLM_SET_FLAG((_l), 1ULL << 1)
> #define ldlm_clear_block_granted(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 1)
>
> /**
> * Server placed lock on conv list, or a recovering client wants the lock
> * added to the conv list, no questions asked. */
> #define LDLM_FL_BLOCK_CONV 0x0000000000000004ULL // bit 2
> -#define ldlm_is_block_conv(_l) LDLM_TEST_FLAG(( _l), 1ULL << 2)
> -#define ldlm_set_block_conv(_l) LDLM_SET_FLAG(( _l), 1ULL << 2)
> +#define ldlm_is_block_conv(_l) LDLM_TEST_FLAG((_l), 1ULL << 2)
> +#define ldlm_set_block_conv(_l) LDLM_SET_FLAG((_l), 1ULL << 2)
> #define ldlm_clear_block_conv(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 2)
>
> /**
> * Server placed lock on wait list, or a recovering client wants the lock
> * added to the wait list, no questions asked. */
> #define LDLM_FL_BLOCK_WAIT 0x0000000000000008ULL // bit 3
> -#define ldlm_is_block_wait(_l) LDLM_TEST_FLAG(( _l), 1ULL << 3)
> -#define ldlm_set_block_wait(_l) LDLM_SET_FLAG(( _l), 1ULL << 3)
> +#define ldlm_is_block_wait(_l) LDLM_TEST_FLAG((_l), 1ULL << 3)
> +#define ldlm_set_block_wait(_l) LDLM_SET_FLAG((_l), 1ULL << 3)
> #define ldlm_clear_block_wait(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 3)
>
> /** blocking or cancel packet was queued for sending. */
> #define LDLM_FL_AST_SENT 0x0000000000000020ULL // bit 5
> -#define ldlm_is_ast_sent(_l) LDLM_TEST_FLAG(( _l), 1ULL << 5)
> -#define ldlm_set_ast_sent(_l) LDLM_SET_FLAG(( _l), 1ULL << 5)
> +#define ldlm_is_ast_sent(_l) LDLM_TEST_FLAG((_l), 1ULL << 5)
> +#define ldlm_set_ast_sent(_l) LDLM_SET_FLAG((_l), 1ULL << 5)
> #define ldlm_clear_ast_sent(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 5)
>
> /**
> * Lock is being replayed. This could probably be implied by the fact that
> * one of BLOCK_{GRANTED,CONV,WAIT} is set, but that is pretty dangerous. */
> #define LDLM_FL_REPLAY 0x0000000000000100ULL // bit 8
> -#define ldlm_is_replay(_l) LDLM_TEST_FLAG(( _l), 1ULL << 8)
> -#define ldlm_set_replay(_l) LDLM_SET_FLAG(( _l), 1ULL << 8)
> +#define ldlm_is_replay(_l) LDLM_TEST_FLAG((_l), 1ULL << 8)
> +#define ldlm_set_replay(_l) LDLM_SET_FLAG((_l), 1ULL << 8)
> #define ldlm_clear_replay(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 8)
>
> /** Don't grant lock, just do intent. */
> #define LDLM_FL_INTENT_ONLY 0x0000000000000200ULL // bit 9
> -#define ldlm_is_intent_only(_l) LDLM_TEST_FLAG(( _l), 1ULL << 9)
> -#define ldlm_set_intent_only(_l) LDLM_SET_FLAG(( _l), 1ULL << 9)
> +#define ldlm_is_intent_only(_l) LDLM_TEST_FLAG((_l), 1ULL << 9)
> +#define ldlm_set_intent_only(_l) LDLM_SET_FLAG((_l), 1ULL << 9)
> #define ldlm_clear_intent_only(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 9)
>
> /** lock request has intent */
> #define LDLM_FL_HAS_INTENT 0x0000000000001000ULL // bit 12
> -#define ldlm_is_has_intent(_l) LDLM_TEST_FLAG(( _l), 1ULL << 12)
> -#define ldlm_set_has_intent(_l) LDLM_SET_FLAG(( _l), 1ULL << 12)
> +#define ldlm_is_has_intent(_l) LDLM_TEST_FLAG((_l), 1ULL << 12)
> +#define ldlm_set_has_intent(_l) LDLM_SET_FLAG((_l), 1ULL << 12)
> #define ldlm_clear_has_intent(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 12)
>
> /** flock deadlock detected */
> @@ -113,28 +113,28 @@
>
> /** discard (no writeback) on cancel */
> #define LDLM_FL_DISCARD_DATA 0x0000000000010000ULL // bit 16
> -#define ldlm_is_discard_data(_l) LDLM_TEST_FLAG(( _l), 1ULL << 16)
> -#define ldlm_set_discard_data(_l) LDLM_SET_FLAG(( _l), 1ULL << 16)
> +#define ldlm_is_discard_data(_l) LDLM_TEST_FLAG((_l), 1ULL << 16)
> +#define ldlm_set_discard_data(_l) LDLM_SET_FLAG((_l), 1ULL << 16)
> #define ldlm_clear_discard_data(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 16)
>
> /** Blocked by group lock - wait indefinitely */
> #define LDLM_FL_NO_TIMEOUT 0x0000000000020000ULL // bit 17
> -#define ldlm_is_no_timeout(_l) LDLM_TEST_FLAG(( _l), 1ULL << 17)
> -#define ldlm_set_no_timeout(_l) LDLM_SET_FLAG(( _l), 1ULL << 17)
> +#define ldlm_is_no_timeout(_l) LDLM_TEST_FLAG((_l), 1ULL << 17)
> +#define ldlm_set_no_timeout(_l) LDLM_SET_FLAG((_l), 1ULL << 17)
> #define ldlm_clear_no_timeout(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 17)
>
> /**
> * Server told not to wait if blocked. For AGL, OST will not send glimpse
> * callback. */
> #define LDLM_FL_BLOCK_NOWAIT 0x0000000000040000ULL // bit 18
> -#define ldlm_is_block_nowait(_l) LDLM_TEST_FLAG(( _l), 1ULL << 18)
> -#define ldlm_set_block_nowait(_l) LDLM_SET_FLAG(( _l), 1ULL << 18)
> +#define ldlm_is_block_nowait(_l) LDLM_TEST_FLAG((_l), 1ULL << 18)
> +#define ldlm_set_block_nowait(_l) LDLM_SET_FLAG((_l), 1ULL << 18)
> #define ldlm_clear_block_nowait(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 18)
>
> /** return blocking lock */
> #define LDLM_FL_TEST_LOCK 0x0000000000080000ULL // bit 19
> -#define ldlm_is_test_lock(_l) LDLM_TEST_FLAG(( _l), 1ULL << 19)
> -#define ldlm_set_test_lock(_l) LDLM_SET_FLAG(( _l), 1ULL << 19)
> +#define ldlm_is_test_lock(_l) LDLM_TEST_FLAG((_l), 1ULL << 19)
> +#define ldlm_set_test_lock(_l) LDLM_SET_FLAG((_l), 1ULL << 19)
> #define ldlm_clear_test_lock(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 19)
>
> /**
> @@ -143,81 +143,81 @@
> * is for clients (like liblustre) that cannot be expected to reliably
> * response to blocking AST. */
> #define LDLM_FL_CANCEL_ON_BLOCK 0x0000000000800000ULL // bit 23
> -#define ldlm_is_cancel_on_block(_l) LDLM_TEST_FLAG(( _l), 1ULL << 23)
> -#define ldlm_set_cancel_on_block(_l) LDLM_SET_FLAG(( _l), 1ULL << 23)
> +#define ldlm_is_cancel_on_block(_l) LDLM_TEST_FLAG((_l), 1ULL << 23)
> +#define ldlm_set_cancel_on_block(_l) LDLM_SET_FLAG((_l), 1ULL << 23)
> #define ldlm_clear_cancel_on_block(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 23)
>
> /**
> * measure lock contention and return -EUSERS if locking contention is high */
> #define LDLM_FL_DENY_ON_CONTENTION 0x0000000040000000ULL // bit 30
> -#define ldlm_is_deny_on_contention(_l) LDLM_TEST_FLAG(( _l), 1ULL << 30)
> -#define ldlm_set_deny_on_contention(_l) LDLM_SET_FLAG(( _l), 1ULL << 30)
> +#define ldlm_is_deny_on_contention(_l) LDLM_TEST_FLAG((_l), 1ULL << 30)
> +#define ldlm_set_deny_on_contention(_l) LDLM_SET_FLAG((_l), 1ULL << 30)
> #define ldlm_clear_deny_on_contention(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 30)
>
> /**
> * These are flags that are mapped into the flags and ASTs of blocking
> * locks Add FL_DISCARD to blocking ASTs */
> #define LDLM_FL_AST_DISCARD_DATA 0x0000000080000000ULL // bit 31
> -#define ldlm_is_ast_discard_data(_l) LDLM_TEST_FLAG(( _l), 1ULL << 31)
> -#define ldlm_set_ast_discard_data(_l) LDLM_SET_FLAG(( _l), 1ULL << 31)
> +#define ldlm_is_ast_discard_data(_l) LDLM_TEST_FLAG((_l), 1ULL << 31)
> +#define ldlm_set_ast_discard_data(_l) LDLM_SET_FLAG((_l), 1ULL << 31)
> #define ldlm_clear_ast_discard_data(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 31)
>
> /**
> * Used for marking lock as a target for -EINTR while cp_ast sleep emulation
> * + race with upcoming bl_ast. */
> #define LDLM_FL_FAIL_LOC 0x0000000100000000ULL // bit 32
> -#define ldlm_is_fail_loc(_l) LDLM_TEST_FLAG(( _l), 1ULL << 32)
> -#define ldlm_set_fail_loc(_l) LDLM_SET_FLAG(( _l), 1ULL << 32)
> +#define ldlm_is_fail_loc(_l) LDLM_TEST_FLAG((_l), 1ULL << 32)
> +#define ldlm_set_fail_loc(_l) LDLM_SET_FLAG((_l), 1ULL << 32)
> #define ldlm_clear_fail_loc(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 32)
>
> /**
> * Used while processing the unused list to know that we have already
> * handled this lock and decided to skip it. */
> #define LDLM_FL_SKIPPED 0x0000000200000000ULL // bit 33
> -#define ldlm_is_skipped(_l) LDLM_TEST_FLAG(( _l), 1ULL << 33)
> -#define ldlm_set_skipped(_l) LDLM_SET_FLAG(( _l), 1ULL << 33)
> +#define ldlm_is_skipped(_l) LDLM_TEST_FLAG((_l), 1ULL << 33)
> +#define ldlm_set_skipped(_l) LDLM_SET_FLAG((_l), 1ULL << 33)
> #define ldlm_clear_skipped(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 33)
>
> /** this lock is being destroyed */
> #define LDLM_FL_CBPENDING 0x0000000400000000ULL // bit 34
> -#define ldlm_is_cbpending(_l) LDLM_TEST_FLAG(( _l), 1ULL << 34)
> -#define ldlm_set_cbpending(_l) LDLM_SET_FLAG(( _l), 1ULL << 34)
> +#define ldlm_is_cbpending(_l) LDLM_TEST_FLAG((_l), 1ULL << 34)
> +#define ldlm_set_cbpending(_l) LDLM_SET_FLAG((_l), 1ULL << 34)
> #define ldlm_clear_cbpending(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 34)
>
> /** not a real flag, not saved in lock */
> #define LDLM_FL_WAIT_NOREPROC 0x0000000800000000ULL // bit 35
> -#define ldlm_is_wait_noreproc(_l) LDLM_TEST_FLAG(( _l), 1ULL << 35)
> -#define ldlm_set_wait_noreproc(_l) LDLM_SET_FLAG(( _l), 1ULL << 35)
> +#define ldlm_is_wait_noreproc(_l) LDLM_TEST_FLAG((_l), 1ULL << 35)
> +#define ldlm_set_wait_noreproc(_l) LDLM_SET_FLAG((_l), 1ULL << 35)
> #define ldlm_clear_wait_noreproc(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 35)
>
> /** cancellation callback already run */
> #define LDLM_FL_CANCEL 0x0000001000000000ULL // bit 36
> -#define ldlm_is_cancel(_l) LDLM_TEST_FLAG(( _l), 1ULL << 36)
> -#define ldlm_set_cancel(_l) LDLM_SET_FLAG(( _l), 1ULL << 36)
> +#define ldlm_is_cancel(_l) LDLM_TEST_FLAG((_l), 1ULL << 36)
> +#define ldlm_set_cancel(_l) LDLM_SET_FLAG((_l), 1ULL << 36)
> #define ldlm_clear_cancel(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 36)
>
> /** whatever it might mean */
> #define LDLM_FL_LOCAL_ONLY 0x0000002000000000ULL // bit 37
> -#define ldlm_is_local_only(_l) LDLM_TEST_FLAG(( _l), 1ULL << 37)
> -#define ldlm_set_local_only(_l) LDLM_SET_FLAG(( _l), 1ULL << 37)
> +#define ldlm_is_local_only(_l) LDLM_TEST_FLAG((_l), 1ULL << 37)
> +#define ldlm_set_local_only(_l) LDLM_SET_FLAG((_l), 1ULL << 37)
> #define ldlm_clear_local_only(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 37)
>
> /** don't run the cancel callback under ldlm_cli_cancel_unused */
> #define LDLM_FL_FAILED 0x0000004000000000ULL // bit 38
> -#define ldlm_is_failed(_l) LDLM_TEST_FLAG(( _l), 1ULL << 38)
> -#define ldlm_set_failed(_l) LDLM_SET_FLAG(( _l), 1ULL << 38)
> +#define ldlm_is_failed(_l) LDLM_TEST_FLAG((_l), 1ULL << 38)
> +#define ldlm_set_failed(_l) LDLM_SET_FLAG((_l), 1ULL << 38)
> #define ldlm_clear_failed(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 38)
>
> /** lock cancel has already been sent */
> #define LDLM_FL_CANCELING 0x0000008000000000ULL // bit 39
> -#define ldlm_is_canceling(_l) LDLM_TEST_FLAG(( _l), 1ULL << 39)
> -#define ldlm_set_canceling(_l) LDLM_SET_FLAG(( _l), 1ULL << 39)
> +#define ldlm_is_canceling(_l) LDLM_TEST_FLAG((_l), 1ULL << 39)
> +#define ldlm_set_canceling(_l) LDLM_SET_FLAG((_l), 1ULL << 39)
> #define ldlm_clear_canceling(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 39)
>
> /** local lock (ie, no srv/cli split) */
> #define LDLM_FL_LOCAL 0x0000010000000000ULL // bit 40
> -#define ldlm_is_local(_l) LDLM_TEST_FLAG(( _l), 1ULL << 40)
> -#define ldlm_set_local(_l) LDLM_SET_FLAG(( _l), 1ULL << 40)
> +#define ldlm_is_local(_l) LDLM_TEST_FLAG((_l), 1ULL << 40)
> +#define ldlm_set_local(_l) LDLM_SET_FLAG((_l), 1ULL << 40)
> #define ldlm_clear_local(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 40)
>
> /**
> @@ -233,8 +233,8 @@
> * That change is pretty high-risk, though, and would need a lot more
> * testing. */
> #define LDLM_FL_LVB_READY 0x0000020000000000ULL // bit 41
> -#define ldlm_is_lvb_ready(_l) LDLM_TEST_FLAG(( _l), 1ULL << 41)
> -#define ldlm_set_lvb_ready(_l) LDLM_SET_FLAG(( _l), 1ULL << 41)
> +#define ldlm_is_lvb_ready(_l) LDLM_TEST_FLAG((_l), 1ULL << 41)
> +#define ldlm_set_lvb_ready(_l) LDLM_SET_FLAG((_l), 1ULL << 41)
> #define ldlm_clear_lvb_ready(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 41)
>
> /**
> @@ -245,28 +245,28 @@
> * to know to exclude each other's locks from the calculation as they walk
> * the granted list. */
> #define LDLM_FL_KMS_IGNORE 0x0000040000000000ULL // bit 42
> -#define ldlm_is_kms_ignore(_l) LDLM_TEST_FLAG(( _l), 1ULL << 42)
> -#define ldlm_set_kms_ignore(_l) LDLM_SET_FLAG(( _l), 1ULL << 42)
> +#define ldlm_is_kms_ignore(_l) LDLM_TEST_FLAG((_l), 1ULL << 42)
> +#define ldlm_set_kms_ignore(_l) LDLM_SET_FLAG((_l), 1ULL << 42)
> #define ldlm_clear_kms_ignore(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 42)
>
> /** completion AST to be executed */
> #define LDLM_FL_CP_REQD 0x0000080000000000ULL // bit 43
> -#define ldlm_is_cp_reqd(_l) LDLM_TEST_FLAG(( _l), 1ULL << 43)
> -#define ldlm_set_cp_reqd(_l) LDLM_SET_FLAG(( _l), 1ULL << 43)
> +#define ldlm_is_cp_reqd(_l) LDLM_TEST_FLAG((_l), 1ULL << 43)
> +#define ldlm_set_cp_reqd(_l) LDLM_SET_FLAG((_l), 1ULL << 43)
> #define ldlm_clear_cp_reqd(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 43)
>
> /** cleanup_resource has already handled the lock */
> #define LDLM_FL_CLEANED 0x0000100000000000ULL // bit 44
> -#define ldlm_is_cleaned(_l) LDLM_TEST_FLAG(( _l), 1ULL << 44)
> -#define ldlm_set_cleaned(_l) LDLM_SET_FLAG(( _l), 1ULL << 44)
> +#define ldlm_is_cleaned(_l) LDLM_TEST_FLAG((_l), 1ULL << 44)
> +#define ldlm_set_cleaned(_l) LDLM_SET_FLAG((_l), 1ULL << 44)
> #define ldlm_clear_cleaned(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 44)
>
> /**
> * optimization hint: LDLM can run blocking callback from current context
> * w/o involving separate thread. in order to decrease cs rate */
> #define LDLM_FL_ATOMIC_CB 0x0000200000000000ULL // bit 45
> -#define ldlm_is_atomic_cb(_l) LDLM_TEST_FLAG(( _l), 1ULL << 45)
> -#define ldlm_set_atomic_cb(_l) LDLM_SET_FLAG(( _l), 1ULL << 45)
> +#define ldlm_is_atomic_cb(_l) LDLM_TEST_FLAG((_l), 1ULL << 45)
> +#define ldlm_set_atomic_cb(_l) LDLM_SET_FLAG((_l), 1ULL << 45)
> #define ldlm_clear_atomic_cb(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 45)
>
> /**
> @@ -282,14 +282,14 @@
> * is used when ELC RPC is already prepared and is waiting for rpc_lock,
> * too late to send a separate CANCEL RPC. */
> #define LDLM_FL_BL_AST 0x0000400000000000ULL // bit 46
> -#define ldlm_is_bl_ast(_l) LDLM_TEST_FLAG(( _l), 1ULL << 46)
> -#define ldlm_set_bl_ast(_l) LDLM_SET_FLAG(( _l), 1ULL << 46)
> +#define ldlm_is_bl_ast(_l) LDLM_TEST_FLAG((_l), 1ULL << 46)
> +#define ldlm_set_bl_ast(_l) LDLM_SET_FLAG((_l), 1ULL << 46)
> #define ldlm_clear_bl_ast(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 46)
>
> /** whatever it might mean */
> #define LDLM_FL_BL_DONE 0x0000800000000000ULL // bit 47
> -#define ldlm_is_bl_done(_l) LDLM_TEST_FLAG(( _l), 1ULL << 47)
> -#define ldlm_set_bl_done(_l) LDLM_SET_FLAG(( _l), 1ULL << 47)
> +#define ldlm_is_bl_done(_l) LDLM_TEST_FLAG((_l), 1ULL << 47)
> +#define ldlm_set_bl_done(_l) LDLM_SET_FLAG((_l), 1ULL << 47)
> #define ldlm_clear_bl_done(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 47)
>
> /**
> @@ -297,8 +297,8 @@
> * to aging. Used by MGC locks, they are cancelled only at unmount or
> * by callback. */
> #define LDLM_FL_NO_LRU 0x0001000000000000ULL // bit 48
> -#define ldlm_is_no_lru(_l) LDLM_TEST_FLAG(( _l), 1ULL << 48)
> -#define ldlm_set_no_lru(_l) LDLM_SET_FLAG(( _l), 1ULL << 48)
> +#define ldlm_is_no_lru(_l) LDLM_TEST_FLAG((_l), 1ULL << 48)
> +#define ldlm_set_no_lru(_l) LDLM_SET_FLAG((_l), 1ULL << 48)
> #define ldlm_clear_no_lru(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 48)
>
> /**
> @@ -306,8 +306,8 @@
> *
> * Protected by lock and resource locks. */
> #define LDLM_FL_FAIL_NOTIFIED 0x0002000000000000ULL // bit 49
> -#define ldlm_is_fail_notified(_l) LDLM_TEST_FLAG(( _l), 1ULL << 49)
> -#define ldlm_set_fail_notified(_l) LDLM_SET_FLAG(( _l), 1ULL << 49)
> +#define ldlm_is_fail_notified(_l) LDLM_TEST_FLAG((_l), 1ULL << 49)
> +#define ldlm_set_fail_notified(_l) LDLM_SET_FLAG((_l), 1ULL << 49)
> #define ldlm_clear_fail_notified(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 49)
>
> /**
> @@ -317,14 +317,14 @@
> *
> * Protected by lock and resource locks. */
> #define LDLM_FL_DESTROYED 0x0004000000000000ULL // bit 50
> -#define ldlm_is_destroyed(_l) LDLM_TEST_FLAG(( _l), 1ULL << 50)
> -#define ldlm_set_destroyed(_l) LDLM_SET_FLAG(( _l), 1ULL << 50)
> +#define ldlm_is_destroyed(_l) LDLM_TEST_FLAG((_l), 1ULL << 50)
> +#define ldlm_set_destroyed(_l) LDLM_SET_FLAG((_l), 1ULL << 50)
> #define ldlm_clear_destroyed(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 50)
>
> /** flag whether this is a server namespace lock */
> #define LDLM_FL_SERVER_LOCK 0x0008000000000000ULL // bit 51
> -#define ldlm_is_server_lock(_l) LDLM_TEST_FLAG(( _l), 1ULL << 51)
> -#define ldlm_set_server_lock(_l) LDLM_SET_FLAG(( _l), 1ULL << 51)
> +#define ldlm_is_server_lock(_l) LDLM_TEST_FLAG((_l), 1ULL << 51)
> +#define ldlm_set_server_lock(_l) LDLM_SET_FLAG((_l), 1ULL << 51)
> #define ldlm_clear_server_lock(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 51)
>
> /**
> @@ -335,8 +335,8 @@
> * because it works only for SMP so user needs to add extra macros like
> * LASSERT_SPIN_LOCKED for uniprocessor kernels. */
> #define LDLM_FL_RES_LOCKED 0x0010000000000000ULL // bit 52
> -#define ldlm_is_res_locked(_l) LDLM_TEST_FLAG(( _l), 1ULL << 52)
> -#define ldlm_set_res_locked(_l) LDLM_SET_FLAG(( _l), 1ULL << 52)
> +#define ldlm_is_res_locked(_l) LDLM_TEST_FLAG((_l), 1ULL << 52)
> +#define ldlm_set_res_locked(_l) LDLM_SET_FLAG((_l), 1ULL << 52)
> #define ldlm_clear_res_locked(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 52)
>
> /**
> @@ -345,14 +345,14 @@
> *
> * Protected by lock and resource locks. */
> #define LDLM_FL_WAITED 0x0020000000000000ULL // bit 53
> -#define ldlm_is_waited(_l) LDLM_TEST_FLAG(( _l), 1ULL << 53)
> -#define ldlm_set_waited(_l) LDLM_SET_FLAG(( _l), 1ULL << 53)
> +#define ldlm_is_waited(_l) LDLM_TEST_FLAG((_l), 1ULL << 53)
> +#define ldlm_set_waited(_l) LDLM_SET_FLAG((_l), 1ULL << 53)
> #define ldlm_clear_waited(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 53)
>
> /** Flag whether this is a server namespace lock. */
> #define LDLM_FL_NS_SRV 0x0040000000000000ULL // bit 54
> -#define ldlm_is_ns_srv(_l) LDLM_TEST_FLAG(( _l), 1ULL << 54)
> -#define ldlm_set_ns_srv(_l) LDLM_SET_FLAG(( _l), 1ULL << 54)
> +#define ldlm_is_ns_srv(_l) LDLM_TEST_FLAG((_l), 1ULL << 54)
> +#define ldlm_set_ns_srv(_l) LDLM_SET_FLAG((_l), 1ULL << 54)
> #define ldlm_clear_ns_srv(_l) LDLM_CLEAR_FLAG((_l), 1ULL << 54)
>
> /** Flag whether this lock can be reused. Used by exclusive open. */
> diff --git a/drivers/staging/lustre/lustre/include/lustre_lib.h b/drivers/staging/lustre/lustre/include/lustre_lib.h
> index 43ee9f0..2a4294d 100644
> --- a/drivers/staging/lustre/lustre/include/lustre_lib.h
> +++ b/drivers/staging/lustre/lustre/include/lustre_lib.h
> @@ -313,7 +313,7 @@ static inline void obd_ioctl_freedata(char *buf, int len)
> #define OBD_IOC_CLIENT_RECOVER _IOW ('f', 133, OBD_IOC_DATA_TYPE)
> #define OBD_IOC_PING_TARGET _IOW ('f', 136, OBD_IOC_DATA_TYPE)
>
> -#define OBD_IOC_DEC_FS_USE_COUNT _IO ('f', 139 )
> +#define OBD_IOC_DEC_FS_USE_COUNT _IO ('f', 139)
> #define OBD_IOC_NO_TRANSNO _IOW ('f', 140, OBD_IOC_DATA_TYPE)
> #define OBD_IOC_SET_READONLY _IOW ('f', 141, OBD_IOC_DATA_TYPE)
> #define OBD_IOC_ABORT_RECOVERY _IOR ('f', 142, OBD_IOC_DATA_TYPE)
> @@ -557,9 +557,9 @@ do { \
> if (__timeout == 0) { \
> schedule(); \
> } else { \
> - long interval = info->lwi_interval? \
> + long interval = info->lwi_interval ? \
> min_t(long, \
> - info->lwi_interval,__timeout):\
> + info->lwi_interval, __timeout) : \
> __timeout; \
> long remaining = schedule_timeout(interval);\
> __timeout = cfs_time_sub(__timeout, \
> diff --git a/drivers/staging/lustre/lustre/include/lustre_net.h b/drivers/staging/lustre/lustre/include/lustre_net.h
> index 48ad60b..3341b5d 100644
> --- a/drivers/staging/lustre/lustre/include/lustre_net.h
> +++ b/drivers/staging/lustre/lustre/include/lustre_net.h
> @@ -2065,7 +2065,7 @@ struct ptlrpc_service_part {
> * rqbd list and incoming requests waiting for preprocess,
> * threads starting & stopping are also protected by this lock.
> */
> - spinlock_t scp_lock __cfs_cacheline_aligned;
> + spinlock_t scp_lock __cfs_cacheline_aligned;
> /** total # req buffer descs allocated */
> int scp_nrqbds_total;
> /** # posted request buffers for receiving */
> diff --git a/drivers/staging/lustre/lustre/include/obd.h b/drivers/staging/lustre/lustre/include/obd.h
> index 9ad8c26..76a2546 100644
> --- a/drivers/staging/lustre/lustre/include/obd.h
> +++ b/drivers/staging/lustre/lustre/include/obd.h
> @@ -1300,7 +1300,7 @@ struct md_ops {
> const struct lu_fid *,
> struct ptlrpc_request **);
> int (*m_setattr)(struct obd_export *, struct md_op_data *, void *,
> - int , void *, int, struct ptlrpc_request **,
> + int, void *, int, struct ptlrpc_request **,
> struct md_open_data **mod);
> int (*m_sync)(struct obd_export *, const struct lu_fid *,
> struct obd_capa *, struct ptlrpc_request **);
> diff --git a/drivers/staging/lustre/lustre/include/obd_support.h b/drivers/staging/lustre/lustre/include/obd_support.h
> index 18aec79..3986ca4 100644
> --- a/drivers/staging/lustre/lustre/include/obd_support.h
> +++ b/drivers/staging/lustre/lustre/include/obd_support.h
> @@ -585,7 +585,7 @@ do { \
> (!HAS_FAIL_ALLOC_FLAG || obd_alloc_fail_rate == 0 || \
> !obd_alloc_fail(ptr, #ptr, "km", size, \
> __FILE__, __LINE__) || \
> - OBD_FREE_RTN0(ptr)))){ \
> + OBD_FREE_RTN0(ptr)))) { \
> OBD_ALLOC_POST(ptr, size, "kmalloced"); \
> } \
> } while (0)
> diff --git a/drivers/staging/lustre/lustre/libcfs/tracefile.c b/drivers/staging/lustre/lustre/libcfs/tracefile.c
> index effa2af..5a2e5ea 100644
> --- a/drivers/staging/lustre/lustre/libcfs/tracefile.c
> +++ b/drivers/staging/lustre/lustre/libcfs/tracefile.c
> @@ -124,7 +124,7 @@ int cfs_trace_refill_stock(struct cfs_trace_cpu_data *tcd, gfp_t gfp,
> * from here: this will lead to infinite recursion.
> */
>
> - for (i = 0; i + tcd->tcd_cur_stock_pages < TCD_STOCK_PAGES ; ++ i) {
> + for (i = 0; i + tcd->tcd_cur_stock_pages < TCD_STOCK_PAGES ; ++i) {
> struct cfs_trace_page *tage;
>
> tage = cfs_tage_alloc(gfp);
> @@ -370,7 +370,7 @@ int libcfs_debug_vmsg2(struct libcfs_debug_msg_data *msgdata,
> /* indent message according to the nesting level */
> while (depth-- > 0) {
> *(debug_buf++) = '.';
> - ++ tage->used;
> + ++tage->used;
> }
>
> strcpy(debug_buf, file);
> diff --git a/drivers/staging/lustre/lustre/llite/file.c b/drivers/staging/lustre/lustre/llite/file.c
> index dcd0c6d..ef7c470 100644
> --- a/drivers/staging/lustre/lustre/llite/file.c
> +++ b/drivers/staging/lustre/lustre/llite/file.c
> @@ -270,7 +270,7 @@ static int ll_md_close(struct obd_export *md_exp, struct inode *inode,
> int lockmode;
> __u64 flags = LDLM_FL_BLOCK_GRANTED | LDLM_FL_TEST_LOCK;
> struct lustre_handle lockh;
> - ldlm_policy_data_t policy = {.l_inodebits = {MDS_INODELOCK_OPEN}};
> + ldlm_policy_data_t policy = { .l_inodebits = {MDS_INODELOCK_OPEN} };
> int rc = 0;
>
> /* clear group lock, if present */
> @@ -2670,7 +2670,7 @@ ll_file_flock(struct file *file, int cmd, struct file_lock *file_lock)
> };
> struct md_op_data *op_data;
> struct lustre_handle lockh = {0};
> - ldlm_policy_data_t flock = {{0}};
> + ldlm_policy_data_t flock = { {0} };
> __u64 flags = 0;
> int rc;
> int rc2 = 0;
> diff --git a/drivers/staging/lustre/lustre/llite/llite_internal.h b/drivers/staging/lustre/lustre/llite/llite_internal.h
> index ec8fff4..2de64c2 100644
> --- a/drivers/staging/lustre/lustre/llite/llite_internal.h
> +++ b/drivers/staging/lustre/lustre/llite/llite_internal.h
> @@ -71,7 +71,7 @@ struct ll_dentry_data {
> struct rcu_head lld_rcu_head;
> };
>
> -#define ll_d2d(de) ((struct ll_dentry_data*)((de)->d_fsdata))
> +#define ll_d2d(de) ((struct ll_dentry_data *)((de)->d_fsdata))
>
> #define LLI_INODE_MAGIC 0x111d0de5
> #define LLI_INODE_DEAD 0xdeadd00d
> diff --git a/drivers/staging/lustre/lustre/llite/lproc_llite.c b/drivers/staging/lustre/lustre/llite/lproc_llite.c
> index 486dca6..2fc24a9 100644
> --- a/drivers/staging/lustre/lustre/llite/lproc_llite.c
> +++ b/drivers/staging/lustre/lustre/llite/lproc_llite.c
> @@ -1325,7 +1325,7 @@ void ll_rw_stats_tally(struct ll_sb_info *sbi, pid_t pid,
> lprocfs_oh_clear(&io_extents->pp_extents[cur].pp_w_hist);
> }
>
> - for(i = 0; (count >= (1 << LL_HIST_START << i)) &&
> + for (i = 0; (count >= (1 << LL_HIST_START << i)) &&
> (i < (LL_HIST_MAX - 1)); i++);
> if (rw == 0) {
> io_extents->pp_extents[cur].pp_r_hist.oh_buckets[i]++;
> diff --git a/drivers/staging/lustre/lustre/llite/rw.c b/drivers/staging/lustre/lustre/llite/rw.c
> index 991d20c..3e28255 100644
> --- a/drivers/staging/lustre/lustre/llite/rw.c
> +++ b/drivers/staging/lustre/lustre/llite/rw.c
> @@ -656,7 +656,7 @@ static int ll_read_ahead_pages(const struct lu_env *env,
> page_idx, mapping);
> if (rc == 1) {
> (*reserved_pages)--;
> - count ++;
> + count++;
> } else if (rc == -ENOLCK)
> break;
> } else if (stride_ria) {
> @@ -890,7 +890,7 @@ static void ras_update_stride_detector(struct ll_readahead_state *ras,
> if (!stride_io_mode(ras) && (stride_gap != 0 ||
> ras->ras_consecutive_stride_requests == 0)) {
> ras->ras_stride_pages = ras->ras_consecutive_pages;
> - ras->ras_stride_length = stride_gap +ras->ras_consecutive_pages;
> + ras->ras_stride_length = stride_gap+ras->ras_consecutive_pages;
> }
> LASSERT(ras->ras_request_index == 0);
> LASSERT(ras->ras_consecutive_stride_requests == 0);
> @@ -902,7 +902,7 @@ static void ras_update_stride_detector(struct ll_readahead_state *ras,
> }
>
> ras->ras_stride_pages = ras->ras_consecutive_pages;
> - ras->ras_stride_length = stride_gap +ras->ras_consecutive_pages;
> + ras->ras_stride_length = stride_gap+ras->ras_consecutive_pages;
>
> RAS_CDEBUG(ras);
> return;
> diff --git a/drivers/staging/lustre/lustre/llite/statahead.c b/drivers/staging/lustre/lustre/llite/statahead.c
> index f97371d..8f4cca1 100644
> --- a/drivers/staging/lustre/lustre/llite/statahead.c
> +++ b/drivers/staging/lustre/lustre/llite/statahead.c
> @@ -659,7 +659,7 @@ static void ll_post_statahead(struct ll_statahead_info *sai)
> * revalidate.
> */
> /* unlinked and re-created with the same name */
> - if (unlikely(!lu_fid_eq(&minfo->mi_data.op_fid2, &body->fid1))){
> + if (unlikely(!lu_fid_eq(&minfo->mi_data.op_fid2, &body->fid1))) {
> entry->se_inode = NULL;
> iput(child);
> child = NULL;
> @@ -1215,7 +1215,7 @@ do_it:
> while (1) {
> l_wait_event(thread->t_ctl_waitq,
> !sa_received_empty(sai) ||
> - sai->sai_sent == sai->sai_replied||
> + sai->sai_sent == sai->sai_replied ||
> !thread_is_running(thread),
> &lwi);
>
> diff --git a/drivers/staging/lustre/lustre/llite/vvp_io.c b/drivers/staging/lustre/lustre/llite/vvp_io.c
> index a659962..75a8ea2 100644
> --- a/drivers/staging/lustre/lustre/llite/vvp_io.c
> +++ b/drivers/staging/lustre/lustre/llite/vvp_io.c
> @@ -698,7 +698,7 @@ static int vvp_io_fault_start(const struct lu_env *env,
>
> /* return +1 to stop cl_io_loop() and ll_fault() will catch
> * and retry. */
> - result = +1;
> + result = 1;
> goto out;
> }
>
> diff --git a/drivers/staging/lustre/lustre/lmv/lmv_obd.c b/drivers/staging/lustre/lustre/lmv/lmv_obd.c
> index c9e0536e..f8146ac 100644
> --- a/drivers/staging/lustre/lustre/lmv/lmv_obd.c
> +++ b/drivers/staging/lustre/lustre/lmv/lmv_obd.c
> @@ -1849,7 +1849,7 @@ static int lmv_early_cancel(struct obd_export *exp, struct md_op_data *op_data,
> struct obd_device *obd = exp->exp_obd;
> struct lmv_obd *lmv = &obd->u.lmv;
> struct lmv_tgt_desc *tgt;
> - ldlm_policy_data_t policy = {{0}};
> + ldlm_policy_data_t policy = { {0} };
> int rc = 0;
>
> if (!fid_is_sane(fid))
> diff --git a/drivers/staging/lustre/lustre/lov/lov_obd.c b/drivers/staging/lustre/lustre/lov/lov_obd.c
> index c5c67d9..2a2fd8d 100644
> --- a/drivers/staging/lustre/lustre/lov/lov_obd.c
> +++ b/drivers/staging/lustre/lustre/lov/lov_obd.c
> @@ -1672,7 +1672,7 @@ static int fiemap_calc_last_stripe(struct lov_stripe_md *lsm, u64 fm_start,
> break;
> }
> *stripe_count = j;
> - last_stripe = (start_stripe + j - 1) %lsm->lsm_stripe_count;
> + last_stripe = (start_stripe + j - 1) % lsm->lsm_stripe_count;
> }
>
> return last_stripe;
> @@ -1862,7 +1862,7 @@ static int lov_fiemap(struct lov_obd *lov, __u32 keylen, void *key,
> fm_local->fm_start = lun_start;
> fm_local->fm_flags &= ~FIEMAP_FLAG_DEVICE_ORDER;
> memcpy(&fm_key->fiemap, fm_local, sizeof(*fm_local));
> - *vallen=fiemap_count_to_size(fm_local->fm_extent_count);
> + *vallen = fiemap_count_to_size(fm_local->fm_extent_count);
> rc = obd_get_info(NULL,
> lov->lov_tgts[ost_index]->ltd_exp,
> keylen, key, vallen, fm_local, lsm);
> diff --git a/drivers/staging/lustre/lustre/obdclass/cl_io.c b/drivers/staging/lustre/lustre/obdclass/cl_io.c
> index fd1a4c5..da0066a 100644
> --- a/drivers/staging/lustre/lustre/obdclass/cl_io.c
> +++ b/drivers/staging/lustre/lustre/obdclass/cl_io.c
> @@ -307,7 +307,7 @@ static void cl_io_locks_sort(struct cl_io *io)
> */
> default:
> LBUG();
> - case +1:
> + case 1:
> list_move_tail(&curr->cill_linkage,
> &prev->cill_linkage);
> done = 0;
> @@ -335,7 +335,7 @@ int cl_queue_match(const struct list_head *queue,
>
> list_for_each_entry(scan, queue, cill_linkage) {
> if (cl_lock_descr_match(&scan->cill_descr, need))
> - return +1;
> + return 1;
> }
> return 0;
> }
> @@ -353,7 +353,7 @@ static int cl_queue_merge(const struct list_head *queue,
> CDEBUG(D_VFSTRACE, "lock: %d: [%lu, %lu]\n",
> scan->cill_descr.cld_mode, scan->cill_descr.cld_start,
> scan->cill_descr.cld_end);
> - return +1;
> + return 1;
> }
> return 0;
>
> @@ -600,7 +600,7 @@ 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))
> - result = +1;
> + result = 1;
> else {
> list_add(&link->cill_linkage, &io->ci_lockset.cls_todo);
> result = 0;
> @@ -918,7 +918,7 @@ int cl_io_submit_sync(const struct lu_env *env, struct cl_io *io,
> */
> cl_page_list_for_each(pg, &queue->c2_qin) {
> pg->cp_sync_io = NULL;
> - cl_sync_io_note(anchor, +1);
> + cl_sync_io_note(anchor, 1);
> }
>
> /* wait for the IO to be finished. */
> diff --git a/drivers/staging/lustre/lustre/obdclass/cl_lock.c b/drivers/staging/lustre/lustre/obdclass/cl_lock.c
> index b081167..7c6583a 100644
> --- a/drivers/staging/lustre/lustre/obdclass/cl_lock.c
> +++ b/drivers/staging/lustre/lustre/obdclass/cl_lock.c
> @@ -2027,7 +2027,7 @@ static struct cl_lock *cl_lock_hold_mutex(const struct lu_env *env,
> cl_lock_mutex_get(env, lock);
> if (lock->cll_state < CLS_FREEING &&
> !(lock->cll_flags & CLF_CANCELLED)) {
> - cl_lock_hold_mod(env, lock, +1);
> + cl_lock_hold_mod(env, lock, 1);
> lu_ref_add(&lock->cll_holders, scope, source);
> lu_ref_add(&lock->cll_reference, scope, source);
> break;
> @@ -2115,7 +2115,7 @@ void cl_lock_hold_add(const struct lu_env *env, struct cl_lock *lock,
> LINVRNT(cl_lock_invariant(env, lock));
> LASSERT(lock->cll_state != CLS_FREEING);
>
> - cl_lock_hold_mod(env, lock, +1);
> + cl_lock_hold_mod(env, lock, 1);
> cl_lock_get(lock);
> lu_ref_add(&lock->cll_holders, scope, source);
> lu_ref_add(&lock->cll_reference, scope, source);
> @@ -2157,7 +2157,7 @@ void cl_lock_user_add(const struct lu_env *env, struct cl_lock *lock)
> LINVRNT(cl_lock_is_mutexed(lock));
> LINVRNT(cl_lock_invariant(env, lock));
>
> - cl_lock_used_mod(env, lock, +1);
> + cl_lock_used_mod(env, lock, 1);
> }
> EXPORT_SYMBOL(cl_lock_user_add);
>
> diff --git a/drivers/staging/lustre/lustre/obdclass/cl_object.c b/drivers/staging/lustre/lustre/obdclass/cl_object.c
> index 163fe0c..fb9a854 100644
> --- a/drivers/staging/lustre/lustre/obdclass/cl_object.c
> +++ b/drivers/staging/lustre/lustre/obdclass/cl_object.c
> @@ -661,7 +661,7 @@ static int cl_env_store_init(void) {
> CFS_HASH_MAX_THETA,
> &cl_env_hops,
> CFS_HASH_RW_BKTLOCK);
> - return cl_env_hash != NULL ? 0 :-ENOMEM;
> + return cl_env_hash != NULL ? 0 : -ENOMEM;
> }
>
> static void cl_env_store_fini(void)
> diff --git a/drivers/staging/lustre/lustre/obdclass/class_obd.c b/drivers/staging/lustre/lustre/obdclass/class_obd.c
> index 2c705d7..130c4e3 100644
> --- a/drivers/staging/lustre/lustre/obdclass/class_obd.c
> +++ b/drivers/staging/lustre/lustre/obdclass/class_obd.c
> @@ -138,7 +138,7 @@ int obd_alloc_fail(const void *ptr, const char *name, const char *type,
> if (ptr == NULL ||
> (cfs_rand() & OBD_ALLOC_FAIL_MASK) < obd_alloc_fail_rate) {
> CERROR("%s%salloc of %s (%llu bytes) failed at %s:%d\n",
> - ptr ? "force " :"", type, name, (__u64)size, file,
> + ptr ? "force " : "", type, name, (__u64)size, file,
> line);
> CERROR("%llu total bytes and %llu total pages"
> " (%llu bytes) allocated by Lustre\n",
> @@ -470,7 +470,7 @@ int obd_init_checks(void)
> return -EOVERFLOW;
> }
> if (do_div(div64val, 256) != (u64val & 255)) {
> - CERROR("do_div(%#llx,256) != %llu\n", u64val, u64val &255);
> + CERROR("do_div(%#llx,256) != %llu\n", u64val, u64val & 255);
> return -EOVERFLOW;
> }
> if (u64val >> 8 != div64val) {
> diff --git a/drivers/staging/lustre/lustre/obdclass/genops.c b/drivers/staging/lustre/lustre/obdclass/genops.c
> index 0ca7309..bc614dc 100644
> --- a/drivers/staging/lustre/lustre/obdclass/genops.c
> +++ b/drivers/staging/lustre/lustre/obdclass/genops.c
> @@ -545,7 +545,7 @@ struct obd_device *class_find_client_obd(struct obd_uuid *tgt_uuid,
> strlen(typ_name)) == 0)) {
> if (obd_uuid_equals(tgt_uuid,
> &obd->u.cli.cl_target_uuid) &&
> - ((grp_uuid)? obd_uuid_equals(grp_uuid,
> + ((grp_uuid) ? obd_uuid_equals(grp_uuid,
> &obd->obd_uuid) : 1)) {
> read_unlock(&obd_dev_lock);
> return obd;
> @@ -1078,7 +1078,7 @@ void __class_export_add_lock_ref(struct obd_export *exp, struct ldlm_lock *lock)
> LCONSOLE_WARN("setting export %p for lock %p which already has export %p\n",
> exp, lock, lock->l_exp_refs_target);
> }
> - if ((lock->l_exp_refs_nr ++) == 0) {
> + if ((lock->l_exp_refs_nr++) == 0) {
> list_add(&lock->l_exp_refs_link, &exp->exp_locks_list);
> lock->l_exp_refs_target = exp;
> }
> @@ -1488,7 +1488,7 @@ int obd_export_evict_by_uuid(struct obd_device *obd, const char *uuid)
> EXPORT_SYMBOL(obd_export_evict_by_uuid);
>
> #if LUSTRE_TRACKS_LOCK_EXP_REFS
> -void (*class_export_dump_hook)(struct obd_export*) = NULL;
> +void (*class_export_dump_hook)(struct obd_export *) = NULL;
> EXPORT_SYMBOL(class_export_dump_hook);
> #endif
>
> diff --git a/drivers/staging/lustre/lustre/obdclass/llog.c b/drivers/staging/lustre/lustre/obdclass/llog.c
> index facc835..78d9226 100644
> --- a/drivers/staging/lustre/lustre/obdclass/llog.c
> +++ b/drivers/staging/lustre/lustre/obdclass/llog.c
> @@ -326,7 +326,7 @@ repeat:
> * swabbing is done at the beginning of the loop. */
> for (rec = (struct llog_rec_hdr *)buf;
> (char *)rec < buf + LLOG_CHUNK_SIZE;
> - rec = (struct llog_rec_hdr *)((char *)rec + rec->lrh_len)){
> + rec = (struct llog_rec_hdr *)((char *)rec + rec->lrh_len)) {
>
> CDEBUG(D_OTHER, "processing rec 0x%p type %#x\n",
> rec, rec->lrh_type);
> @@ -522,7 +522,7 @@ int llog_reverse_process(const struct lu_env *env,
> rec = (void *)rec + rec->lrh_len;
> if (LLOG_REC_HDR_NEEDS_SWABBING(rec))
> lustre_swab_llog_rec(rec);
> - idx ++;
> + idx++;
> }
> LASSERT(idx == index);
> tail = (void *)rec + rec->lrh_len - sizeof(*tail);
> diff --git a/drivers/staging/lustre/lustre/obdclass/obd_config.c b/drivers/staging/lustre/lustre/obdclass/obd_config.c
> index 93805ac..953e4c8 100644
> --- a/drivers/staging/lustre/lustre/obdclass/obd_config.c
> +++ b/drivers/staging/lustre/lustre/obdclass/obd_config.c
> @@ -1451,7 +1451,7 @@ int class_config_llog_handler(const struct lu_env *env,
> lustre_cfg_bufs_init(&bufs, lcfg);
>
> if (clli && clli->cfg_instance &&
> - LUSTRE_CFG_BUFLEN(lcfg, 0) > 0){
> + LUSTRE_CFG_BUFLEN(lcfg, 0) > 0) {
> inst = 1;
> inst_len = LUSTRE_CFG_BUFLEN(lcfg, 0) +
> sizeof(clli->cfg_instance) * 2 + 4;
> diff --git a/drivers/staging/lustre/lustre/osc/osc_request.c b/drivers/staging/lustre/lustre/osc/osc_request.c
> index 12113df..cf2739c 100644
> --- a/drivers/staging/lustre/lustre/osc/osc_request.c
> +++ b/drivers/staging/lustre/lustre/osc/osc_request.c
> @@ -1586,7 +1586,7 @@ static int osc_brw_fini_request(struct ptlrpc_request *req, int rc)
> char *router;
> cksum_type_t cksum_type;
>
> - cksum_type = cksum_type_unpack(body->oa.o_valid &OBD_MD_FLFLAGS?
> + cksum_type = cksum_type_unpack(body->oa.o_valid&OBD_MD_FLFLAGS ?
> body->oa.o_flags : 0);
> client_cksum = osc_checksum_bulk(rc, aa->aa_page_count,
> aa->aa_ppga, OST_READ,
> @@ -1654,7 +1654,7 @@ static int osc_brw_redo_request(struct ptlrpc_request *request,
> "redo for recoverable error %d", rc);
>
> rc = osc_brw_prep_request(lustre_msg_get_opc(request->rq_reqmsg) ==
> - OST_WRITE ? OBD_BRW_WRITE :OBD_BRW_READ,
> + OST_WRITE ? OBD_BRW_WRITE : OBD_BRW_READ,
> aa->aa_cli, aa->aa_oa,
> NULL /* lsm unused by osc currently */,
> aa->aa_page_count, aa->aa_ppga,
> @@ -3101,7 +3101,7 @@ static int osc_import_event(struct obd_device *obd,
>
> /* See bug 7198 */
> if (ocd->ocd_connect_flags & OBD_CONNECT_REQPORTAL)
> - imp->imp_client->cli_request_portal =OST_REQUEST_PORTAL;
> + imp->imp_client->cli_request_portal = OST_REQUEST_PORTAL;
>
> rc = obd_notify_observer(obd, obd, OBD_NOTIFY_OCD, NULL);
> break;
> diff --git a/drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c b/drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c
> index 53f9af1..84eb3da 100644
> --- a/drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c
> +++ b/drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c
> @@ -54,7 +54,7 @@ static struct ll_rpc_opcode {
> { OST_SETATTR, "ost_setattr" },
> { OST_READ, "ost_read" },
> { OST_WRITE, "ost_write" },
> - { OST_CREATE , "ost_create" },
> + { OST_CREATE, "ost_create" },
> { OST_DESTROY, "ost_destroy" },
> { OST_GET_INFO, "ost_get_info" },
> { OST_CONNECT, "ost_connect" },
> --
> 1.8.3.1
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists