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