[Devel] [PATCH vz9 v1 55/63] dm-ploop: rework bat completion logic

[Pavel Tikhomirov]

On 1/24/25 23:36, Alexander Atanasov wrote:
> @@ -900,6 +902,7 @@ static void ploop_advance_local_after_bat_wb(struct ploop *ploop,
>   	if (wait_llist_pending) {
>   		llist_for_each_safe(pos, t, wait_llist_pending) {
>   			pio = list_entry((struct list_head *)pos, typeof(*pio), list);
> +			INIT_LIST_HEAD(&pio->list);
>   			list_add(&pio->list, &list);
>   		}
>   	}

Excess list initialization. There is no point in initializing the 
list_head before inserting it into list. Please fix it in all other 
places, not just this one.

-- 
Best regards, Tikhomirov Pavel
Senior Software Developer, Virtuozzo.