On Sun, Sep 27, 2026 at 05:24:46PM +0200, Antheas Kapenekakis wrote:
On Sun, 27 Sept 2026 at 16:06, Sasha Levin <[email protected]> wrote:
If someone adds their sign-off without understanding what it certifies,
that's on them. We can't have agents add it for them just because some
people would ignore its meaning anyway.
I didn't say we should make the agent add it. I said if someone wants
to do it it is up to them. Not you, not the mainline kernel config,
not this patch. And disabling it does nothing because the user does
not understand what a sign off is anyway and will add it later or
submit without it. In which case you get broken patches anyway and you
solved nothing.
Again, that's on the user. Your argument is saying that we should just drop
this whole DCO nonsense because no one understands it anyway.
We can't stop enforcing things just because users don't bother reading the
docs.
Nothing is stopping you from changing this behavior to your liking either, but
at that point it makes you a downstream consumer on the kernel, so you get to
carry that delta.
If you think a hand-written AGENTS.md would do better, feel free to
write one and propose it. For now, this one seems to be working just
fine.
No, I am fine with one not existing and it would be my preference
actually. Ideally it would be added to .gitignore as well.
That just does not address your need of steering novel contributor
plus LLM, which I tried to help with. But if you do not want to put
more effort than a symlink then not breaking our workflows is more
important so this patch should not merge.
All I care about is having all these "agents" follow the rules and guidelines
that the kernel community set around LLM contributions. I'm not trying to
improve the quality or make it more efficient. This patch is purely about
having the majority of LLM agents follow our rules out of the box.
If you want to spend a few months trying to massage an AGENTS.md to the liking
of your LLMs-of-choice, then go at it.
--
Thanks,
Sasha
