Re: NAPI (aka RX Polling) for natsemi?

2005-07-22 Thread Patrick McHardy
Francois Romieu wrote: Daniel Higgins <[EMAIL PROTECTED]> : [napi for natsemi] Can you please fill a bugzilla entry for it at bugzilla.kernel.org ? It is bugzilla, not featurewishzilla. If some external patches cause problem I suggest to contact their authors. - To unsubscribe from this list:

Re: NAPI (aka RX Polling) for natsemi?

2005-07-22 Thread Patrick McHardy
Francois Romieu wrote: Daniel Higgins [EMAIL PROTECTED] : [napi for natsemi] Can you please fill a bugzilla entry for it at bugzilla.kernel.org ? It is bugzilla, not featurewishzilla. If some external patches cause problem I suggest to contact their authors. - To unsubscribe from this list:

Re: Is this a bug in linux-2.6.12 ipsec code function xfrm4_rcv_encap ??

2005-07-22 Thread Patrick McHardy
k8 s wrote: I AM SORRY FOR THE PREVIOUS MAIL. I am correcting my previous mail. Infact I see only One race(not three as was wrongly pointed out). I commented out the section once again where the race might be. / Race Here . The

Re: 2.6.12 netfilter: local packets marked as invalid

2005-07-12 Thread Patrick McHardy
patch is fine, thanks. ACKed-by: Patrick McHardy <[EMAIL PROTECTED]> Regards Patrick - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Ple

Re: 2.6.12 netfilter: local packets marked as invalid

2005-07-12 Thread Patrick McHardy
]: Revert nf_reset change Revert the nf_reset change that caused so much trouble, drop conntrack references manually before packets are queued to packet sockets. Signed-off-by: Phil Oester <[EMAIL PROTECTED]> Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]>

Re: kernel guide to space

2005-07-12 Thread Patrick McHardy
Denis Vlasenko wrote: text with 8-char tabs: struct s { int n; /* comment */ unsigned int u; /* comment */ }; Same text viewed with tabs set to 4-char width: struct s { int n; /* comment */ unsigned int u; /* comment */ }; Comments are not aligned

Re: 2.6.12 netfilter: local packets marked as invalid

2005-07-12 Thread Patrick McHardy
patch is fine, thanks. ACKed-by: Patrick McHardy [EMAIL PROTECTED] Regards Patrick - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http

Re: kernel guide to space

2005-07-12 Thread Patrick McHardy
Denis Vlasenko wrote: text with 8-char tabs: struct s { int n; /* comment */ unsigned int u; /* comment */ }; Same text viewed with tabs set to 4-char width: struct s { int n; /* comment */ unsigned int u; /* comment */ }; Comments are not aligned

Re: 2.6.12 netfilter: local packets marked as invalid

2005-07-12 Thread Patrick McHardy
]: Revert nf_reset change Revert the nf_reset change that caused so much trouble, drop conntrack references manually before packets are queued to packet sockets. Signed-off-by: Phil Oester [EMAIL PROTECTED] Signed-off-by: Patrick McHardy [EMAIL PROTECTED] --- commit

Re: 2.6.12 netfilter: local packets marked as invalid

2005-07-08 Thread Patrick McHardy
Daniel Drake wrote: When retrying the telnet test, this appears in the logs: Jul 8 14:53:04 dsd inv IN=lo OUT= MAC=00:00:00:00:00:00:00:00:00:00:00:00:08:00 SRC=127.0.0.1 DST=127.0.0.1 LEN=40 TOS=0x10 PREC=0x00 TTL=64 ID=15 DF PROTO=TCP SPT=80 DPT=58950 WINDOW=0 RES=0x00 ACK RST URGP=0 Does

Re: 2.6.12 netfilter: local packets marked as invalid

2005-07-08 Thread Patrick McHardy
Daniel Drake wrote: When retrying the telnet test, this appears in the logs: Jul 8 14:53:04 dsd inv IN=lo OUT= MAC=00:00:00:00:00:00:00:00:00:00:00:00:08:00 SRC=127.0.0.1 DST=127.0.0.1 LEN=40 TOS=0x10 PREC=0x00 TTL=64 ID=15 DF PROTO=TCP SPT=80 DPT=58950 WINDOW=0 RES=0x00 ACK RST URGP=0 Does

Re: fix for ISDN ippp filtering

2005-04-21 Thread Patrick McHardy
Karsten Keil wrote: Hi, We do not longer use DLT_LINUX_SLL for activ/pass filters but DLT_PPP_WITHDIRECTION witch need 1 as outbound flag. Please apply. Won't this break compatibility with old ipppd binaries? Regards Patrick - To unsubscribe from this list: send the line "unsubscribe

Re: fix for ISDN ippp filtering

2005-04-21 Thread Patrick McHardy
Karsten Keil wrote: Hi, We do not longer use DLT_LINUX_SLL for activ/pass filters but DLT_PPP_WITHDIRECTION witch need 1 as outbound flag. Please apply. Won't this break compatibility with old ipppd binaries? Regards Patrick - To unsubscribe from this list: send the line unsubscribe linux-kernel

Re: ACPI/HT or Packet Scheduler BUG?

2005-04-17 Thread Patrick McHardy
Herbert Xu wrote: On Sat, Apr 16, 2005 at 01:06:39PM +0200, Thomas Graf wrote: qdisc_destroy can still be invoked without qdisc_tree_lock via the deletion of a class when it calls qdisc_destroy to destroy its leaf qdisc. Indeed. Fortuantely HTB seems to be safe as it calls sch_tree_lock which is

Re: ACPI/HT or Packet Scheduler BUG?

2005-04-17 Thread Patrick McHardy
Herbert Xu wrote: On Sat, Apr 16, 2005 at 01:06:39PM +0200, Thomas Graf wrote: qdisc_destroy can still be invoked without qdisc_tree_lock via the deletion of a class when it calls qdisc_destroy to destroy its leaf qdisc. Indeed. Fortuantely HTB seems to be safe as it calls sch_tree_lock which is

Re: [5/9] [TUN] Fix check for underflow

2005-03-16 Thread Patrick McHardy
, one for 2.6.10, one for 2.6.11. Regards Patrick # This is a BitKeeper generated diff -Nru style patch. # # ChangeSet # 2005/03/10 18:20:44-08:00 [EMAIL PROTECTED] # [IPV4]: Fix crash while reading /proc/net/route caused by stale pointers # # Signed-off-by: Patrick McHardy <[EMAIL PROTEC

Re: [5/9] [TUN] Fix check for underflow

2005-03-16 Thread Patrick McHardy
, one for 2.6.10, one for 2.6.11. Regards Patrick # This is a BitKeeper generated diff -Nru style patch. # # ChangeSet # 2005/03/10 18:20:44-08:00 [EMAIL PROTECTED] # [IPV4]: Fix crash while reading /proc/net/route caused by stale pointers # # Signed-off-by: Patrick McHardy [EMAIL PROTECTED

Re: [PATCH] Export dev_get_flags in net/core/dev.c to fix missing symbols

2005-03-15 Thread Patrick McHardy
Alex Tribble wrote: [EMAIL PROTECTED], 2005-03-15 13:46:12-06:00, [EMAIL PROTECTED] Export dev_get_flags to fix missing symbols in ipv6.ko The same patch is already in Linus' tree: ChangeSet 1.2186, 2005/03/15 10:16:32-08:00, [EMAIL PROTECTED] [NET]: Need to export dev_get_flags() to

Re: [PATCH] Export dev_get_flags in net/core/dev.c to fix missing symbols

2005-03-15 Thread Patrick McHardy
Alex Tribble wrote: [EMAIL PROTECTED], 2005-03-15 13:46:12-06:00, [EMAIL PROTECTED] Export dev_get_flags to fix missing symbols in ipv6.ko The same patch is already in Linus' tree: ChangeSet 1.2186, 2005/03/15 10:16:32-08:00, [EMAIL PROTECTED] [NET]: Need to export dev_get_flags() to

Re: Badness in local_bh_enable at kernel/softirq.c:140

2005-03-12 Thread Patrick McHardy
Kimmo Sundqvist wrote: Mar 12 10:39:45 shadowgate Badness in local_bh_enable at kernel/softirq.c:140 Mar 12 10:39:45 shadowgate [] local_bh_enable+0x64/0x70 Mar 12 10:39:45 shadowgate [] isdn_ppp_xmit+0xf7/0x7e0 [isdn] Mar 12 10:39:45 shadowgate [] isdn_net_xmit+0x186/0x1d0 [isdn] Mar 12 10:39:45

Re: Badness in local_bh_enable at kernel/softirq.c:140

2005-03-12 Thread Patrick McHardy
Kimmo Sundqvist wrote: Mar 12 10:39:45 shadowgate Badness in local_bh_enable at kernel/softirq.c:140 Mar 12 10:39:45 shadowgate [c0114ca4] local_bh_enable+0x64/0x70 Mar 12 10:39:45 shadowgate [c486afd7] isdn_ppp_xmit+0xf7/0x7e0 [isdn] Mar 12 10:39:45 shadowgate [c485d646] isdn_net_xmit+0x186/0x1d0

Re: Last night Linus bk - netfilter busted?

2005-03-11 Thread Patrick McHardy
Herbert Xu wrote: Patrick McHardy <[EMAIL PROTECTED]> wrote: You're right, good catch. IPT_RETURN is interpreted internally by ip_tables, but since the value changed it isn't recognized by ip_tables anymore and returned to nf_iterate() as NF_REPEAT. This patch restores the old value. Plea

Re: Last night Linus bk - netfilter busted?

2005-03-11 Thread Patrick McHardy
patch. # # ChangeSet # 2005/03/11 21:41:01+01:00 [EMAIL PROTECTED] # [NETFILTER]: Fix iptables userspace compatibility breakage # # Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> # # include/linux/netfilter_ipv6/ip6_tables.h # 2005/03/11 21:40:52+01:00 [EMAIL PROTECTED]

Re: Last night Linus bk - netfilter busted?

2005-03-11 Thread Patrick McHardy
Dmitry Torokhov wrote: My box gets stuck while booting (actually starting ntpd) whith tonight pull from Linus. It looks like it is spinning in ipt_do_table when I do SysRq-P. No call trace though. Please post your ruleset and .config. A backtrace would also be useful. Anyone else seeing it? Any

Re: Last night Linus bk - netfilter busted?

2005-03-11 Thread Patrick McHardy
Dmitry Torokhov wrote: My box gets stuck while booting (actually starting ntpd) whith tonight pull from Linus. It looks like it is spinning in ipt_do_table when I do SysRq-P. No call trace though. Please post your ruleset and .config. A backtrace would also be useful. Anyone else seeing it? Any

Re: Last night Linus bk - netfilter busted?

2005-03-11 Thread Patrick McHardy
patch. # # ChangeSet # 2005/03/11 21:41:01+01:00 [EMAIL PROTECTED] # [NETFILTER]: Fix iptables userspace compatibility breakage # # Signed-off-by: Patrick McHardy [EMAIL PROTECTED] # # include/linux/netfilter_ipv6/ip6_tables.h # 2005/03/11 21:40:52+01:00 [EMAIL PROTECTED] +1 -1

Re: Last night Linus bk - netfilter busted?

2005-03-11 Thread Patrick McHardy
Herbert Xu wrote: Patrick McHardy [EMAIL PROTECTED] wrote: You're right, good catch. IPT_RETURN is interpreted internally by ip_tables, but since the value changed it isn't recognized by ip_tables anymore and returned to nf_iterate() as NF_REPEAT. This patch restores the old value. Please fix

Re: 2.6.11 on AMD64 traps

2005-03-09 Thread Patrick McHardy
Michal Vanco wrote: On Wednesday 09 March 2005 20:45, Patrick McHardy wrote: This patch should fix it. The crash is caused by stale pointers, the pointers in fib_iter_state are not reloaded after seq->stop() followed by seq->start(pos > 0). Well. Trap vanished after applying t

Re: 2.6.11 on AMD64 traps

2005-03-09 Thread Patrick McHardy
ute caused by stale pointers # # Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> # # net/ipv4/fib_hash.c # 2005/03/09 20:41:37+01:00 [EMAIL PROTECTED] +11 -1 # [IPV4]: Fix crash while reading /proc/net/route caused by stale pointers # # Signed-off-by: Patrick McHardy <[EM

Re: 2.6.11 on AMD64 traps

2005-03-09 Thread Patrick McHardy
pointers # # Signed-off-by: Patrick McHardy [EMAIL PROTECTED] # # net/ipv4/fib_hash.c # 2005/03/09 20:41:37+01:00 [EMAIL PROTECTED] +11 -1 # [IPV4]: Fix crash while reading /proc/net/route caused by stale pointers # # Signed-off-by: Patrick McHardy [EMAIL PROTECTED] # diff -Nru a/net/ipv4

Re: 2.6.11 on AMD64 traps

2005-03-09 Thread Patrick McHardy
Michal Vanco wrote: On Wednesday 09 March 2005 20:45, Patrick McHardy wrote: This patch should fix it. The crash is caused by stale pointers, the pointers in fib_iter_state are not reloaded after seq-stop() followed by seq-start(pos 0). Well. Trap vanished after applying this patch, but another

Re: Insmod error ip_nat_ftp

2005-03-05 Thread Patrick McHardy
govind raj wrote: /lib/modules/2.4.29/kernel/net/ipv4/netfilter/ip_nat_ftp.o: init_module: Device or resource busy You probably already have in already statically linked in. Check your .config. Regards Patrick - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body

Re: Insmod error ip_nat_ftp

2005-03-05 Thread Patrick McHardy
govind raj wrote: /lib/modules/2.4.29/kernel/net/ipv4/netfilter/ip_nat_ftp.o: init_module: Device or resource busy You probably already have in already statically linked in. Check your .config. Regards Patrick - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of

Re: [netfilter-core] [PATCH 2.6.10 1/1] netfilter: fix crash on nat+icmp packets

2005-03-02 Thread Patrick McHardy
mukesh agrawal wrote: The cause of the crash is that udp_manip_pkt reads *pskb into iph before calling skb_ip_make_writable, and fails to update iph after the call. Since skb_ip_make_writable may delete the original skb when it makes a copy, a page fault may occur when udp_manip_pkt later

Re: [netfilter-core] [PATCH 2.6.10 1/1] netfilter: fix crash on nat+icmp packets

2005-03-02 Thread Patrick McHardy
mukesh agrawal wrote: The cause of the crash is that udp_manip_pkt reads *pskb into iph before calling skb_ip_make_writable, and fails to update iph after the call. Since skb_ip_make_writable may delete the original skb when it makes a copy, a page fault may occur when udp_manip_pkt later

Re: iptables and ip_conntrack_tuple.h compile fix

2005-02-01 Thread Patrick McHardy
Tompa Septimius Paul wrote: Hi, I try to recompile iptables iptables-1.2.11 with kernel 2.6.11-rc2 (and mm2) running and I don't succeed. It complains about /usr/src/linux/include/linux/netfilter_ipv4/ip_conntrack_tuple.h after this small changes iptables is compiling again. I just added a

Re: iptables and ip_conntrack_tuple.h compile fix

2005-02-01 Thread Patrick McHardy
Tompa Septimius Paul wrote: Hi, I try to recompile iptables iptables-1.2.11 with kernel 2.6.11-rc2 (and mm2) running and I don't succeed. It complains about /usr/src/linux/include/linux/netfilter_ipv4/ip_conntrack_tuple.h after this small changes iptables is compiling again. I just added a

Re: Memory leak in 2.6.11-rc1?

2005-01-30 Thread Patrick McHardy
YOSHIFUJI Hideaki / [EMAIL PROTECTED] wrote: In article <[EMAIL PROTECTED]> (at Mon, 31 Jan 2005 15:11:32 +1100), Herbert Xu <[EMAIL PROTECTED]> says: Patrick McHardy <[EMAIL PROTECTED]> wrote: Ok, final decision: you are right :) conntrack also defragments locally genera

Re: Memory leak in 2.6.11-rc1?

2005-01-30 Thread Patrick McHardy
Patrick McHardy wrote: Russell King wrote: I don't know if the code is using fragment lists in ip_fragment(), but on reading the code a question comes to mind: if we have a list of fragments, does each fragment skb have a valid (and refcounted) dst pointer before ip_fragment() does it's job

Re: Memory leak in 2.6.11-rc1?

2005-01-30 Thread Patrick McHardy
Patrick McHardy wrote: Russell King wrote: I don't know if the code is using fragment lists in ip_fragment(), but on reading the code a question comes to mind: if we have a list of fragments, does each fragment skb have a valid (and refcounted) dst pointer before ip_fragment() does it's job

Re: Memory leak in 2.6.11-rc1?

2005-01-30 Thread Patrick McHardy
Russell King wrote: I don't know if the code is using fragment lists in ip_fragment(), but on reading the code a question comes to mind: if we have a list of fragments, does each fragment skb have a valid (and refcounted) dst pointer before ip_fragment() does it's job? If yes, then isn't the

Re: 2.6.11-rc2 TCP ignores PMTU ICMP (Re: Linux 2.6.11-rc2)

2005-01-27 Thread Patrick McHardy
David S. Miller wrote: I've forwarded this to netfilter-devel for inspection. Thanks for collecting all the data points so well. Here is the fix for everyone. Please report back if it doesn't solve the problem. Thanks. = net/ipv4/netfilter/ip_nat_proto_tcp.c 1.10 vs edited = ---

Re: 2.6.11-rc2 TCP ignores PMTU ICMP (Re: Linux 2.6.11-rc2)

2005-01-27 Thread Patrick McHardy
David S. Miller wrote: I've forwarded this to netfilter-devel for inspection. Thanks for collecting all the data points so well. Here is the fix for everyone. Please report back if it doesn't solve the problem. Thanks. = net/ipv4/netfilter/ip_nat_proto_tcp.c 1.10 vs edited = ---

Re: 2.6.11-rc2 complains badly aboud badness in local_bh_enable

2005-01-23 Thread Patrick McHardy
Jasper Spaans wrote: I'm seeing a similar problem on my machine - one that does not know what ppp is. Main suspect is the network bridging code in combination with iptables; the first lines of the message: The patch which caused this has already been reverted. # This is a BitKeeper generated diff

Re: 2.6.11-rc2 complains badly aboud badness in local_bh_enable

2005-01-23 Thread Patrick McHardy
Jasper Spaans wrote: I'm seeing a similar problem on my machine - one that does not know what ppp is. Main suspect is the network bridging code in combination with iptables; the first lines of the message: The patch which caused this has already been reverted. # This is a BitKeeper generated diff

<    1   2   3   4   5   6