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. > > > > 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]>
