On 4/21/2025 2:35 PM, Mimi Zohar wrote:
On Mon, 2025-04-21 at 13:40 -0700, steven chen wrote:
On 4/21/2025 7:18 AM, Mimi Zohar wrote:
On Mon, 2025-04-21 at 09:51 -0400, Mimi Zohar wrote:
On Fri, 2025-04-18 at 12:36 +0800, Baoquan He wrote:
On 04/15/25 at 07:10pm, steven chen wrote:
From: Steven Chen <[email protected]>
^^^^^^
As James Bottomley previously explained[1], if you haven't made any changes to
Tushar's patch, then the very first line of the patch description would be
"From: Tushar Sugandhi <[email protected]>" followed by a blank line.
If there is a minor change, you would add "<your email address>: explanation".
For example:
Steven Chen <[email protected]>: modified patch description
To clarify: This line would be included below with your Signed-off-by tag.
[1]
https://lore.kernel.org/lkml/58e70121aaee33679ac295847197c1e5511b2a81.ca...@hansenpartnership.com/
Implement kimage_map_segment() to enable IMA to map the measurement log
list to the kimage structure during the kexec 'load' stage. This function
gathers the source pages within the specified address range, and maps them
to a contiguous virtual address range.
This is a preparation for later usage.
Implement kimage_unmap_segment() for unmapping segments using vunmap().
From: Tushar Sugandhi <[email protected]>
^^^^^^
Neither "Author:" nor "From:" belong here. Please remove.
Signed-off-by: Tushar Sugandhi <[email protected]>
^^^^^^^
Having Tushar's "Signed-off-by" tag and yours below indicate that you modified
the original author's patch.
To clarify: "Just" having Tushar's "Signed-off-by" tag and yours below indicate
that you modified the original author's patch.
Hi Mimi,
I will update it in next version.
Sigh, after reviewing your discussion with Baoquan, I'm not sure whether my
comment this morning added more confusion.
Option 1: Include a single "From:" line at the very top to change the author
from yourself to someone else. The following from line would make Tushar the
patch author: From: Tushar Sugandhi <[email protected]>
In addition, any minor changes you made should be added before your Signed-off-
by tag. For example: Steven Chen <[email protected]>: modified patch
description
Option 2: As mentioned previously and now discussed with Baoquan, adding "Co-
developed-by:"
Just wandering are you done reviewing or still need more time?
Yes, I just finished reviewing/testing. The patch descriptions are looking much
better. As Baoquan reminded you, please remember to update the "ima: define and
call ima_alloc_kexec_file_buf()" patch description.
thanks,
Mimi
Thanks a lot!
Cc: Eric Biederman <[email protected]>
Cc: Baoquan He <[email protected]>
Cc: Vivek Goyal <[email protected]>
Cc: Dave Young <[email protected]>
Signed-off-by: steven chen <[email protected]>
^^^^^
The signing on this patch is a little confusing. I can't see who is the
real author, who is the co-author, between you and Tushar. You may need
to refer to Documentation/process/5.Posting.rst to make that clear.
Acked-by: Baoquan He <[email protected]>
