Re: [ipvs] BUG: soft lockup detected on CPU#3!

2007-05-25 Thread Horms
+0x24a/0x260 [x86_64_start_kernel+358/368] _sinittext+0x166/0x170 -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More majordomo info at http

Re: [ipvs] BUG: soft lockup detected on CPU#3!

2007-05-25 Thread Horms
with different configs. -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - 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: [ipvs] BUG: soft lockup detected on CPU#3!

2007-05-28 Thread Horms
On Sat, May 26, 2007 at 11:22:40AM +0900, Horms wrote: On Fri, May 25, 2007 at 09:30:52AM +, Sebastien Estienne wrote: I didn't try 2.6.21 yet, but using ubuntu dapper kernel (2.6.15) i can't reproduce the bug. When i was using feisty kernel (2.6.20), i can reproduce in less than 5

Re: [ipvs] BUG: soft lockup detected on CPU#3!

2007-05-28 Thread Horms
On Mon, May 28, 2007 at 04:20:32PM +0200, Sebastien Estienne wrote: On 5/28/07, Horms [EMAIL PROTECTED] wrote: On Sat, May 26, 2007 at 11:22:40AM +0900, Horms wrote: On Fri, May 25, 2007 at 09:30:52AM +, Sebastien Estienne wrote: I didn't try 2.6.21 yet, but using ubuntu dapper

[patch 0/4] ip_vs_ftp cleanups

2006-09-01 Thread Horms
Hi, This series of four patches has several minor cleanups for the options to the ip_vs_ftp modules. -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED

[patch 3/4] Make sure ip_vs_ftp ports are valid

2006-09-01 Thread Horms
; + } ret = register_ip_vs_app_inc(app, app-protocol, ports[i]); if (ret) break; -- -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - To unsubscribe from this list: send the line unsubscribe netdev in the body

[patch 1/4] Document the ports option to ip_vs_ftp in kernel-parameters.txt

2006-09-01 Thread Horms
] When an interrupt is not handled search all handlers for it. Intended to get systems with badly broken -- -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - To unsubscribe from this list: send the line unsubscribe netdev in the body

[patch 4/4] remove the debug option go ip_vs_ftp

2006-09-01 Thread Horms
: loaded support on port[%d] = %d\n, - app-name, i, ports[i]); + IP_VS_INFO(%s: loaded support on port[%d] = %d\n, + app-name, i, ports[i]); } if (ret) -- -- Horms H: http://www.vergenet.net/~horms/ W: http

[patch 2/4] auto-help for ip_vs_ftp

2006-09-01 Thread Horms
-- -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - 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 3/4] Make sure ip_vs_ftp ports are valid

2006-09-03 Thread Horms
On Mon, Sep 04, 2006 at 01:09:59AM +0200, Patrick McHardy wrote: Horms wrote: I'm not entirely sure what happens in the case of a valid port, at best it'll be silently ignored. This patch ignores them a little more verbosely. Signed-Off-By: Simon Horman [EMAIL PROTECTED] Index: linux

Re: [patch 3/4] Make sure ip_vs_ftp ports are valid

2006-09-03 Thread Horms
On Mon, Sep 04, 2006 at 09:44:02AM +0900, Horms wrote: On Mon, Sep 04, 2006 at 01:09:59AM +0200, Patrick McHardy wrote: Horms wrote: I'm not entirely sure what happens in the case of a valid port, at best it'll be silently ignored. This patch ignores them a little more verbosely

Re: [patch 3/4] Make sure ip_vs_ftp ports are valid

2006-09-20 Thread Horms
On Wed, Sep 20, 2006 at 12:29:45PM +0200, Patrick McHardy wrote: Horms wrote: Here is the revised patch. [IPVS] Make sure ip_vs_ftp ports are valid I'm not entirely sure what happens in the case of a valid port, at best it'll be silently ignored. This patch ensures that the port

Re: ipvs locahost client patch for 2.6?

2006-09-20 Thread Horms
effort. -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - 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 0/2] [IPVS]: Make sure ip_vs_ftp ports are valid (again)

2006-09-28 Thread Horms
On Wed, Sep 27, 2006 at 10:53:54PM -0700, David Miller wrote: From: Horms [EMAIL PROTECTED] Date: Wed, 20 Sep 2006 23:44:57 +0900 there are two patches floating around for this problem. It seems that the first incarntation has been committed to Linus's tree

Re: [PATCH] [IPVS] transparent proxying

2006-12-17 Thread Horms
On Wed, Nov 29, 2006 at 11:46:22PM +0900, Horms wrote: On Wed, Nov 29, 2006 at 03:15:23PM +0100, Thomas Graf wrote: [split] This patch seems to be based on an old tree, I've renamed nfmark to mark in net-2.6.20. The term fwmark and nfmark shouldn't be used anymore. Sorry, I based

[PATCH] [IPVS] replace if .. goto with while

2006-12-17 Thread Horms
I guess that this code used to be more complex, but replacing the goto with a while seems to make things a bit more readable. Or in other words, two fairly gratuitous goto are removed. On a related note, I wonder if there should be a limit to how many times it tries. Signed-Off-By: Simon Horman

Re: [PATCH] [IPVS] replace if .. goto with while

2007-01-03 Thread Horms
On Tue, Jan 02, 2007 at 12:38:39AM -0800, David Miller wrote: From: Horms [EMAIL PROTECTED] Date: Mon, 18 Dec 2006 12:11:11 +0900 I guess that this code used to be more complex, but replacing the goto with a while seems to make things a bit more readable. Or in other words, two fairly

Re: [rfc][patch] ipvs: use proper timeout instead of fixed value

2006-05-04 Thread Horms
= pp-timeout_table[cp-state]; ip_vs_conn_put(cp); if (p buffer+buflen) { -- Horms http://www.vergenet.net/~horms/ - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL

Re: [rfc][patch] ipvs: use proper timeout instead of fixed value

2006-05-04 Thread Horms
On Thu, May 04, 2006 at 10:51:11PM -0400, Andy Gospodarek wrote: On Fri, May 05, 2006 at 09:47:56AM +0900, Horms wrote: On Thu, May 04, 2006 at 04:11:16PM -0400, Andy Gospodarek wrote: Instead of using the default timeout of 3 minutes, this uses the timeout specific to the protocol

Re: [rfc][patch] ipvs: use proper timeout instead of fixed value

2006-05-06 Thread Horms
On Fri, May 05, 2006 at 02:57:26PM -0400, Andy Gospodarek wrote: On Fri, May 05, 2006 at 12:20:54PM +0900, Horms wrote: Sorry, I missunderstood your patch completely the first time around. Yes I think this is an excellent idea. Assuming its tested and works I'm happy to sign off

Re: [rfc][patch] ipvs: use proper timeout instead of fixed value

2006-05-08 Thread Horms
On Sun, May 07, 2006 at 11:13:33PM -0400, Andy Gospodarek wrote: On Sun, May 07, 2006 at 01:38:40PM +0900, Horms wrote: On Fri, May 05, 2006 at 02:57:26PM -0400, Andy Gospodarek wrote: On Fri, May 05, 2006 at 12:20:54PM +0900, Horms wrote: Sorry, I missunderstood your patch

[IPVS] remove dead code

2005-12-05 Thread Horms
ip_vs_app.c | 28 ip_vs_lblc.c | 27 --- ip_vs_lblcr.c | 27 --- ip_vs_proto_tcp.c | 22 -- 4 files changed, 104 deletions(-) Please apply -- Horms [IPVS] remove dead code

[PATCH-2.6] cleanup IP_VS_DBG statements

2005-12-29 Thread Horms
Dave, please apply. -- Horms tree 60d9ec661356d90c826b9e95d9d1feac04df0ceb parent d7ba5e23dc0661bc1e79593e0a6fbd424def3dcb author Horms [EMAIL PROTECTED] Fri, 30 Dec 2005 12:58:16 +0900 committer Horms [EMAIL PROTECTED] Fri, 30 Dec 2005 12:58:16 +0900 [PATCH-2.6] cleanup IP_VS_DBG

Re: [2.6 patch] fix ipvs compilation

2006-01-05 Thread Horms
are remnants of the 2.4 kernel and said new abstractions. I will try to move some functionality into ip_vs.h, which should be the only place with a large number of includes anyway. Of course Horms has to ack all of this, but I don't think he'll object, so long as everything still compiles

[PATCH] Remove suprious use of goto out: in icmp_reply

2006-01-31 Thread Horms
in this function. This patch removes out. Signed-Off-By: Horms [EMAIL PROTECTED] diff --git a/net/ipv4/icmp.c b/net/ipv4/icmp.c index be5a519..eadbd55 100644 --- a/net/ipv4/icmp.c +++ b/net/ipv4/icmp.c @@ -385,7 +385,7 @@ static void icmp_reply(struct icmp_bxm * u32 daddr

[PATCH] [IPV4] Document icmp_errors_use_inbound_ifaddr sysctl

2006-02-01 Thread Horms
the ordering of the options in sysctl.txt, so I took a wild guess about where it fits. Signed-Off-By: Horms [EMAIL PROTECTED] diff --git a/Documentation/networking/ip-sysctl.txt b/Documentation/networking/ip-sysctl.txt index 2b7cf19..26364d0 100644 --- a/Documentation/networking/ip-sysctl.txt

Re: [PATCH][RFC] Shrink ip_vs_*.c includes

2006-02-06 Thread Horms
/ip_vs_conn.o] Error 1 make[2]: *** [net/ipv4/ipvs] Error 2 make[1]: *** [net/ipv4] Error 2 make: *** [net] Error 2 -- Horms - 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

Re: [PATCH][RFC] Shrink ip_vs_*.c includes

2006-02-06 Thread Horms
of it needed to be built. So with that in mind, could you continue in the vein of your original patch? -- Horms Hello, This patch against the latest GIT HEAD reduces the includes to the necessary ones. I've compile-tested it against following (and s/y/m/) configuration on x86: CONFIG_IP_VS=y

Re: [PATCH][RFC] Shrink ip_vs_*.c includes

2006-02-06 Thread Horms
as what I posted yesterday. -- Horms - 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] [IPVS] Shrink ip_vs_*.c includes

2006-02-06 Thread Horms
Dave, please apply. -- Horms Hello, This patch against the latest GIT HEAD reduces the includes to the necessary ones. I've compile-tested it against following (and s/y/m/) configuration on x86: CONFIG_IP_VS=y CONFIG_IP_VS_DEBUG=y CONFIG_IP_VS_TAB_BITS=12 CONFIG_IP_VS_PROTO_TCP=y

Re: [PATCH] [IPVS] Shrink ip_vs_*.c includes

2006-02-07 Thread Horms
On Wed, Feb 08, 2006 at 12:19:32PM +1100, Herbert Xu wrote: Horms [EMAIL PROTECTED] wrote: Dave, please apply. Looks bogus to me. Why are we removing linux/modules.h from ip_vs_app.c when it uses things like EXPORT_SYMBOL? Given that the code still compiles, I guess linux/modules.h

Re: [PATCH] [IPVS] Shrink ip_vs_*.c includes

2006-02-07 Thread Horms
On Wed, Feb 08, 2006 at 01:36:11PM +1100, Herbert Xu wrote: Horms [EMAIL PROTECTED] wrote: Looks bogus to me. Why are we removing linux/modules.h from ip_vs_app.c when it uses things like EXPORT_SYMBOL? Given that the code still compiles, I guess linux/modules.h is included in some

Re: [PATCH] [IPVS] Shrink ip_vs_*.c includes

2006-02-08 Thread Horms
On Tue, Feb 07, 2006 at 09:07:34PM -0800, David S. Miller wrote: From: Horms [EMAIL PROTECTED] Date: Wed, 8 Feb 2006 12:09:29 +0900 Unfortunately this seems like it is going to be more tedious than we first thought. I would guess writing some sort of tool to analyse symbols and headers

[PATCH,2.4,SECURITY,NET] orinoco: CVE-2005-3180: Information leakage due to incorrect padding

2006-02-08 Thread Horms
made a backport below, with the only semi-significant change being including the ALIGN macro in orinoco.c, as it doesn't exist in 2.4. As yet untested Signed-off-by: Horms [EMAIL PROTECTED] index 0c06b14..b99edd3 100644 --- a/drivers/net/wireless/hermes.c +++ b/drivers/net/wireless/hermes.c

[PATCH] [IPVS] use msleep_interruptable() instead of ssleep() aka msleep()

2006-11-27 Thread Horms
Dean Manners notices that when an IPVS synchonisation daemons are started the system load slowly climbs up to 1. This seems to be related to the call to ssleep(1) (aka msleep(1000) in the main loop. Replacing this with a call to msleep_interruptable() seems to make the problem go away. Though I'm

Re: [PATCH] [IPVS] use msleep_interruptable() instead of ssleep() aka msleep()

2006-11-28 Thread Horms
On Tue, Nov 28, 2006 at 10:35:01AM +0200, Julian Anastasov wrote: Hello, On Tue, 28 Nov 2006, Horms wrote: Dean Manners notices that when an IPVS synchonisation daemons are started the system load slowly climbs up to 1. This seems to be related to the call to ssleep(1) (aka

[IPVS] make ip_vs_sync.c = 80col wide

2006-11-28 Thread Horms
Make ip_vs_sync.c = 80col wide Signed-off-by: Simon Horman [EMAIL PROTECTED] Index: linux-2.6/net/ipv4/ipvs/ip_vs_sync.c === --- linux-2.6.orig/net/ipv4/ipvs/ip_vs_sync.c 2006-11-29 09:53:51.0 +0900 +++

[PATCH] [IPVS] transparent proxying

2006-11-28 Thread Horms
This seems to be a pretty clean solution to a real problem. Ultimately I would like to see IPVS move into the forward chain. This seems to be a nice way to explore that, without breaking any existing setups. -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ [IPVS

Re: [PATCH] [IPVS] transparent proxying

2006-11-29 Thread Horms
On Wed, Nov 29, 2006 at 03:15:23PM +0100, Thomas Graf wrote: * Horms [EMAIL PROTECTED] 2006-11-29 15:21 This seems to be a pretty clean solution to a real problem. Ultimately I would like to see IPVS move into the forward chain. This seems to be a nice way to explore that, without

[PATCH] [BRIDGE]: netfilter missing symbol has_bridge_parent

2006-02-12 Thread Horms
that the following patch will resolve this problem. -- Horms [BRIDGE]: netfilter missing symbol has_bridge_parent 5dce971acf2ae20c80d5e9d1f6bbf17376870911 in Linus' tree, otherwise known as bridge-netfilter-races-on-device-removal.patch in 2.5.15.4 removed has_bridge_parent, however this symbol is still called

Re: [PATCH] [BRIDGE]: netfilter missing symbol has_bridge_parent

2006-02-13 Thread Horms
On Mon, Feb 13, 2006 at 10:59:07AM -0800, Stephen Hemminger wrote: Horms patch was the best of the three fixes. Dave, already applied Harald's version, so this patch converts that to the better one. Thanks, I was working of Linus' i2.6 tree rather than Dave's net-2.6, so I didn't see Harald's

[PATCH] net: ne2k.c won't compile if pci_clone_list is const

2006-03-23 Thread Horms
: pci_clone_list causes a section type conflict ~/ gcc --version gcc (GCC) 4.0.3 (Debian 4.0.3-1) ~/ dpkg gcc-4.0 | grep Version Version: 4.0.3-1 Signed-Off-By: Horms [EMAIL PROTECTED] ne2k-pci.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) cee0890cc97247b6a9decd94f5dc0719ac8f0b1b diff --git

[PATCH] LVS: Send ICMP unreachable responses to end-users when real-servers are removed

2007-02-10 Thread Horms
Hi, this is a small patch by Janusz Krzysztofik to ip_route_output_slow() that allows VIP-less LVS linux director to generate packets originating From VIP if sysctl_ip_nonlocal_bind is set. In a nutshell, the intention is for an LVS linux director to be able to send ICMP unreachable responses

IPVS Sysctl Variables

2006-06-30 Thread Horms
are and what they do, you don't have to check if you don't want to), is it appropriate for inclusion in Documentation/networking/, and if so, should it be in ip-sysctl.txt, or elsewhere? http://www.linuxvirtualserver.org/docs/sysctl.html -- Horms H: http

[PATCH] ipvs: Add sysctl documentation

2006-07-02 Thread Horms
-levels.txt from IPVS source tree into description of ipvs_debug * Minor spelling fixes * Further editing more than welcome * DaveM, do you need a 2.4 version of this document, it will likely be a slightly different list of options? Signed-Off-By: Horms [EMAIL PROTECTED] Documentation

Re: [PATCH] ipvs: Add sysctl documentation

2006-07-04 Thread Horms
On Mon, Jul 03, 2006 at 07:36:47PM -0700, David Miller wrote: From: Horms [EMAIL PROTECTED] Date: Mon, 3 Jul 2006 11:31:30 +0900 * Derived from http://www.linuxvirtualserver.org/docs/sysctl.html, v1.4 maintained by Wensong Zhang * Adjusted preample to match ip-sysctl.txt

Re: [patch] ipvs: force read of atomic_t in while loop

2007-08-08 Thread Horms
-usecnt) 1) {}; + IP_VS_WAIT_WHILE(atomic_read(svc-usecnt) 1); /* call the update_service, because server weight may be changed */ svc-scheduler-update_service(svc); -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - To unsubscribe from this list

[patch 0/2] ipvs: avoid overcommit on the standby, take III

2007-11-04 Thread horms
previous attempt, which happily showed the bogus bits that I know about have been fixed. -- -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - To unsubscribe from this list: send the line unsubscribe netdev in the body of a message to [EMAIL PROTECTED] More

[patch 2/2] ipvs: Syncrhonise Closing of Connections

2007-11-04 Thread horms
PROTECTED] Signed-off-by: Simon Horman [EMAIL PROTECTED] --- Thu, 01 Nov 2007 18:25:10 +0900, Horms * Redifed for net-2.6 * Ran through scripts/checkpatch.pl and fixed up everything that it complains about except the use of volatile, as its in keeping with other fields in the structure

[patch 1/2] ipvs: Bind connections on stanby if the destination exists

2007-11-04 Thread horms
if it is a normal entry. */ -- -- Horms H: http://www.vergenet.net/~horms/ W: http://www.valinux.co.jp/en/ - 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