[PATCH 42/56] staging/lustre/include: Adjust comment style

From: Oleg Drokin This fixes most of the "Block comments use a trailing */ on a separate line" checkpatch warnings, also some slight reformats of structures or comments at places. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/cl_object.h | 69 ++ dri

[PATCH 31/56] staging/lustre: Remove unused struct quota_body

From: Oleg Drokin It's only used for communications between servers. Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_idl.h | 31 --- .../lustre/lustre/include/lustre_req_layout.h | 3 -- drivers/staging/lustre/lustre/ptlrpc/layout.c | 36 -

[PATCH 38/56] staging/lustre: Remove dead code in ldlm_lock_destroy_internal

From: Oleg Drokin This code has been commented out since end of 2012, so it's clearly not needed here. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_lock.c b/

[PATCH 40/56] staging/lustre: Remove unused lli_open_count from struct ll_inode_info

From: Oleg Drokin Capabilities code was the only user and it is long gone. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/llite_internal.h | 6 ++ drivers/staging/lustre/lustre/llite/llite_lib.c | 1 - 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/

[PATCH 35/56] staging/lustre: Remove index file transfer structures

From: Oleg Drokin struct idx_info, struct lu_idxpage, lustre_swab_idx_info, request formats and related defines are all unused and could only happen for between-servers communications, so remove them from the client code. Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_i

[PATCH 37/56] staging/lustre: Remove last_rcvd-file related data

From: Oleg Drokin last_rcvd (last received) is a special server-side file on the Lustre FS that stores information about server filesystem and also list of connected clients and their state. There's no point in having any of tha ton the client, though. Signed-off-by: Oleg Drokin --- .../stagin

[PATCH 33/56] staging/lustre: S_DIRSYNC is always defined in kernel

From: Oleg Drokin No need to check for it. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lustre/include/lustre/l

[PATCH 36/56] staging/lustre: Remove server data from struct obd_export

From: Oleg Drokin Despite it's name struct obd_export could exist on the client too (used for reverse exports), but all the actual server bits are not needed there, so could be dropped. This is mostly various target-related stuff. Signed-off-by: Oleg Drokin --- .../staging/lustre/lustre/includ

[PATCH 32/56] staging/lustre: Remove lquota_glb_rec, lquota_slv_rec and lquota_acct_rec

From: Oleg Drokin These server-only structures and some supporting defines are unused on the client. Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_idl.h | 52 -- drivers/staging/lustre/lustre/ptlrpc/wiretest.c| 44 -- 2 file

[PATCH 34/56] staging/lustre: FMODE_READ and FMODE_WRITE are always defined in the kernel

From: Oleg Drokin So no need to see if they are. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lustre/include/l

[PATCH 30/56] staging/lustre: Remove duplicated quota defines

From: Oleg Drokin QUOTABLOCK_BITS, QUOTABLOCK_SIZE and toqb are only used on the server and are copied from kernel code anyway. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 12 drivers/staging/lustre/lustre/ptlrpc/wiretest.c

[PATCH 29/56] staging/lustre: Remove *_CONNECT_SUPPORTED defines

From: Oleg Drokin These defines are only used on the servers to determine what connect flags corresponding servers support: MDT_CONNECT_SUPPORTED OST_CONNECT_SUPPORTED MGS_CONNECT_SUPPORTED ECHO_CONNECT_SUPPORTED Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_idl.h

[PATCH 27/56] staging/lustre: Remove FSF mailing address from GPL notices

From: Oleg Drokin This patch removes remaining several places that still had that extra paragraph included in Lustre code. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/include/linux/libcfs/libcfs_cpu.h | 5 - drivers/staging/lustre/include/linux/libcfs/linux/linux-cpu.h | 5 -

[PATCH 25/56] staging/lustre: Get rid of C99 comments in lustre_idl.h

From: Oleg Drokin lustre_idl.h contains various protocol data. In order to avoid reusal of values that might have meant something else in the past, the old no longer used values are commented out, convert the comments into regular comments from C99 where appropriate. Signed-off-by: Oleg Drokin

[PATCH 22/56] staging/lustre: Get rid of ldlm_cancel_flags_t typedef

From: Oleg Drokin Replace it with enum ldlm_cancel_flags Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_dlm.h | 18 ++ drivers/staging/lustre/lustre/include/obd.h| 2 +- drivers/staging/lustre/lustre/include/obd_class.h | 2 +- drivers/st

[PATCH 20/56] staging/lustre: Remove ldlm_appetite_t typedef

From: Oleg Drokin Replace it with enum ldlm_appetite Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_dlm.h | 8 drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/lustre/

[PATCH 19/56] staging/lustre: Get rid of ldlm_error_t typedef

From: Oleg Drokin Replace it with enum ldlm_error Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_dlm.h | 8 drivers/staging/lustre/lustre/ldlm/ldlm_flock.c| 4 ++-- drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 4 ++-- drivers/staging/lustre/lus

[PATCH 18/56] staging/lustre: Get rid of lustre_quota_version typedef

From: Oleg Drokin Does not appear to be used anywhere. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_user.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_user.h b/drivers/staging

[PATCH 12/56] staging/lustre: Get rid of obd_cmd_t typedef

From: Oleg Drokin Replace with enum obd_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lustr

[PATCH 11/56] staging/lustre: Get rid of mgs_cmd_t typedef

From: Oleg Drokin Replace it with enum mgs_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lu

[PATCH 15/56] staging/lustre/ldlm: Remove unused lr_most_restr from struct ldlm_resource

From: Oleg Drokin This member comes from the dawn of time and is no longer needed Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_dlm.h | 3 --- drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 3 --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 1 - 3 f

[PATCH 14/56] staging/lustre: Get rid of sec_cmd_t typedef

From: Oleg Drokin Replace with enum sec_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lustr

[PATCH 10/56] staging/lustre: Remove ldlm type/mode typedefs

From: Oleg Drokin Replace ldlm_mode_t with enum ldlm_mode, ldlm_type_t with enum ldlm_type Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_idl.h | 14 drivers/staging/lustre/lustre/include/lustre_dlm.h | 39 -- drivers/staging/lustre/lus

[PATCH 01/56] staging/lustre/fld: Remove useless typedefs

From: Oleg Drokin Typedefs fld_hash_func_t and fld_scan_func_t are used exectly once in the following structure so don't really help anything at all. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/fld/fld_internal.h | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-)

[PATCH 08/56] staging/lustre: Get rid of mds_reint_t, mdt_reint_t typedefs

From: Oleg Drokin Replace with enum mdt_reint_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre

[PATCH 09/56] staging/lustre: Get rid of ldlm_cmd_t typedef

From: Oleg Drokin Replace it with enum ldlm_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/l

[PATCH 06/56] staging/lustre: Get rid of quota_cmd_t typedef

From: Oleg Drokin Replace it with enum quota_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/

[PATCH 07/56] staging/lustre: Get rid of mds_cmd_t typedef

From: Oleg Drokin Replace it with enum mds_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lu

[PATCH 02/56] staging/lustre: Get rid of client_obd_lock_t typedef

From: Oleg Drokin Replace it with struct client_obd_lock Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/linux/obd.h | 12 ++-- drivers/staging/lustre/lustre/include/obd.h | 4 ++-- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/stagi

[PATCH 04/56] stagig/lustre: Get rid of cksum_type_t typedef

From: Oleg Drokin Replace it with enum cksum_type Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- drivers/staging/lustre/lustre/include/obd.h | 2 +- drivers/staging/lustre/lustre/include/obd_cksum.h | 12 ++--

[PATCH 00/56] More Lustre cleanups.

From: Oleg Drokin This is another round of Lustre cleanups mostly aimed at fixing checkpatch warnings, but also removing unneeded code and fixign other things I am noticing along the way. Oleg Drokin (56): staging/lustre/fld: Remove useless typedefs staging/lustre: Get rid of client_obd_lock

[PATCH 05/56] staging/lustre: Get rid of ost_cmd_t typedef

From: Oleg Drokin Replace it with enum ost_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lu

[PATCH v2 01/84] staging/lustre/ldlm: Remove unused lr_most_restr from struct ldlm_resource

From: Oleg Drokin This member comes from the dawn of time and is no longer needed Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_dlm.h | 3 --- drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 3 --- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 1 - 3 f

[PATCH v2 09/84] staging/lustre: Get rid of mds_reint_t, mdt_reint_t typedefs

From: Oleg Drokin Replace with enum mdt_reint_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre

[PATCH v2 08/84] staging/lustre: Get rid of mds_cmd_t typedef

From: Oleg Drokin Replace it with enum mds_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lu

[PATCH v2 03/84] staging/lustre: Get rid of client_obd_lock_t typedef

From: Oleg Drokin Replace it with struct client_obd_lock Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/linux/obd.h | 12 ++-- drivers/staging/lustre/lustre/include/obd.h | 4 ++-- 2 files changed, 8 insertions(+), 8 deletions(-) diff --git a/drivers/stagi

[PATCH v2 23/84] staging/lustre/llite: Get rid of se_stat_t typedef

From: Oleg Drokin Replace it with enum se_stat Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/statahead.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/statahead.c b/drivers/staging/lustre/lustre/llite/st

[PATCH v2 21/84] staging/lustre: Get rid of ldlm_ns_type_t typedef

From: Oleg Drokin Replace it with enum ldlm_ns_type Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_dlm.h | 6 +++--- drivers/staging/lustre/lustre/ldlm/ldlm_lib.c | 2 +- drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 4 ++-- 3 files changed, 6 insertion

[PATCH v2 46/84] staging/lustre/ldlm: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/ldlm/interval_tree.c | 6 +- drivers/staging/lustre/lustre/ldlm/ldlm_extent.c | 12 +- drivers/staging/lustre/lus

[PATCH v2 33/84] staging/lustre: S_DIRSYNC is always defined in kernel

From: Oleg Drokin No need to check for it. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lustre/include/lustre/l

[PATCH v2 49/84] staging/lustre/lov: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/lov/lov_ea.c | 3 +- drivers/staging/lustre/lustre/lov/lov_internal.h | 6 +- drivers/staging/lustre/lustre/lo

[PATCH v2 19/84] staging/lustre: Get rid of ldlm_error_t typedef

From: Oleg Drokin Replace it with enum ldlm_error Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_dlm.h | 8 drivers/staging/lustre/lustre/ldlm/ldlm_flock.c| 4 ++-- drivers/staging/lustre/lustre/ldlm/ldlm_internal.h | 4 ++-- drivers/staging/lustre/lus

[PATCH v2 35/84] staging/lustre: Remove index file transfer structures

From: Oleg Drokin struct idx_info, struct lu_idxpage, lustre_swab_idx_info, request formats and related defines are all unused and could only happen for between-servers communications, so remove them from the client code. Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_i

[PATCH v2 43/84] staging/lustre/fld: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/fld/fld_cache.c| 9 ++--- drivers/staging/lustre/lustre/fld/fld_internal.h | 24 dri

[PATCH v2 12/84] staging/lustre: Get rid of mgs_cmd_t typedef

From: Oleg Drokin Replace it with enum mgs_cmd Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lu

[PATCH v2 71/84] staging/lustre: Remove RELEASEPAGE_ARG_TYPE compat macro

From: Oleg Drokin Not used in the kernel proper. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/rw26.c | 7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/rw26.c b/drivers/staging/lustre/lustre/llite/rw26.c index 64

[PATCH v2 67/84] staging/lustre: Remove liblustre references from explanations

From: Oleg Drokin Since liblustre is no longer with us, referencing to it in the explanations only makes things less clear Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/cl_object.h | 4 ++-- drivers/staging/lustre/lustre/include/lustre_lite.h | 2 +- drivers/staging/lu

[PATCH v2 62/84] staging/lustre/fld: Fix style vs open parenthesis alignment

From: Oleg Drokin This mostly fixes checkpatch complaints about "Alignment should match open parenthesis" Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/fld/fld_cache.c | 14 +++--- drivers/staging/lustre/lustre/fld/fld_request.c | 17 +++-- drivers/staging/

[PATCH v2 50/84] staging/lustre/mdc: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/mdc/mdc_lib.c | 3 +- drivers/staging/lustre/lustre/mdc/mdc_locks.c | 87 - drivers/stag

[PATCH v2 57/84] staging/lustre: Remove stray space before newline in messages

From: Oleg Drokin There were some messages in Lustre that ended with a space followed by a newline which is pointless, so remove the extra spaces. Found with checkpatch. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/lloop.c | 2 +- drivers/staging/lustre/lustre/llite/

[PATCH v2 73/84] staging/lustre/ptlrpc: Fix style vs open parenthesis alignment

From: Oleg Drokin This mostly fixes checkpatch complaints about "Alignment should match open parenthesis" Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/ptlrpc/client.c | 40 +--- drivers/staging/lustre/lustre/ptlrpc/import.c | 21 +++ drivers/staging/lustre

[PATCH v2 77/84] staging/lustre/mdc: Fix style vs open parenthesis alignment

From: Oleg Drokin This mostly fixes checkpatch complaints about "Alignment should match open parenthesis" Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/mdc/mdc_internal.h | 2 +- drivers/staging/lustre/lustre/mdc/mdc_locks.c| 26 +++- drivers/staging/lus

[PATCH v2 78/84] staging/lustre/mgc: Fix style vs open parenthesis alignment

From: Oleg Drokin This mostly fixes checkpatch complaints about "Alignment should match open parenthesis" Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/mgc/mgc_request.c | 28 - 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/drivers/stagi

[PATCH v2 79/84] staging/lustre/obdecho: Fix style vs open parenthesis alignment

From: Oleg Drokin This mostly fixes checkpatch complaints about "Alignment should match open parenthesis" Signed-off-by: Oleg Drokin --- .../staging/lustre/lustre/obdecho/echo_client.c| 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/stagi

[PATCH v2 80/84] staging/lustre/lmv: Fix style vs open parenthesis alignment

From: Oleg Drokin This mostly fixes checkpatch complaints about "Alignment should match open parenthesis" Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/lmv/lmv_intent.c | 2 +- drivers/staging/lustre/lustre/lmv/lmv_internal.h | 4 ++-- drivers/staging/lustre/lustre/lmv/lmv_o

[PATCH v2 82/84] staging/lustre: Rework MAX_DIO_SIZE macro

From: Oleg Drokin KMALLOC_MAX_SIZE is always defined in the kernel, so no point in checking for it, just use t directly. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/rw26.c | 10 ++ 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/drivers/staging/lust

[PATCH v2 76/84] staging/lustre/lov: Fix style vs open parenthesis alignment

From: Oleg Drokin This mostly fixes checkpatch complaints about "Alignment should match open parenthesis" Signed-off-by: Oleg Drokin --- .../staging/lustre/lustre/lov/lov_cl_internal.h| 87 +++--- drivers/staging/lustre/lustre/lov/lov_ea.c | 2 +- drivers/staging/l

[PATCH v2 81/84] staging/lustre: Fix lines that start with spaces

From: Oleg Drokin Found with checkpatch. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_cfg.h | 2 +- drivers/staging/lustre/lustre/include/lustre_dlm.h | 4 ++-- drivers/staging/lustre/lustre/llite/llite_nfs.c| 6 +++--- drivers/staging/lustre/lustre/llite/

[PATCH v2 84/84] staging/lustre/ptlrpc: Fix ENABLE_PINGER ifdef

From: Oleg Drokin As it is ENABLE_PINGER is never defined, but in reality it's the code that is now compiled out that should be used since all other instances were converted like that too. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/ptlrpc/import.c | 28 ---

[PATCH v2 83/84] staging/lustre: Remove unneeded {} in lprocfs_stats_unlock()

From: Oleg Drokin Since there's only one call in those if () else branches, the braces are not really necessary. Highlighted by checkpatch. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lprocfs_status.h | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) d

[PATCH v2 74/84] staging/lustre/osc: Fix style vs open parenthesis alignment

From: Oleg Drokin This mostly fixes checkpatch complaints about "Alignment should match open parenthesis" Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/osc/lproc_osc.c | 20 ++--- drivers/staging/lustre/lustre/osc/osc_cache.c | 18 +--- drivers/staging/lustre

[PATCH v2 72/84] staging/lustre: NEED_QUOTA_DEFS is never defined, drop it.

From: Oleg Drokin Signed-off-by: Oleg Drokin --- .../staging/lustre/lustre/include/lustre/lustre_user.h | 16 1 file changed, 16 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_user.h b/drivers/staging/lustre/lustre/include/lustre/lustre_user.h

[PATCH v2 70/84] staging/lustre: MS_POSIXACL is always defined, don't check for it

From: Oleg Drokin Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/llite_lib.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_lib.c b/drivers/staging/lustre/lustre/llite/llite_lib.c index 5b155b2..db9072c 100644 --- a/drivers/

[PATCH v2 68/84] staging/lustre/lmv: Remove commented out MDS selection policies

From: Oleg Drokin The new DNE code does not use them either so they are not longer needed. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/lmv/lmv_obd.c | 45 - 1 file changed, 45 deletions(-) diff --git a/drivers/staging/lustre/lustre/lmv/lmv_obd.c b

[PATCH v2 75/84] staging/lustre/obdclass: Fix style vs open parenthesis alignment

From: Oleg Drokin This mostly fixes checkpatch complaints about "Alignment should match open parenthesis" Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/obdclass/cl_io.c | 20 ++-- drivers/staging/lustre/lustre/obdclass/cl_lock.c | 21 ++-- drivers/staging/lustre/lustre/o

[PATCH v2 58/84] staging/lustre: Remove commented out obd functions

From: Oleg Drokin obd_register_page_removal_cb, obd_unregister_page_removal_cb obd_register_lock_cancel_cb, obd_unregister_lock_cancel_cb Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/obd_class.h | 51 --- 1 file changed, 51 deletions(-) diff --git a

[PATCH v2 56/84] staging/lustre: Fix indentation mistakes

From: Oleg Drokin none look like bugs or bugs-inducing, mostly just off by a couple of spaces all around. Signed-off-by: Oleg Drokin --- .../staging/lustre/lustre/include/lprocfs_status.h | 2 +- .../lustre/lustre/include/lustre/lustre_idl.h | 8 ++-- .../lustre/lustre/include/lustre/lu

[PATCH v2 61/84] staging/lustre/include: Fix style of function declarations

From: Oleg Drokin This mostly fixes checkpatch complaints about "Alignment should match open parenthesis" and "space prohibited between function name and open parenthesis" Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/cl_object.h | 410 ++--- .../staging

[PATCH v2 69/84] staging/lustre: F_SETLKW64 F_SETLK64 F_GETLK64 are always defined

From: Oleg Drokin Remove the ifdefs for the around usage. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/file.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/file.c b/drivers/staging/lustre/lustre/llite/file.c index 477dc0e..2

[PATCH v2 59/84] staging/lustre: Remove unused members of struct ll_sb_info

From: Oleg Drokin ll_orphan_dentry_list and ll_conn_chain are not used and could be removed. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/llite_internal.h | 2 -- drivers/staging/lustre/lustre/llite/llite_lib.c | 4 2 files changed, 6 deletions(-) diff --git a/

[PATCH v2 65/84] staging/lustre: Convert ptlrpc_at_check_timed to void

From: Oleg Drokin The only caller was not looking at the return value, and liblustre, that cared about it is not part of the kernel client. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/ptlrpc/service.c | 10 -- 1 file changed, 4 insertions(+), 6 deletions(-) diff --git

[PATCH v2 55/84] staging/lustre/ptlrpc: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/ptlrpc/connection.c | 3 +- drivers/staging/lustre/lustre/ptlrpc/events.c | 51 ++--- drivers/staging/lust

[PATCH v2 54/84] staging/lustre/osc: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/osc/osc_cache.c | 168 ++--- .../staging/lustre/lustre/osc/osc_cl_internal.h| 33 ++-- driver

[PATCH v2 60/84] staging/lustre/llite: Fix style vs open parenthesis alignment

From: Oleg Drokin This mostly fixes checkpatch complaints about "Alignment should match open parenthesis" Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/dcache.c | 3 +- drivers/staging/lustre/lustre/llite/dir.c | 6 +- drivers/staging/lustre/lustre/llite/f

[PATCH v2 66/84] staging/lustre: Remove misleading liblustre comments.

From: Oleg Drokin These two comments certainly refer to some ifdefed code that is no longer present, so remove them too. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/lov/lov_pool.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/lov/lov_pool.c

[PATCH v2 64/84] staging/lustre: Remove unused lustre_build_lock_params()

From: Oleg Drokin And also struct lustre_rw_params that is only referenced by it. Signed-off-by: Oleg Drokin --- .../staging/lustre/lustre/include/lustre_lite.h| 48 -- 1 file changed, 48 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre_lite.h b/

[PATCH v2 63/84] staging/lustre: Remove unused liblustre_check_services prototype

From: Oleg Drokin liblustre_check_services is no longer present in the tree. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_net.h | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/include/lustre_net.h b/drivers/staging/lustre/lustre/

[PATCH v2 47/84] staging/lustre/llite: Adjust comments to better conform to coding style

li_och_mutex); it->it_create_mode |= M_CHECK_STALE; rc = ll_intent_file_open(file->f_path.dentry, NULL, 0, it); @@ -654,9 +667,11 @@ restart: /* md_intent_lock() didn't get a request ref if there was an * ope

[PATCH v2 51/84] staging/lustre/mgc: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/mgc/mgc_request.c | 71 - 1 file changed, 46 insertions(+), 25 deletions(-) diff --git a/drive

[PATCH v2 52/84] staging/lustre/obdclass: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/obdclass/cl_io.c | 13 +++-- drivers/staging/lustre/lustre/obdclass/cl_lock.c | 24 +--- drivers/staging/lust

[PATCH v2 53/84] staging/lustre/obdecho: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/obdecho/echo_client.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/lust

[PATCH v2 44/84] staging/lustre/fid: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/fid/fid_request.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/staging/lustre/lus

[PATCH v2 39/84] staging/lustre: MAY_NOT_BLOCK is always defined, don't check for it.

From: Oleg Drokin Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/file.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/file.c b/drivers/staging/lustre/lustre/llite/file.c index bbe6ebf..ba65fc0 100644 --- a/drivers/staging/lustre/lu

[PATCH v2 41/84] staging/lustre: Reformat comments and structures in lustre_idl.h

From: Oleg Drokin This file mostly contains various on the wire data and got significantly mangled in transition. This patch rectifies the situation somewhat. Also fixes the comment style for block comments. Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_idl.h | 28

[PATCH v2 36/84] staging/lustre: Remove server data from struct obd_export

From: Oleg Drokin Despite it's name struct obd_export could exist on the client too (used for reverse exports), but all the actual server bits are not needed there, so could be dropped. This is mostly various target-related stuff. Signed-off-by: Oleg Drokin --- .../staging/lustre/lustre/includ

[PATCH v2 40/84] staging/lustre: Remove unused lli_open_count from struct ll_inode_info

From: Oleg Drokin Capabilities code was the only user and it is long gone. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/llite_internal.h | 6 ++ drivers/staging/lustre/lustre/llite/llite_lib.c | 1 - 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/

[PATCH v2 37/84] staging/lustre: Remove last_rcvd-file related data

From: Oleg Drokin last_rcvd (last received) is a special server-side file on the Lustre FS that stores information about server filesystem and also list of connected clients and their state. There's no point in having any of tha ton the client, though. Signed-off-by: Oleg Drokin --- .../stagin

[PATCH v2 38/84] staging/lustre: Remove dead code in ldlm_lock_destroy_internal

From: Oleg Drokin This code has been commented out since end of 2012, so it's clearly not needed here. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/ldlm/ldlm_lock.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_lock.c b/

[PATCH v2 42/84] staging/lustre/include: Adjust comment style

From: Oleg Drokin This fixes most of the "Block comments use a trailing */ on a separate line" checkpatch warnings, also some slight reformats of structures or comments at places. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/cl_object.h | 69 ++ dri

[PATCH v2 48/84] staging/lustre/lmv: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/lmv/lmv_fld.c| 3 ++- drivers/staging/lustre/lustre/lmv/lmv_intent.c | 9 --- drivers/staging/lustre/lustre/

[PATCH v2 45/84] staging/lustre/lclient: Adjust comments to better conform to coding style

From: Oleg Drokin This patch fixes "Block comments use a trailing */ on a separate line" warnings from checkpatch. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/lclient/glimpse.c| 3 ++- drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 24 ++ .../sta

[PATCH v2 28/84] staging/lustre: Remove server-only values from enum lu_dirent_attrs

From: Oleg Drokin These values cannot be seen on the client so we probably should not even be aware of them on the client. Signed-off-by: Oleg Drokin --- .../staging/lustre/lustre/include/lustre/lustre_idl.h| 16 1 file changed, 16 deletions(-) diff --git a/drivers/stagin

[PATCH v2 32/84] staging/lustre: Remove lquota_glb_rec, lquota_slv_rec and lquota_acct_rec

From: Oleg Drokin These server-only structures and some supporting defines are unused on the client. Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_idl.h | 52 -- drivers/staging/lustre/lustre/ptlrpc/wiretest.c| 44 -- 2 file

[PATCH v2 31/84] staging/lustre: Remove unused struct quota_body

From: Oleg Drokin It's only used for communications between servers. Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_idl.h | 31 --- .../lustre/lustre/include/lustre_req_layout.h | 3 -- drivers/staging/lustre/lustre/ptlrpc/layout.c | 36 -

[PATCH v2 30/84] staging/lustre: Remove duplicated quota defines

From: Oleg Drokin QUOTABLOCK_BITS, QUOTABLOCK_SIZE and toqb are only used on the server and are copied from kernel code anyway. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 12 drivers/staging/lustre/lustre/ptlrpc/wiretest.c

[PATCH v2 34/84] staging/lustre: FMODE_READ and FMODE_WRITE are always defined in the kernel

From: Oleg Drokin So no need to see if they are. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/staging/lustre/lustre/include/lustre/lustre_idl.h b/drivers/staging/lustre/lustre/include/l

[PATCH v2 26/84] staging/lustre: Remove server-side changelog defines

From: Oleg Drokin CHANGELOG_MAGIC, CHANGELOG_MINMASK, CHANGELOG_ALLMASK, CHANGELOG_DEFMASK and CHANGELOG_USER_PREFIX are all unused on the client. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 11 --- 1 file changed, 11 deletions(-) diff --

[PATCH v2 25/84] staging/lustre: Get rid of C99 comments in lustre_idl.h

From: Oleg Drokin lustre_idl.h contains various protocol data. In order to avoid reusal of values that might have meant something else in the past, the old no longer used values are commented out, convert the comments into regular comments from C99 where appropriate. Signed-off-by: Oleg Drokin

[PATCH v2 29/84] staging/lustre: Remove *_CONNECT_SUPPORTED defines

From: Oleg Drokin These defines are only used on the servers to determine what connect flags corresponding servers support: MDT_CONNECT_SUPPORTED OST_CONNECT_SUPPORTED MGS_CONNECT_SUPPORTED ECHO_CONNECT_SUPPORTED Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_idl.h

[PATCH v2 27/84] staging/lustre: Remove FSF mailing address from GPL notices

From: Oleg Drokin This patch removes remaining several places that still had that extra paragraph included in Lustre code. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/include/linux/libcfs/libcfs_cpu.h | 5 - drivers/staging/lustre/include/linux/libcfs/linux/linux-cpu.h | 5 -

<    1   2   3   4   5   6   7   8   9   10   >