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
next prev parent 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