Linux HAM/Amateur Radio development
 help / color / mirror / Atom feed
From: David Ranch <linux-hams@trinnet.net>
To: Lars Kellogg-Stedman <lars@oddbit.com>,
	Chris Maness <christopher.maness@gmail.com>
Cc: Duoming Zhou <duoming@zju.edu.cn>,
	linux-hams@vger.kernel.org, dan.carpenter@linaro.org
Subject: Re: Kernel 6.9.1 AX.25 Crash
Date: Tue, 21 May 2024 08:53:06 -0700	[thread overview]
Message-ID: <f27710b7-253c-79ce-e214-2b8251aceead@trinnet.net> (raw)
In-Reply-To: <r3hwkyx6os3eacimhqvrs4cv4ssqajpxp3zkiqqidu6pjkj2v6@slrtgnartz3f>


Hello Lars,

> Well, partly because you're the first person to confirm that it works
> for someone besides me :). Dan (Cross) has offered to test it out as
> well; if I can get a couple of confirmations that it's working, I'll
> probably go ahead and submit it to netdev.
>
> The bigger issue has been that there are very few people interested in
> the ax.25 support in the kernel; there's not a real maintainer, so it's
> difficult to get code approved. There were some concerns expressed that
> maybe this isn't the *correct* fix, but I would argue that even if it's
> not the most correct fix we should try to get it in anyway, since
> otherwise ax.25 is completely broken.

First off, Lars, I wanted to thank you for getting some of these REAL 
fixes actually committed into the mainline.  You've done something that 
many others could not!  That said, I wanted to chime in here about your 
commend about "very few people interested in the a.25 support in the 
kernel".  That's definitely not true.

I and several others tried for many quarters and years to try to get 
some of these initial toxic commits that came in around the 4.19.x 
timeframe fixed but there was never any real meaningful responses and 
fixes.  I do agree that the lack of a real maintainer aka steward for 
the mkiss / ax25 / netrom / rose kernel code really hurts the ongoing 
health of the stack.  I wish I could take up that job but I don't have 
the required skillset.  The larger issue here is that random Linux 
kernel developers are periodically changing various parts of kernel 
infrastructure and when they try to make "updates" to the AX.25 code, 
the mindset is that they only consider a successful kernel compile as a 
"code pass".  That's VERY bad.  The other significant issue is that the 
kernel branch maintainers / powers at be seem to just accept these 
changes without any real scrutiny and don't require any real testing 
results to confirm things still work.  I had previously asked how some 
of these developers had really tested their code when they offered 
official patches here on the vger list and never received *any* 
meaningful responses.  I've asked if there is some form of a per-commit 
or daily CI build and regression test environment that I could offer up 
some basic toxicity test scenarios to.  No response.  It's been hugely 
frustrating for YEARS now and all of the "silently suffering" Linux 
packet users have resorted to still run ancient Linux distros like 
Debian Wheezy to keep their systems fully operational.

I am hugely grateful to you (Lars) on being able to submit some 
meaningful fixes and was able to get them committed to the mainline.  
The next major hurdle that I would love to get feedback on is how to get 
these new changes but also create a set of backported fixes committed 
into various Linux distributions kernels that don't strictly follow the 
mainline kernel.  My personal interest is in the Raspberry Pi OS kernel 
and the Canonical Ubuntu 22.04 / 20.04 kernels.  If people have 
recommendations / contacts to try / etc...please contact me offline and 
I will work on trying to get these fixes committed.

--David
KI6ZHD
Maintainer of the Linpac AX.25 terminal program
Advocate of the Direwolf multi-platform AX.25 software based TNC
AMPR 44-net Coordinator for the Silicon Valley, USA region


  parent reply	other threads:[~2024-05-21 16:18 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-01  6:02 [PATCH net] ax25: Fix refcount leak issues of ax25_dev Duoming Zhou
2024-05-01 17:33 ` Markus Elfring
2024-05-01 17:43 ` Dan Carpenter
2024-05-02  4:35   ` duoming
2024-05-02  7:56     ` Dan Carpenter
2024-05-02  9:30       ` Paolo Abeni
2024-05-02  1:29 ` Lars Kellogg-Stedman
2024-05-03 20:36   ` Dan Carpenter
2024-05-03 23:40     ` Lars Kellogg-Stedman
2024-05-04 12:16       ` Dan Carpenter
2024-05-04 22:16         ` Lars Kellogg-Stedman
2024-05-07  3:18           ` Lars Kellogg-Stedman
2024-05-07  8:08             ` Dan Carpenter
2024-05-07  9:04               ` duoming
2024-05-08 18:27                 ` Dan Carpenter
2024-05-09  1:40               ` duoming
2024-05-23 12:30               ` Lars Kellogg-Stedman
2024-05-07  6:38     ` Dan Carpenter
2024-05-15  9:52   ` duoming
2024-05-16 19:20     ` Lars Kellogg-Stedman
     [not found]       ` <CANnsUMHTZ_P4-C2iGdbakcp_Xk5c-aCO5kYEvaBdOcsaSnK5Pg@mail.gmail.com>
2024-05-20 14:42         ` Kernel 6.9.1 AX.25 Crash Lars Kellogg-Stedman
2024-05-20 14:49           ` Chris Maness
2024-05-21  0:11           ` Chris Maness
2024-05-21  0:48             ` Lars Kellogg-Stedman
2024-05-21  1:38               ` Chris Maness
2024-05-21  2:12               ` Dan Cross
2024-05-21  3:07                 ` Chris Maness
2024-05-21 11:47                   ` Dave Hibberd
2024-05-21 12:03                     ` Chris Maness
2024-05-21 19:20                       ` Mike Quin
2024-05-21 23:02                         ` Chris Maness
2024-05-21 23:07                           ` Mike Quin
2024-05-21 23:17                             ` Chris Maness
2024-05-21 15:53               ` David Ranch [this message]
2024-05-21 16:28                 ` Chris Maness
2024-05-21 18:12                 ` Lars Kellogg-Stedman
2024-05-22 15:32                   ` Dan Cross
2024-05-22 16:07                     ` Chris Maness
2024-05-22 17:06                       ` David Ranch
2024-05-22 18:04                         ` Dan Cross
2024-05-22 17:54                       ` Dan Cross
2024-05-22 18:17                     ` Lars Kellogg-Stedman
2024-05-22 18:37                       ` Dan Cross
2024-05-22 19:23                         ` Lars Kellogg-Stedman
2024-05-22 22:22                           ` David Ranch
2024-05-22 22:32                             ` Lars Kellogg-Stedman
2024-05-23  0:54                             ` Userspace AX.25 stack library [was Re: Kernel 6.9.1 AX.25 Crash] Stuart Longland VK4MSL
2024-05-23 16:32                               ` Dan Cross
2024-05-22 16:10                   ` Kernel 6.9.1 AX.25 Crash David Ranch
2024-05-22 18:13                     ` Lars Kellogg-Stedman
2024-06-07 12:01                   ` Lars Kellogg-Stedman
2024-06-07 13:49                     ` David Ranch
2024-06-08  1:59                       ` Lars Kellogg-Stedman
2024-06-08 15:23                         ` David Ranch
2024-05-23 10:30               ` Dan Carpenter
2024-05-27 17:59           ` Chris Maness
2024-05-04 11:04 ` [PATCH net] ax25: Fix refcount leak issues of ax25_dev Dan Carpenter
  -- strict thread matches above, loose matches on Subject: below --
2024-05-20  2:55 Kernel 6.9.1 AX.25 Crash Chris Maness

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f27710b7-253c-79ce-e214-2b8251aceead@trinnet.net \
    --to=linux-hams@trinnet.net \
    --cc=christopher.maness@gmail.com \
    --cc=dan.carpenter@linaro.org \
    --cc=duoming@zju.edu.cn \
    --cc=lars@oddbit.com \
    --cc=linux-hams@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox