On 6/17/24 05:00, Cyril Hrubis wrote: > Signed-off-by: Cyril Hrubis <chrubis@xxxxxxx> > --- [...] > + > + errors=$(_dmesg_since_test_start |grep -c "operation not supported error, dev .*WRITE_ZEROES") > + nit:- space on the both side of '|' before grep would be nice, can be done at the time of applying the patch ... Looks good. Reviewed-by: Chaitanya Kulkarni <kch@xxxxxxxxxx> -ck