b4 review experience

[Chris Samuel <chris@csamuel.org>]

Hi there,

First of all thanks so much for this work, it looks really nice!  I'll 
also preface this by saying I'm not a kernel person, I'm not even really 
a programmer, just a sysadmin who likes to track what's going on in 
certain parts of the kernel.

Having seen Jon Corbet's mentions of the new "b4 review" feature I 
decided to try "b4" out from the git repo and I've got to say I really 
like it. It took me a minute to get to grips with it but I'm enjoying 
how it lets me look at patchsets and comments on them.

One thing I've noticed is that if the patch set is large then I can't 
scroll the list of patches on the left hand side of the review view with 
the mouse, though I can use the arrow keys to go through the patches.

A similar issue that if I run the CI (to get checkpatch and Sashiko 
input) I can scroll the list with the mouse, but I can't select any of 
the patches with the mouse. I can move through the list with the arrow 
keys but when I get to the bottom of the screen it doesn't scroll. I can 
scroll with the mouse to see where it's got to, but as soon as I press 
an arrow key it resets to the start of the list.

The only other thing I'll mention is that with the checkpatch output 
it'd be handy (for the complaints about lines in the commit message) if 
it showed the context like it does when you run it at the command line.

For my experiments I've been using two of the MPAM patchsets:

https://lore.kernel.org/lkml/20260319162225.378485-1-ben.horgan@arm.com/

That set has the checkpatch failure init.

https://lore.kernel.org/lkml/20260313144617.3420416-1-ben.horgan@arm.com/

That is the large (40 patch) set I mention.

But again, I really like this and for alpha code it's pretty good!

Thank you, all the best,
Chris
-- 
Chris Samuel  :  http://www.csamuel.org/  :  Philadelphia, PA, USA