Hi Christoph and Steven,
Have addressed your review comment to update the patch description and title for this patch. Can you please provide your ack to this patch. Thanks, ~ Vaibhav Vaibhav Jain <vaib...@linux.ibm.com> writes: > 'seq_buf' provides a very useful abstraction for writing to a string > buffer without needing to worry about it over-flowing. However even > though the API has been stable for couple of years now its still not > exported to kernel loadable modules limiting its usage. > > Hence this patch proposes update to 'seq_buf.c' to mark > seq_buf_printf() which is part of the seq_buf API to be exported to > kernel loadable GPL modules. This symbol will be used in later parts > of this patch-set to simplify content creation for a sysfs attribute. > > Cc: Piotr Maziarz <piotrx.mazi...@linux.intel.com> > Cc: Cezary Rojewski <cezary.rojew...@intel.com> > Cc: Christoph Hellwig <h...@infradead.org> > Cc: Steven Rostedt <rost...@goodmis.org> > Cc: Borislav Petkov <b...@alien8.de> > Signed-off-by: Vaibhav Jain <vaib...@linux.ibm.com> > --- > Changelog: > > v7..v8: > * Updated the patch title [ Christoph Hellwig ] > * Updated patch description to replace confusing term 'external kernel > modules' to 'kernel lodable modules'. > > Resend: > * Added ack from Steven Rostedt > > v6..v7: > * New patch in the series > --- > lib/seq_buf.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/lib/seq_buf.c b/lib/seq_buf.c > index 4e865d42ab03..707453f5d58e 100644 > --- a/lib/seq_buf.c > +++ b/lib/seq_buf.c > @@ -91,6 +91,7 @@ int seq_buf_printf(struct seq_buf *s, const char *fmt, ...) > > return ret; > } > +EXPORT_SYMBOL_GPL(seq_buf_printf); > > #ifdef CONFIG_BINARY_PRINTF > /** > -- > 2.26.2 > --
