On Wed, 29 Oct 2014 17:56:08 -0400 Steven Rostedt <[email protected]> wrote:
> From: Joe Perches <[email protected]> > > [ REQUEST FOR ACKS ] Can any of the DLM maintainers give me an Acked-by for this? Thanks! -- Steve > > Convert the seq_printf output with constant strings to seq_puts. > > Link: > http://lkml.kernel.org/p/b416b016f4a6e49115ba736cad6ea2709a8bc1c4.1412031505.git....@perches.com > > Cc: Christine Caulfield <[email protected]> > Cc: David Teigland <[email protected]> > Cc: [email protected] > Signed-off-by: Joe Perches <[email protected]> > Signed-off-by: Steven Rostedt <[email protected]> > --- > fs/dlm/debug_fs.c | 12 +++++------- > 1 file changed, 5 insertions(+), 7 deletions(-) > > diff --git a/fs/dlm/debug_fs.c b/fs/dlm/debug_fs.c > index 3bf460894088..eea64912c9c0 100644 > --- a/fs/dlm/debug_fs.c > +++ b/fs/dlm/debug_fs.c > @@ -279,7 +279,7 @@ static void print_format3(struct dlm_rsb *r, struct > seq_file *s) > print_name = 0; > } > > - seq_printf(s, "%s", print_name ? "str " : "hex"); > + seq_puts(s, print_name ? "str " : "hex"); > > for (i = 0; i < r->res_length; i++) { > if (print_name) > @@ -353,7 +353,7 @@ static void print_format4(struct dlm_rsb *r, struct > seq_file *s) > print_name = 0; > } > > - seq_printf(s, "%s", print_name ? "str " : "hex"); > + seq_puts(s, print_name ? "str " : "hex"); > > for (i = 0; i < r->res_length; i++) { > if (print_name) > @@ -390,23 +390,21 @@ static int table_seq_show(struct seq_file *seq, void > *iter_ptr) > break; > case 2: > if (ri->header) { > - seq_printf(seq, "id nodeid remid pid xid exflags " > - "flags sts grmode rqmode time_ms " > - "r_nodeid r_len r_name\n"); > + seq_puts(seq, "id nodeid remid pid xid exflags flags > sts grmode rqmode time_ms r_nodeid r_len r_name\n"); > ri->header = 0; > } > print_format2(ri->rsb, seq); > break; > case 3: > if (ri->header) { > - seq_printf(seq, "version rsb 1.1 lvb 1.1 lkb 1.1\n"); > + seq_puts(seq, "version rsb 1.1 lvb 1.1 lkb 1.1\n"); > ri->header = 0; > } > print_format3(ri->rsb, seq); > break; > case 4: > if (ri->header) { > - seq_printf(seq, "version 4 rsb 2\n"); > + seq_puts(seq, "version 4 rsb 2\n"); > ri->header = 0; > } > print_format4(ri->rsb, seq); -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
