Thread (15 messages) 15 messages, 5 authors, 2024-05-29

Re: [PATCH v4] ax25: Fix refcount imbalance on inbound connections

From: Dan Carpenter <hidden>
Date: 2024-05-29 14:34:26
Also in: linux-hams

[ 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:
quoted
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/ (local)
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help