On Fri 2014-11-07 13:37:29, Steven Rostedt wrote: > > Some more touch ups. > > -- Steve > > From 7e724556c21178a9890b31ff57f69761b41f435a Mon Sep 17 00:00:00 2001 > From: "Steven Rostedt (Red Hat)" <rost...@goodmis.org> > Date: Wed, 29 Oct 2014 15:26:09 -0400 > Subject: [PATCH] tracing: Have seq_buf use full buffer > > Currently seq_buf is full when all but one byte of the buffer is > filled. Change it so that the seq_buf is full when all of the > buffer is filled. > > Some of the functions would fill the buffer completely and report > everything was fine. This was inconsistent with the max of size - 1. > Changing this to be max of size makes all functions consistent. > > Link: http://lkml.kernel.org/r/20141104160222.502133...@goodmis.org > > Tested-by: Jiri Kosina <jkos...@suse.cz> > Acked-by: Jiri Kosina <jkos...@suse.cz> > Signed-off-by: Steven Rostedt <rost...@goodmis.org>
Reviewed-by: Petr Mladek <pmla...@suse.cz> This patch might need a refresh if you add the extra buffer overflow check into seq_buf_set_overflow() in the 3rd patch. Anyway, changes in this patch looks good. I am happy with all the changes in the patchset. Thanks a lot for driving it forward. Best Regards, Petr -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
