Problematic culture around Signed-off-by

[Ian Molton]

Hi folks,

I've been away from kernel development for a bit, but I've returned and
I'm troubled by what seems to be an entrenched and widespread (IMO)
misuse of the "Signed-off-by:" in commits.

I've now either been asked to sign off RFC quality patches "because its
quicker" on more than one occasion in the last week or so, and I've seen
others signing off code which clearly has no hope of going anywhere near
the kernel. (eg. // commented out lines)

I was of the impression that Signed-off-by: was intended to be used on
essentially *finished* commits, indicating both readiness for inclusion
upstream and ones ownership of the copyright.

Even if the intent is *purely* a copyright isue, Signing off
*everything* surely makes it far too easy for people to get junk into
the kernel.

I've actually been deliberately NOT signing off my code because I *DONT*
want it surreptitiously applied with my name on it unless its 100% spot on.

WTH is going on? How can we fix this?

Is Signed-off-by: a lost cause for code quality?

Do we need to add a

Ready-for-mainline: {yes,no} field as well?

Have I just missed something? Maybe I'm just in a minority here? Whats
going on?

-Ian