Linux Security Modules development
 help / color / mirror / Atom feed
* [PATCH] Documentation: fix typo and formattting in security/credentials.rst
@ 2026-05-06  5:49 Mayank Gite
  2026-05-06 17:31 ` Randy Dunlap
  0 siblings, 1 reply; 7+ messages in thread
From: Mayank Gite @ 2026-05-06  5:49 UTC (permalink / raw)
  To: Paul Moore
  Cc: Mayank Gite, Serge Hallyn, Jonathan Corbet, Shuah Khan,
	linux-security-module, linux-doc, linux-kernel

- Fixes a typo in "Keys and keyrings" section. Replaces "keying" with
  "keyring".
- Updates formatting of keyring types.

Signed-off-by: Mayank Gite <drapl0n@protonmail.com>
---
 Documentation/security/credentials.rst | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/security/credentials.rst b/Documentation/security/credentials.rst
index d0191c8b8060..4996838491b1 100644
--- a/Documentation/security/credentials.rst
+++ b/Documentation/security/credentials.rst
@@ -189,9 +189,9 @@ The Linux kernel supports the following types of credentials:
      be searched for the desired key.  Each process may subscribe to a number
      of keyrings:
 
-	Per-thread keying
-	Per-process keyring
-	Per-session keyring
+	- Per-thread keyring
+	- Per-process keyring
+	- Per-session keyring
 
      When a process accesses a key, if not already present, it will normally be
      cached on one of these keyrings for future accesses to find.
-- 
2.53.0


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] Documentation: fix typo and formattting in security/credentials.rst
  2026-05-06  5:49 [PATCH] Documentation: fix typo and formattting in security/credentials.rst Mayank Gite
@ 2026-05-06 17:31 ` Randy Dunlap
  0 siblings, 0 replies; 7+ messages in thread
From: Randy Dunlap @ 2026-05-06 17:31 UTC (permalink / raw)
  To: Mayank Gite, Paul Moore
  Cc: Serge Hallyn, Jonathan Corbet, Shuah Khan, linux-security-module,
	linux-doc, linux-kernel

Hi--

On 5/5/26 10:49 PM, Mayank Gite wrote:
> - Fixes a typo in "Keys and keyrings" section. Replaces "keying" with
>   "keyring".
> - Updates formatting of keyring types.
> 
> Signed-off-by: Mayank Gite <drapl0n@protonmail.com>

Something (protonmail?) split some lines of your patch so that it
cannot be applied:

patch: **** malformed patch at line 24: tials:

See https://lore.kernel.org/linux-doc/20260506054914.162806-1-drapl0n@protonmail.com/raw

You can see if Documentation/process/email-clients.rst can help you any,
or just do an web search for using Proton mail to send plain text patches,
but using Proton might be difficult (IDK).

> ---
>  Documentation/security/credentials.rst | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/security/credentials.rst b/Documentation/security/credentials.rst
> index d0191c8b8060..4996838491b1 100644
> --- a/Documentation/security/credentials.rst
> +++ b/Documentation/security/credentials.rst
> @@ -189,9 +189,9 @@ The Linux kernel supports the following types of credentials:
>       be searched for the desired key.  Each process may subscribe to a number
>       of keyrings:
>  
> -	Per-thread keying
> -	Per-process keyring
> -	Per-session keyring
> +	- Per-thread keyring
> +	- Per-process keyring
> +	- Per-session keyring
>  
>       When a process accesses a key, if not already present, it will normally be
>       cached on one of these keyrings for future accesses to find.

The change itself looks good - if you can find a good way to email it.

thanks.
-- 
~Randy


^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH] Documentation: fix typo and formattting in security/credentials.rst
@ 2026-05-06 22:59 Mayank Gite
  2026-05-07  0:36 ` Randy Dunlap
  0 siblings, 1 reply; 7+ messages in thread
From: Mayank Gite @ 2026-05-06 22:59 UTC (permalink / raw)
  To: Paul Moore
  Cc: Mayank Gite, Serge Hallyn, Jonathan Corbet, Shuah Khan,
	linux-security-module, linux-doc, linux-kernel

- Fixes a typo in "Keys and keyrings" section. Replaces "keying" with
  "keyring".
- Updates formatting of keyring types.

Signed-off-by: Mayank Gite <drapl0n.kernel@gmail.com>
---
 Documentation/security/credentials.rst | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/security/credentials.rst b/Documentation/security/credentials.rst
index d0191c8b8060..4996838491b1 100644
--- a/Documentation/security/credentials.rst
+++ b/Documentation/security/credentials.rst
@@ -189,9 +189,9 @@ The Linux kernel supports the following types of credentials:
      be searched for the desired key.  Each process may subscribe to a number
      of keyrings:
 
-	Per-thread keying
-	Per-process keyring
-	Per-session keyring
+	- Per-thread keyring
+	- Per-process keyring
+	- Per-session keyring
 
      When a process accesses a key, if not already present, it will normally be
      cached on one of these keyrings for future accesses to find.
-- 
2.53.0

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] Documentation: fix typo and formattting in security/credentials.rst
  2026-05-06 22:59 Mayank Gite
@ 2026-05-07  0:36 ` Randy Dunlap
  2026-05-07 17:48   ` John Doe
  0 siblings, 1 reply; 7+ messages in thread
From: Randy Dunlap @ 2026-05-07  0:36 UTC (permalink / raw)
  To: Mayank Gite, Paul Moore
  Cc: Serge Hallyn, Jonathan Corbet, Shuah Khan, linux-security-module,
	linux-doc, linux-kernel

Hi,

On 5/6/26 3:59 PM, Mayank Gite wrote:
> - Fixes a typo in "Keys and keyrings" section. Replaces "keying" with
>   "keyring".
> - Updates formatting of keyring types.
> 
> Signed-off-by: Mayank Gite <drapl0n.kernel@gmail.com>

Acked-by: Randy Dunlap <rdunlap@infradead.org>
Tested-by: Randy Dunlap <rdunlap@infradead.org>

> ---
In the future, right here, after that "---" line, it should say
something like

v2: use 'git send-email' so that that patch isn't corrupted

and in the Subject: it should say
[PATCH v2] ...

Thanks.

>  Documentation/security/credentials.rst | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/security/credentials.rst b/Documentation/security/credentials.rst
> index d0191c8b8060..4996838491b1 100644
> --- a/Documentation/security/credentials.rst
> +++ b/Documentation/security/credentials.rst
> @@ -189,9 +189,9 @@ The Linux kernel supports the following types of credentials:
>       be searched for the desired key.  Each process may subscribe to a number
>       of keyrings:
>  
> -	Per-thread keying
> -	Per-process keyring
> -	Per-session keyring
> +	- Per-thread keyring
> +	- Per-process keyring
> +	- Per-session keyring
>  
>       When a process accesses a key, if not already present, it will normally be
>       cached on one of these keyrings for future accesses to find.

-- 
~Randy

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] Documentation: fix typo and formattting in security/credentials.rst
  2026-05-07  0:36 ` Randy Dunlap
@ 2026-05-07 17:48   ` John Doe
  2026-05-07 18:31     ` Jonathan Corbet
  0 siblings, 1 reply; 7+ messages in thread
From: John Doe @ 2026-05-07 17:48 UTC (permalink / raw)
  To: Randy Dunlap
  Cc: Paul Moore, Serge Hallyn, Jonathan Corbet, Shuah Khan,
	linux-security-module, linux-doc, linux-kernel

On Wed, May 06, 2026 at 05:36:53PM -0700, Randy Dunlap wrote:
> Hi,
> 
> On 5/6/26 3:59 PM, Mayank Gite wrote:
> > - Fixes a typo in "Keys and keyrings" section. Replaces "keying" with
> >   "keyring".
> > - Updates formatting of keyring types.
> > 
> > Signed-off-by: Mayank Gite <drapl0n.kernel@gmail.com>
> 
> Acked-by: Randy Dunlap <rdunlap@infradead.org>
> Tested-by: Randy Dunlap <rdunlap@infradead.org>
> 
> > ---
> In the future, right here, after that "---" line, it should say
> something like
> 
> v2: use 'git send-email' so that that patch isn't corrupted
> 
> and in the Subject: it should say
> [PATCH v2] ...
> 
> Thanks.
> 
> >  Documentation/security/credentials.rst | 6 +++---
> >  1 file changed, 3 insertions(+), 3 deletions(-)
> > 
> > diff --git a/Documentation/security/credentials.rst b/Documentation/security/credentials.rst
> > index d0191c8b8060..4996838491b1 100644
> > --- a/Documentation/security/credentials.rst
> > +++ b/Documentation/security/credentials.rst
> > @@ -189,9 +189,9 @@ The Linux kernel supports the following types of credentials:
> >       be searched for the desired key.  Each process may subscribe to a number
> >       of keyrings:
> >  
> > -	Per-thread keying
> > -	Per-process keyring
> > -	Per-session keyring
> > +	- Per-thread keyring
> > +	- Per-process keyring
> > +	- Per-session keyring
> >  
> >       When a process accesses a key, if not already present, it will normally be
> >       cached on one of these keyrings for future accesses to find.
> 
> -- 
> ~Randy

Hi,

Thank you for the feedback and acknowledgment. I thought that since my
earlier patch was invalid/corrupted, I should submit a new one rather than
version 2.

I understand the procedure now and will ensure that future revisions follow
the guidelines.

Thanks.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] Documentation: fix typo and formattting in security/credentials.rst
  2026-05-07 17:48   ` John Doe
@ 2026-05-07 18:31     ` Jonathan Corbet
  2026-05-07 19:35       ` Mayank Gite
  0 siblings, 1 reply; 7+ messages in thread
From: Jonathan Corbet @ 2026-05-07 18:31 UTC (permalink / raw)
  To: John Doe, Randy Dunlap
  Cc: Paul Moore, Serge Hallyn, Shuah Khan, linux-security-module,
	linux-doc, linux-kernel

John Doe <drapl0n.kernel@gmail.com> writes:

> Thank you for the feedback and acknowledgment. I thought that since my
> earlier patch was invalid/corrupted, I should submit a new one rather than
> version 2.
>
> I understand the procedure now and will ensure that future revisions follow
> the guidelines.

Along those lines, seeing multiple names associated with the same email
account is a bit disconcerting.  We are not generally interested in
taking patches submitted under pseudonyms...

jon

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] Documentation: fix typo and formattting in security/credentials.rst
  2026-05-07 18:31     ` Jonathan Corbet
@ 2026-05-07 19:35       ` Mayank Gite
  0 siblings, 0 replies; 7+ messages in thread
From: Mayank Gite @ 2026-05-07 19:35 UTC (permalink / raw)
  To: Jonathan Corbet
  Cc: Randy Dunlap, Paul Moore, Serge Hallyn, Shuah Khan,
	linux-security-module, linux-doc, linux-kernel

On Thu, May 07, 2026 at 12:31:48PM -0600, Jonathan Corbet wrote:
> John Doe <drapl0n.kernel@gmail.com> writes:
> 
> > Thank you for the feedback and acknowledgment. I thought that since my
> > earlier patch was invalid/corrupted, I should submit a new one rather than
> > version 2.
> >
> > I understand the procedure now and will ensure that future revisions follow
> > the guidelines.
> 
> Along those lines, seeing multiple names associated with the same email
> account is a bit disconcerting.  We are not generally interested in
> taking patches submitted under pseudonyms...

While configuring mutt, I forgot to change the default placeholder value
for the "realname" variable, resulting in an inconsistent sender name.

I have corrected the configuration and verified that my environment is
now set up appropriately.

Sorry for the confusion.

Mayank

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2026-05-07 19:35 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-06  5:49 [PATCH] Documentation: fix typo and formattting in security/credentials.rst Mayank Gite
2026-05-06 17:31 ` Randy Dunlap
  -- strict thread matches above, loose matches on Subject: below --
2026-05-06 22:59 Mayank Gite
2026-05-07  0:36 ` Randy Dunlap
2026-05-07 17:48   ` John Doe
2026-05-07 18:31     ` Jonathan Corbet
2026-05-07 19:35       ` Mayank Gite

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox