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 |
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
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?
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
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
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
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 --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
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(+)