public inbox for linux-hams@vger.kernel.org
 help / color / mirror / Atom feed
From: Bernard Pidoux <bernard.pidoux@upmc.fr>
To: Cathryn Mataga <cathryn@junglevision.com>,
	linux-hams <linux-hams@vger.kernel.org>
Subject: [PATCH] xfbbd:  Fix issue with xfbbd and AX25 radio connections
Date: Sun, 14 Nov 2010 19:58:04 +0100	[thread overview]
Message-ID: <4CE0313C.3080804@upmc.fr> (raw)

Hi Cathryn,

Many thanks for the patches.
I had noticed the SIGPIPE bug when in GDB session.
However xfbbd did not fail usually when not under GDB.
Anyway, its nice you cured the problem.
Other bugs were also very annoying and I am gratefull that
your perspicacy found the reasons I did not discovered.

I applied your patch into new FBB pre release xd704r12.

http://f6bvp.free.fr/logiciels/BBS-f6fbb/xd704r12-src.tar.bz2

See CHANGES file for important notices about the use of xfbbd
on 64 bits and SMP systems.

73s de Bernard, f6bvp



>Patch versus version xfbbd version r11
>
>Same patch file, Except I changed the email subject.
>
>
>
>1. SOCK_MAXCHAN (drv_sock.c) now set to (MAXVOIES)  This was set to 50, which was less than MAXVOIES, and the code was
>accessing data off the end of this table.  This was causing strange problems.  After I made this change the gateway
>function started working again.  If anyone was having trouble with xfbbd and RF radio connections, this change may
>possibly fix their problem.
>2.  Several minor changes designed to eliminate "Bad file descriptor" error messages.  These occurred at the time
>of housekeeping due to a minor bug.  The messages actually turned out to be harmless, as far as I can tell.
>3.  Only use the signal handler for signals that have functionality in the code.  This is a simplification
>to prevent bugs.  Other signals are set to SIG_IGN.




             reply	other threads:[~2010-11-14 18:58 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-14 18:58 Bernard Pidoux [this message]
2013-07-05 21:20 ` [PATCH] xfbbd: Run the same data files on 64-bit Cathryn Mataga
2013-07-06 21:24   ` Pidoux
2013-07-07  0:15     ` Cathryn Mataga
2013-07-16  3:09     ` Cathryn Mataga
  -- strict thread matches above, loose matches on Subject: below --
2010-05-27 12:33 [PATCH 4/11] drivers/net/hamradio: Eliminate a NULL pointer dereference Julia Lawall
2010-05-27 23:29 ` David Miller
2010-11-09 19:09   ` xfbbd oddness Cathryn Mataga
2010-11-10  2:54     ` Cathryn Mataga
2010-11-12 21:23       ` Cathryn Mataga
2010-11-14  6:52         ` Cathryn Mataga
2010-11-14  6:58           ` [PATCH] xfbbd: Fix issue with xfbbd and AX25 radio connections Cathryn Mataga

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=4CE0313C.3080804@upmc.fr \
    --to=bernard.pidoux@upmc.fr \
    --cc=cathryn@junglevision.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