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
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 insertions(+), 13 deletions(-)
diff --git a/drivers/st
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
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
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 with
kernel build infor
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
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
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 --
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
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
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
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
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
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
From: Oleg Drokin
This patchset fixes some minor Lustre problems and also
updates the internal version.
Please consider.
Oleg Drokin (5):
staging/lustre/obdecho: Remove always true condition
staging/lustre/obdecho: Better handle invalid create requests
staging/lustre: Adjust import state
From: Oleg Drokin
LUSTRE_VERSION_ALLOWED_OFFSET is only used on the server to disallow
connection of old userspace clients.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_ver.h | 9 +++--
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/sta
From: Oleg Drokin
While initial code drop was corresponding to 2.3.64, lots of
different changes went in since then, and also quite a bunch of fixes,
but almost none of the new features.
Code-wise we are almost at 2.5.0, so let's call it 2.4.60 - this is a
version that was never reached in the ac
From: Oleg Drokin
Dan Carpenter noticed that since we already checked for
(oa->o_valid & OBD_MD_FLID) == 0, that means
(oa->o_valid & OBD_MD_FLID) is always true after that so
no point in checking for it.
Signed-off-by: Oleg Drokin
Reported-by: Dan Carpenter
---
drivers/staging/lustre/lustre/
From: Oleg Drokin
New test scripts expect spaces around state names and square brackets
when parsing debugfs output, so add them to avoid false failures.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/obdclass/lprocfs_status.c | 14 +++---
1 file changed, 7 insertions(+),
From: Oleg Drokin
When gettng an invalid create request in echo code (wrong group,
or no group at all), just return an error instead of crashing.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/obdecho/echo_client.c | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
From: Oleg Drokin
commit 020ecc6f3229 ("staging: lustre: llite: Remove IS_ERR tests")
managed to break one of the converted checks by losing a dereference
causing the condition to be wrong and hence leading
to a crash later on in case of actual error.
IS_ERR_OR_NULL(*inode) got converted into !in
From: Oleg Drokin
This is a server-side request printing function, so we don't
really need it on the client.
Signed-off-by: Oleg Drokin
Reported-by: Arnd Bergmann
---
.../staging/lustre/lustre/include/lprocfs_status.h | 4 ---
.../staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c| 30 -
From: Oleg Drokin
It's only used on the server
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/include/lustre_req_layout.h | 3 --
drivers/staging/lustre/lustre/ptlrpc/layout.c | 62 --
2 files changed, 65 deletions(-)
diff --git a/d
From: Oleg Drokin
All of them but the ccc_transient_page_prep are unused, so remove
the unused ones.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lclient.h| 19 -
drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 48 --
2 files changed
From: Oleg Drokin
Quite a bunch of them are only used on the server.
lu_object_put_nocache, lu_object_invariant, lu_dev_del_linkage,
lu_context_tags_update, lu_context_tags_clear, lu_session_tags_update,
lu_session_tags_clear, lu_env_refill_by_tags, lu_printk_printer,
lu_object_assign_fid, lu_obj
From: Oleg Drokin
These seem to be unused.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lclient.h| 4
drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 15 ---
2 files changed, 19 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/
From: Oleg Drokin
These client request/import functions are not used anywhere,
so drop them.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_net.h | 13 ---
drivers/staging/lustre/lustre/ptlrpc/client.c | 95 --
d
From: Oleg Drokin
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_fid.h | 51 --
1 file changed, 51 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/lustre_fid.h
b/drivers/staging/lustre/lustre/include/lustre_fid.h
index bbe3ff8..
From: Oleg Drokin
Does not appear to be used anywhere.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lclient.h| 1 -
drivers/staging/lustre/lustre/lclient/lcommon_cl.c | 7 ---
2 files changed, 8 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/lc
From: Oleg Drokin
This patch is a first stab at trying to remove structure fields
from obd_export and obd structures that are only used on the
server or make sense on the server. These include tracking
requests in recovery, various recovery stages, lists of
recovered and not yet recovered clients
From: Oleg Drokin
Those functions are not present anywhere in the client code.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_fid.h | 24 --
1 file changed, 24 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/lustre_fid.h
b/driv
From: Oleg Drokin
nid_hash is used on export to faster find clients based on
their NID. There's no use for that on the client.
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/include/lustre_export.h | 1 -
drivers/staging/lustre/lustre/include/obd.h| 2 -
drivers/staging/lu
From: Oleg Drokin
Since we know lcs_srv is always NULL, just get rid of it completely
and fix up all the code to assumee it was never there.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/fid/fid_request.c| 48 --
drivers/staging/lustre/lustre/fid/lproc_fi
From: Oleg Drokin
In fact they could be static as they are only used inside
this file, so remove EXPORT_SYMBOL and declarations.
Also seq_client_init is always called with srv = NULL, so just
drop this argument.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/fid/fid_request.c
From: Arnd Bergmann
This is a large patch to remove all dead code from obdclass and
ptlrpc, purely removing functions and declarations now, so
despite the size should be easy enough to verify.
Signed-off-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/include/lustre/lustre_
From: Oleg Drokin
Also while we are at it, remove seq_site_fini forward declaration
as there's no such function anymore.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/fid/fid_request.c| 51 --
drivers/staging/lustre/lustre/include/lustre_fid.h | 3 --
2
From: Oleg Drokin
They are no longer used anywhere.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/obdclass/linux/linux-obdo.c | 137 -
1 file changed, 137 deletions(-)
diff --git a/drivers/staging/lustre/lustre/obdclass/linux/linux-obdo.
From: Oleg Drokin
All users are only in the server code
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/include/lustre_disk.h| 2 --
drivers/staging/lustre/lustre/obdclass/obd_mount.c | 27 --
2 files changed, 29 deletions(-)
diff
From: Oleg Drokin
Drop unused functions.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_sec.h | 20 --
drivers/staging/lustre/lustre/ptlrpc/sec.c | 116
drivers/staging/lustre/lustre/ptlrpc/sec_bulk.c| 329 -
From: Oleg Drokin
These functions are not even referenced in any header files anymore.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/obdclass/obdo.c | 170 --
1 file changed, 170 deletions(-)
diff --git a/drivers/staging/lustr
From: Oleg Drokin
These functions are mostly used on the server.
class_uuid2obd, get_devices_count, class_obd_list, class_conn2obd,
class_conn2cliimp, class_connected_export, obd_exports_barrier,
kuc_* (kernel-userspace communications).
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
--
From: Oleg Drokin
It's not referenced anywhere anymore.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c | 5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/staging/lustre/lustre/ldlm/ldlm_lockd.c
b/drivers/staging/lustre/l
From: Oleg Drokin
No callers left.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_handles.h | 1 -
drivers/staging/lustre/lustre/obdclass/lustre_handles.c | 13 -
2 files changed, 14 deletions(-)
diff --git a/drivers/stagi
From: Oleg Drokin
No point in retaining it if it's unused.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_net.h | 1 -
drivers/staging/lustre/lustre/ptlrpc/ptlrpcd.c | 42 --
2 files changed, 43 deletions(-)
dif
From: Oleg Drokin
cl_object_header_fini, cl_object_has_locks, cl_attr2lvb,
cl_page_list_own, cl_page_list_unmap, cl_2queue_assume,
cl_io_print, cl_enqueue,
also cleanup extern declarations in cl_object.h
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/i
From: Oleg Drokin
rs_batch is used on the server only.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_net.h | 6 +-
drivers/staging/lustre/lustre/ptlrpc/service.c | 145 -
2 files changed, 1 insertion(+), 150 de
From: Oleg Drokin
It's only used on the server.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_lib.h | 1 -
drivers/staging/lustre/lustre/obdclass/statfs_pack.c | 14 --
2 files changed, 15 deletions(-)
diff --git a/drive
From: Oleg Drokin
This function is only used on the server where real high-priority
requests actually exist.
This deletes ptlrpc_hpreq_handler() and ptlrpc_request_change_export()
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/lustre_net.h | 3
From: Oleg Drokin
Apparently this is only used only on the metadata server.
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/include/lustre_eacl.h| 4 -
drivers/staging/lustre/lustre/obdclass/acl.c | 119 -
2 files changed, 12
From: Oleg Drokin
ptlrpc_service_health_check is only used on a service, so
it makes no point to retain it in the client code.
Also removing it's helpers: ptlrpc_svcpt_health_check and
ptlrpc_nrs_req_peek_nolock
Reported-by: Arnd Bergmann
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/
From: Oleg Drokin
This set removes a bunch of lustre code that is no lonver relevant,
is server specific or is otherwise unused.
Please consider.
More to come later.
Arnd Bergmann (1):
staging/lustre: remove lots of dead code
Oleg Drokin (28):
staging/lustre: Remove unused target_print_re
From: Oleg Drokin
This is take two for the compat stuff removal.
Oleg Drokin (6):
staging/lustre: Drop SEEK_* definition checks
staging/lustre: remove unused ll_quota_on and ll_quota_off
staging/lustre: Drop FMODE_UNSIGNED_OFFSET define
staging/lustre: Use hlist primitives directly
sta
From: Oleg Drokin
Get rid of ll_d_hlist* compat defines.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/linux/lustre_compat25.h | 9 -
drivers/staging/lustre/lustre/llite/dcache.c | 3 +--
drivers/staging/lustre/lustre/llite/namei.c
From: Oleg Drokin
They are noop anyways.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/linux/lustre_compat25.h | 5 -
drivers/staging/lustre/lustre/llite/dir.c | 4
2 files changed, 9 deletions(-)
diff --git a/drivers/staging/lustre/lustre/i
From: Oleg Drokin
It's not really used anywhere.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/linux/lustre_compat25.h | 4
1 file changed, 4 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/linux/lustre_compat25.h
b/drivers/staging/lustre/lustre/in
From: Oleg Drokin
These primitives are long deprecated and unused.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/linux/lustre_compat25.h | 5 -
drivers/staging/lustre/lustre/llite/llite_lib.c | 5 +
drivers/staging/lustre/lustre/llite/vvp_io.c
From: Oleg Drokin
SEEK_DATA and SEEK_HOLE are always defined in the kernel,
drop the definition checks
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/linux/lustre_compat25.h | 7 ---
1 file changed, 7 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/li
From: Oleg Drokin
They are not used anywhere, so safe to drop.
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/include/linux/lustre_compat25.h | 31 --
1 file changed, 31 deletions(-)
diff --git a/drivers/staging/lustre/lustre/include/linux/lustre_compat25.h
b/drivers/s
From: Andreas Dilger
Now that the lustre.org domain has been liberated we can again
use that for the main website URL and mailing list.
Also update the URL for userspace tools downloads and Git repo.
Signed-off-by: Andreas Dilger
Signed-off-by: Oleg Drokin
---
MAINTAINERS
From: Oleg Drokin
It turns out if you call ldlm_pools_fini without completing the
ldlm_pools_init, then attempt to unregister not yet registered
shrinkers makes the kernel very unhappy.
So make sure we have them registered first.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/ldl
From: Oleg Drokin
In the past when Lustre did its own allocation amounts tracking
the results were shown in sysctl as current and overall max
number of bytes and pages allocated. Now that we don't
track these, remove the sysctls.
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/obdclass/linux/
From: Dmitry Eremin
They are just fancy module parameters wrappers,
so just the same functionality now would be accessible via
/sys/module/libcfs/parameters/libcfs_console_{min,max}_delay
Also install compatibility symlinks
Signed-off-by: Dmitry Eremin
Signed-off-by: Oleg Drokin
---
drivers/
From: Oleg Drokin
It's just a fancy libcfs_debug_mb module parameter wrapper,
so just add debug buffer size check and resizing and the same
functionality now would be accessible via
/sys/module/libcfs/parameters/libcfs_debug_mb
Also add a symlink for backwards compatibility.
Signed-off-by: Oleg
From: Oleg Drokin
/proc/sys/lnet/console_ratelimit, debug_path and panic_on_lbug
are module parameters with no special magic accessible via
/sys/module/libcfs/parameters/libcfs_console_ratelimit,
/sys/module/libcfs/parameters/libcfs_debug_file_path and
/sys/module/libcfs/parameters/libcfs_panic_o
From: Dmitry Eremin
Remove portal_enter_debugger because it's not used any more.
Signed-off-by: Dmitry Eremin
---
drivers/staging/lustre/lustre/libcfs/debug.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/lustre/lustre/libcfs/debug.c
b/drivers/staging/lustre/lustre/lib
From: Oleg Drokin
Parameters in lnet sysctl are of debug quantity, so let's move them
to debugfs instead.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/libcfs/module.c | 86 +--
1 file changed, 56 insertions(+), 30 deletions(-)
diff --git a/drivers/stagi
From: Dmitry Eremin
It is no longer used anywhere.
Signed-off-by: Dmitry Eremin
---
drivers/staging/lustre/include/linux/libcfs/libcfs_debug.h | 1 -
drivers/staging/lustre/lustre/libcfs/debug.c | 3 ---
drivers/staging/lustre/lustre/libcfs/module.c | 12 --
From: Dmitry Eremin
/proc/sys/lnet/lnet_memused
Remove memory tracking for LNet.
Remove redundant enums definition.
Signed-off-by: Dmitry Eremin
---
.../lustre/include/linux/libcfs/libcfs_private.h | 28 +--
.../staging/lustre/lnet/klnds/o2iblnd/o2iblnd.c| 12 ---
.../sta
From: Oleg Drokin
Since all of the variables from /proc/sys/lustre were moved to
/sys/fs/lustre, get rid of the remaining infrastructure.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/obd_class.h | 1 -
drivers/staging/lustre/lustre/obdclass/class_obd.c | 1 -
.../lus
From: Oleg Drokin
ldlm_timeout is used server-side to determine AST timeouts,
so it makes no sense on the client, esp. since it's not really used
anywhere.
Remove all traces of it except from the config where make
it a noop.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/
From: Oleg Drokin
module parameter libcfs_console_backoff accessible through
/sys/module/libcfs/parameters/libcfs_console_backoff would
do the same thing, just add a special "uintpos" parameter
type to disallow 0 values too.
Also add a symlink to the module parameter variable for
backwards compat
From: Oleg Drokin
debug_peer_on_timeout, dump_on_timeout and dump_on_eviction
controls from /proc/sys/lustre to /sys/fs/lustre
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/obdclass/linux/linux-sysctl.c| 28 ++
drivers/staging/lustre/sysfs-fs-lustre | 26
From: Oleg Drokin
This is the first step of moving lustre sysctls from
/proc/sys/lustre to /sys/fs/lustre
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/include/obd_class.h | 4 +-
drivers/staging/lustre/lustre/obdclass/class_obd.c | 6 +-
.../lustre/lustre/obdclass/linux/linu
From: Oleg Drokin
This helps to avoid use after free on unmount.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/lov/lov_obd.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/lustre/lustre/lov/lov_obd.c
b/drivers/staging/lustre/lustre/lo
From: Oleg Drokin
max_dirty_mb is now a parameter in /sys/fs/lustre/
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/obdclass/linux/linux-sysctl.c| 74 +-
drivers/staging/lustre/sysfs-fs-lustre | 12
2 files changed, 43 insertions(+), 43 deletions(-)
From: Oleg Drokin
Adaptive Timeouts controls are being moved from /proc/sys/lustre
to /sys/fs/lustre
Signed-off-by: Oleg Drokin
---
.../lustre/lustre/obdclass/linux/linux-sysctl.c| 45 --
drivers/staging/lustre/sysfs-fs-lustre | 53 ++
2 file
From: Oleg Drokin
It was used to control allocation failure rate, but there is
in-kernel way of doing that that's more versatile too.
This is going to remove just the sysctl, the underlying variable
will be removed once all OBD_ALLOC* macros removal patchseries land.
Signed-off-by: Oleg Drokin
From: Oleg Drokin
Dan Carpenter noticed that procfs conversion patches introduced
a bug where should kobject_create_and_add, an error is not returned
from class_procfs_init.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/obdclass/linux/linux-module.c | 2 +-
1 file changed, 1 ins
From: Oleg Drokin
This patch series removes the sysctls (also under /proc) from Lustre.
/proc/sys/lnet is moved to debugfs and
/proc/sys/lustre content is moved to /sys/fs/lustre
Additionally we now have the tools that understand the new layout,
and so first few patches fix problems introduced b
From: Oleg Drokin
ldebugfs_remove is usually called on directories with files passed in
as attributes, so simple debugfs_remove failes on them as not empty
Switch to debugfs_remove_recursive.
This fixes a number of problems where a new filesystem is mounted after
being unmounted first.
Signed-o
From: Oleg Drokin
Apparently kstrtouint could return not just -EINVAL, but also -ERANGE,
so make sure we just check the return value for something negative.
Noticed by Dan Carpenter
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/libcfs/debug.c | 2 +-
1 file changed, 1 insertio
From: Dmitry Eremin
They are just fancy module parameters wrappers,
so just the same functionality now would be accessible via
/sys/module/libcfs/parameters/libcfs_console_{min,max}_delay
Also install compatibility symlinks
Signed-off-by: Dmitry Eremin
Signed-off-by: Oleg Drokin
---
drivers/
From: Dmitry Eremin
Remove portal_enter_debugger because it's not used any more.
Signed-off-by: Dmitry Eremin
---
drivers/staging/lustre/lustre/libcfs/debug.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/staging/lustre/lustre/libcfs/debug.c
b/drivers/staging/lustre/lustre/lib
From: Oleg Drokin
Rediffed the last few patches as per Greg's instructions.
Also Added a folow on fix for the uint_min_max param to
properly take into account kstrtouint range of error return values.
Please consider.
Dmitry Eremin (2):
staging/lustre/libcfs: get rid of
debugfs/lnet/consol
From: Oleg Drokin
It's just a fancy libcfs_debug_mb module parameter wrapper,
so just add debug buffer size check and resizing and the same
functionality now would be accessible via
/sys/module/libcfs/parameters/libcfs_debug_mb
Also add a symlink for backwards compatibility.
Signed-off-by: Oleg
From: Oleg Drokin
ldlm_namespace_sysfs_unregister needs to be called ldlm_namespace_free_post
so that we don't have this dangling object there after the namespace
has disappeared.
Signed-off-by: Oleg Drokin
---
drivers/staging/lustre/lustre/ldlm/ldlm_resource.c | 1 +
1 file changed, 1 inserti
From: Oleg Drokin
In debugfs it's not enough to just set file mode to read-only to
deny write access to a file, instead just don't provide
the write method unless write access is really requested.
Signed-off-by: Oleg Drokin
---
I assume allowing run-time changes via /sys/module is preferrable,
From: Oleg Drokin
roundup_pow_of_two return when called on a zero argument is
undefined, so don't call it like that.
This fixes a problem introduced by commit 322489d9d551
("staging/lustre: Use roundup_pow_of_two() in LNetEQAlloc()")
since 0 is a valid count parameter for LNetEQAlloc. Also manif
From: Oleg Drokin
It turns out that unlike procfs, debugfs does not really enforce
permissions for root (similar to regular filesystems), so we need
to ensure we are not providing ->write() method to read-only files
and ->read() method for write-only files at registration.
This fixes a couple of
From: Oleg Drokin
These two patches tie some loose ends from the Lustre debugfs conversion,
but while investigating them I also accumulated some questions
that would be good to get answers for.
1. Unlike procfs, debugfs does not really guard your back and if root
comes in and tries to write to a
From: Oleg Drokin
Turns out we mistakenly export some pretty-wide-reaching debugfs
functions as EXPORT_SYMBOL instead of EXPORT_SYMBOL_GPL as we should,
so this patch rectifies the situation.
Signed-off-by: Oleg Drokin
---
.../staging/lustre/lustre/obdclass/lprocfs_status.c| 18 +--
From: Oleg Drokin
drm_debugfs_create_files and drm_debugfs_remove_files
should only be available for GPL-code only as per Greg-KH
Signed-off-by: Oleg Drokin
---
drivers/gpu/drm/drm_debugfs.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/drm_debugfs.c b
s could make some sense.
If you turn the problem upside down to solve the representation question
first, maybe there's a way forward defining the "power tree" in terms of
regulators, and then adding something in struct regulator that spams
readers with timestamped results if the
Hi -
Running perf test on 3.10-rc6 gives some unexpected behaviours.
This is on a dual Cortex A9 board with perf userland cross-built from
same tree as the kernel.
root@aa9-eb:~# perf test
1: vmlinux symtab matches kallsyms: FAILED!
2: detect open syscall event
On 26 June 2013 21:39, Prabhat Kumar Ravi wrote:
> Hi,
>
> On Wed, Jun 26, 2013 at 4:39 PM, Andy Green wrote:
>> Hi -
>>
>> Running perf test on 3.10-rc6 gives some unexpected behaviours.
>>
>> This is on a dual Cortex A9 board with perf userland cros
From: Andy Green
warning-elimination: android: binder
This commit in mainline (now) causes a couple of warnings
commit 975a1ac9a9fe65d66ee1726c0db6dc58e53d232a
Author: Arve Hjønnevåg
Date: Tue Oct 16 15:29:53 2012 -0700
Staging: android: binder: Add some tracepoints
This patch fixes
up of the PRCMU code
and on the multiplatform support for OMAP with mailbox enabled. We
can definitely collaborate on the improvements. Andy Green would also
be interested, as he is also looking into adopting the mailbox API.
To clarify Jassi works on my team, after I wrote two Mailbox drivers for
op to mediate,
Jassi is suggesting being able to do the client locking at your layer as
a primitive will simplify things, not least get rid of the mediation
driver. Your layer has concept of completion and notifier already so it
seems it wouldn't take much more.
-Andy
--
Andy Green | Fujit
ailbox usage.
At least, this real scenario should be interesting to think about before
rejecting ^^
-Andy
--
Andy Green | Fujitsu Landing Team Leader
Linaro.org │ Open source software for ARM SoCs | Follow Linaro
http://facebook.com/pages/Linaro/155974581091106 -
http://twitter.com/#!/linaro
iled into the kernel. I've downloaded the latest Tulip
driver but haven't tried it yet.
* I've spent many hours trying to get this to work, and my manager is
getting desparate.
Do you have any ideas?
Thank you VERY, VERY much,
Avi
==
501 - 600 of 1193 matches
Mail list logo
