Re: [PATCH net] net: bcmasp: fix memory leak when bringing down if
From: Simon Horman
Date: Wed Apr 17 2024 - 12:19:52 EST
On Mon, Apr 15, 2024 at 09:46:44PM +0200, Markus Elfring wrote:
> >>> 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?
> >
> > The change description makes sense to me. Can you be a bit more specific as to what isn't clear here?
>
> Spelling suggestions:
> + … forget to reclaim …
> + … This leads to …
Markus, let's cut to the chase.
What portion of your responses of this thread were produced
by an LLM or similar technology?
The suggestions in your second email are correct.
But, ironically, your first response appears to be grammatically incorrect.
Specifically:
* What does "improvable" mean in this context?
* "How do you think about to avoid typos?"
is, in my opinion, grammatically incorrect.
And, FWIW, I see no typos.
* "Would another imperative wording be more desirable?"
is, in my opinion, also grammatically incorrect.
And yet your comment is ostensibly about grammar.
I'm sorry, but this strikes me as absurd.