Thread (15 messages) 15 messages, 4 authors, 2016-11-26
STALE3470d
Revisions (3)
  1. v1 [diff vs current]
  2. v1 current
  3. v1 [diff vs current]

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

From: Greg KH <hidden>
Date: 2016-11-26 17:19:22

On Sat, Nov 26, 2016 at 11:56:17AM -0500, Walt Feasel wrote:
On Sat, Nov 26, 2016 at 12:05:50PM +0100, Greg KH wrote:
quoted
On Sat, Nov 26, 2016 at 03:43:18AM -0500, Walt Feasel wrote:
quoted
Make checkpatch modifications to include:

Multiple blank lines
Blank line after declarations
Space after cast
WARN_ON vs BUG_ON
Align parenthesis
Comparison to NULL
Line continuations

Walt Feasel (7):
  security: apparmor: apparmorfs.c Multiple blank lines
  security: apparmor: apparmorfs.c Blank line after declarations
  security: apparmor: apparmorfs.c Space after cast
  security: apparmor: apparmorfs.c WARN_ON vs BUG_ON
  security: apparmor: apparmorfs.c Align parenthesis
  security: apparmor: apparmorfs.c Comparison to NULL
  security: apparmor: apparmorfs.c Line continuations
Why are you sending apparmor patches to the kernelnewbies list and not
the maintainers and subsystem list for this code instead?

thanks,

greg k-h
Because as stated in my patch notes I am not sure how to
fix certain cases.
I have not been able to find reference material other than
other patches and discussions for certain cases and have
to try and reason why.
You wanted me to move out of staging so I have looked to
other areas but do not wish to send in ridiculous patches
and have me seen as being malicious.
Since this is kernelnewbies mailing list isn't that it's
purpose or am I move on again?
Feel free to skip my post should they trouble you.
Just send the patches to the proper people and list that
get_maintainer.pl shows you to, that's all, no need to bother
kernelnewbies with it.

good luck!

greg k-h
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help