linux-kernel - Re: [PATCH] Change the document about iowait

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20161119103916.3a562069@lwn.net>
Date:   Sat, 19 Nov 2016 10:39:16 -0700
From:   Jonathan Corbet <corbet@....net>
To:     Chao Fan <fanc.fnst@...fujitsu.com>
Cc:     <linux-doc@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <peterz@...radead.org>, <akpm@...ux-foundation.org>,
        <izumi.taku@...fujitsu.com>, Cao Jin <caoj.fnst@...fujitsu.com>
Subject: Re: [PATCH] Change the document about iowait

On Mon, 14 Nov 2016 11:42:41 +0800
Chao Fan <fanc.fnst@...fujitsu.com> wrote:

> >Sorry for the delay on this.  Life has been...challenging...
> >  
> >> Signed-off-by: Cao Jin <caoj.fnst@...fujitsu.com>
> >> Signed-off-by: Chao Fan <fanc.fnst@...fujitsu.com>  
> >
> >What does this signoff chain mean?    
> 
> Hi jon,
> 
> I made this patch. He is my colleague who helped me a lot on
> investigating why this value decrease, so I added his s-o-b.
> I didn't know how to put the position, but both of us are
> responsible for this patch. Should I change and resend it?

The meaning of Signed-off-by is defined in the DOC, as seen in
Documentation/submitting-patches.rst.  In this case, the better approach
would have been to credit Cao Jin in the changelog, since he didn't write
the patch.

For a little documentation tweak like this it's not that big a deal,
though, so I simply gone ahead and applied it.

Thanks,

jon