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
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.
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 -
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
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
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 --
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
---
From: Oleg Drokin
of the two macroses is contained the LUSTRE_RELEASE was unused and
BUILD_VERSION is too generically named and was replaced with
LUSTRE_VERSION_STRING with all the users (provided by lustre_ver.h)
With in-kernel client the build version should be combined
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
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
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
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
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
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,
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 ++--
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
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
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
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 +-
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
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
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
From: Oleg Drokin
Replace with enum llog_op_type
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
From: Oleg Drokin
Replace it with struct lu_fid
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/include/lustre/lustre_idl.h | 2 +-
.../lustre/lustre/include/lustre/lustre_user.h | 24 +++---
2 files changed, 13
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
From: Oleg Drokin
Object update was originally planned to be used for cross-server
updates, so nothign to do with the client. The idea was
later significantly reworked anyway so even wire structs should
not be retained.
This gets rid of update_cmd_t, UPDATE and
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
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
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
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
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
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
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 ---
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
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
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 +-
From: Oleg Drokin
Replace it with direct reference of enum loc_flags
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lu_object.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
From: Oleg Drokin
Due to unfortunate reordering that broke the build in the middle,
I am resending the previous patchset with some extra added patches.
This contains various Lustre cleanups and coding style fixes,
also hunts out various ifdefs that we don't really need
and
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 +-
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,
From: Oleg Drokin
of the two macroses is contained the LUSTRE_RELEASE was unused and
BUILD_VERSION is too generically named and was replaced with
LUSTRE_VERSION_STRING with all the users (provided by lustre_ver.h)
With in-kernel client the build version should be combined
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
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
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
---
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 +-
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
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 ++---
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
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
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
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
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
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
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 --
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
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
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
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
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 ++---
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 +++--
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 |
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
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 --
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
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
---
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
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
---
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
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 ++-
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
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.
e_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
* open error, so don't do cleanup on the request here
-* (bug 3430) */
+* (bug 3430)
+
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 ++--
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 -
From: Oleg Drokin
Object update was originally planned to be used for cross-server
updates, so nothing to do with the client. The idea was
later significantly reworked anyway so even wire structs should
not be retained.
This gets rid of update_cmd_t, UPDATE and
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
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 +-
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
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 +-
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
From: Oleg Drokin
Replace it with struct lu_fid
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/include/lustre/lustre_idl.h | 2 +-
.../lustre/lustre/include/lustre/lustre_user.h | 24 +++---
2 files changed, 13
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
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
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 ---
From: Oleg Drokin
Replace with enum llog_op_type
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre/lustre_idl.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
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 +-
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
From: Oleg Drokin
Replace it with direct reference of enum loc_flags
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lu_object.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git
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
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
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
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 +-
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
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
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
From: Oleg Drokin
Majority of this patch is Lustre code style cleanups.
NULL comparison style is adjusted for the whole lustre tree
(but not lnet not to interfere with James patches in flight).
Some server-only code was removed in the process.
lustre/libcfs in addition
From: Oleg Drokin
Lines that were too long for not good reason were shortened in this patch.
Found with checkpatch.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/libcfs/linux/linux-debug.c | 6 --
From: Oleg Drokin
Replace it with enum cfs_trace_buf_type
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/libcfs/linux/linux-tracefile.c | 2 +-
drivers/staging/lustre/lustre/libcfs/tracefile.h | 6 +++---
2 files changed, 4
From: Oleg Drokin
checkpatch complains that the trailing */ on the multiline comments
should be on it's own line.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/libcfs/debug.c | 18 +-
From: Oleg Drokin
No need for an empty line after opening curvy brace and no need
for an empty line before the closing one too.
Remove them.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/libcfs/linux/linux-crypto.c| 2 --
201 - 300 of 754 matches
Mail list logo