On Mon, Apr 29, 2024, Zide Chen wrote:
> Currently, the migration worker delays 1-10 us, assuming that one
> KVM_RUN iteration only takes a few microseconds.  But if the CPU low
> power wakeup latency is large enough, for example, hundreds or even
> thousands of microseconds deep C-state exit latencies on x86 server
> CPUs, it may happen that it's not able to wakeup the target CPU before
> the migration worker starts to migrate the vCPU thread to the next CPU.
> 
> If the system workload is light, most CPUs could be at a certain low
> power state, which may result in less successful migrations and fail the
> migration/KVM_RUN ratio sanity check.  But this is not supposed to be
> deemed a test failure.
> 
> This patch adds a command line option to skip the sanity check in
> this case.
> 
> Signed-off-by: Zide Chen <[email protected]>
> Co-developed-by: donsheng <[email protected]>

This needs Dongsheng's SoB, and your SoB should come last.  And the attributed
name for any tag should use the person's full name.  Given that I have emails 
from
Dongsheng that show up as "Dongsheng Zhang", I _assume_ "donsheng" is 
incomplete,
but that's a big assumption on my part.

Dongsheng, can you provide your explicit SoB, with how you want your name to 
show
up?  Thanks!

>From Documentation/process/submitting-patches.rst:

  Co-developed-by: states that the patch was co-created by multiple developers;
  it is used to give attribution to co-authors (in addition to the author
  attributed by the From: tag) when several people work on a single patch.  
Since
  Co-developed-by: denotes authorship, every Co-developed-by: must be 
immediately
  followed by a Signed-off-by: of the associated co-author.  Standard sign-off
  procedure applies, i.e. the ordering of Signed-off-by: tags should reflect the
  chronological history of the patch insofar as possible, regardless of whether
  the author is attributed via From: or Co-developed-by:.  Notably, the last
  Signed-off-by: must always be that of the developer submitting the patch.
  
  Note, the From: tag is optional when the From: author is also the person (and
  email) listed in the From: line of the email header.
  
  Example of a patch submitted by the From: author::
  
          <changelog>
  
          Co-developed-by: First Co-Author <[email protected]>
          Signed-off-by: First Co-Author <[email protected]>
          Co-developed-by: Second Co-Author <[email protected]>
          Signed-off-by: Second Co-Author <[email protected]>
          Signed-off-by: From Author <[email protected]>
  
  Example of a patch submitted by a Co-developed-by: author::
  
          From: From Author <[email protected]>
  
          <changelog>
  
          Co-developed-by: Random Co-Author <[email protected]>
          Signed-off-by: Random Co-Author <[email protected]>
          Signed-off-by: From Author <[email protected]>
          Co-developed-by: Submitting Co-Author <[email protected]>
          Signed-off-by: Submitting Co-Author <[email protected]>

Reply via email to