On Thu, 12 Feb 2026 at 06:17, Linus Torvalds <[email protected]> wrote: > > On Wed, 11 Feb 2026 at 11:45, Dave Airlie <[email protected]> wrote: > > > > Hi all, > > This is a public inbox, you can also git clone > > > > https://gitlab.freedesktop.org/drm/ai-reviews-public-inbox > > What an odd format that is, and I don't have anything that reads it > natively, so I just did a one-liner script for it: > > git show $(git rev-list HEAD | sed 's/$/:m/')
public-inbox-v2 I think it's called. lei q -o ai-rev -I https://lore.gitlab.freedesktop.org/drm-ai-reviews "(rt:1.day.ago..)" will produce a local mailbox with the reviews in it that you can point mutt at as well There are also tools I think that will inject things from lei into gmail, but I haven't played there yet. > > and having done that I think the review output format is not exactly > lovely, but whatever. Well the idea is to have it use lore so that users can get it using lei/b4 into mboxes so it fits their workflow. I just published the public inbox git tree in case it makes it easier for someone to use lei with. > But the details in reviews look mostly pretty good to me. I don't know > the code in question well enough to say whether they are useful to > *you*, but it certainly doesn't look bad to me. > > The one review I reacted to was because I *do* know the code enough. > So when Claude reacted to this nonsense patch with that whole "Fix > passing zero to ERR_PTR" thing: > > - return ERR_PTR(err); > + return err ? ERR_PTR(err) : NULL; > > *muy* reaction is that it's the opposite of a "fix" and we should get > rid of that whole And I think as I figure out using the prompts properly etc these sort of things will go away. > However, Claude's other reaction to it is actually much more > interesting than my initial "that's just a nonsense patch". So while > I disagree with the claude on a detail, I don't think it was overall > bad. Dave.
