alt.hn

3/18/2026 at 4:17:44 PM

Google Engineers Launch "Sashiko" for Agentic AI Code Review of the Linux Kernel

 https://www.phoronix.com/news/Sashiko-Linux-AI-Code-Review

by speckx

3/18/2026 at 5:39:48 PM

Better to link to the site itself, or one of the reviews?

For an example of a review (picked pretty much at random) see: https://sashiko.dev/#/patchset/20260318151256.2590375-1-andr...

The original patch series corresponding to that is: https://lkml.org/lkml/2026/3/18/1600

Edit: Here's a simpler and better example of a review: https://sashiko.dev/#/patchset/20260318110848.2779003-1-liju...

I'm very glad they're not spamming the mailing list.

by rwmj

3/18/2026 at 5:48:35 PM

That is both really useful and a great example of why they should have stopped writing code in C decades ago. So many kernel bugs have arisen from people adding early returns without thinking about the cleanup functions, a problem that many other language platforms handle automatically on scope exit.

by jeffbee

3/19/2026 at 7:59:25 AM

You don't even need an LLM for this stuff. GCC has the __cleanup__ attribute, and kernel static analyzers like Smatch have been catching missing unlocks for a decade now. People just ignore linter warnings when submitting patches, so the language itself isn't really the issue. The LLM is basically just acting as a talking linter that can explain the error in plain English

by KurSix

3/19/2026 at 7:03:12 PM

Linux doesn't have any of: sufficient testing, sufficient static analysis, or sufficient pre-commit code review. Under those conditions, which I take as a given because it's their project and we can't just swap out the leaders with more tasteful leaders, adding this type of third-party review feedback strikes me as valuable. Perhaps, to your point, it would also be possible to simply run static analyzers on new proposed commits.

by jeffbee

3/18/2026 at 6:34:06 PM

Must we do this on every thread about the Linux kernel?

by overfeed

3/18/2026 at 8:25:26 PM

The beatings will continue until morale improves

by RobRivera

3/18/2026 at 9:45:11 PM

yeah but Linux is love, linux is life. if you really want to get the beatings going:

Rust > C and GNU/Linux should be Rust.

by vpShane

3/19/2026 at 3:08:14 AM

Ironically C is safer than Rust (if you compile it with Fil-C)

by Ferret7446

3/18/2026 at 10:50:23 PM

also vim > emacs

by ugh123

3/18/2026 at 6:53:12 PM

[flagged]

by richwater

3/18/2026 at 8:40:11 PM

> stopped writing code in C decades ago.

And what were they supposed to use in 2006? Free Pascal? Ada?

by nurettin

3/18/2026 at 8:53:18 PM

Someone suggested C++ and you should see the response from Linus

https://harmful.cat-v.org/software/c++/linus

by greenavocado

3/18/2026 at 11:10:33 PM

Of course I specifically avoided invoking that language's name within the context of kernel programming in fear of summoning a Linus.

And he's so right. I didn't think like that back then, but new/delete (which have to be overloaded for kernel) behind allocators behind containers, vtables, =0, uninitialized members, unhandled ctor errors, template magic, "sometimes rvo", compiler hints, "sometimes reinterpret cast", 3rd party libraries, it would have been a disaster 20 years ago. Now he's being nice to Rust partially to spite that lang I love some more.

by nurettin

3/18/2026 at 6:54:24 PM

Looks cool, but this site is a bit difficult for me to grok.

I think the table might be slightly inside-out? The Status column appears to show internal pipeline states ("Pending", "In Review") that really only matter to the system, while Findings are buried in the column on the far right. For example, one reviewed patchset with a critical and a high finding is just causally hanging out below the fold. I couldn't immediately find a way to filter or search for severe findings.

It might help to separate unreviewed patches from reviewed ones, and somehow wire the findings into the visual hierarchy better. Or perhaps I'm just off base and this is targeting a very specific Linux kernel community workflow/mindset.

Just my 1c.

by withinrafael

3/18/2026 at 7:10:27 PM

I think it's just a dashboard, not meant to be used as is.

Reviewers are more likely to instead subscribe to get the review inline, and then potentially incorporate that with their feedback.

by tonfa

3/18/2026 at 7:57:19 PM

[flagged]

by fdghrtbrt

3/18/2026 at 7:56:21 PM

> Sashiko was able to find around 53% of bugs

That's cool. Another interesting metric, however, would be the false positive ratio: like, I could just build a bogus system that simply marks everything as a bug and then claim "my system found 100% of all bugs!"

In practice, not just the recall of a bug finding system is important but also its precision: if human reviewers get spammed with piles of alleged bug reports by something like Sashiko, most of which turn out not to be bugs at all, that noise binds resources and could undermine trust in the usefulness of the system.

by kleiba

3/18/2026 at 9:12:28 PM

They mention false positives as well on github: The rate of false positives is harder to measure, but based on limited manual reviews it's well within 20% range and the majority of it is a gray zone.

by i_cannot_hack

3/19/2026 at 4:48:35 PM

That 20% figure is actually better than it sounds. Coverity on kernel-scale C codebases typically lands in the 40-60% false positive range... "not wrong but not the bug you'd prioritize" is different from a true false positive.

by riteshkew1001

3/19/2026 at 6:57:28 AM

Hard to measure, how? Either something is a bug or not - otherwise how would you be able to count anything at all?

by kleiba

3/19/2026 at 2:59:42 PM

Assign each line of code a bugginess factor then count those exceeding an arbitrary threshold obviously.

by lstodd

3/18/2026 at 5:30:05 PM

I think this is a great and interesting project. However, I hope that they're not doing this to submit patches to the kernel. It would be much better to layer in additional tests to exploit bugs and defects for verification of existance/fixes.

(Also tests can be focused per defect.. which prevents overload)

From some of the changes I'm seeing: This looks like it's doing style and structure changes, which for a codebase this size is going to add drag to existing development. (I'm supportive of cleanups.. but done on an automated basis is a bad idea)

I.e. https://sashiko.dev/#/message/20260318170604.10254-1-erdemhu...

by monksy

3/18/2026 at 5:35:58 PM

No, it's reviewing patches posted on LKML and offering suggestions. The original patch posted corresponding to your link was this, which was (presumably!) written by a human:

https://lkml.org/lkml/2026/3/9/1631

by rwmj

3/18/2026 at 5:41:49 PM

Style and structure is not the goal here, the reason people are interested in it is to find bugs.

Having said that, if it can save maintainers time it could be useful. It's worth slowing contribution down if it lets maintainers get more reviews done, since the kernel is bottlenecked much more on maintainer time than on contributor energy.

My experience with using the prototype is that it very rarely comments with "opinions" it only identifies functional issues. So when you get false positives it's usually of the form "the model doesn't understand the code" or "the model doesn't understand the context" rather than "I'm getting spammed with pointless advice about C programming preferences". This may be a subsystem-specific thing, as different areas of the codebase have different prompts. (May also be that my coding style happens to align with its "preferences").

by bjackman

3/18/2026 at 8:59:33 PM

I find it interesting that this is written in Rust (not golang) and co-authored with Claude (not gemini)

by throwa356262

3/18/2026 at 11:12:53 PM

It's written in rust, but why do you believe it was co-authored with Claude? The README in github specifically says:

> This project was built using Gemini CLI

https://github.com/sashiko-dev/sashiko

by dgacmu

3/19/2026 at 12:11:57 AM

Claude snitches on you in your commits. You can just look at the history.

by adampunk

3/19/2026 at 12:55:25 AM

Only two commits have 'Co-Authored-By: Claude' and they're both PR contributions from a non-google email.

by gcommer

3/19/2026 at 12:58:45 AM

How do the kernel devs feel about this? Cause that seems to be the sticking point for external AI “help” - the open source devs hate it

Seems to be a well funded effort though so maybe it’s better?

by Havoc

3/18/2026 at 10:01:09 PM

> Roman reports that Sashiko was able to find around 53% of bugs based on an unfiltered set of 1,000 recent upstream Linux kernel issues with "Fixes: " tag

What does this mean?

by simianwords

3/18/2026 at 10:26:42 PM

47% of recent bugs would go unnoticed if we relied solely on this tool. But we might find more, and faster.

by spiderfarmer

3/18/2026 at 9:55:04 PM

Looks like a great new tool to help ship less bugs!

Nitpicking on this though:

> "In my measurement, Sashiko was able to find 53% of bugs based on a completely unfiltered set of 1000 recent upstream issues based on "Fixes:" tags (using Gemini 3.1 Pro). Some might say that 53% is not that impressive, but 100% of these issues were missed by human reviewers."

That'd assume 100% of the issues that were fixed and used for training were not fixed following a human review. I don't buy it: it's extremely common to have a dev notice a bug in the code, without a user having ever reported the bug.

I think the wording meant to say: "... but 100% of these issues were first missed by humans".

My point being: the original code review by a human ain't the only code review by a human. Or put it this way: it's not as if we were writing code, shipping it, then never ever looking at that line of code again unless a bug report were to come out. It's not how development works.

by TacticalCoder

3/18/2026 at 8:26:01 PM

the separation between who writes and who reviews is the whole thing. I do same at smaller scale — one model writes code, different model reviews it. self-review misses things, same reason you don't review your own PRs

by mika-el

3/19/2026 at 3:10:03 AM

[dead]

by bmd1905

3/18/2026 at 6:27:00 PM

[dead]

by takahitoyoneda

3/19/2026 at 12:51:49 AM

[dead]

by whiteclawonso36

3/19/2026 at 12:49:02 AM

[dead]

by balinha_8864

3/18/2026 at 5:08:33 PM

[dead]

by Heer_J

3/18/2026 at 6:16:14 PM

[dead]

by ratrace

3/18/2026 at 9:07:34 PM

[flagged]

by michaelchen58

3/18/2026 at 9:34:15 PM

[flagged]

by goatyishere25

3/18/2026 at 6:32:41 PM

[flagged]

by quantium1628

3/18/2026 at 5:23:04 PM

oh god can we not

by 4fterd4rk

3/18/2026 at 5:28:28 PM

What's your concern?

by smlacy

3/18/2026 at 5:31:43 PM

Have you ever programmed with AI? It needs a lot of hand holding for even simple things sometimes. Forgets basic input, does all kinds of brain dead stuff it should know not to do.

>"good catch - thanks for pointing that out"

by htx80nerd

3/18/2026 at 5:35:21 PM

Can you clarify how, at all, that’s relevant to the article?

by lame-robot-hoax

3/18/2026 at 6:03:10 PM

Both the curl and the SQLite project have been overburdened by AI bug reports. Unless the Google engineers take great care to review each potential bug for validity the same fate might apply here. There have been a lot of news regarding open source projects being stuffed to the brim with low effort and high cost merge requests or issues. You just don't see all the work that is caused unless you have to deal with the fallout...

by ablob

3/18/2026 at 7:12:47 PM

This project has nothing to do with bug reports... it's an opt-in tool for reviewing proposed changes that kernel developers can decide to use (if they find it useful).

by tonfa

3/18/2026 at 5:46:07 PM

Well, if it doesn't find anything it's just a waste of time at best.

by jamesnorden

3/18/2026 at 8:32:20 PM

Prevention paradox.

by danielbln

3/18/2026 at 5:34:56 PM

i think it's a skill.

by asadm

3/18/2026 at 5:31:16 PM

well tbf code review is probably the most useful part of "AI coding", if it catches even a single bug you missed its worth it, plus false positives would waste dev time but not pollute the kernel

by __tidu

3/18/2026 at 7:25:32 PM

They would have completely redesigned Google Gerrit.

by qainsights

3/19/2026 at 7:53:18 AM

Written in Rust, tests a C kernel, using the Google Gemini API... classic 2026. I'd bet 90% of the actual useful work this agent does is just dumb pattern-matching for typical vulnerabilities (use-after-free, uninitialized vars) that the model memorized straight out of the CVE database

by KurSix

3/19/2026 at 9:07:39 AM

Arguably that's still very useful :)

by nasretdinov

3/18/2026 at 5:46:32 PM

Now they want to kill the Linux kernel. :(

We've already seen how bug bounty projects were closed by AI spam; I think it was curl? Or some other project I don't remember right now.

I think AI tools should be required, by law, to verify that what they report is actually a true bug rather than some hypothetical, hallucinated context-dependent not-quite-a-real-bug bug.

by shevy-java

3/18/2026 at 7:11:27 PM

It's not forced upon anyone, it's a tool that patch authors or reviewers can use if they want to.

by tonfa

3/19/2026 at 8:13:03 AM

Those incidents with curl and sqlite were caused by a mob of script kiddies dumping source code into ChatGPT and spamming bug bounties for quick cash. This tool is built by actual Google devs who are active on the kernel mailing list. They know perfectly well that if they start spamming LKML with hallucinations, they'll get patch acceptance blocked for their entire corporation

by KurSix