Skip to contentSkip to conversation

AI reviews are getting emailed out now

ANN Jakub Kicinski 2 participants 2 messages Sep 1 – Sep 2

Subsystems (2)
lore
Share & tools
lore

Jakub Kicinski

author maintainer
#0 lore raw

Hi!

As mentioned previously we were trying to get false positive rate low enough to start sending AI reviews out automatically.

According to the stats for the last 3 weeks 70-80% of the patches we are applying have no issue with severity higher than Low (50-60% are completely clean). Only around 20% of patches we mark as Changes Requested are have issues <Low. Let's call that good enough.

The embargo on the reviews and time when review is emailed out follows the same timeline as maintainer reviews. Delays are added if author does not review other people's patches and if CI finds issues. Paolo explained this a bit at netdev.conf.

Please let us know if you have any feedback, especially if you are a reviewer.

maintainer
#1 lore raw
Quoted 8 lines · Jakub Kicinski

As mentioned previously we were trying to get false positive rate low enough to start sending AI reviews out automatically.

According to the stats for the last 3 weeks 70-80% of the patches we are applying have no issue with severity higher than Low (50-60% are completely clean). Only around 20% of patches we mark as Changes Requested are have issues <Low. Let's call that good enough.

Quoted 4 lines

The embargo on the reviews and time when review is emailed out follows the same timeline as maintainer reviews. Delays are added if author does not review other people's patches and if CI finds issues. Paolo explained this a bit at netdev.conf.

FTR the related slides are available here:

https://netdevconf.info/0x1A/docs/netdev-0x1a-paper49-talk-slides/Device%20driver%20workshop%20(2).pdf

beware: the link could be updated, please check the conference page.

/P