On Thursday, 21 November 2024 10:13:15 CET Remi Pommarel wrote: > Looks right to me, I would even simplify that more for readability with: [...] > -del: > - list_del(&entry->list); > - kmem_cache_free(batadv_tt_change_cache, entry); > - bat_priv->tt.local_changes--; > discard: > kmem_cache_free(batadv_tt_change_cache, tt_change_node); > goto unlock; >
The "discard" is unused. But the rest looks good.
Kind regards,
Sven
signature.asc
Description: This is a digitally signed message part.
