netdev - Re: [PATCH] rxrpc: Neaten logging macros and add KERN

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1512065211.19952.112.camel@perches.com>
Date:   Thu, 30 Nov 2017 10:06:51 -0800
From:   Joe Perches <joe@...ches.com>
To:     David Miller <davem@...emloft.net>
Cc:     linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
        dhowells@...hat.com, linux-afs <linux-afs@...ts.infradead.org>
Subject: Re: [PATCH] rxrpc: Neaten logging macros and add KERN_DEBUG logging
 level

On Thu, 2017-11-30 at 12:31 -0500, David Miller wrote:
> From: Joe Perches <joe@...ches.com>
> Date: Thu, 30 Nov 2017 08:21:14 -0800
> 
> > When enabled, the current debug logging does not have a KERN_<LEVEL>.
> > Add KERN_DEBUG to the logging macros.
> > 
> > Miscellanea:
> > 
> > o Remove #define redundancy and neaten the macros a bit
> > 
> > Signed-off-by: Joe Perches <joe@...ches.com>
> 
> Please also CC: the proper mailing list and maintainer for rxrpc.
> 
> The MAINTAINERS entry for rxrpc needs to include more than just
> net/rxrpc/af_rxrpc.c, David please fix this up as I know Joe is going
> to use the excuse that he didn't CC: you and the AFS list because of
> this.

There is no listed rxrpc maintainer.

David Howells entry is:

AFS FILESYSTEM & AF_RXRPC SOCKET DOMAIN
M:	David Howells <dhowells@...hat.com>
L:	linux-afs@...ts.infradead.org
S:	Supported
F:	fs/afs/
F:	include/net/af_rxrpc.h
F:	net/rxrpc/af_rxrpc.c
W:	https://www.infradead.org/~dhowells/kafs/

which seems pretty specific to AFS and its rxrpc kernel interface.

Excuse is probably the wrong word here David.
Rationale maybe...

cheers, Joe