Re: [PATCH] ARM: hw_breakpoint: Handle inexact watchpoint addresses

On Thu, Aug 06, 2020 at 04:41:44PM +0100, Russell King - ARM Linux admin wrote:

On Thu, Aug 06, 2020 at 08:05:10AM -0700, Doug Anderson wrote:

quoted

Yet if I go check mainline the patch is not there.  This came to my
attention since we had my patch picked to the Chrome OS 4.19 tree and
suddenly recently got a stable merge conflict with "ARM: 8986/1:
hw_breakpoint: Don't invoke overflow handler on uaccess watchpoints".

Anyone know what happened here?

Yes.  Stephen Rothwell raised a complaint against it, which you were
copied with:

quoted

Hi all,

Commit

  116375be0461 ("ARM: 8944/1: hw_breakpoint: Handle inexact watchpoint addresses")

is missing a Signed-off-by from its author.

My reply to Stephen's email was:

quoted

Thanks Stephen, patch dropped.

It looks like Doug used his "m.disordat.com" address to submit the
patch through the web interface, and there was no From: in the patch
itself, so that was used as the patch author.  However, as you spotted,
it was signed off using Doug's "chromium.org" address.

I think it's time to make the patch system a bit more strict, checking
that the submission address is mentioned in a signed-off-by tag
somewhere in the commit message.

Doug, the patch system does have your "chromium.org" address, if that's
the one you want to use as the author, please submit using that instead.
Thanks.

Russell.

Neither email got a response from you, so the patch was dropped and
nothing further happened.

I should've also said, there's two ways to avoid that problem:

1. Provide a From: line in the standard way to tell the patch system
   who the author of the patch is (the author defaults to the known
   login email address.)

2. Update your login email address in the system to the one you
   normally author patches.

-- 
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last!

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel