Hello, Tao.

On Fri, 18 Sep 2026 13:49:57 +0800, Tao Cui wrote:

> +     mutex_lock(&iocost_bpf_reg_lock);
> +     {
> +             struct iocost_bpf_model *other;
> +
> +             list_for_each_entry(other, &iocost_bpf_models, list) {

No need for a block to scope a variable. Declare it at the top of the
function, or in the body if it has to sit inside an #ifdef block. Same for
the other blocks in this patch and the cursor block in the 2x example
model.

Thanks.

--
tejun

Reply via email to