Re: [PATCH] net: Marking struct pernet_operations __net_initdata was inappropriate

2007-10-26 Thread Eric W. Biederman
David Miller <[EMAIL PROTECTED]> writes: > > Applied, thanks Eric. > > Although juding by his comments I though that Denis had different > plans in mind to fix this. He might. Somehow I wasn't on that thread so I missed it until after I sent this patch. Reading through that thread again it look

Re: [PATCH] netns: Fix get_net_ns_by_pid

2007-10-26 Thread David Miller
From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Fri, 26 Oct 2007 17:29:41 -0600 > > The pid namespace patches changed the semantics of > find_task_by_pid without breaking the compile resulting > in get_net_ns_by_pid doing the wrong thing. > > So switch to using the intended find_task_by_vpid.

Re: [PATCH] net: Marking struct pernet_operations __net_initdata was inappropriate

2007-10-26 Thread David Miller
From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Fri, 26 Oct 2007 17:45:33 -0600 > > It is not safe to to place struct pernet_operations in a special section. > We need struct pernet_operations to last until we call > unregister_pernet_subsys. > Which doesn't happen until module unload. > > So

Re: [2.6 patch] net/ipv{4,6}/esp{4,6}.c must #include

2007-10-26 Thread David Miller
From: Adrian Bunk <[EMAIL PROTECTED]> Date: Sat, 27 Oct 2007 04:34:48 +0200 > This patch fixes the following compile errors in some configurations: > > <-- snip --> > > ... > CC net/ipv4/esp4.o > /home/bunk/linux/kernel-2.6/git/linux-2.6/net/ipv4/esp4.c: In function > 'esp_output': > /

Re: [PATCH] tcp v6: fix softnet build breakage

2007-10-26 Thread David Miller
From: Jeff Garzik <[EMAIL PROTECTED]> Date: Fri, 26 Oct 2007 23:42:40 -0400 > > net/ipv6/tcp_ipv6.c: In function 'tcp_v6_rcv': > net/ipv6/tcp_ipv6.c:1736: error: implicit declaration of function > 'get_softnet_dma' > net/ipv6/tcp_ipv6.c:1736: warning: assignment makes pointer from integer > witho

[PATCH] tcp v6: fix softnet build breakage

2007-10-26 Thread Jeff Garzik
net/ipv6/tcp_ipv6.c: In function 'tcp_v6_rcv': net/ipv6/tcp_ipv6.c:1736: error: implicit declaration of function 'get_softnet_dma' net/ipv6/tcp_ipv6.c:1736: warning: assignment makes pointer from integer without a cast Signed-off-by: Jeff Garzik <[EMAIL PROTECTED]> --- diff --git a/net/ipv6/tcp_

[2.6 patch] net/ipv{4,6}/esp{4,6}.c must #include

2007-10-26 Thread Adrian Bunk
This patch fixes the following compile errors in some configurations: <-- snip --> ... CC net/ipv4/esp4.o /home/bunk/linux/kernel-2.6/git/linux-2.6/net/ipv4/esp4.c: In function 'esp_output': /home/bunk/linux/kernel-2.6/git/linux-2.6/net/ipv4/esp4.c:113: error: implicit declaration of f

Re: [2.6 patch] always export sysctl_{r,w}mem_max

2007-10-26 Thread Eric W. Biederman
David Miller <[EMAIL PROTECTED]> writes: > From: Rick Jones <[EMAIL PROTECTED]> > Date: Fri, 26 Oct 2007 16:31:47 -0700 > >> Eric W. Biederman wrote: >> > Adrian Bunk <[EMAIL PROTECTED]> writes: >> > >> > >> >>This patch fixes the following build error with CONFIG_SYSCTL=n: >> >> >> >><-- snip

Re: [2.6 patch] always export sysctl_{r,w}mem_max

2007-10-26 Thread David Miller
From: Rick Jones <[EMAIL PROTECTED]> Date: Fri, 26 Oct 2007 16:46:36 -0700 > David Miller wrote: > > If DLM really wants minimum, it can use SO_SNDBUFFORCE and > > SO_RCVBUFFORCE socket options and use whatever limits it > > likes. > > > > But even this is questionable. > > Drift... > > Is that

[PATCH] net: Marking struct pernet_operations __net_initdata was inappropriate

2007-10-26 Thread Eric W. Biederman
It is not safe to to place struct pernet_operations in a special section. We need struct pernet_operations to last until we call unregister_pernet_subsys. Which doesn't happen until module unload. So marking struct pernet_operations is a disaster for modules in two ways. - We discard it before we

Re: [2.6 patch] always export sysctl_{r,w}mem_max

2007-10-26 Thread Rick Jones
David Miller wrote: If DLM really wants minimum, it can use SO_SNDBUFFORCE and SO_RCVBUFFORCE socket options and use whatever limits it likes. But even this is questionable. Drift... Is that something netperf should be using though? Right now it uses the regular SO_[SND|RCV]BUF calls and is

Re: [NETNS] Oops in register_pernet_operations() with CONFIG_NET_NS=n

2007-10-26 Thread Eric W. Biederman
David Miller <[EMAIL PROTECTED]> writes: > Thanks for doing this. > > But this appears to be still discussed, so I'll give > Denis and others another day to work out the fix they > want to include. At this point I think all that really needs to happen is to remove __net_initdata. The function at

Re: [2.6 patch] always export sysctl_{r,w}mem_max

2007-10-26 Thread David Miller
From: Rick Jones <[EMAIL PROTECTED]> Date: Fri, 26 Oct 2007 16:31:47 -0700 > Eric W. Biederman wrote: > > Adrian Bunk <[EMAIL PROTECTED]> writes: > > > > > >>This patch fixes the following build error with CONFIG_SYSCTL=n: > >> > >><-- snip --> > >> > >>... > >>ERROR: "sysctl_rmem_max" [fs/dlm

Re: [2.6 patch] always export sysctl_{r,w}mem_max

2007-10-26 Thread Rick Jones
Eric W. Biederman wrote: Adrian Bunk <[EMAIL PROTECTED]> writes: This patch fixes the following build error with CONFIG_SYSCTL=n: <-- snip --> ... ERROR: "sysctl_rmem_max" [fs/dlm/dlm.ko] undefined! ERROR: "sysctl_wmem_max" [drivers/net/rrunner.ko] undefined! ERROR: "sysctl_rmem_max" [driv

[PATCH] netns: Fix get_net_ns_by_pid

2007-10-26 Thread Eric W. Biederman
The pid namespace patches changed the semantics of find_task_by_pid without breaking the compile resulting in get_net_ns_by_pid doing the wrong thing. So switch to using the intended find_task_by_vpid. Combined with Denis' earlier patch to make netlink traffic fully synchronous the inadvertent r

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

2007-10-26 Thread Roland Dreier
OK, a couple quick review comments and a process comment too: - First step in the driver is to kill off a lot of the #ifdefs: > +#ifdef IRQF_SHARED The upstream driver really shouldn't have compatibility gunk for older kernels... just make it build against the kernel it's in. > +#ifdef OFED_

Re: [2.6 patch] always export sysctl_{r,w}mem_max

2007-10-26 Thread Eric W. Biederman
Adrian Bunk <[EMAIL PROTECTED]> writes: > This patch fixes the following build error with CONFIG_SYSCTL=n: > > <-- snip --> > > ... > ERROR: "sysctl_rmem_max" [fs/dlm/dlm.ko] undefined! > ERROR: "sysctl_wmem_max" [drivers/net/rrunner.ko] undefined! > ERROR: "sysctl_rmem_max" [drivers/net/rrunner

[2.6 patch] always export sysctl_{r,w}mem_max

2007-10-26 Thread Adrian Bunk
This patch fixes the following build error with CONFIG_SYSCTL=n: <-- snip --> ... ERROR: "sysctl_rmem_max" [fs/dlm/dlm.ko] undefined! ERROR: "sysctl_wmem_max" [drivers/net/rrunner.ko] undefined! ERROR: "sysctl_rmem_max" [drivers/net/rrunner.ko] undefined! make[2]: *** [__modpost] Error 1 <--

Re: Files, sockets, and closing

2007-10-26 Thread Al Viro
On Fri, Oct 26, 2007 at 03:09:01PM -0700, Stephen Hemminger wrote: > > close() from another thread is not a way to abort blocked accept(). Never > > promised to be that. Just as close() from another thread is not a way to > > abort blocked write() or read() or sendmsg() or... > > The problem is

Re: [PATCH] skye/skge: sparse fix - data can't ever be bigger than LONG_MAX / HZ

2007-10-26 Thread Kok, Auke
Stephen Hemminger wrote: > On Fri, 26 Oct 2007 15:10:28 -0700 > Auke Kok <[EMAIL PROTECTED]> wrote: > >> Trivial replacement - use INT_MAX instead here. >> >> Signed-off-by: Auke Kok <[EMAIL PROTECTED]> >> Cc: [EMAIL PROTECTED] > > Acked-by: Stephen Hemminger <[EMAIL PROTECTED]> > > Sure that wo

[PATCH] pcnet: fix sparse triviality

2007-10-26 Thread Auke Kok
Since data can never exceed u32, it can't even be larger than LONG_MAX/HZ. Signed-off-by: Auke Kok <[EMAIL PROTECTED]> Cc: [EMAIL PROTECTED] --- drivers/net/pcnet32.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/net/pcnet32.c b/drivers/net/pcnet32.c index

Re: [PATCH] skye/skge: sparse fix - data can't ever be bigger than LONG_MAX / HZ

2007-10-26 Thread Stephen Hemminger
On Fri, 26 Oct 2007 15:10:28 -0700 Auke Kok <[EMAIL PROTECTED]> wrote: > Trivial replacement - use INT_MAX instead here. > > Signed-off-by: Auke Kok <[EMAIL PROTECTED]> > Cc: [EMAIL PROTECTED] Acked-by: Stephen Hemminger <[EMAIL PROTECTED]> Sure that works. Note: original code was copied from e

[PATCH] skye/skge: sparse fix - data can't ever be bigger than LONG_MAX / HZ

2007-10-26 Thread Auke Kok
Trivial replacement - use INT_MAX instead here. Signed-off-by: Auke Kok <[EMAIL PROTECTED]> Cc: [EMAIL PROTECTED] --- drivers/net/sk98lin/skethtool.c |4 ++-- drivers/net/skge.c |8 drivers/net/sky2.c |8 3 files changed, 10 insertions(+),

Re: Files, sockets, and closing

2007-10-26 Thread Stephen Hemminger
On Fri, 26 Oct 2007 22:45:13 +0100 Al Viro <[EMAIL PROTECTED]> wrote: > On Fri, Oct 26, 2007 at 02:03:19PM -0700, Stephen Hemminger wrote: > > Looking at this bug: > > http://bugzilla.kernel.org/show_bug.cgi?id=9149 > > > > Exposes some rather deep issues in the filesystem/socket/inet/tcp > > lay

Re: [PATCH] Remove pointless casts from void pointers,

2007-10-26 Thread Lennert Buytenhek
On Fri, Oct 26, 2007 at 05:40:22AM -0400, Jeff Garzik wrote: > arch/arm/mach-pxa/ssp.c|2 +- > arch/arm/mach-s3c2410/usb-simtec.c |2 +- > arch/arm/plat-omap/mailbox.c |2 +- FWIW Acked-by: Lennert Buytenhek <[EMAIL PROTECTED]> - To unsubscri

Re: Files, sockets, and closing

2007-10-26 Thread Al Viro
On Fri, Oct 26, 2007 at 02:03:19PM -0700, Stephen Hemminger wrote: > Looking at this bug: > http://bugzilla.kernel.org/show_bug.cgi?id=9149 > > Exposes some rather deep issues in the filesystem/socket/inet/tcp > layering. It seems that sys_close() zaps the file table entry, but > since each thread

Re: [PATCH v2] 2.6.24-rc1 remove architecture warning compiling fealnx on ia64

2007-10-26 Thread Rick Jones
Overall that poor driver is more than a little bit ugly :) Well, given some of the code I've put into netperf over the years I wasn't going to be the first to cast a stone :) but I won't disagree with the assesment :) Looking at the large chain of highly-similar values, I would tend to prefe

[PATCH v2] 2.6.24-rc1 remove architecture warning compiling fealnx on ia64

2007-10-26 Thread Jeff Garzik
Rick Jones wrote: The likelihood of one of these being used on an ia64 box is epsilon but it would still be nice to get rid of the warning: #warning Processor architecture undefined! when compiling on same. So, pick some likely similar architectures and follow those leads. Compile tested onl

[PATCH] 2.6.24-rc1 remove architecture warning compiling fealnx on ia64

2007-10-26 Thread Rick Jones
The likelihood of one of these being used on an ia64 box is epsilon but it would still be nice to get rid of the warning: #warning Processor architecture undefined! when compiling on same. So, pick some likely similar architectures and follow those leads. Compile tested only. Signed-off-by: Ri

Files, sockets, and closing

2007-10-26 Thread Stephen Hemminger
Looking at this bug: http://bugzilla.kernel.org/show_bug.cgi?id=9149 Exposes some rather deep issues in the filesystem/socket/inet/tcp layering. It seems that sys_close() zaps the file table entry, but since each thread has a separate reference, the actual tcp_close() doesn't happen until the last

Re: [0/3] Distributed storage. Mirror algo extension for automatic recovery.

2007-10-26 Thread Andrew Morton
On Thu, 18 Oct 2007 23:17:41 +0400 Evgeniy Polyakov <[EMAIL PROTECTED]> wrote: > I'm pleased to announce sixth release of the distributed storage > subsystem, which allows to form a storage on top of remote and local > nodes, which in turn can be exported to another storage as a node to > form tre

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

2007-10-26 Thread Francois Romieu
Stephen Hemminger <[EMAIL PROTECTED]> : > Don't call napi_disable if not configured. > And make sure that any misuse of napi_xxx in future fails > with a compile error. Disable napi polling early and remove the useless poll_locked logic. > > Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]>

Re: [PATCH 1] net: fix and typo's

2007-10-26 Thread Kok, Auke
Roel Kluin wrote: > Kok, Auke wrote: > >> Ack this e1000e change here! > >> (PS since there was only 1 netdriver patch here and the rest is wireless, I >> would >> have suggested splitting this patch up in two and sending them to the >> wireless >> maintainer and netdevice maintainer separately

Re: [PATCH 1] net: fix and typo's

2007-10-26 Thread Roel Kluin
Kok, Auke wrote: > Ack this e1000e change here! > (PS since there was only 1 netdriver patch here and the rest is wireless, I > would > have suggested splitting this patch up in two and sending them to the wireless > maintainer and netdevice maintainer separately. But I'm sure this will get > p

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

2007-10-26 Thread Stephen Hemminger
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]> --- a/drivers/net/r8169.c 2007-10-24 21:38:43.0 -0700 +++ b/drivers/net/r8169.c 2007-10-26 11:27:02.0

[PATCH 5, last for now] net: unlock before return

2007-10-26 Thread Roel Kluin
To David Miller: this was the one you asked to send. I removed the incorrect one (net/bridge/netfilter/ebtables.c). Note the drivers/net/cris/eth_v10.c, which you may not yet have reviewed, But I checked, and there was not a similar comment above the header of the function :) Roel -- unlock

Re: [PATCH 1] net: fix and typo's

2007-10-26 Thread Stephen Hemminger
On Fri, 26 Oct 2007 19:07:10 +0200 Roel Kluin <[EMAIL PROTECTED]> wrote: > A few patches with changes to net code. I have sent these to the lkml > previously, but they were not yet merged. I am fairly new to kernel > programming, so it is possible that I make some mistakes. I'll explain my > rati

[PATCH 4] Unlock before BUG, but preserve normal operation

2007-10-26 Thread Roel Kluin
I am a bit uncertain about this, so if you have comments on this, it is much appreciated. The rationale: in the default case the BUG() kills the current process, but with the lock still held this causes the system to hang; therefore the unlock before the BUG(). When CONFIG_BUG is disabled - e.g.

Re: 2.6.24-rc1 fails with lockup - /sbin/ifconfig / inet_ioctl() / dev_close() / rtl8169_down()

2007-10-26 Thread Ingo Molnar
* Stephen Hemminger <[EMAIL PROTECTED]> wrote: > > hm, from your log it appears that lockdep did not find anything, > > still the hang does trigger. > > > > it's /sbin/ifconfig and inet_ioctl() / dev_close() / rtl8169_down() > > that seems to be hanging. I've extracted the relevant backtrace

Re: [PATCH 1] net: fix and typo's

2007-10-26 Thread Kok, Auke
Roel Kluin wrote: > A few patches with changes to net code. I have sent these to the lkml > previously, but they were not yet merged. I am fairly new to kernel > programming, so it is possible that I make some mistakes. I'll explain my > rationale, please nack if incorrect, an additional bit of ex

Re: [PATCH] dev_change_name: ignore changes to same name

2007-10-26 Thread Rick Jones
David Miller wrote: From: Stephen Hemminger <[EMAIL PROTECTED]> Date: Wed, 24 Oct 2007 06:44:45 -0700 Prevent error/backtrace from dev_rename() when changing name of network device to the same name. This is a common situation with udev and other scripts that bind addr to device. Signed-off-by

Re: stateless 1:1 NAT

2007-10-26 Thread Florin Andrei
Patrick McHardy wrote: Florin Andrei wrote: OpenBSD 4.1 as a firewall fails even worse in this test case (it freezes instantly). OpenBSD 4.2 works fine under the UDP flood, as if nothing happened. And Linux 2.6.23? :) Same as 2.6.18, actually maybe a little bit worse than .18: the current

[PATCH 1] net: fix and typo's

2007-10-26 Thread Roel Kluin
A few patches with changes to net code. I have sent these to the lkml previously, but they were not yet merged. I am fairly new to kernel programming, so it is possible that I make some mistakes. I'll explain my rationale, please nack if incorrect, an additional bit of explanation is appreciated e

[PATCH 3] isdn: incorrect assignments in if

2007-10-26 Thread Roel Kluin
incorrect assignments in if Signed-off-by: Roel Kluin <[EMAIL PROTECTED]> --- diff --git a/drivers/isdn/hisax/elsa.c b/drivers/isdn/hisax/elsa.c index 948a9b2..ed610ed 100644 --- a/drivers/isdn/hisax/elsa.c +++ b/drivers/isdn/hisax/elsa.c @@ -883,7 +883,7 @@ setup_elsa_isa(struct IsdnC

[PATCH 2] isdn: fix 'and' typo's

2007-10-26 Thread Roel Kluin
This patch is similar to the former, but affects isdn drivers. also note these changes in drivers/isdn/i4l/isdn_ttyfax.c: - if (!info->faxonline & 1) /* not outgoing connection */ + if (!info->faxonline) /* not outgoing connection */ ... - i

Re: Bad TCP checksum error

2007-10-26 Thread Rick Jones
Checksum Offload on the NIC(s) can complicate things. First, if you are tracing on the sender, the tracepoint is before the NIC has computed the full checksum. IIRC only a partial checksum is passed-down to the NIC when CKO is in use. So, making certain your trace is from the "wire" or the re

Re: 2.6.24-rc1 fails with lockup - /sbin/ifconfig / inet_ioctl() / dev_close() / rtl8169_down()

2007-10-26 Thread Stephen Hemminger
On Fri, 26 Oct 2007 08:37:33 +0200 Ingo Molnar <[EMAIL PROTECTED]> wrote: > > * Romano Giannetti <[EMAIL PROTECTED]> wrote: > > > > Does this help? > > > > I tried this, but although I have the D-state processes, I cannot see > > any debug trace now. Results are at: > > > > http://www.dea.ica

[PATCH v4.3] FEC - fast ethernet controller for mpc52xx

2007-10-26 Thread Domen Puncer
On 26/10/07 07:18 -0700, Dale Farnsworth wrote: > On Fri, Oct 26, 2007 at 01:59:09PM +0200, Domen Puncer wrote: > > +static irqreturn_t mpc52xx_fec_tx_interrupt(int irq, void *dev_id) > > +{ > > + struct net_device *dev = dev_id; > > + struct mpc52xx_fec_priv *priv = netdev_priv(dev); > > + > >

Re: Bad TCP checksum error

2007-10-26 Thread Gaurav Aggarwal
Hi All, I have dome some changes in the code. Now instead of calculating the TCP checksum from scratch, I was just recalculating it by incremental update as mentioned in RFC 1624. Good thing is that now I was able to get the correct IP header checksum but still TCP checksum value is corrupted. Int

Re: [PATCH] net: Saner thash_entries default with much memory

2007-10-26 Thread akepner
On Fri, Oct 26, 2007 at 05:21:31PM +0200, Jean Delvare wrote: > > This is just one way to limit the hash size, there are others; I am not > familiar enough with the TCP code to decide which is best. Thus, I > would welcome the proposals of alternatives. > Yeah, the existing way of sizing the

Re: [PATCH 2/2] [CRYPTO] users: Fix up scatterlist conversion errors

2007-10-26 Thread Vlad Yasevich
Herbert Xu wrote: > [CRYPTO] users: Fix up scatterlist conversion errors > > This patch fixes the errors made in the users of the crypto layer during > the sg_init_table conversion. It also adds a few conversions that were > missing altogether. > > Signed-off-by: Herbert Xu <[EMAIL PROTECTED]> >

RE: [2.6 patch] unexport softnet_data

2007-10-26 Thread Nelson, Shannon
>-Original Message- >From: David Miller [mailto:[EMAIL PROTECTED] > >From: Adrian Bunk <[EMAIL PROTECTED]> >Date: Wed, 24 Oct 2007 18:24:25 +0200 > >> The EXPORT_PER_CPU_SYMBOL(softnet_data) is no longer used. >> >> Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> > >I wanted to apply this,

Re: [PATCH] net: Saner thash_entries default with much memory

2007-10-26 Thread Andi Kleen
On Fri, Oct 26, 2007 at 05:21:31PM +0200, Jean Delvare wrote: > I know that /proc/net/tcp is > deprecated in favor of tcp_diag, however at the moment netstat only > knows of the former. Even tcp_diag will be slow when all slots are dumped. It's a fundamental problem of the data structure. /proc ha

[PATCH] net: Saner thash_entries default with much memory

2007-10-26 Thread Jean Delvare
On systems with a very large amount of memory, the heuristics in alloc_large_system_hash() result in a very large TCP established hash table: 16 millions of entries for a 128 GB ia64 system. This makes reading from /proc/net/tcp pretty slow (well over a second) and as a result netstat is slow on th

Re: [PATCH] Remove pointless casts from void pointers,

2007-10-26 Thread Holger Schurig
> drivers/net/wireless/libertas/if_cs.c |2 +- ACK - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

[PATCH 2/2] [CRYPTO] users: Fix up scatterlist conversion errors

2007-10-26 Thread Herbert Xu
[CRYPTO] users: Fix up scatterlist conversion errors This patch fixes the errors made in the users of the crypto layer during the sg_init_table conversion. It also adds a few conversions that were missing altogether. Signed-off-by: Herbert Xu <[EMAIL PROTECTED]> --- drivers/crypto/padlock-sha.

[PATCH 1/2] [CRYPTO] tcrypt: Move sg_init_table out of timing loops

2007-10-26 Thread Herbert Xu
[CRYPTO] tcrypt: Move sg_init_table out of timing loops This patch moves the sg_init_table out of the timing loops for hash algorithms so that it doesn't impact on the speed test results. Signed-off-by: Herbert Xu <[EMAIL PROTECTED]> --- crypto/tcrypt.c | 20 ++-- 1 files chan

[0/2] [CRYPTO] users: Fix up remaining sg issues

2007-10-26 Thread Herbert Xu
Hi Dave: Here's a couple of patches to fix up the crypto users with respect to the scatterlist change. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <[EMAIL PROTECTED]> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbe

Re: [PATCH] Remove pointless casts from void pointers,

2007-10-26 Thread Josh Boyer
On Fri, 26 Oct 2007 05:40:22 -0400 (EDT) Jeff Garzik <[EMAIL PROTECTED]> wrote: > mostly in and around irq handlers. > > Signed-off-by: Jeff Garzik <[EMAIL PROTECTED]> > --- > drivers/serial/uartlite.c |2 +- Acked-by: Josh Boyer <[EMAIL PROTECTED] - To unsubscribe from thi

Re: [PATCH v4.2] FEC - fast ethernet controller for mpc52xx

2007-10-26 Thread Dale Farnsworth
On Fri, Oct 26, 2007 at 01:59:09PM +0200, Domen Puncer wrote: > +static irqreturn_t mpc52xx_fec_tx_interrupt(int irq, void *dev_id) > +{ > + struct net_device *dev = dev_id; > + struct mpc52xx_fec_priv *priv = netdev_priv(dev); > + > + spin_lock(&priv->lock); > + > + while (bcom_buf

Re: [2.6.25 patch] the planned eepro100 removal

2007-10-26 Thread David Acker
Kok, Auke wrote: Jeff Garzik wrote: Bill Davidsen wrote: Adrian Bunk wrote: This patch contains the planned removal of the eepro100 driver. Are the e100 people satisfied that e100 now handles all known cases? I Nope. There are still e100 work outstanding that means we cannot kill eepro100.

Re: [PATCH] Remove pointless casts from void pointers,

2007-10-26 Thread Dmitry Torokhov
On 10/26/07, Jeff Garzik <[EMAIL PROTECTED]> wrote: > drivers/input/touchscreen/h3600_ts_input.c |4 ++-- Acked-by: Dmitry Torokhov <[EMAIL PROTECTED]> -- Dmitry - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo i

[PATCH] ehea: add kexec support

2007-10-26 Thread Jan-Bernd Themann
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 kexec starts a new kernel. Signed-off-by: Jan-Bernd Themann <[EMAIL

[IFGROUPv5 3/3] Netfilter Interface group match

2007-10-26 Thread Laszlo Attila Toth
Interface group values can be checked on both input and output interfaces. Signed-off-by: Laszlo Attila Toth <[EMAIL PROTECTED]> --- include/linux/netfilter/xt_ifgroup.h | 18 + net/netfilter/Kconfig| 16 + net/netfilter/Makefile |1 + net/netfilter/x

[IFGROUPv5 2/3] Interface group: core (netlink) part

2007-10-26 Thread Laszlo Attila Toth
Interface groups let handle different interfaces together especially in netfilter modules. Modified net device structure and netlink interface. Signed-off-by: Laszlo Attila Toth <[EMAIL PROTECTED]> --- include/linux/if_link.h |2 ++ include/linux/netdevice.h |2 ++ net/core/rtnetlink.c

[IFGROUPv5 iptables] Interface group match

2007-10-26 Thread Laszlo Attila Toth
Interface group values can be checked on both input and output interfaces with optional mask. Signed-off-by: Laszlo Attila Toth <[EMAIL PROTECTED]> --- Makefile |2 libip6t_ifgroup.man | 36 +++ libipt_ifgroup.man | 36 +++ libxt_ifgroup.c

[IFGROUPv5 iproute 2/2] Interface group as new ip link option

2007-10-26 Thread Laszlo Attila Toth
Interfaces can be grouped and each group has an unique positive integer ID. It can be set via ip link. Symbolic names can be specified in /etc/iproute2/rt_ifgroup. Signed-off-by: Laszlo Attila Toth <[EMAIL PROTECTED]> --- include/linux/if_link.h |2 + include/rt_names.h |2 + ip/ipad

[IFGROUPv5 1/3] rtnetlink: setlink changes are unprotected; with single notification

2007-10-26 Thread Laszlo Attila Toth
In do_setlink the device changes don't need to be protected. Notification is sent at the end of the function once if any modification occured and once if an address has been changed. Signed-off-by: Laszlo Attila Toth <[EMAIL PROTECTED]> --- net/core/rtnetlink.c | 32

[IFGROUPv5 0/3 (+3)] Interface group patches

2007-10-26 Thread Laszlo Attila Toth
Hi Dave, This is the 5th version of our interface group patches. The first patch is a fix in the rtnl socket interface. An u_int32_t member was added to net devices indicating the interface group number of the device which can be get/set via netlink. The xt_ifgroup netfilter match is for checki

[IFGROUPv5 iproute 1/2] Added IFLA_NET_NS_PID as in kernel v2.6.24-rc1

2007-10-26 Thread Laszlo Attila Toth
Signed-off-by: Laszlo Attila Toth <[EMAIL PROTECTED]> --- include/linux/if_link.h |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/linux/if_link.h b/include/linux/if_link.h index 23b3a8e..c948395 100644 --- a/include/linux/if_link.h +++ b/include/linux/if_link.h @@

Re: [PATCH] Remove pointless casts from void pointers,

2007-10-26 Thread John W. Linville
On Fri, Oct 26, 2007 at 05:40:22AM -0400, Jeff Garzik wrote: > mostly in and around irq handlers. > > Signed-off-by: Jeff Garzik <[EMAIL PROTECTED]> > --- > drivers/net/wireless/airo.c|5 +++-- > drivers/net/wireless/hostap/hostap_hw.c|2 +- > drivers/net/wireless/lib

Re: [PATCH] Remove always-true tests in irq handlers

2007-10-26 Thread Ralf Baechle
On Fri, Oct 26, 2007 at 05:40:24AM -0400, Jeff Garzik wrote: > In these drivers, dev_id is always non-NULL. > > Signed-off-by: Jeff Garzik <[EMAIL PROTECTED]> > --- > arch/ia64/hp/sim/simeth.c |5 - > arch/mips/pmc-sierra/msp71xx/msp_hwbutton.c |5 + Acked-by: R

[PATCH v4.2] FEC - fast ethernet controller for mpc52xx

2007-10-26 Thread Domen Puncer
On 25/10/07 13:29 -0700, Dale Farnsworth wrote: > On Thu, Oct 25, 2007 at 09:41:14PM +0200, Domen Puncer wrote: > > On 25/10/07 11:57 -0700, Dale Farnsworth wrote: > > > Domen wrote: > > > > > use your platform's dma mapping functions, rather than virt_to_phys() > > > > > > > > > > it might be the

Re: [NETNS] Oops in register_pernet_operations() with CONFIG_NET_NS=n

2007-10-26 Thread David Miller
From: Benjamin Thery <[EMAIL PROTECTED]> Date: Fri, 26 Oct 2007 13:41:55 +0200 > David Miller wrote: > > From: [EMAIL PROTECTED] (Eric W. Biederman) > > Date: Thu, 25 Oct 2007 11:21:55 -0600 > > > >>> By the way, I think that we can in the case of undefined CONFIG_NET_NS > >>> reduce register to

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

2007-10-26 Thread Maxim Levitsky
On Thursday 25 October 2007 19:02:12 Alan Stern wrote: > On Thu, 25 Oct 2007, Maxim Levitsky wrote: > > > Hi, > > > > Recently, trying to fix saa7134 suspend/resume problems I found that there > > is a race between IRQ handler and .suspend , and that I cant let driver > > access the device > >

Re: [NETNS] Oops in register_pernet_operations() with CONFIG_NET_NS=n

2007-10-26 Thread Benjamin Thery
David Miller wrote: > From: [EMAIL PROTECTED] (Eric W. Biederman) > Date: Thu, 25 Oct 2007 11:21:55 -0600 > >>> By the way, I think that we can in the case of undefined CONFIG_NET_NS >>> reduce register to calling ->init method and unregister to calling >>> ->exit method. >>> >>> This is a correct

Re: [PATCH] remove __net_initdata attribute

2007-10-26 Thread David Miller
From: "Denis V. Lunev" <[EMAIL PROTECTED]> Date: Fri, 26 Oct 2007 15:28:32 +0400 > Dave, I am sorry, plz disregard the patch :( > > - if the file is a part of the kernel, __exit (and pernet_unregister) > will never called and should be dropped > - if the file is a part of a module, __init is NO

Re: [PATCH] remove __net_initdata attribute

2007-10-26 Thread David Miller
From: "Denis V. Lunev" <[EMAIL PROTECTED]> Date: Fri, 26 Oct 2007 13:56:52 +0400 > The discussion with Eric W. Biederman reveals that the concept for > __net_initdata is bogus. The ->exit method of pernet_operations > should be called during module exit. > > So, discard this attribute. > > Signe

Re: [NETNS] Oops in register_pernet_operations() with CONFIG_NET_NS=n

2007-10-26 Thread David Miller
From: [EMAIL PROTECTED] (Eric W. Biederman) Date: Thu, 25 Oct 2007 11:21:55 -0600 > > By the way, I think that we can in the case of undefined CONFIG_NET_NS > > reduce register to calling ->init method and unregister to calling > > ->exit method. > > > > This is a correct thing at least for now an

Re: [PATCH] NetLabel: correct usage of RCU locking

2007-10-26 Thread David Miller
From: Paul Moore <[EMAIL PROTECTED]> Date: Thu, 25 Oct 2007 09:52:38 -0400 > This fixes some awkward, and perhaps even problematic, RCU lock usage in the > NetLabel code as well as some other related trivial cleanups found when > looking through the RCU locking. Most of the changes involve removi

Re: [PATCH net-2.6] [TCP]: fix D-SACK cwnd handling

2007-10-26 Thread David Miller
From: Ryousei Takano <[EMAIL PROTECTED]> Date: Thu, 25 Oct 2007 17:11:47 +0900 (JST) > In the current net-2.6 kernel, handling FLAG_DSACKING_ACK is broken. > The flag is cleared to 1 just after FLAG_DSACKING_ACK is set. > > if (found_dup_sack) > flag |= FLAG_DSACKING_ACK;

Re: [PATCH] remove __net_initdata attribute

2007-10-26 Thread Denis V. Lunev
Dave, I am sorry, plz disregard the patch :( - if the file is a part of the kernel, __exit (and pernet_unregister) will never called and should be dropped - if the file is a part of a module, __init is NOT dropped on the module loading stage So, we can still make embedded people happy :) Than

Re: [PATCH] napi: use non-interruptible sleep in napi_disable

2007-10-26 Thread David Miller
From: Jeff Garzik <[EMAIL PROTECTED]> Date: Thu, 25 Oct 2007 01:10:32 -0400 > Benjamin Herrenschmidt wrote: > > The current napi_disable() uses msleep_interruptible() but doesn't > > (and can't) exit in case there's a signal, thus ending up doing a > > hot spin without a cpu_relax. Use uninterrupt

Re: [2.6 patch] net/sctp/auth.c: make 3 functions static

2007-10-26 Thread David Miller
From: Vlad Yasevich <[EMAIL PROTECTED]> Date: Wed, 24 Oct 2007 16:27:38 -0400 > Adrian Bunk wrote: > > This patch makes three needlessly global functions static. > > > > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> > > ACK. Thanks for catching these.. Applied, thanks everyone. - To unsubscri

Re: [2.6 patch] unexport softnet_data

2007-10-26 Thread David Miller
From: Adrian Bunk <[EMAIL PROTECTED]> Date: Wed, 24 Oct 2007 18:24:25 +0200 > The EXPORT_PER_CPU_SYMBOL(softnet_data) is no longer used. > > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> I wanted to apply this, but in validing the patch I noticed what appears to be an omission in TCP ipv6. It

Re: [2.6 patch] #if 0 sctp_update_copy_cksum()

2007-10-26 Thread David Miller
From: Vlad Yasevich <[EMAIL PROTECTED]> Date: Wed, 24 Oct 2007 16:30:34 -0400 > Adrian Bunk wrote: > > sctp_update_copy_cksum() is no longer used. > > > > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> > > ACK. Applied, thanks! - To unsubscribe from this list: send the line "unsubscribe netdev"

Re: [2.6 patch] unexport icmpmsg_statistics

2007-10-26 Thread David Miller
From: David Stevens <[EMAIL PROTECTED]> Date: Wed, 24 Oct 2007 13:43:14 -0700 > So, I stand corrected, and sorry about the histrionics. Since > these are arguably ICMP errors, and since errors is the only > thing being MIB-counted in DCCP and SCTP, then it now looks > ok to me as-is, and also ok t

Re: [2.6 patch] unexport sock_enable_timestamp

2007-10-26 Thread David Miller
From: Adrian Bunk <[EMAIL PROTECTED]> Date: Wed, 24 Oct 2007 18:24:05 +0200 > sock_enable_timestamp() no longer has any modular users. > > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> Applied. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAI

Re: [2.6 patch] make tcp_match_skb_to_sack() static

2007-10-26 Thread David Miller
From: "Ilpo_Järvinen" <[EMAIL PROTECTED]> Date: Thu, 25 Oct 2007 00:39:23 +0300 (EEST) > On Wed, 24 Oct 2007, Adrian Bunk wrote: > > > tcp_match_skb_to_sack() can become static. > > > > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> > > > [...snip...] > > Thanks, I should have noticed that righ

Re: [2.6 patch] make ircomm_tty static

2007-10-26 Thread David Miller
From: Adrian Bunk <[EMAIL PROTECTED]> Date: Wed, 24 Oct 2007 18:23:55 +0200 > ircomm_tty can now become static. > > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> Applied. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More major

Re: [PATCH] dev_change_name: ignore changes to same name

2007-10-26 Thread David Miller
From: Stephen Hemminger <[EMAIL PROTECTED]> Date: Wed, 24 Oct 2007 06:44:45 -0700 > Prevent error/backtrace from dev_rename() when changing > name of network device to the same name. This is a common > situation with udev and other scripts that bind addr to device. > > Signed-off-by: Stephen Hemm

Re: Distributed storage. Move away from char device ioctls.

2007-10-26 Thread Evgeniy Polyakov
Returning back to this, since block based storage, which can act as a shared storage/transport layer, is ready with 5'th release of the DST. My couple of notes on proposed data distribution algorithm in FS. On Sun, Sep 16, 2007 at 03:07:11AM -0400, Kyle Moffett ([EMAIL PROTECTED]) wrote: > >I ac

[PATCH] remove __net_initdata attribute

2007-10-26 Thread Denis V. Lunev
The discussion with Eric W. Biederman reveals that the concept for __net_initdata is bogus. The ->exit method of pernet_operations should be called during module exit. So, discard this attribute. Signed-off-by: Denis V. Lunev <[EMAIL PROTECTED]> diff --git a/drivers/net/loopback.c b/drivers/net/

Re: [PATCH 2/3] [NET_CLS_ACT] Use skb_act_clone

2007-10-26 Thread David Miller
From: jamal <[EMAIL PROTECTED]> Date: Sun, 21 Oct 2007 15:20:20 -0400 > A resubmit against net-2.6 from about an hour ago. Applied. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majo

Re: [PATCH 3/3] [NET_DOC] Document some simple rules for actions

2007-10-26 Thread David Miller
From: jamal <[EMAIL PROTECTED]> Date: Sun, 21 Oct 2007 15:22:17 -0400 > Against net-2.6 Applied. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 1/3] [NET_CLS_ACT] Introduce skb_act_clone

2007-10-26 Thread David Miller
From: jamal <[EMAIL PROTECTED]> Date: Sun, 21 Oct 2007 15:18:46 -0400 > A resubmit against net-2.6 from about an hour ago. Applied. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majo

[PATCH] NETDRVR, USB: declance, lp486e, s3c2410_udc: minor irq handler cleanups

2007-10-26 Thread Jeff Garzik
declance: - irq handlers do not take 'const int' as first arg - add KERN_ prefix lp486e: - neaten irq handler top of function formatting - remove pointless void* casts s3c2410_udc: - 'irq' argument is merely used in place of a constant; replace it

[PATCH] drivers/net/irda/au1k_ir: fix obvious irq handler bugs

2007-10-26 Thread Jeff Garzik
interrupt handlers return a return value these days. Also, kill always-true test and unneeded void* cast. Signed-off-by: Jeff Garzik <[EMAIL PROTECTED]> --- drivers/net/irda/au1k_ir.c | 11 --- 1 files changed, 4 insertions(+), 7 deletions(-) diff --git a/drivers/net/irda/au1k_ir.c b/

[PATCH] Remove always-true tests in irq handlers

2007-10-26 Thread Jeff Garzik
In these drivers, dev_id is always non-NULL. Signed-off-by: Jeff Garzik <[EMAIL PROTECTED]> --- arch/ia64/hp/sim/simeth.c |5 - arch/mips/pmc-sierra/msp71xx/msp_hwbutton.c |5 + drivers/net/cpmac.c |3 --- drivers/net/ucc_geth.c

[PATCH] Remove pointless casts from void pointers,

2007-10-26 Thread Jeff Garzik
mostly in and around irq handlers. Signed-off-by: Jeff Garzik <[EMAIL PROTECTED]> --- arch/arm/mach-pxa/ssp.c|2 +- arch/arm/mach-s3c2410/usb-simtec.c |2 +- arch/arm/plat-omap/mailbox.c |2 +- arch/ia64/sn/pci/pcibr/pcibr_provider.c|6

  1   2   >