On Sun, May 15, 2022 at 3:31 AM Shreenidhi Shedi <yessh...@gmail.com> wrote: > > This commit fixes almost all styling issues reported by checkpatch.pl > script. > > Signed-off-by: Shreenidhi Shedi <ssh...@vmware.com> > --- > kernel/audit.c | 205 ++++++++++++++++++++++++++++++------------------- > 1 file changed, 127 insertions(+), 78 deletions(-)
Please don't send me patches like this unless you are an established contributor to the audit subsystem, and even then I would strongly encourage you to think long and hard before hitting send. Most of this patch is just comment formatting churn, which is okay if you are doing something else in a nearby section of code, but a +200 line patch of such changes is not okay. The printk ratelimiting changes are probably okay to submit in a separate patch, but I didn't look at all of them. Although some of the pr_cont() to pr_info() conversions warrant a closer look. The EXPORT_SYMBOL() relocations are probably also okay is sent as a separate patch. -- paul-moore.com -- Linux-audit mailing list Linux-audit@redhat.com https://listman.redhat.com/mailman/listinfo/linux-audit