Re: [PATCH net] net: bcmasp: fix memory leak when bringing down if
From: Markus Elfring
Date: Sun Apr 14 2024 - 07:24:23 EST
Can it be nicer to use the word “interface” instead of “if”
in the summary phrase?
> When bringing down the TX rings we flush the rings but forget to
> reclaimed the flushed packets. This lead to a memory leak since we
> do not free the dma mapped buffers. …
I find this change description improvable.
* How do you think about to avoid typos?
* Would another imperative wording be more desirable?
Regards,
Markus