public inbox for linux-hams@vger.kernel.org
 help / color / mirror / Atom feed
From: "Bernard, f6bvp" <f6bvp@free.fr>
To: Ralf Baechle DL5RB <ralf@linux-mips.org>
Cc: linux-hams@vger.kernel.org
Subject: Re: [PATCH 0/7] ROSE: Misc fixes
Date: Fri, 22 Jul 2011 11:10:46 +0200	[thread overview]
Message-ID: <4E293E96.3030708@free.fr> (raw)
In-Reply-To: <20110720175950.GA13753@linux-mips.org>

Hello Ralf,

Replacement patch applied and install went fine.
Nearly 24 hours after running the new "atomic" rose code
on two systems, I observe that 'use' parameter is 0, something I never 
observed before. It used to be positive on some link and rarely
displaying a high number meaning underflow negative values.

> /proc/net/rose_neigh
> addr  callsign  dev  count use mode restart  t0  tf digipeaters
> 00020 F6BVP-5   ax0      1   0  DTE      no   0   0
> 00019 F6BVP-7   ax0      2   0  DTE      no   0   0
> 00018 F6BVP-9   ax2      2   0  DTE      no   0   0
> 00017 F3KT-11   ax0      3   0  DTE     yes   0   0
> 00016 F8COJ-11  ax0      2   0  DCE     yes   0   0
> 00015 F6GGY-9   ax0      3   0  DTE      no   0   0
> 00014 F4BWT-11  ax0      4   0  DTE      no   0   0
> 00013 F1MVP-5   ax0      2   0  DTE      no   0   0
> 00012 F5KBW-9   ax0      6   0  DTE      no   0   0
> 00011 IR5AG-13  ax0      2   0  DTE      no   0   0
> 00010 K4GBB-9   ax0      4   0  DCE     yes   0   0
> 00009 KD4YAL-9  ax0      2   0  DTE      no   0   0
> 00008 KP4DJT-9  ax0      3   0  DTE     yes   0   0
> 00007 P43L-4    ax0      5   0  DCE     yes   0   0
> 00006 VK2XB-2   ax0      2   0  DTE      no   0   0
> 00005 VK2TV-2   ax0      2   0  DTE      no   0   0
> 00004 VK7HDM-5  ax0      2   0  DTE      no   0   0
> 00003 YN1BBS-9  ax0      5   0  DTE      no   0   0
> 00002 TI2HAS-9  ax0      5   0  DTE      no   0   0
> 00001 RSLOOP-0  ???      1   2  DCE     yes   0   0

I guess this is a pretty good thing.

I will try removing rose module and report if any progress
on that point too.

Bernard


Le 20/07/2011 19:59, Ralf Baechle DL5RB a écrit :
> On Wed, Jul 20, 2011 at 07:15:33PM +0200, Bernard, f6bvp wrote:
>
>> All seven patch applied.
>> However there are compile errors in rose_route.c
>>
>> f6bvp-8 linux]# make modules
>> CHK include/linux/version.h
>> CHK include/generated/utsrelease.h
>> CALL scripts/checksyscalls.sh
>> CC [M] net/rose/af_rose.o
>> CC [M] net/rose/rose_dev.o
>> CC [M] net/rose/rose_in.o
>> CC [M] net/rose/rose_link.o
>> CC [M] net/rose/rose_loopback.o
>> CC [M] net/rose/rose_out.o
>> CC [M] net/rose/rose_route.o
>> net/rose/rose_route.c: In function ‘rose_add_node’:
>> net/rose/rose_route.c:104: erreur: incompatible type for argument 2
>> of ‘atomic_add_return’
>> /usr/src/linux-2.6.39/arch/x86/include/asm/atomic.h:173: note:
>> expected ‘struct atomic_t *’ but argument is of type ‘atomic_t’
>> net/rose/rose_route.c: In function ‘rose_add_loopback_neigh’:
>> net/rose/rose_route.c:391: erreur: incompatible type for argument 2
>> of ‘atomic_add_return’
>> /usr/src/linux-2.6.39/arch/x86/include/asm/atomic.h:173: note:
>> expected ‘struct atomic_t *’ but argument is of type ‘atomic_t’
>> make[2]: *** [net/rose/rose_route.o] Erreur 1
>> make[1]: *** [net/rose] Erreur 2
>> make: *** [net] Erreur 2
>
> Odd, the timestamps of my build tree must have been scrambled.  I
> get the same error messages but only after a full make clean and the
> error is quite obvious.
>
> I'm going to send a replacement match for the defective patch in
> a minute.
>
>    Ralf
--
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-07-22  9:10 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-20  9:00 [PATCH 0/7] ROSE: Misc fixes Ralf Baechle
2011-07-20  0:21 ` [PATCH 1/7] NET: ROSE: Fix race in SIOCRSSL2CALL ioctl accessing userspace Ralf Baechle
2011-07-20  0:21 ` [PATCH 2/7] NET: ROSE: Factor our common code from functions Ralf Baechle
2011-07-20  0:37 ` [PATCH 3/7] NET: ROSE: Protect rose_callsign with a spinlock Ralf Baechle
2013-08-06 17:57   ` [PATCH] ax25tools mheard : don't display empty records f6bvp@free
2013-08-07  8:17     ` Thomas Osterried
2013-08-07 10:06       ` f6bvp@free
2013-08-09  8:10     ` Thomas Osterried
2011-07-20  8:11 ` [PATCH 4/7] NET: ROSE: Make neighbour->use atomic Ralf Baechle
2011-07-20  8:11 ` [PATCH 5/7] NET: ROSE: Make rose_neigh_no atomic Ralf Baechle
2011-07-20 18:09   ` [PATCH v2 " Ralf Baechle
2011-07-20  8:11 ` [PATCH 6/7] NET: ROSE: Move return statements hidden behind an if to their own line Ralf Baechle
2011-07-20  8:11 ` [PATCH 7/7] NET: ROSE: Fix formatting Ralf Baechle
2011-07-20 17:15 ` [PATCH 0/7] ROSE: Misc fixes Bernard, f6bvp
2011-07-20 17:59   ` Ralf Baechle DL5RB
2011-07-22  9:10     ` Bernard, f6bvp [this message]
2011-07-22 10:56       ` Ralf Baechle DL5RB
2011-07-22 16:12         ` f6bvp
2011-07-23 13:28           ` Ralf Baechle DL5RB
2011-07-29 22:32         ` f6bvp
2011-08-08 13:40           ` f6bvp
2011-08-08 14:06             ` Ralf Baechle
2011-08-08 15:33               ` f6bvp
2011-08-19 13:07                 ` 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=4E293E96.3030708@free.fr \
    --to=f6bvp@free.fr \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox