[Cluster-devel] DLM patches from PLD Linux

2012-11-02 Thread Jacek Konieczny
Hello, These are three of the patches I have applied to dlm when packaging it for PLD Linux. I think they could be applied upstream. Greets, Jacek

[Cluster-devel] [PATCH 2/3] Run dlm.service after sys-kernel-config.mount

2012-11-02 Thread Jacek Konieczny
dlm_controld requires configfs mounted. systemd can take care for that, but it does it better if the order is set in the unit file. Signed-off-by: Jacek Konieczny jaj...@jajcus.net --- init/dlm.service | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/init/dlm.service b/init

[Cluster-devel] [PATCH 3/3] Do not crash uncleanly on SIGINT and SIGHUP

2012-11-02 Thread Jacek Konieczny
that, but when someone tries this by mistake, this is not a reason to crash. Signed-off-by: Jacek Konieczny jaj...@jajcus.net --- dlm_controld/main.c | 9 + 1 file changed, 9 insertions(+) diff --git a/dlm_controld/main.c b/dlm_controld/main.c index 6eafa49..fd469f9 100644

[Cluster-devel] [PATCH 1/3] Fix linking order

2012-11-02 Thread Jacek Konieczny
Objects should generally be listed before the libraries they require. DLM building fails for me without these changes. Signed-off-by: Jacek Konieczny jaj...@jajcus.net --- dlm_controld/Makefile | 4 ++-- dlm_tool/Makefile | 2 +- fence/Makefile| 2 +- libdlm/Makefile | 8

[Cluster-devel] cluster4 dlm dlm_stonith – should it really fence by turning node off?

2012-11-03 Thread Jacek Konieczny
Hello, The dlm_stonith fencing helper is really convenient when Pacemaker is in use. Though, it doesn't quite work as I would expect – when fencing is needed it requests a node to be turned off instead of rebooting. And it doesn't handle unfencing – so automatic recovery is not possible

[Cluster-devel] [PATCH 1/2] --foreground option added to dlm_controld

2012-11-03 Thread Jacek Konieczny
Foreground is the preferred way to run services from systemd and it is easier to provide startup notification this way. Signed-off-by: Jacek Konieczny jaj...@jajcus.net --- dlm_controld/dlm_controld.8 |3 +++ dlm_controld/dlm_daemon.h |1 + dlm_controld/main.c |7 ++- 3

[Cluster-devel] cluster4 dlm: startup notification for systemd

2012-11-03 Thread Jacek Konieczny
Hello, The two patches: [PATCH 1/2] --foreground option added to dlm_controld [PATCH 2/2] Startup notification by sd_notify() add startup notification for the systemd service unit. This way startup of services depending on DLM can be properly serialized. Currently dlm_controld forks

[Cluster-devel] [PATCH 2/2] Startup notification by sd_notify()

2012-11-03 Thread Jacek Konieczny
additional dependency, the feature my by disabled on compile time by calling 'make' with 'USE_SD_NOTIFY=no'. Signed-off-by: Jacek Konieczny jaj...@jajcus.net --- dlm_controld/Makefile |8 dlm_controld/main.c |8 init/dlm.service |5 +++-- 3 files changed, 19 insertions

[Cluster-devel] [PATCH] dlm_stonith_{off, reboot} aliases for fence helper

2012-11-05 Thread Jacek Konieczny
Not so pretty hack to allow rebooting instead of halting the node fenced by dlm_stonith. 'dlm_stonith_reboot' can be now used instead of 'dlm_stonith' in the dlm.conf file to request node reboot. 'dlm_stonith_off' alias is also provided to explicitly request power-off. Signed-off-by: Jacek

Re: [Cluster-devel] cluster4 dlm dlm_stonith ??? should it really fence by turning node off?

2012-11-05 Thread Jacek Konieczny
My workaround for the missing functionality (or wrong default): [PATCH] dlm_stonith_{off,reboot} aliases for fence helper Greets, Jacek

Re: [Cluster-devel] [PATCH] dlm_stonith_{off, reboot} aliases for fence helper

2012-11-05 Thread Jacek Konieczny
On Mon, Nov 05, 2012 at 02:30:33PM -0500, David Teigland wrote: On Mon, Nov 05, 2012 at 07:05:22PM +0100, Jacek Konieczny wrote: - rv = stonith_api_kick_helper(nodeid, 300, 1); + rv = stonith_api_kick_helper(nodeid, 300, turn_off); I'd like it to be reboot, but seeing the arg as bool

[Cluster-devel] [PATCH] dlm: Allow 64-bit request when CONFIG_COMPAT is on

2013-02-19 Thread Jacek Konieczny
Even if kernel is compiled for 32-bit ABI compatibility it still should able to process 64-bit dlm_write_request struct. The regression was introduced by: b75bc91 dlm: check the maximum size of a request from user Signed-off-by: Jacek Konieczny jaj...@jajcus.net --- fs/dlm/user.c | 4 1