<< All versions

Skill v1.0.0

currentLLM-judged scan90/100
gvago/unofficial-qodo-review-skills/kernel-change-intent
──Details
PublishedSeptember 28, 2026 at 08:32 PM
Content Hashsha256:761fcebe01c4c489...
Git SHA
──Files
Files (1 file, 2.8 KB)
SKILL.md2.8 KBactive
SKILL.md · 66 lines · 2.8 KB

version: "1.0.0" name: kernel-change-intent description: "Use when reviewing a Linux kernel PR diff, verifies commit goal and implementation claims (Sashiko stages 1-2): design soundness, UAPI/ABI breakage, and commit-message-vs-code mismatches in the CHANGED code only. Part of the linux-kernel-review suite." license: Apache-2.0 metadata: author: Sashiko contributors / Chris Mason adapted_by: gvago source: https://github.com/sashiko-dev/sashiko (Apache-2.0), references from masoncl/review-prompts (MIT) variant: review version: 1.0.0


kernel-change-intent

Lens for commit goal and implementation-claim verification (Sashiko stages 1-2). Runs no tools. Judges whether the change's concept is sound and whether the code actually does what the commit message claims.

How to apply

  • Read the commit message first, then verify every claim against the diff.

Do not trust the message; assume it may be wrong or even intentionally misleading.

  • Question assumptions. Consider simpler alternative designs.
  • Judge changed lines and their direct blast radius only.

Rule index

  • I1. Architectural soundness. Flag fundamentally flawed concepts,

long-term maintainability hazards, and violations of established kernel design principles. Consider system-wide implications.

  • I2. UAPI and backwards compatibility. Any UAPI breakage or ABI change

without proper deprecation is action_required (Critical-class).

  • I3. Claim-vs-code match. Every claim in the commit message must be fully

realized in the diff. Flag missing pieces: a core change without updating corresponding callers, a struct change without updating all initializers, incomplete implementations, undocumented side-effects.

  • I4. API contract completeness. When defining or modifying structures

containing function pointers, verify all logically required callbacks are implemented. Check for interface omissions.

  • I5. Semantic soundness of arguments. Verify arguments passed to external

subsystems (kobjects, netdevs, etc.) are valid and semantically correct: non-empty strings, correct sizes, correct format specifiers, correct bitwise operations, sound bounds math.

Mapping a finding to the contract

  • action_level: action_required for UAPI breakage or a proven

claim/code mismatch that changes behavior; remediation_recommended for maintainability or design concerns.

  • category: Correctness for mismatches, Maintainability for design.
  • evidence: quote the commit-message claim and cite the code that

contradicts or fails to implement it.

What NOT to flag

  • Low-level memory or locking errors, other lenses own those.
  • Stylistic disagreement with a design that violates no kernel principle.
  • Commit message wording nits with no code impact.

Sourcing

Stage text adapted from Sashiko (Apache-2.0). No reference files needed; this lens is pure reasoning.

All versions