[PATCH 0/7] security: apparmor: apparmorfs.c Checkpatch mods

Joel Fernandes agnel.joel at gmail.com
Sat Nov 26 17:41:49 EST 2016


On Nov 26, 2016 1:28 PM, "Bjørn Mork" <bjorn at mork.no> wrote:
>
> Greg KH <greg at kroah.com> writes:
> > On Sat, Nov 26, 2016 at 01:02:14PM -0500, Walt Feasel wrote:
> >
> >> So kernelnewbies is not to be used to learn about how to fix
> >> checkpatch type warning?
> >
> > Maybe, but really, if you have a well-formed patch, just send it to the
> > correct maintainers, it's up to them to accept it or not, that's their
> > job :)
>
> And I think the most important reason for doing that is because only
> they can answer the questions.  You need to know what the code does to
> be able to answer things like "is it OK to replace BUG_ON with WARN_ON
> here?". Although that is preferable according to checkpatch, it's not
> necessarily a 1-to-1 replacement. The error path changes, and the
> existing code is likely not tested or developed with the new path in
> mind.
>
> Nobody(?) in kernelnewbies knows anything about the apparmor code.  It's
> not that we don't want to answer. We just can't.
>

I think a better approach is to document your thoughts possibly in
Documentation/ than posting random patches to teach people what YOU think
needs learning.

Also people can already, if they need to, learn from patches posted on LKML
about what THEY want to learn :).

Thanks,
Joel

>
> Bjørn
>
>
>
> _______________________________________________
> Kernelnewbies mailing list
> Kernelnewbies at kernelnewbies.org
> https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.kernelnewbies.org/pipermail/kernelnewbies/attachments/20161126/f95e227c/attachment.html 


More information about the Kernelnewbies mailing list