Re: [PATCH net 3/4] can: raw: instantly reject disabled CAN frames

On 15.01.26 13:26, Paolo Abeni wrote:

On 1/15/26 10:18 AM, Oliver Hartkopp wrote:

quoted

And I was wondering why my patch was marked "yellow"

https://patchwork.kernel.org/project/netdevbpf/patch/20260114105212.1034554-4-mkl@pengutronix.de/

The AI review marked the patch as "yellow" but the review result was not
accessible until midnight.

A direct feedback to the authors would be helpful.

The AI review is intentionally "revealed" in PW after a grace period to
avoid random people sending unreviewed/half-finished patches to the ML
just to get the AI review.

I insisted to raise such grace period to 24h to align with the maximum
re-submit rate, but I did not consider carefully the trusted PR cases.

Thanks for the explanation!

IMO the grace period is generally fine to control the re-submit rate.

Btw. automatically informing only the author (who's very likely in 
charge to provide a fix) would be helpful. E.g. when a PR/patchset is 
processed completely.

That would at least avoid patches from random people but would give more 
time to the author to think about his faults ;-)

Best regards,
Oliver