They got to him

you are viewing a single comment's thread
view the rest of the comments
[–] 8 points 9 hours ago* (6 children)

Cool assumption. But pretty sure it's not how things work at all but sounds great...

What actually happens is that you have a long list of multiple maintainers reviewing , and multiple branches for testing, and multiple layers of review too.

https://www.kernel.org/doc/html/next/process/maintainers.html . See that long list of maintainers for a start.

The actual policy is that everything ends up getting reviewed by a human. If I post a patch, it doesn't go straight to master

If it's not being properly tested and maintained, it gets dropped irrespective of what it is (remember devfs?)

Also, I'm not sure about the kernel specifically, but on projects I submitted code to, they often wanted complex patches broken down if possible.

  • source
  • parent
  • hideshow 6 child comments
  • [–] 21 points 6 hours ago (1 child)

    Yes this means the review process is working. What is important to track is if LLM code causes a disproportionately large number of reviews or reviews that take longer because whoever generated the code doesn't actually know what it is doing. Hopefully not the case here but it does happen a lot with LLM coding.

  • source
  • parent
  • hideshow 1 child comment
  • [–] 2 points 8 hours ago (3 children)

    XZ Utils made it past all those checks with malicious code from a human.

    If they’d been more careful with their tarball commits it probably would have made it into the final and commercial distros.

    It made into Homebrew for instance.

  • source
  • parent
  • hideshow 3 child comments
  • [–] 11 points 8 hours ago (2 children)

    Correct me if I'm wrong, but that's not the kernel...

  • source
  • parent
  • hideshow 2 child comments