[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <962afcda-8f67-400f-b3eb-951bf2e46fb7@moroto.mountain>
Date: Wed, 29 May 2024 17:34:20 +0300
From: Dan Carpenter <dan.carpenter@...aro.org>
To: Lars Kellogg-Stedman <lars@...bit.com>
Cc: Paolo Abeni <pabeni@...hat.com>, linux-hams@...r.kernel.org,
netdev@...r.kernel.org, Duoming Zhou <duoming@....edu.cn>
Subject: Re: [PATCH v4] ax25: Fix refcount imbalance on inbound connections
[ Sorry for the delay on sending this. My email daemon messed up so
all my outgoing mail has been going to /dev/null for the past few
days. Resending everything... -dan ]
On Tue, May 28, 2024 at 12:06:39PM -0400, Lars Kellogg-Stedman wrote:
> On Tue, May 28, 2024 at 11:40:38AM GMT, Paolo Abeni wrote:
> > Note that the fixes tag above is still wrong - the hash must be 12
> > chars long, see:
>
> I had spotted that, thanks. Just waiting to see if there are any real
> change requests before re-submitting.
Okay. It looks good to me. Sorry for all the headache.
1) The Fixes tag points to the wrong commit, though, right? The one
you have here doesn't make sense and it doesn't match the bisect.
2) Can we edit the commitmessage a bit to say include what you wrote
about "but rather bind/accept" being paired. We increment in bind
and we should increment in accept as well. It's the same.
3) The other thing that I notice is that Duoming dropped part of his
commit when he resent v6.
https://lore.kernel.org/all/5c61fea1b20f3c1596e4fb46282c3dedc54513a3.1715065005.git.duoming@zju.edu.cn/
That part of the commit was correct. Maybe it wasn't necessary but it
feels right and it's more readable and it's obviously harmless. I can
resend that.
regards,
dan carpenter
Powered by blists - more mailing lists