All of lore.kernel.org
 help / color / mirror / Atom feed
From: f6bvp <f6bvp@free.fr>
To: ralf@linux-mips.org
Cc: davem@davemloft.net, linux-hams@vger.kernel.org,
	C Schuman <k4gbb1@embarqmail.com>
Subject: Re: [PATCH] [ROSE] AX25 packet routing improvement
Date: Fri, 21 Jan 2011 12:23:34 +0100	[thread overview]
Message-ID: <4D396CB6.50209@free.fr> (raw)
In-Reply-To: <4D0F9C6B.4020608@free.fr>

Hi Ralf,

One month ago I sent two patches for ROSE aimed at
improving rose network routing, especially when
a user sends a connect request on a local node.

They have been implemented on a dozen of ROSE FPAC nodes in the
United States and other countries in the world and seem to be
quite efficient and cause no problem.

As I have no feedback since, I am a little bit puzzled.
Could you tell us if you have any restrictive opinion about
applying these patches ?

73 de Bernard, f6bvp


Le 20/12/2010 19:11, f6bvp a écrit :
> Hi Ralf,
>
> Here is a new patch that completes the one I sent recently.
>
> I hope you will find time to have a look at both patches
> and approve them.
>
> ----------
>
> [PATCH] [ROSE] [AX25] finding routes simplification
>
> With previous patch, rose_get_neigh() routine
> investigates the full list of neighbor nodes
> until it finds or not an already connected node whether
> it is called locally or through a level 3 transit frame.
> If no routes are opened through an adjacent connected node
> then a classical connect request is attempted.
>
> Then there is no more reason for an extra loop such
> as the one removed by this patch.
>
> Signed-off-by: Bernard Pidoux <f6bvp@free.fr>
>
>
>
> Le 02/12/2010 13:05, f6bvp a écrit :
>> Hi,
>>
>> Sorry for this second post.
>> I found that spin_lock_bh was missing in the first loop !
>>
>>
>> [PATCH] [ROSE] rose AX25 packet routing improvement
>>
>> FPAC AX25 packet application is using Linux kernel ROSE
>> routing skills in order to connect or send packets to remote stations
>> knowing their ROSE address via a network of interconnected nodes.
>>
>> Each FPAC node has a ROSE routing table that Linux ROSE module is
>> looking at each time a ROSE frame is relayed by the node or when
>> a connect request to a neighbor node is received.
>>
>> A previous patch improved the system time response by looking at
>> already established routes each time the system was looking for a
>> route to relay a frame. If a neighbor node routing the destination
>> address was already connected, then the frame would be sent
>> through him. If not, a connection request would be issued.
>>
>> The present patch extends the same routing capability to a connect
>> request asked by a user locally connected into an FPAC node.
>> Without this patch, a connect request was not well handled unless it
>> was directed to an immediate connected neighbor of the local node.
>>
>> Implemented at a number of ROSE FPAC node stations, the present patch 
>> improved dramatically FPAC ROSE routing time response and efficiency.
>>
>> Signed-off-by: Bernard Pidoux <f6bvp@free.fr>
>>
>>
>

--
To unsubscribe from this list: send the line "unsubscribe linux-hams" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2011-01-21 11:23 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-12-02 12:05 [PATCH] [ROSE] AX25 packet routing improvement f6bvp
2010-12-20 18:11 ` f6bvp
2011-01-21 11:23   ` f6bvp [this message]
2010-12-21 10:38 ` Can AX25 socket debug message be removed ? f6bvp
2010-12-21 17:33   ` Ralf Baechle DL5RB
2010-12-22  9:43     ` Bernard Pidoux F6BVP
2011-02-15 14:25     ` f6bvp
2010-12-21 10:41 ` Can ROSE " Bernard Pidoux
2011-02-14 17:52 ` [PATCH] [ROSE] AX25 packet routing improvement Bernard Pidoux
2011-02-14 21:35   ` David Miller
  -- strict thread matches above, loose matches on Subject: below --
2010-12-02 11:33 f6bvp

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=4D396CB6.50209@free.fr \
    --to=f6bvp@free.fr \
    --cc=davem@davemloft.net \
    --cc=k4gbb1@embarqmail.com \
    --cc=linux-hams@vger.kernel.org \
    --cc=ralf@linux-mips.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 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.