[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aIcCBZ6MVX1OHKpw@lappy>
Date: Mon, 28 Jul 2025 00:52:21 -0400
From: Sasha Levin <sashal@...nel.org>
To: Greg KH <gregkh@...uxfoundation.org>
Cc: corbet@....net, linux-doc@...r.kernel.org, workflows@...r.kernel.org,
josh@...htriplett.org, kees@...nel.org,
konstantin@...uxfoundation.org, linux-kernel@...r.kernel.org,
rostedt@...dmis.org
Subject: Re: [PATCH 2/4] agents: add core development references
On Mon, Jul 28, 2025 at 06:24:44AM +0200, Greg KH wrote:
>On Sun, Jul 27, 2025 at 03:58:00PM -0400, Sasha Levin wrote:
>> Signed-off-by: Sasha Levin <sashal@...nel.org>
>> ---
>> Documentation/agents/core.rst | 28 ++++++++++++++++++++++++++++
>> Documentation/agents/index.rst | 3 ++-
>> Documentation/agents/main.rst | 5 +++++
>> 3 files changed, 35 insertions(+), 1 deletion(-)
>> create mode 100644 Documentation/agents/core.rst
>
>I know I can't take patches without any changelog text, maybe the
>documentation maintainer is more lenient? :)
>
>>
>> diff --git a/Documentation/agents/core.rst b/Documentation/agents/core.rst
>> new file mode 100644
>> index 000000000000..da171dde1f9d
>> --- /dev/null
>> +++ b/Documentation/agents/core.rst
>> @@ -0,0 +1,28 @@
>> +.. SPDX-License-Identifier: GPL-2.0
>> +
>> +========================================
>> +Core Linux Kernel Development References
>> +========================================
>> +
>> +Essential documentation for Linux kernel development:
>> +
>> +How to do Linux kernel development
>> +----------------------------------
>> +
>> +The comprehensive guide for becoming a Linux kernel developer and learning to work with the kernel development community.
>> +
>> +See :ref:`Documentation/process/howto.rst <process_howto>`
>> +
>> +Submitting patches
>> +------------------
>> +
>> +The essential guide to getting your code into the kernel, covering everything from patch formatting to the submission process.
>> +
>> +See :ref:`Documentation/process/submitting-patches.rst <submittingpatches>`
>> +
>> +Submission checklist
>> +--------------------
>> +
>> +A checklist of items to review before submitting code to ensure patches are accepted more quickly.
>> +
>> +See :ref:`Documentation/process/submit-checklist.rst <submitchecklist>`
>
>Why isn't the agent already reading all of this in the documentation
>tree? Why must it be told to read it again? Do we not properly index
>this well enough in our documentation tree already?
So by default the agent doesn't read anything into it's context unless
it thinks it's relevant to the task at hand.
Imagine a junior engineer you ask to fix a spelling mistake and commit
the result: would it think on it's own that it needs to read the rules
about which tags to add to commit messages?
Probably not.
Just as with humans, we need to tell them to read the goddamn docs :)
An an example, look at the wiki page for LF's kernel mentorship
program[1]. The first step it lists is:
Complete the project prerequisite tasks assigned to you when you apply for the project:
Step 1: You should have completed the A Beginner’s Guide to
Linux Kernel Development e-course during the planning phase.
Upload the certificate of completion.
Where the course is basically going over all the processes and rules
around kernel development ("This course is intended for developers who
need to know about the Linux kernel development process and the explicit
and implicit “rules of the road”.").
This patchset is effectively the equivalent of the course for agents :)
[1] https://wiki.linuxfoundation.org/lkmp
[2] https://training.linuxfoundation.org/training/a-beginners-guide-to-linux-kernel-development-lfd103/
--
Thanks,
Sasha
Powered by blists - more mailing lists