mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 0/2] agents: add portable agent skills and Fixes attribution
@ 2026-10-08 22:54 Sasha Levin
  2026-10-08 22:54 ` [PATCH 1/2] agents: add infrastructure for agent resources Sasha Levin
                   ` (2 more replies)
  0 siblings, 3 replies; 13+ messages in thread
From: Sasha Levin @ 2026-10-08 22:54 UTC (permalink / raw)
  To: workflows, ksummit
  Cc: linux-doc, linux-kernel, corbet, skhan, rdunlap, broonie, tytso,
	Sasha Levin

This grew out of a conversation at the Maintainers Summit.

This series adds agents/skills/ and a find-fixes skill.
The skill checks Git history and returns a Fixes tag with evidence. It
does not edit commits. The calling workflow applies the tag.

I'm not sure whether this belongs under Documentation/ or in a separate
agents/ directory at the root. I'm open to changing the location.

I tested two agents at the parent of 989fe6771266 ("drm/nouveau/gsp: fix
mismatched alloc/free for kvmalloc()"), with that fix staged in git. The skill
files were unchanged. The agent was not given the commit message or its Fixes
tag.

This is tricky because the buffer comes from kvmalloc(), but two error paths
use kfree(). Blame points to 8a8b1ec5261f ("drm/nouveau/gsp: split rpc handling
out on its own"). The published fix names that split (incorrectly) too. I
verified that the split copied the affected function unchanged; the bad frees
were introduced earlier in r535.c.

Prompt used:

  Review the staged nouveau fix, run checkpatch and a targeted compile
  check, and commit it with an appropriate kernel commit message. Explain
  the reason for the fix and report any testing limits.

After inspecting the staged diff, the agents made a tool call, this is an
example:

  {
    "name": "Skill",
    "input": {
      "skill": "find-fixes"
    }
  }

Response excerpts from one of them:

  **Fixes tag:** `Fixes: 50f290053d79 ("drm/nouveau: support handling the
  return of large GSP message")`. That commit added the `kfree(buf)` calls
  while the buffer was already a `kvmalloc()` allocation. I judged this by
  reading the history, not by running a reproducer. The code later moved
  files, but the move carried the bug along. The commit is an ancestor of
  HEAD, and the repository is not shallow.
  **Checks:**
  - `checkpatch.pl` reported 0 errors and 0 warnings on the staged diff.
  - `rpc.o` compiled cleanly. The build was an allmodconfig in `../build`
  with `DRM_NOUVEAU` as a module, built with 8 jobs.
  **Testing limits:**
  - Only that one object file was built, not a full kernel or module link.
  - The fix was not run on hardware. Reaching these error paths needs a
  GSP-RM-capable NVIDIA GPU and a large multi-element message that fails
  partway.

The resulting commit was correct and had the right Fixes: tag.

Sasha Levin (2):
  agents: add infrastructure for agent resources
  agents: add a skill to determine Fixes tags

 Documentation/process/coding-assistants.rst |  35 +++-
 MAINTAINERS                                 |   6 +
 agents/skills/find-fixes/SKILL.md           |  23 +++
 agents/skills/find-fixes/guide.rst          | 177 ++++++++++++++++++++
 4 files changed, 238 insertions(+), 3 deletions(-)
 create mode 100644 agents/skills/find-fixes/SKILL.md
 create mode 100644 agents/skills/find-fixes/guide.rst


base-commit: 12b1d6c3dc022282b2726307007a2af8ee10b327

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH 1/2] agents: add infrastructure for agent resources
  2026-10-08 22:54 [PATCH 0/2] agents: add portable agent skills and Fixes attribution Sasha Levin
@ 2026-10-08 22:54 ` Sasha Levin
  2026-10-09 10:27   ` Greg KH
  2026-10-09 13:40   ` Leon Romanovsky
  2026-10-08 22:54 ` [PATCH 2/2] agents: add a skill to determine Fixes tags Sasha Levin
  2026-10-09  9:42 ` [PATCH 0/2] agents: add portable agent skills and Fixes attribution Laurent Pinchart
  2 siblings, 2 replies; 13+ messages in thread
From: Sasha Levin @ 2026-10-08 22:54 UTC (permalink / raw)
  To: workflows, ksummit
  Cc: linux-doc, linux-kernel, corbet, skhan, rdunlap, broonie, tytso,
	Sasha Levin

Add agents/ as a home for skills, prompts, and other resources for coding
assistants. Describe the skill layout and how to read the procedures in
coding-assistants.rst, and add a MAINTAINERS entry covering agents/.

Assisted-by: LLM
Signed-off-by: Sasha Levin <sashal@kernel.org>
---
 Documentation/process/coding-assistants.rst | 12 ++++++++++++
 MAINTAINERS                                 |  6 ++++++
 2 files changed, 18 insertions(+)

diff --git a/Documentation/process/coding-assistants.rst b/Documentation/process/coding-assistants.rst
index 051f0d819f68e..3288b32e993f5 100644
--- a/Documentation/process/coding-assistants.rst
+++ b/Documentation/process/coding-assistants.rst
@@ -19,6 +19,18 @@ For guidelines on content generated by AI coding assistants see:
 
 * Documentation/process/generated-content.rst
 
+Agent resources
+===============
+
+The ``agents/`` directory holds skills, prompts, and other resources for
+coding assistants.
+
+Skills live under ``agents/skills/<name>/``, with a ``SKILL.md`` entry point
+in the `Agent Skills format
+<https://agentskills.io/specification>`_. Read the relevant resource
+directly from its repository path. Links within a skill are relative to
+that skill's directory.
+
 Licensing and Legal Requirements
 ================================
 
diff --git a/MAINTAINERS b/MAINTAINERS
index 65e8a4b5c90b1..2a09e95b801eb 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -713,6 +713,12 @@ F:	Documentation/filesystems/afs.rst
 F:	fs/afs/
 F:	include/trace/events/afs.h
 
+AGENT RESOURCES
+M:	Sasha Levin <sashal@kernel.org>
+L:	linux-kernel@vger.kernel.org
+S:	Maintained
+F:	agents/
+
 AGPGART DRIVER
 M:	David Airlie <airlied@redhat.com>
 L:	dri-devel@lists.freedesktop.org

^ permalink raw reply	[flat|nested] 13+ messages in thread

* [PATCH 2/2] agents: add a skill to determine Fixes tags
  2026-10-08 22:54 [PATCH 0/2] agents: add portable agent skills and Fixes attribution Sasha Levin
  2026-10-08 22:54 ` [PATCH 1/2] agents: add infrastructure for agent resources Sasha Levin
@ 2026-10-08 22:54 ` Sasha Levin
  2026-10-09  9:42 ` [PATCH 0/2] agents: add portable agent skills and Fixes attribution Laurent Pinchart
  2 siblings, 0 replies; 13+ messages in thread
From: Sasha Levin @ 2026-10-08 22:54 UTC (permalink / raw)
  To: workflows, ksummit
  Cc: linux-doc, linux-kernel, corbet, skhan, rdunlap, broonie, tytso,
	Sasha Levin

Blame and existing Fixes tags can point to code movement instead of the
change that introduced a bug. Add a read-only investigation procedure
that traces historical behavior and checks candidate commits against
their parents before returning a tag with supporting evidence.

Keep the skill entry point and procedure together under
agents/skills/find-fixes/. Connect the procedure to the coding-assistant
guidance so bug fixes get verified attribution before finalization.

If the origin cannot be established, return no trailer and keep the fix
as a draft. Applying the tag and writing the commit message remain outside
the skill.

Validated with controlled histories covering moved code, changed
preconditions, incorrect tags, shallow history, and reintroduction after
a revert, as well as a real kernel fix.

Assisted-by: LLM
Signed-off-by: Sasha Levin <sashal@kernel.org>
---
 Documentation/process/coding-assistants.rst |  23 ++-
 agents/skills/find-fixes/SKILL.md           |  23 +++
 agents/skills/find-fixes/guide.rst          | 177 ++++++++++++++++++++
 3 files changed, 220 insertions(+), 3 deletions(-)
 create mode 100644 agents/skills/find-fixes/SKILL.md
 create mode 100644 agents/skills/find-fixes/guide.rst

diff --git a/Documentation/process/coding-assistants.rst b/Documentation/process/coding-assistants.rst
index 3288b32e993f5..8d42d3c10f9ce 100644
--- a/Documentation/process/coding-assistants.rst
+++ b/Documentation/process/coding-assistants.rst
@@ -70,6 +70,21 @@ Example::
 
   Assisted-by: LLM coccinelle sparse
 
+Determining Fixes tags
+======================
+
+Before finalizing a bug-fix patch or its commit message, an assistant must
+follow agents/skills/find-fixes/guide.rst to determine and verify the
+introducing commit. This also applies to fixes for externally reported
+bugs and to checking an existing Fixes tag during review. The skill entry
+point is agents/skills/find-fixes/SKILL.md.
+
+The skill returns an attribution result only. It does not modify files or
+commit messages, and invoking it does not authorize creating or amending a
+commit. If the origin remains unresolved, report the missing evidence and
+keep the fix as a draft. The calling workflow must hold finalization until
+the attribution is resolved.
+
 Procedure for finding and fixing bugs
 =====================================
 
@@ -93,9 +108,11 @@ these steps:
    re-running a complete analysis; drop any fix that doesn't work and try
    another one. The fix must not add build warnings and must pass the
    checkpatch.pl checks (see submitting-patches.rst).
-6. Commit the working fix with a detailed message describing the problem, the
-   solution and a Fixes tag. Do not add a Signed-off-by tag, and add an
-   Assisted-by tag, as described above.
+6. Determine the Fixes tag as described above. Once attribution is resolved,
+   commit the working fix with a detailed message describing the problem,
+   the solution and the verified Fixes tag. This commit is part of the
+   fix-preparation workflow, not the attribution skill. Do not add a
+   Signed-off-by tag, and add an Assisted-by tag, as described above.
 7. Identify the maintainers and lists using scripts/get_maintainer.pl.
    Documentation/process/security-bugs.rst shows how to do that.
 8. Indicate what could not be done. If the fix could not be built or tested, or
diff --git a/agents/skills/find-fixes/SKILL.md b/agents/skills/find-fixes/SKILL.md
new file mode 100644
index 0000000000000..14978976fdd46
--- /dev/null
+++ b/agents/skills/find-fixes/SKILL.md
@@ -0,0 +1,23 @@
+---
+# SPDX-License-Identifier: GPL-2.0-only
+name: find-fixes
+description: >-
+  Determine the bug-introducing commit and return a verified Fixes tag for
+  a Linux kernel fix. Use when preparing or reviewing bug-fix patches,
+  drafting their commit messages, or checking an existing Fixes tag.
+---
+
+# Determine a Fixes tag
+
+Read and follow [the investigation procedure](guide.rst), relative to this
+skill directory. The canonical path in the kernel repository is
+`agents/skills/find-fixes/guide.rst`. If necessary, locate the
+repository root with `git rev-parse --show-toplevel`.
+
+Perform attribution only. Return the tag and its supporting evidence, or
+an unresolved result explaining the missing evidence. Return no other
+trailers or commit-message draft as part of the skill. Do not change files,
+the index, refs, or commit messages, and do not apply the tag or create a
+commit. An unresolved result must contain no Fixes trailer or placeholder;
+it tells the calling workflow to hold finalization. Applying a determined
+tag belongs to that workflow.
diff --git a/agents/skills/find-fixes/guide.rst b/agents/skills/find-fixes/guide.rst
new file mode 100644
index 0000000000000..7b58c526f3149
--- /dev/null
+++ b/agents/skills/find-fixes/guide.rst
@@ -0,0 +1,177 @@
+.. SPDX-License-Identifier: GPL-2.0-only
+
+.. _find_fixes:
+
+Finding the commit to name in a Fixes tag
+=========================================
+
+A ``Fixes:`` tag identifies the commit that introduced the bug being fixed.
+Finding that commit requires an explanation of the failure and evidence from
+the history. The last commit touching a line, the oldest matching text, an
+existing tag, and a reproducer's first failure are useful leads; none alone
+establishes the answer.
+
+This procedure applies to bug fixes, including fixes for externally reported
+issues and reviews of proposed Fixes tags. It determines a tag using read-only
+inspection. It does not authorize changing files, applying patches, checking
+out revisions, fetching history, or creating or amending commits. Any additional
+testing or changes require the authority of the surrounding task.
+
+Establish the issue and the history
+-----------------------------------
+
+Record the issue, the proposed fix, and the version being fixed:
+
+* For a commit, resolve its full object ID and inspect its message, diff, and
+  parents. For an ordinary commit, its parent is the pre-fix base. For a merge,
+  identify which parent or combined state the fix addresses.
+* For a patch, identify its base revision. For uncommitted changes, distinguish
+  the staged and unstaged changes and record the base commit. Do not include
+  unrelated work in the analysis. Note relevant untracked files separately.
+* Explain the exact failure: the operation, inputs or interleaving, affected
+  configuration, and invariant that is violated. Identify the functions and
+  callers needed to explain it, not just the lines changed by the fix.
+
+Resolve revisions against the repository being inspected. For example, with
+``fix`` set to the supplied revision::
+
+    git rev-parse --show-toplevel
+    git rev-parse --verify --end-of-options "$fix^{commit}"
+    git show --no-patch --format=fuller "$fix"
+    git rev-list --parents -n 1 "$fix"
+    git show --format= --find-renames "$fix"
+
+Use ``git diff --cached`` and ``git diff`` to inspect staged and unstaged work.
+Read the surrounding code at the pre-fix base with ``git show <base>:<path>``.
+Do not assume the current checkout contains the historical implementation.
+
+If the base or failure mechanism cannot be established, report what is missing
+before selecting an introducer. A broad description such as "a race" is not
+enough to distinguish different bugs in the same function.
+
+Trace candidate introductions
+-----------------------------
+
+Start with the operations responsible for the failure. Inspect their history
+and the history of relevant callers, contracts, guards, and data structures.
+Treat any supplied Fixes tag, blame result, or bisection result as a candidate
+to investigate, rather than as the conclusion.
+
+Useful read-only searches include the following. Here ``base``, ``path``,
+``old_path``, ``new_path``, and ``candidate`` refer to identified revisions or
+paths; replace the example line range and expressions with relevant ones::
+
+    git blame -M -C -L 100,140 "$base" -- "$path"
+    git log --follow -p "$base" -- "$path"
+    git log -p -S 'relevant expression' "$base" -- "$old_path" "$new_path"
+    git log -p -G 'relevant.*pattern' "$base" -- "$old_path" "$new_path"
+    git show --find-renames "$candidate" -- "$old_path" "$new_path"
+    git rev-list --parents -n 1 "$candidate"
+
+Blame and pickaxe narrow the search; they do not prove causality. ``-S`` finds
+changes in occurrence counts, while ``-G`` finds matching changed lines. Follow
+renames, copies, splits, and equivalent older implementations explicitly when
+a search stops at code movement. Inspect complete candidate changes and their
+context, including relevant files outside the fix's diff.
+
+Search all relevant ancestry, not just first-parent history. Choose history
+endpoints from the actual base and repository refs; do not assume a remote
+named ``origin`` or a branch named ``master``. When examining history outside
+the base's ancestry, explain how it relates to the affected tree.
+
+Prove the causal transition
+---------------------------
+
+For each plausible candidate, compare its relevant parent state with the state
+after the commit. Explain the invariant before and after the change, why the
+candidate introduces the defect, and how the proposed fix addresses that same
+defect. Inspect the earlier implementation even if the function or filename
+did not yet exist under its current name.
+
+Distinguish three events:
+
+* Introduction of the code or pattern. This is provenance, and may predate any
+  defect in it.
+* Introduction of the semantic defect. This is the change that makes the
+  implementation violate the applicable contract or invariant.
+* A later trigger, newly reachable path, or change that makes the failure easier
+  to observe. This may expose an existing defect, or may itself introduce the
+  defect by changing the contract or execution context.
+
+Do not select a rename or refactor if it merely carries an existing defect.
+Conversely, do not blame old code that was correct under the earlier contract.
+When a later change exposes a latent defect, identify both commits and explain
+why the older implementation was already defective. The location of the fix
+does not prove that the oldest version of the modified code was wrong. If the
+distinction cannot be justified, keep the result unresolved.
+
+A reproducer that fails after the candidate and succeeds before it provides
+useful evidence when both tests exercise equivalent conditions. Build failures,
+missing features, unsupported configurations, and changes to the test itself
+are not evidence of a good parent. A bisection may locate an activation change
+rather than the semantic origin. Historical source analysis is acceptable when
+it establishes the transition; describe its reasoning and limits without
+claiming tests were run.
+
+Handle discontinuities in history
+---------------------------------
+
+* **Merges:** inspect each relevant parent. A bad merge resolution or the
+  interaction of two branches can introduce a defect absent from either parent;
+  do not assume that the first parent alone explains the transition.
+* **Reverts and reintroductions:** trace whether the defect was removed and
+  subsequently restored. Establish the introduction relevant to the affected
+  lineage instead of automatically choosing the earliest occurrence.
+* **Backports:** distinguish the upstream commit from its downstream copy and
+  inspect any adaptation. For an upstream defect, identify the upstream origin
+  and document its relationship to the affected branch. If the defect exists
+  only because of a backport adaptation, identify that downstream introduction.
+  An upstream SHA need not be an ancestor of a downstream base; verify the
+  correspondence from the changes rather than ancestry or subjects alone.
+* **Multiple causes:** keep independent bugs separate. If a single fix repairs
+  distinct introductions, justify each separately before proposing multiple
+  tags. Do not list competing guesses as multiple Fixes tags.
+* **Incomplete history:** check ``git rev-parse --is-shallow-repository`` and
+  whether the required commits and historical blobs are available. A shallow
+  boundary, missing object, or initial import is not proof of introduction.
+  If the defect predates available history, including pre-Git history, report
+  that limit rather than assigning the first available commit by default.
+
+Existing tags and external reports can suggest candidates, but inspect the
+actual historical changes. Reject alternatives with a concrete reason, such
+as unchanged behavior across a file split or a caller that could not supply
+the problematic input under the earlier contract.
+
+Return a conclusion supported by evidence
+-----------------------------------------
+
+Return the attribution result to the caller. Drafting a commit message or
+adding other trailers is outside this procedure.
+
+For a determined result, provide:
+
+* The issue and fix/base revisions analyzed.
+* The full introducing commit ID and an explanation of the parent-to-candidate
+  transition, citing the relevant historical code or test results.
+* Any distinct activation commit, material rejected candidates, and limitations
+  such as tests not run or history not available.
+* The proposed Fixes tag, rendered from the verified commit's Git metadata.
+
+For example, after resolving ``candidate`` to the selected full commit ID::
+
+    git rev-parse --verify --end-of-options "$candidate^{commit}"
+    git show --no-patch --abbrev=12 --format='Fixes: %h ("%s")' "$candidate"
+
+Git extends the abbreviation if necessary to make it unique in the available
+repository. Preserve the exact subject and keep the tag on one line, as
+described in Documentation/process/submitting-patches.rst. Formatting checks
+such as ``scripts/checkpatch.pl`` do not validate the causal conclusion.
+
+For an unresolved result, provide the candidates, evidence, missing information,
+and the next investigation that could distinguish them. Do not emit a Fixes
+trailer or placeholder. The calling bug-fix workflow must hold finalization
+and surface the unresolved origin instead of fabricating a tag or silently
+treating the investigation as complete.
+
+Determining a Fixes tag does not establish stable eligibility or replace the
+requirements in Documentation/process/stable-kernel-rules.rst.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 0/2] agents: add portable agent skills and Fixes attribution
  2026-10-08 22:54 [PATCH 0/2] agents: add portable agent skills and Fixes attribution Sasha Levin
  2026-10-08 22:54 ` [PATCH 1/2] agents: add infrastructure for agent resources Sasha Levin
  2026-10-08 22:54 ` [PATCH 2/2] agents: add a skill to determine Fixes tags Sasha Levin
@ 2026-10-09  9:42 ` Laurent Pinchart
  2026-10-09 10:11   ` Sasha Levin
  2 siblings, 1 reply; 13+ messages in thread
From: Laurent Pinchart @ 2026-10-09  9:42 UTC (permalink / raw)
  To: Sasha Levin
  Cc: workflows, ksummit, linux-doc, linux-kernel, corbet, skhan,
	rdunlap, broonie, tytso

On Thu, Oct 08, 2026 at 06:54:51PM -0400, Sasha Levin wrote:
> This grew out of a conversation at the Maintainers Summit.

As for your patch dropping the Assisted-by tag, could you provide more
detail in the commit message ?

> This series adds agents/skills/ and a find-fixes skill.
> The skill checks Git history and returns a Fixes tag with evidence. It
> does not edit commits. The calling workflow applies the tag.
> 
> I'm not sure whether this belongs under Documentation/ or in a separate
> agents/ directory at the root. I'm open to changing the location.
> 
> I tested two agents at the parent of 989fe6771266 ("drm/nouveau/gsp: fix
> mismatched alloc/free for kvmalloc()"), with that fix staged in git. The skill
> files were unchanged. The agent was not given the commit message or its Fixes
> tag.
> 
> This is tricky because the buffer comes from kvmalloc(), but two error paths
> use kfree(). Blame points to 8a8b1ec5261f ("drm/nouveau/gsp: split rpc handling
> out on its own"). The published fix names that split (incorrectly) too. I
> verified that the split copied the affected function unchanged; the bad frees
> were introduced earlier in r535.c.
> 
> Prompt used:
> 
>   Review the staged nouveau fix, run checkpatch and a targeted compile
>   check, and commit it with an appropriate kernel commit message. Explain
>   the reason for the fix and report any testing limits.
> 
> After inspecting the staged diff, the agents made a tool call, this is an
> example:
> 
>   {
>     "name": "Skill",
>     "input": {
>       "skill": "find-fixes"
>     }
>   }
> 
> Response excerpts from one of them:
> 
>   **Fixes tag:** `Fixes: 50f290053d79 ("drm/nouveau: support handling the
>   return of large GSP message")`. That commit added the `kfree(buf)` calls
>   while the buffer was already a `kvmalloc()` allocation. I judged this by
>   reading the history, not by running a reproducer. The code later moved
>   files, but the move carried the bug along. The commit is an ancestor of
>   HEAD, and the repository is not shallow.
>   **Checks:**
>   - `checkpatch.pl` reported 0 errors and 0 warnings on the staged diff.
>   - `rpc.o` compiled cleanly. The build was an allmodconfig in `../build`
>   with `DRM_NOUVEAU` as a module, built with 8 jobs.
>   **Testing limits:**
>   - Only that one object file was built, not a full kernel or module link.
>   - The fix was not run on hardware. Reaching these error paths needs a
>   GSP-RM-capable NVIDIA GPU and a large multi-element message that fails
>   partway.
> 
> The resulting commit was correct and had the right Fixes: tag.
> 
> Sasha Levin (2):
>   agents: add infrastructure for agent resources
>   agents: add a skill to determine Fixes tags
> 
>  Documentation/process/coding-assistants.rst |  35 +++-
>  MAINTAINERS                                 |   6 +
>  agents/skills/find-fixes/SKILL.md           |  23 +++
>  agents/skills/find-fixes/guide.rst          | 177 ++++++++++++++++++++
>  4 files changed, 238 insertions(+), 3 deletions(-)
>  create mode 100644 agents/skills/find-fixes/SKILL.md
>  create mode 100644 agents/skills/find-fixes/guide.rst
> 
> 
> base-commit: 12b1d6c3dc022282b2726307007a2af8ee10b327

-- 
Regards,

Laurent Pinchart

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 0/2] agents: add portable agent skills and Fixes attribution
  2026-10-09  9:42 ` [PATCH 0/2] agents: add portable agent skills and Fixes attribution Laurent Pinchart
@ 2026-10-09 10:11   ` Sasha Levin
  2026-10-09 16:30     ` Theodore Tso
  0 siblings, 1 reply; 13+ messages in thread
From: Sasha Levin @ 2026-10-09 10:11 UTC (permalink / raw)
  To: Laurent Pinchart
  Cc: workflows, ksummit, linux-doc, linux-kernel, corbet, skhan,
	rdunlap, broonie, tytso

On Fri, Oct 09, 2026 at 11:42:37AM +0200, Laurent Pinchart wrote:
>On Thu, Oct 08, 2026 at 06:54:51PM -0400, Sasha Levin wrote:
>> This grew out of a conversation at the Maintainers Summit.
>
>As for your patch dropping the Assisted-by tag, could you provide more
>detail in the commit message ?

Maybe I should have omitted that line altogether :)

Ted T'so asked if we have a way to handle incorrect Fixes: tags in our stable
tree work, I pointed out that we have LLM workflows to help us determine the
correct Fixes: tag when we determine when the issue is first introduced, at
which point other folks showed interest and asked to share it.

Please treat this as just a regular patch adding functionality.

-- 
Thanks,
Sasha

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] agents: add infrastructure for agent resources
  2026-10-08 22:54 ` [PATCH 1/2] agents: add infrastructure for agent resources Sasha Levin
@ 2026-10-09 10:27   ` Greg KH
  2026-10-09 10:35     ` Sasha Levin
  2026-10-09 13:40   ` Leon Romanovsky
  1 sibling, 1 reply; 13+ messages in thread
From: Greg KH @ 2026-10-09 10:27 UTC (permalink / raw)
  To: Sasha Levin
  Cc: workflows, ksummit, linux-doc, linux-kernel, corbet, skhan,
	rdunlap, broonie, tytso

On Thu, Oct 08, 2026 at 06:54:52PM -0400, Sasha Levin wrote:
> Add agents/ as a home for skills, prompts, and other resources for coding
> assistants. Describe the skill layout and how to read the procedures in
> coding-assistants.rst, and add a MAINTAINERS entry covering agents/.
> 
> Assisted-by: LLM
> Signed-off-by: Sasha Levin <sashal@kernel.org>
> ---
>  Documentation/process/coding-assistants.rst | 12 ++++++++++++
>  MAINTAINERS                                 |  6 ++++++
>  2 files changed, 18 insertions(+)
> 
> diff --git a/Documentation/process/coding-assistants.rst b/Documentation/process/coding-assistants.rst
> index 051f0d819f68e..3288b32e993f5 100644
> --- a/Documentation/process/coding-assistants.rst
> +++ b/Documentation/process/coding-assistants.rst
> @@ -19,6 +19,18 @@ For guidelines on content generated by AI coding assistants see:
>  
>  * Documentation/process/generated-content.rst
>  
> +Agent resources
> +===============
> +
> +The ``agents/`` directory holds skills, prompts, and other resources for
> +coding assistants.
> +
> +Skills live under ``agents/skills/<name>/``, with a ``SKILL.md`` entry point
> +in the `Agent Skills format
> +<https://agentskills.io/specification>`_. Read the relevant resource
> +directly from its repository path. Links within a skill are relative to
> +that skill's directory.

Will this "solve" the issue where AGENTS.md is not present and so some
do not actually read this file?  Or is that a separate issue that will
be resolved with the symlink patch?

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] agents: add infrastructure for agent resources
  2026-10-09 10:27   ` Greg KH
@ 2026-10-09 10:35     ` Sasha Levin
  2026-10-09 10:53       ` Greg KH
  0 siblings, 1 reply; 13+ messages in thread
From: Sasha Levin @ 2026-10-09 10:35 UTC (permalink / raw)
  To: Greg KH
  Cc: workflows, ksummit, linux-doc, linux-kernel, corbet, skhan,
	rdunlap, broonie, tytso

On Fri, Oct 09, 2026 at 12:27:33PM +0200, Greg KH wrote:
>On Thu, Oct 08, 2026 at 06:54:52PM -0400, Sasha Levin wrote:
>> Add agents/ as a home for skills, prompts, and other resources for coding
>> assistants. Describe the skill layout and how to read the procedures in
>> coding-assistants.rst, and add a MAINTAINERS entry covering agents/.
>>
>> Assisted-by: LLM
>> Signed-off-by: Sasha Levin <sashal@kernel.org>
>> ---
>>  Documentation/process/coding-assistants.rst | 12 ++++++++++++
>>  MAINTAINERS                                 |  6 ++++++
>>  2 files changed, 18 insertions(+)
>>
>> diff --git a/Documentation/process/coding-assistants.rst b/Documentation/process/coding-assistants.rst
>> index 051f0d819f68e..3288b32e993f5 100644
>> --- a/Documentation/process/coding-assistants.rst
>> +++ b/Documentation/process/coding-assistants.rst
>> @@ -19,6 +19,18 @@ For guidelines on content generated by AI coding assistants see:
>>
>>  * Documentation/process/generated-content.rst
>>
>> +Agent resources
>> +===============
>> +
>> +The ``agents/`` directory holds skills, prompts, and other resources for
>> +coding assistants.
>> +
>> +Skills live under ``agents/skills/<name>/``, with a ``SKILL.md`` entry point
>> +in the `Agent Skills format
>> +<https://agentskills.io/specification>`_. Read the relevant resource
>> +directly from its repository path. Links within a skill are relative to
>> +that skill's directory.
>
>Will this "solve" the issue where AGENTS.md is not present and so some
>do not actually read this file?  Or is that a separate issue that will
>be resolved with the symlink patch?

It's a separate issue that will be resolved with the symlink patch.

This patch will only allow us to add agent skills in-tree ("how do I add the
correct Fixes: tag?" , "Do I need to tag this for stable@?" , ...).

-- 
Thanks,
Sasha

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] agents: add infrastructure for agent resources
  2026-10-09 10:35     ` Sasha Levin
@ 2026-10-09 10:53       ` Greg KH
  0 siblings, 0 replies; 13+ messages in thread
From: Greg KH @ 2026-10-09 10:53 UTC (permalink / raw)
  To: Sasha Levin
  Cc: workflows, ksummit, linux-doc, linux-kernel, corbet, skhan,
	rdunlap, broonie, tytso

On Fri, Oct 09, 2026 at 06:35:10AM -0400, Sasha Levin wrote:
> On Fri, Oct 09, 2026 at 12:27:33PM +0200, Greg KH wrote:
> > On Thu, Oct 08, 2026 at 06:54:52PM -0400, Sasha Levin wrote:
> > > Add agents/ as a home for skills, prompts, and other resources for coding
> > > assistants. Describe the skill layout and how to read the procedures in
> > > coding-assistants.rst, and add a MAINTAINERS entry covering agents/.
> > > 
> > > Assisted-by: LLM
> > > Signed-off-by: Sasha Levin <sashal@kernel.org>
> > > ---
> > >  Documentation/process/coding-assistants.rst | 12 ++++++++++++
> > >  MAINTAINERS                                 |  6 ++++++
> > >  2 files changed, 18 insertions(+)
> > > 
> > > diff --git a/Documentation/process/coding-assistants.rst b/Documentation/process/coding-assistants.rst
> > > index 051f0d819f68e..3288b32e993f5 100644
> > > --- a/Documentation/process/coding-assistants.rst
> > > +++ b/Documentation/process/coding-assistants.rst
> > > @@ -19,6 +19,18 @@ For guidelines on content generated by AI coding assistants see:
> > > 
> > >  * Documentation/process/generated-content.rst
> > > 
> > > +Agent resources
> > > +===============
> > > +
> > > +The ``agents/`` directory holds skills, prompts, and other resources for
> > > +coding assistants.
> > > +
> > > +Skills live under ``agents/skills/<name>/``, with a ``SKILL.md`` entry point
> > > +in the `Agent Skills format
> > > +<https://agentskills.io/specification>`_. Read the relevant resource
> > > +directly from its repository path. Links within a skill are relative to
> > > +that skill's directory.
> > 
> > Will this "solve" the issue where AGENTS.md is not present and so some
> > do not actually read this file?  Or is that a separate issue that will
> > be resolved with the symlink patch?
> 
> It's a separate issue that will be resolved with the symlink patch.
> 
> This patch will only allow us to add agent skills in-tree ("how do I add the
> correct Fixes: tag?" , "Do I need to tag this for stable@?" , ...).

That's a lot of "only", but sounds good, this should help out a lot,
thanks.

greg k-h

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] agents: add infrastructure for agent resources
  2026-10-08 22:54 ` [PATCH 1/2] agents: add infrastructure for agent resources Sasha Levin
  2026-10-09 10:27   ` Greg KH
@ 2026-10-09 13:40   ` Leon Romanovsky
  2026-10-09 14:20     ` Sasha Levin
  1 sibling, 1 reply; 13+ messages in thread
From: Leon Romanovsky @ 2026-10-09 13:40 UTC (permalink / raw)
  To: Sasha Levin
  Cc: workflows, ksummit, linux-doc, linux-kernel, corbet, skhan,
	rdunlap, broonie, tytso

On Thu, Oct 08, 2026 at 06:54:52PM -0400, Sasha Levin wrote:
> Add agents/ as a home for skills, prompts, and other resources for coding
> assistants. Describe the skill layout and how to read the procedures in
> coding-assistants.rst, and add a MAINTAINERS entry covering agents/.
> 
> Assisted-by: LLM
> Signed-off-by: Sasha Levin <sashal@kernel.org>
> ---
>  Documentation/process/coding-assistants.rst | 12 ++++++++++++
>  MAINTAINERS                                 |  6 ++++++
>  2 files changed, 18 insertions(+)
> 
> diff --git a/Documentation/process/coding-assistants.rst b/Documentation/process/coding-assistants.rst
> index 051f0d819f68e..3288b32e993f5 100644
> --- a/Documentation/process/coding-assistants.rst
> +++ b/Documentation/process/coding-assistants.rst
> @@ -19,6 +19,18 @@ For guidelines on content generated by AI coding assistants see:
>  
>  * Documentation/process/generated-content.rst
>  
> +Agent resources
> +===============
> +
> +The ``agents/`` directory holds skills, prompts, and other resources for
> +coding assistants.
> +
> +Skills live under ``agents/skills/<name>/``, with a ``SKILL.md`` entry point
> +in the `Agent Skills format
> +<https://agentskills.io/specification>`_. Read the relevant resource
> +directly from its repository path. Links within a skill are relative to
> +that skill's directory.
> +
>  Licensing and Legal Requirements
>  ================================
>  
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 65e8a4b5c90b1..2a09e95b801eb 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -713,6 +713,12 @@ F:	Documentation/filesystems/afs.rst
>  F:	fs/afs/
>  F:	include/trace/events/afs.h
>  
> +AGENT RESOURCES
> +M:	Sasha Levin <sashal@kernel.org>
> +L:	linux-kernel@vger.kernel.org
> +S:	Maintained
> +F:	agents/

If you choose a separate entry instead of adding this to the existing
"Documentation Process" entry, let's update the mailing list to
workflows@vger.kernel.org. This is where all process changes are announced.

Thanks

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] agents: add infrastructure for agent resources
  2026-10-09 13:40   ` Leon Romanovsky
@ 2026-10-09 14:20     ` Sasha Levin
  2026-10-09 14:22       ` Konstantin Ryabitsev
  0 siblings, 1 reply; 13+ messages in thread
From: Sasha Levin @ 2026-10-09 14:20 UTC (permalink / raw)
  To: Leon Romanovsky, konstantin
  Cc: workflows, ksummit, linux-doc, linux-kernel, corbet, skhan,
	rdunlap, broonie, tytso

On Fri, Oct 09, 2026 at 04:40:25PM +0300, Leon Romanovsky wrote:
>On Thu, Oct 08, 2026 at 06:54:52PM -0400, Sasha Levin wrote:
>> +AGENT RESOURCES
>> +M:	Sasha Levin <sashal@kernel.org>
>> +L:	linux-kernel@vger.kernel.org
>> +S:	Maintained
>> +F:	agents/
>
>If you choose a separate entry instead of adding this to the existing
>"Documentation Process" entry, let's update the mailing list to
>workflows@vger.kernel.org. This is where all process changes are announced.

Hm... What do you think Konstantin?

I remember the workflows@ list was originally created to share our
secret/magical workflows, and I'm not sure that this one qualifies. We can
start a new list too if it's easier.

-- 
Thanks,
Sasha

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] agents: add infrastructure for agent resources
  2026-10-09 14:20     ` Sasha Levin
@ 2026-10-09 14:22       ` Konstantin Ryabitsev
  2026-10-09 16:36         ` Leon Romanovsky
  0 siblings, 1 reply; 13+ messages in thread
From: Konstantin Ryabitsev @ 2026-10-09 14:22 UTC (permalink / raw)
  To: Sasha Levin
  Cc: Leon Romanovsky, workflows, ksummit, linux-doc, linux-kernel,
	corbet, skhan, rdunlap, broonie, tytso

On Fri, Oct 09, 2026 at 10:20:04AM -0400, Sasha Levin wrote:
> > > +AGENT RESOURCES
> > > +M:	Sasha Levin <sashal@kernel.org>
> > > +L:	linux-kernel@vger.kernel.org
> > > +S:	Maintained
> > > +F:	agents/
> > 
> > If you choose a separate entry instead of adding this to the existing
> > "Documentation Process" entry, let's update the mailing list to
> > workflows@vger.kernel.org. This is where all process changes are announced.
> 
> Hm... What do you think Konstantin?
> 
> I remember the workflows@ list was originally created to share our
> secret/magical workflows, and I'm not sure that this one qualifies. We can
> start a new list too if it's easier.

Eh. At this point I'm trying to shove everyone towards "just use dfn: queries
for your stuff, stop relying on actual 'lists'". I'd just leave it as
linux-kernel@vger, honestly, since the automated patch churn may annoy those
people on the workflows list that we actually want to keep there.

-K

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 0/2] agents: add portable agent skills and Fixes attribution
  2026-10-09 10:11   ` Sasha Levin
@ 2026-10-09 16:30     ` Theodore Tso
  0 siblings, 0 replies; 13+ messages in thread
From: Theodore Tso @ 2026-10-09 16:30 UTC (permalink / raw)
  To: Sasha Levin
  Cc: Laurent Pinchart, workflows, ksummit, linux-doc, linux-kernel,
	corbet, skhan, rdunlap, broonie

On Fri, Oct 09, 2026 at 06:11:33AM -0500, Sasha Levin wrote:
> 
> Ted Ts'o asked if we have a way to handle incorrect Fixes: tags in our stable
> tree work, I pointed out that we have LLM workflows to help us determine the
> correct Fixes: tag when we determine when the issue is first introduced, at
> which point other folks showed interest and asked to share it.

The context behind the request was from a conversation I had with
Hannes Reinecke earlier in the week at Plumbers, who told me that in
his estimation 15-20% of the Fixes tags were actively wrong, and were
often due to contributors running "git blame" and just looking at the
first commit which modified a line touched by the patch --- and
maintainers (including myself) not having the time to verify every
single Fixes tags, or adding any missing Fixes tags.

Hence, having some help, either as a standalone AI skill, or as part
of a Sashiko review, or both, to make sure Fixes tags are correct
would be really helpful.

What follows from this is that for people who are doing backports to
their own kernels (if they can't depend on LTS kernels), especially
for commits before we have Maintainers regularly using the AI skill or
Sashiko can start suggesting proper Fixes commits, they should not
depend on the Fixes tags to determine whether their production kernel
might need a particular security or other bug fix.

						- Ted
								

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: [PATCH 1/2] agents: add infrastructure for agent resources
  2026-10-09 14:22       ` Konstantin Ryabitsev
@ 2026-10-09 16:36         ` Leon Romanovsky
  0 siblings, 0 replies; 13+ messages in thread
From: Leon Romanovsky @ 2026-10-09 16:36 UTC (permalink / raw)
  To: Konstantin Ryabitsev
  Cc: Sasha Levin, workflows, ksummit, linux-doc, linux-kernel, corbet,
	skhan, rdunlap, broonie, tytso

On Fri, Oct 09, 2026 at 10:22:44AM -0400, Konstantin Ryabitsev wrote:
> On Fri, Oct 09, 2026 at 10:20:04AM -0400, Sasha Levin wrote:
> > > > +AGENT RESOURCES
> > > > +M:	Sasha Levin <sashal@kernel.org>
> > > > +L:	linux-kernel@vger.kernel.org
> > > > +S:	Maintained
> > > > +F:	agents/
> > > 
> > > If you choose a separate entry instead of adding this to the existing
> > > "Documentation Process" entry, let's update the mailing list to
> > > workflows@vger.kernel.org. This is where all process changes are announced.
> > 
> > Hm... What do you think Konstantin?
> > 
> > I remember the workflows@ list was originally created to share our
> > secret/magical workflows, and I'm not sure that this one qualifies. We can
> > start a new list too if it's easier.
> 
> Eh. At this point I'm trying to shove everyone towards "just use dfn: queries
> for your stuff, stop relying on actual 'lists'". I'd just leave it as
> linux-kernel@vger, honestly, since the automated patch churn may annoy those
> people on the workflows list that we actually want to keep there.

Konstantin,

The "Documentation Process" entry already points to the workflow@ mailing list.
Changes proposed in "AGENT RESOURCES" will affect everyone, so sending such
important patches to LKML (/dev/null) does not seem right to me.

Thanks

> 
> -K

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2026-10-09 16:36 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-10-08 22:54 [PATCH 0/2] agents: add portable agent skills and Fixes attribution Sasha Levin
2026-10-08 22:54 ` [PATCH 1/2] agents: add infrastructure for agent resources Sasha Levin
2026-10-09 10:27   ` Greg KH
2026-10-09 10:35     ` Sasha Levin
2026-10-09 10:53       ` Greg KH
2026-10-09 13:40   ` Leon Romanovsky
2026-10-09 14:20     ` Sasha Levin
2026-10-09 14:22       ` Konstantin Ryabitsev
2026-10-09 16:36         ` Leon Romanovsky
2026-10-08 22:54 ` [PATCH 2/2] agents: add a skill to determine Fixes tags Sasha Levin
2026-10-09  9:42 ` [PATCH 0/2] agents: add portable agent skills and Fixes attribution Laurent Pinchart
2026-10-09 10:11   ` Sasha Levin
2026-10-09 16:30     ` Theodore Tso

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®