[Cluster-devel] [GFS2 PATCH] GFS2: Make rename not save dirent location

2014-09-29 Thread Bob Peterson
Hi, This patch fixes a regression in the patch GFS2: Remember directory insert point, commit 2b47dad866d04f14c328f888ba5406057b8c7d33. The problem had to do with the rename function: The function found space for the new dirent, and remembered that location. But then the old dirent was removed,

[Cluster-devel] [PATCH 4/7] dlm: Use seq_puts, remove unnecessary trailing spaces

2014-09-29 Thread Joe Perches
Convert the seq_printf output with constant strings to seq_puts. Remove unnecessary trailing spaces from seq_(printf/puts) output. Signed-off-by: Joe Perches j...@perches.com --- fs/dlm/debug_fs.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/fs/dlm/debug_fs.c

[Cluster-devel] [PATCH 3/7] dlm: Use seq_is_full - remove seq_printf returns

2014-09-29 Thread Joe Perches
The seq_printf return should be ignored and the seq_is_full function should be tested instead. Convert functions returning int to void where seq_printf is used. Signed-off-by: Joe Perches j...@perches.com --- fs/dlm/debug_fs.c | 248 +- 1 file

[Cluster-devel] [PATCH 0/7] seq_printf cleanups

2014-09-29 Thread Joe Perches
seq_printf should return void. Add a public bool seq_is_full function that can be used to shortcut unnecesary seq_printf/seq_puts calls when the seq buffer is full. Start removing the misuses of the seq_printf/seq_puts return value. Patchset brought forward from an unreplied to set of changes