diff mbox series

MAINTAINERS: Add a git and a maintainer entry to keyring subsystems

Message ID 20190910143228.30305-1-jarkko.sakkinen@linux.intel.com (mailing list archive)
State New, archived
Headers show
Series MAINTAINERS: Add a git and a maintainer entry to keyring subsystems | expand

Commit Message

Jarkko Sakkinen Sept. 10, 2019, 2:32 p.m. UTC
Add Jarkko Sakkinen as co-maintainer of KEYS/KEYRINGS and add
git://git.infradead.org/users/jjs/linux-tpmdd.git as the GIT repository
for KEYS-TRUSTED.

Cc: David Howells <dhowells@redhat.com>
Cc: Mimi Zohar <zohar@linux.ibm.com>
Cc: James Bottomley <jejb@linux.ibm.com>
Signed-off-by: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
---
 MAINTAINERS | 2 ++
 1 file changed, 2 insertions(+)

Comments

David Howells Sept. 13, 2019, 3:03 p.m. UTC | #1
Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> wrote:

> Subject: [PATCH] MAINTAINERS: Add a git and a maintainer entry to keyring subsystems

I would recommend splitting the patch in two and putting something like:

	keys: Add Jarkko Sakkinen as co-maintainer

as the subject of the keyrings maintainership one.

Thanks,
David
Joe Perches Sept. 13, 2019, 5:36 p.m. UTC | #2
On Fri, 2019-09-13 at 16:03 +0100, David Howells wrote:
> Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> wrote:
> 
> > Subject: [PATCH] MAINTAINERS: Add a git and a maintainer entry to keyring subsystems
> 
> I would recommend splitting the patch in two and putting something like:
> 
> 	keys: Add Jarkko Sakkinen as co-maintainer
> 
> as the subject of the keyrings maintainership one.

Why is there utility in micro splitting such a trivial patch?
Jarkko Sakkinen Sept. 14, 2019, 12:53 p.m. UTC | #3
On Fri, Sep 13, 2019 at 04:03:53PM +0100, David Howells wrote:
> Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> wrote:
> 
> > Subject: [PATCH] MAINTAINERS: Add a git and a maintainer entry to keyring subsystems
> 
> I would recommend splitting the patch in two and putting something like:
> 
> 	keys: Add Jarkko Sakkinen as co-maintainer
> 
> as the subject of the keyrings maintainership one.

Thanks, I'll refine accordingly.

/Jarkko
Jarkko Sakkinen Sept. 14, 2019, 12:54 p.m. UTC | #4
On Fri, Sep 13, 2019 at 10:36:15AM -0700, Joe Perches wrote:
> On Fri, 2019-09-13 at 16:03 +0100, David Howells wrote:
> > Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com> wrote:
> > 
> > > Subject: [PATCH] MAINTAINERS: Add a git and a maintainer entry to keyring subsystems
> > 
> > I would recommend splitting the patch in two and putting something like:
> > 
> > 	keys: Add Jarkko Sakkinen as co-maintainer
> > 
> > as the subject of the keyrings maintainership one.
> 
> Why is there utility in micro splitting such a trivial patch?

I kind of get this for the MAINTAINERS file so that the changes can
be agreed/disagreed separately.

/Jarkko
David Howells Sept. 16, 2019, 8:52 a.m. UTC | #5
Joe Perches <joe@perches.com> wrote:

> > I would recommend splitting the patch in two and putting something like:
> > 
> > 	keys: Add Jarkko Sakkinen as co-maintainer
> > 
> > as the subject of the keyrings maintainership one.
> 
> Why is there utility in micro splitting such a trivial patch?

Because the keyrings maintainership piece is addressing a major procedural
concern on Linus's part - and I think it needs to go in its own patch with the
subject I suggested.

David
Jarkko Sakkinen Sept. 16, 2019, 6:23 p.m. UTC | #6
On Mon, Sep 16, 2019 at 09:52:51AM +0100, David Howells wrote:
> Joe Perches <joe@perches.com> wrote:
> 
> > > I would recommend splitting the patch in two and putting something like:
> > > 
> > > 	keys: Add Jarkko Sakkinen as co-maintainer
> > > 
> > > as the subject of the keyrings maintainership one.
> > 
> > Why is there utility in micro splitting such a trivial patch?
> 
> Because the keyrings maintainership piece is addressing a major procedural
> concern on Linus's part - and I think it needs to go in its own patch with the
> subject I suggested.

I'll send separate patches tomorrow.

/Jarkko
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index 108ea925a5d9..aa7cedd671d4 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -8937,6 +8937,7 @@  M:	Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
 M:	Mimi Zohar <zohar@linux.ibm.com>
 L:	linux-integrity@vger.kernel.org
 L:	keyrings@vger.kernel.org
+T:	git git://git.infradead.org/users/jjs/linux-tpmdd.git
 S:	Supported
 F:	Documentation/security/keys/trusted-encrypted.rst
 F:	include/keys/trusted-type.h
@@ -8945,6 +8946,7 @@  F:	security/keys/trusted.h
 
 KEYS/KEYRINGS:
 M:	David Howells <dhowells@redhat.com>
+M:	Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
 L:	keyrings@vger.kernel.org
 S:	Maintained
 F:	Documentation/security/keys/core.rst