All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matti Aarnio <matti.aarnio@zmailer.org>
To: Thomas Osterried <thomas@x-berg.in-berlin.de>
Cc: Matti Aarnio <matti.aarnio@zmailer.org>,
	Ralf Baechle DL5RB <ralf@linux-mips.org>,
	linux-hams@vger.kernel.org
Subject: Re: What were the reasons of having IP address at AX.25 interfaces ?
Date: Tue, 1 Apr 2008 12:44:45 +0300	[thread overview]
Message-ID: <20080401094445.GR3700@mea-ext.zmailer.org> (raw)
In-Reply-To: <20080401081726.GB4491@x-berg.in-berlin.de>

On Tue, Apr 01, 2008 at 10:17:26AM +0200, Thomas Osterried wrote:
> > > A while ago I was curious myself for why that is needed and dug into
> > > the CVS history.  I found the IP address wasn't always enforced and the
> > > commit message didn't really explain why the change was needed.
> > > 
> > > 73 de DL5RB op Ralf
> > 
> > I figured it eventually myself, and rewrote  libax25's
> > axconfig/rsconfig/nrconfig functions to not to require IP addresses
> > in AX.25/Netrom/Rose interfaces to be able to find all interface
> > currently defined.
> 
> I think it was a design problem, because the used system call only
> shows those interfaces which are up. 

Are up and with IP addresses.  I am not 100% sure that the used ioctl()
misbehaves in this regard.

> My approach is, to use /proc to find them, inspired by the "ifconfig" tool
> (if i remember correctly).

I wrote similar code myself, and turned the logic of the whole axconfig.c
upside down in order to scan files only once, nor build 

> This simply enhances get_ax25_ifaces() by this method. The code is in
> the CVS version, but not enabled by default (-DFIND_ALL_INTERFACES).
> 
> I hat no problems with it on kernel 2.6. But on kernel 2.4., i observed
> that when an incoming SABM occurs, the kernel did panic.

And recently people have observed kernel panics with ROSE traffic..
Permanently avoiding using some machinery because it breaks for some
users is not a good thing in my mind ..  those broken things will never
get fixed if we don't find them, and report them.


I use only AX.25 UI traffic myself (APRS), thus those SABM problems do
not bite me, but for rare CONS users they are troublesome things indeed..


> 73,
> 	- Thomas  dl9sau

  /Matti Aarnio - OH2MQK

  reply	other threads:[~2008-04-01  9:44 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-03 18:04 What were the reasons of having IP address at AX.25 interfaces ? Matti Aarnio
     [not found] ` <47A60A57.5080402@digitalrealm.net>
2008-02-03 20:28   ` Matti Aarnio
2008-02-04 13:02     ` What were the reasons of having mandatory " Matti Aarnio
2008-02-04 14:38       ` Andi Kleen
2008-02-04 15:02         ` Matti Aarnio
2008-03-31 21:50 ` What were the reasons of having " Ralf Baechle DL5RB
2008-03-31 22:08   ` Matti Aarnio
2008-04-01  8:17     ` Thomas Osterried
2008-04-01  9:44       ` Matti Aarnio [this message]
2008-04-01 10:37         ` Thomas Osterried
2008-04-01 12:09         ` Bernard Pidoux

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=20080401094445.GR3700@mea-ext.zmailer.org \
    --to=matti.aarnio@zmailer.org \
    --cc=linux-hams@vger.kernel.org \
    --cc=ralf@linux-mips.org \
    --cc=thomas@x-berg.in-berlin.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.