atl1 developers wanted

2007-10-28 Thread Chris Snook
Greetings, netdev! Since the Linux Driver Project apparently has a glut of talent, I figured I'd ask for a little help with an existing driver. As some of you may know, Jay Cliburn and I have been working on atl1 since last summer, and we've gotten the driver merged and stable. It does the

Re: [PATCH] proc_fs.h redux

2007-10-28 Thread Alexey Dobriyan
On Sat, Oct 27, 2007 at 03:40:04PM -0700, Joe Perches wrote: On Sat, 2007-10-27 at 23:47 +0400, Alexey Dobriyan wrote: Remove proc_fs.h from headers that doesn't really need it. --- a/arch/powerpc/kernel/process.c +++ b/arch/powerpc/kernel/process.c @@ -17,6 +17,7 @@ #include

[PATCH 2.6.24 1/1]S2io: Fixed memory leak by freeing MSI-X local entry memories when vector allocation fails

2007-10-28 Thread Sivakumar Subramani
- Fixed memory leak by freeing MSI-X local entry memories when vector allocation fails in s2io_add_isr. - Added two utility functions do_rem_msix_isr and do_rem_inta_isr to eliminate code duplication. Signed-off-by: Veena Parat [EMAIL PROTECTED] Signed-off-by: Ramkrishna Vepa [EMAIL PROTECTED]

Re: [PATCH] proc_fs.h redux

2007-10-28 Thread Sam Ravnborg
As a general rule, I think it better to use includes than use naked forward declarations. Quite the opposite - at least in the kernel source. The general rule is that a .h file shall include the .h files which contain declarations used by said .h files. But naked declarations as above is

Re: [PATCH] proc_fs.h redux

2007-10-28 Thread Russell King
On Sat, Oct 27, 2007 at 03:40:04PM -0700, Joe Perches wrote: and forward declarations of struct proc_dir_entry; struct file_operations; As a general rule, I think it better to use includes than use naked forward declarations. If you go down that route, you end up with _lots_ of circular

b44 ssb link errors on sparc

2007-10-28 Thread Meelis Roos
MODPOST 434 modules ERROR: ioremap [drivers/ssb/ssb.ko] undefined! ERROR: iounmap [drivers/ssb/ssb.ko] undefined! ERROR: dma_alloc_coherent [drivers/net/b44.ko] undefined! ERROR: dma_get_cache_alignment [drivers/net/b44.ko] undefined! ERROR: dma_free_coherent [drivers/net/b44.ko] undefined!

Re: [PATCH] proc_fs.h redux

2007-10-28 Thread Bernhard Walle
* Russell King [EMAIL PROTECTED] [2007-10-28 11:34]: If you go down that route, you end up with _lots_ of circular dependencies - header file X needs Y needs Z which needs X. We've been there, several times. It very quickly becomes quite unmaintainable - you end up with hard to predict

Re: [PATCH] proc_fs.h redux

2007-10-28 Thread Russell King
On Sun, Oct 28, 2007 at 12:59:52PM +0100, Bernhard Walle wrote: * Russell King [EMAIL PROTECTED] [2007-10-28 11:34]: If you go down that route, you end up with _lots_ of circular dependencies - header file X needs Y needs Z which needs X. We've been there, several times. It very

Re: [PATCH] proc_fs.h redux

2007-10-28 Thread Bernhard Walle
* Russell King [EMAIL PROTECTED] [2007-10-28 14:04]: On Sun, Oct 28, 2007 at 12:59:52PM +0100, Bernhard Walle wrote: * Russell King [EMAIL PROTECTED] [2007-10-28 11:34]: If you go down that route, you end up with _lots_ of circular dependencies - header file X needs Y needs Z which

Re: [PATCH] Fix/Rewrite of the mipsnet driver

2007-10-28 Thread Ralf Baechle
On Sun, Oct 28, 2007 at 04:38:46AM +, Thiemo Seufer wrote: Hello All, currently the mipsnet driver fails after transmitting a number of packages because SKBs are allocated but never freed. I fixed that and coudn't refrain from removing the most egregious warts. - mipsnet.h folded

[PATCH 00/04] pull request for 'upstream-jeff' branch

2007-10-28 Thread Francois Romieu
Please pull from branch 'upstream-jeff' in repository git://git.kernel.org/pub/scm/linux/kernel/git/romieu/netdev-2.6.git upstream-jeff to get the changes below. Distance from 'master' (96fd4cd3e40e240f0c385af87f58e74da8b7099a) -

[PATCH 01/04] r8169: napi config

2007-10-28 Thread Francois Romieu
Don't call napi_disable if not configured and make sure that any misuse of napi_xxx in future fails with a compile error. Signed-off-by: Stephen Hemminger [EMAIL PROTECTED] Signed-off-by: Francois Romieu [EMAIL PROTECTED] Cc: Edward Hsu [EMAIL PROTECTED] --- drivers/net/r8169.c |4 1

[PATCH 02/04] r8169: remove poll_locked logic

2007-10-28 Thread Francois Romieu
Disabling napi polling early is well enough. Signed-off-by: Stephen Hemminger [EMAIL PROTECTED] Signed-off-by: Francois Romieu [EMAIL PROTECTED] Cc: Edward Hsu [EMAIL PROTECTED] --- drivers/net/r8169.c | 12 1 files changed, 4 insertions(+), 8 deletions(-) diff --git

[PATCH 03/04] ipg: missing Kconfig dependency

2007-10-28 Thread Francois Romieu
Fix for the error below while linking vmlinux: [...] drivers/built-in.o: In function `ipg_ioctl': drivers/net/ipg.c:2148: undefined reference to `generic_mii_ioctl' drivers/built-in.o: In function `ipg_get_settings': drivers/net/ipg.c:2181: undefined reference to `mii_ethtool_gset'

[PATCH 04/04] ipg: Kconfig whitepaces/tab damages

2007-10-28 Thread Francois Romieu
Signed-off-by: Francois Romieu [EMAIL PROTECTED] Cc: Avuton Olrich [EMAIL PROTECTED] Cc: Jesse Huang [EMAIL PROTECTED] Cc: Sorbica Shieh [EMAIL PROTECTED] --- drivers/net/Kconfig | 12 ++-- 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/net/Kconfig

Re: [PATCH] proc_fs.h redux

2007-10-28 Thread Roman Zippel
Hi, On Sunday 28 October 2007, Russell King wrote: On Sat, Oct 27, 2007 at 03:40:04PM -0700, Joe Perches wrote: and forward declarations of struct proc_dir_entry; struct file_operations; As a general rule, I think it better to use includes than use naked forward declarations. If

RE: [PATCH 1/14 v2] nes: module and device initialization

2007-10-28 Thread Glenn Grundstrom
OK, on a process level, my plan is to pull the current driver into a neteffect branch in my git tree with the intention of merging it for 2.6.25. I'll let you know when that's ready (probably early next week). I'll probably do some cleanups there, and you can send me cleanup/fix patches

Re: [linux-pm] QUESTION: How to fix race between .suspend routine and watchdog timer

2007-10-28 Thread Maxim Levitsky
On Sunday 28 October 2007 00:24:10 Alan Stern wrote: On Sat, 27 Oct 2007, Maxim Levitsky wrote: Use del_timer_sync(). It guarantees that when it returns, the timer will be stopped and the timer routine will no longer be running on any CPU. Even if the timer re-enables itself,

Re: [PATCH] Fix/Rewrite of the mipsnet driver

2007-10-28 Thread Thiemo Seufer
Ralf Baechle wrote: On Sun, Oct 28, 2007 at 04:38:46AM +, Thiemo Seufer wrote: Hello All, currently the mipsnet driver fails after transmitting a number of packages because SKBs are allocated but never freed. I fixed that and coudn't refrain from removing the most egregious

[RFC][BNX2X] .h files rewrite

2007-10-28 Thread Eliezer Tamir
Dave, Please take a look and tell me if this is better than the last attempt. Main changes: * Rewrite of the init code: 1. Full rewrite of the generated code into something easier to read. 2. Removed zeroes by adding a zero command. 3. Big chucks of data are read by the

Re: [PATCH] - trivial - Improve appletalk checksum calculation

2007-10-28 Thread Urs Thuermann
Herbert Xu [EMAIL PROTECTED] writes: But your code differs significantly from Stephen's version. However, if it is correct it does look like a good improvement. So please write a simple test program. It can't that bad since there are only 65536 values to test :) I think the code is simple

Re: [PATCH] Fix/Rewrite of the mipsnet driver

2007-10-28 Thread Stephen Hemminger
On Sun, 28 Oct 2007 20:03:08 + Thiemo Seufer [EMAIL PROTECTED] wrote: Ralf Baechle wrote: On Sun, Oct 28, 2007 at 04:38:46AM +, Thiemo Seufer wrote: Hello All, currently the mipsnet driver fails after transmitting a number of packages because SKBs are allocated but

Re: [PATCH] Fix/Rewrite of the mipsnet driver

2007-10-28 Thread Thiemo Seufer
Stephen Hemminger wrote: On Sun, 28 Oct 2007 20:03:08 + Thiemo Seufer [EMAIL PROTECTED] wrote: Ralf Baechle wrote: On Sun, Oct 28, 2007 at 04:38:46AM +, Thiemo Seufer wrote: Hello All, currently the mipsnet driver fails after transmitting a number of packages

Re: [PATCH] Fix/Rewrite of the mipsnet driver

2007-10-28 Thread Ralf Baechle
On Sun, Oct 28, 2007 at 01:22:04PM -0700, Stephen Hemminger wrote: -#define MIPSNET_INTCTL_TXDONE ((uint32_t)(1 0)) -#define MIPSNET_INTCTL_RXDONE ((uint32_t)(1 1)) -#define MIPSNET_INTCTL_TESTBIT((uint32_t)(1 31)) -#define MIPSNET_INTCTL_ALLSOURCES

Re: 2.6.24-rc1 kills onboard r8169 (rtl8111b) NIC

2007-10-28 Thread Francois Romieu
Sergey S. Kostyliov [EMAIL PROTECTED] : [...] After booting 2.6.24-rc1 on my home machine I lost onboard rtl8111b network card. It is neither functional nor even visible by lspci anymore. Moreover rebooting to previous 2.6.23-rc9 didn't fix it - card was still not available. It should

Re: [PATCH] - trivial - Improve appletalk checksum calculation

2007-10-28 Thread Joe Perches
On Sun, 2007-10-28 at 21:18 +0100, Urs Thuermann wrote: I have written a short test program to show this and tested it on i686 (gcc-4.2.2) and powerpc (gcc-3.4.5) without optimization and with -O6. Thanks for that. BTW, shouldn't unsigned long be replaced by unsigned int to avoid 64-bit

Re: [PATCH] ehea: add kexec support

2007-10-28 Thread Michael Ellerman
On Fri, 2007-10-26 at 14:37 +0200, Jan-Bernd Themann wrote: eHEA resources that are allocated via H_CALLs have a unique identifier each. These identifiers are necessary to free the resources. A reboot notifier is used to free all eHEA resources before the indentifiers get lost, i.e before

Re: [PATCH] r8169: don't call napi_disable if not doing NAPI

2007-10-28 Thread Romano Giannetti
On Fri, 2007-10-26 at 11:33 -0700, Stephen Hemminger wrote: Don't call napi_disable if not configured. And make sure that any misuse of napi_xxx in future fails with a compile error. Will test as soon as possible (been without internet in the week end). Thanks. As a bonus, I tried more

HFSC dangerous behaviour (not a bug)

2007-10-28 Thread Denys
Hi All During testing i found very strange thing. After applying even example shaper: http://linux-ip.net/tc/hfsc.en/ - # Example from Figure 1. tc qdisc add dev eth0 root handle 1: hfsc tc class add dev eth0 parent 1: classid 1:1 hfsc sc rate 1000kbit ul

HFSC dangerous behaviour (not a bug)

2007-10-28 Thread Denys
Additionally, it doesn't show rate in stats (so it is difficult to measure, how much is really using each class). qdisc hfsc 1: root default 200 Sent 1392761062 bytes 965768 pkt (dropped 52, overlimits 1620539 requeues 0) rate 0bit 0pps backlog 0b 0p requeues 0 qdisc bfifo 200: parent 1:200

Bonding in active-backup mode with arp monitoring on Xen

2007-10-28 Thread Tsutomu Fujii
Hi. I was trying to use bonding from a guest OS on xen in active-backup mode with arp monitoring, but the virtual server can't communicate to the peer server via bonding interface. The network structure is the following. +---+ |virtual I/F(eth0) | - have

Re: [PATCH] - trivial - Improve appletalk checksum calculation

2007-10-28 Thread Joe Perches
On Sun, 2007-10-28 at 14:01 -0700, Joe Perches wrote: probably unsigned long should be u16. Or not. x86 u16 performs much worse than u32 (~ 50% worse) - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at

[PATCH 2.6.24-rc] [IPV6] NDISC: Fix setting base_reachable_time_ms variable.

2007-10-28 Thread YOSHIFUJI Hideaki / 吉藤英明
This bug was introduced by the commit d12af679bcf8995a237560bdf7a4d734f8df5dbb. Signed-off-by: YOSHIFUJI Hideaki [EMAIL PROTECTED] --- diff --git a/net/ipv6/ndisc.c b/net/ipv6/ndisc.c index 20cfc90..36f7dbf 100644 --- a/net/ipv6/ndisc.c +++ b/net/ipv6/ndisc.c @@ -1670,7 +1670,7 @@ int

RE: [PATCH] ucc_geth: add support for netpoll

2007-10-28 Thread Li Yang-r58472
-Original Message- From: Anton Vorontsov [mailto:[EMAIL PROTECTED] Sent: Saturday, October 27, 2007 10:38 PM To: Sergei Shtylyov Cc: Anton Vorontsov; netdev@vger.kernel.org; Li Yang-r58472; [EMAIL PROTECTED]; [EMAIL PROTECTED] Subject: Re: [PATCH] ucc_geth: add support for netpoll

Re: [UDP6]: Restore sk_filter optimisation

2007-10-28 Thread Mitsuru Chinen
Hello Herbert, Let me ask a question about this patch. After this patch was applied, 2 of the protocol stack behaviors were changed when it receives a UDP datagram with broken checksum: 1. udp6InDatagrams is incremented instead of udpInErrors 2. In userland, recvfrom() replies an error with

Re: [UDP6]: Restore sk_filter optimisation

2007-10-28 Thread YOSHIFUJI Hideaki / 吉藤英明
In article [EMAIL PROTECTED] (at Mon, 29 Oct 2007 15:33:20 +0900), Mitsuru Chinen [EMAIL PROTECTED] says: Hello Herbert, Let me ask a question about this patch. After this patch was applied, 2 of the protocol stack behaviors were changed when it receives a UDP datagram with broken