Re: Problem with GVRP on eth while having a bridge

2013-02-07 Thread Patrick McHardy
On Thu, Feb 07, 2013 at 11:56:38AM +0100, Stefan Priebe - Profihost AG wrote: > Hello list, > > this was tested using vanilla 3.7.6 kernel. > > When i add a vlan to an ethernet device supporting gvrp everything is > fine until this ethernet device also had a bridge. > > This works fine as long a

Re: Problem with GVRP on eth while having a bridge

2013-02-07 Thread Patrick McHardy
On Thu, Feb 07, 2013 at 12:58:01PM +0100, Stefan Priebe - Profihost AG wrote: > Hi Patrick, > > thanks for your fast response. > > > So please describe the entire GVRP message exchange that's going > > on. > I've attached two small pcap files. > > has_bridge.pcap: is where eth1 has a bridge and

Re: cups slow on linux-2.6.24

2008-01-30 Thread Patrick McHardy
Jeff Chua wrote: On Jan 31, 2008 10:23 AM, Jeff Chua <[EMAIL PROTECTED]> wrote: Again, using latest linux, one with 17311393f969090ab060540bd9dbe7dc885a76d5 reverted, and the other without. Sorry, here's the attached output files. Thanks. In the dump we can see that connections reus

Re: cups slow on linux-2.6.24

2008-01-30 Thread Patrick McHardy
Jeff Chua wrote: On Jan 31, 2008 10:41 AM, Patrick McHardy <[EMAIL PROTECTED]> wrote: Thanks. In the dump we can see that connections reusing ports always have their first SYN dropped and retransmissted three seconds later. I'm not sure whats causing this yet, do you have any fir

Re: cups slow on linux-2.6.24

2008-01-31 Thread Patrick McHardy
Jozsef Kadlecsik wrote: Hi Jeff, On Thu, 31 Jan 2008, Jeff Chua wrote: On the bad run, I got the following message ... boston kernel: nf_ct_tcp: invalid packed ignored IN= OUT= SRC=127.0.0.1 DST=127.0.0.1 LEN=60 TOS=0x00 PREC=0x00 TTL=64 ID=8162 DF PROTO=TCP SPT=1016 DPT=515 SEQ=3834958843 AC

Re: cups slow on linux-2.6.24

2008-01-31 Thread Patrick McHardy
David Newall wrote: I'm not debating that checksums are wrong. The question was how and where? It's not as if there are any unreliable communication paths in a loopback interface, so it's surprising that they could be wrong. How? Where? As I said, loopback doesn't perform full checksum calcula

Re: [build bug] net/sched/cls_flow.c:598: error: 'struct tcf_ematch_tree' has no member named 'hdr'

2008-02-01 Thread Patrick McHardy
networking merge i guess. Oops, thanks Ingo. Attached patch should fix it. commit 28383868d7999539f35ae0c2dfbde103a1d3 Author: Patrick McHardy <[EMAIL PROTECTED]> Date: Fri Feb 1 16:50:44 2008 +0100 [NET_SCHED]: cls_flow: fix build error with CONFIG_NET_EMATCH=n Rep

Re: Typo in net/netfilter/xt_iprange.c (git tree)

2008-02-02 Thread Patrick McHardy
Jiri Moravec wrote: Function iprange_mt4 belong to IPv4 family - AF_INET. Right? .name = "iprange", .revision = 1, .family= AF_INET6, <-- Typo? .match = iprange_mt4, Fixed, thanks Jiri. -- To unsubscribe from this list: send the line "unsub

Re: cups slow on linux-2.6.24

2008-02-05 Thread Patrick McHardy
Jozsef Kadlecsik wrote: On Tue, 5 Feb 2008, Jeff Chua wrote: On Feb 5, 2008 4:17 AM, Jozsef Kadlecsik <[EMAIL PROTECTED]> wrote: Actively closed connections are not handled properly, i.e. the initiator of the active close should not be taken into account. So could you give a try to the patch

Re: Netfilter fixes to 2.6.24-git

2008-02-10 Thread Patrick McHardy
David Miller wrote: From: Jan Engelhardt <[EMAIL PROTECTED]> Date: Sun, 10 Feb 2008 22:02:35 +0100 (CET) I have been unable to reach the netfilter and net maintainers the past week regarding inclusion of patches, but most importantly a group of fixes at [0]-[3]. I am kind of at a loss here but

Re: cups slow on linux-2.6.24

2008-02-14 Thread Patrick McHardy
David Miller wrote: From: Jozsef Kadlecsik <[EMAIL PROTECTED]> Date: Thu, 14 Feb 2008 16:02:29 +0100 (CET) Hi, On Sun, 10 Feb 2008, Jeff Chua wrote: Please note the lastest git commit is missing one part (which was in Jozsef's original patch). Sorry everyone, that's my fault: the patch I sen

Re: [GIT] Networking

2013-05-02 Thread Patrick McHardy
On Thu, May 02, 2013 at 02:45:52AM -0400, David Miller wrote: > From: Linus Torvalds > Date: Wed, 1 May 2013 21:55:38 -0700 > > > I think the positions of those bits matter, and adding > > NETIF_F_HW_VLAN_STAG_*_BIT randomly in the middle broke things. That's > > backed up by the fact that we hav

Re: [GIT] Networking

2013-05-02 Thread Patrick McHardy
On Thu, May 02, 2013 at 04:16:25AM -0400, David Miller wrote: > From: Patrick McHardy > Date: Thu, 2 May 2013 09:03:37 +0200 > > > I'll also have a look at this. > > By the mere existence of /sys/devices/${DEV_PATH}/net/${netdev_name}/flags > we have to preserve

Re: [2.6 patch] remove include/linux/netfilter_ipv4/ipt_SAME.h

2008-02-18 Thread Patrick McHardy
Adrian Bunk wrote: This patch removes the no longer used include/linux/netfilter_ipv4/ipt_SAME.h We kept it around because old iptables binaries need it to build. The kernel no longer supports it, but people might still wish to use a distributor-built iptables binary with old kernels. It will

Re: [RFC PATCH 1/8] [NET]: uninline skb_put, de-bloats a lot

2008-02-20 Thread Patrick McHardy
Ilpo Järvinen wrote: ~500 files changed ... kernel/uninlined.c: skb_put | +104 1 function changed, 104 bytes added, diff: +104 vmlinux.o: 869 functions changed, 198 bytes added, 111003 bytes removed, diff: -110805 This change is INCOMPLETE, I think that the call to cur

Re: [RFC PATCH 3/8] [NET]: uninline dev_alloc_skb, de-bloats a lot

2008-02-20 Thread Patrick McHardy
Jan Engelhardt wrote: On Feb 20 2008 15:47, Ilpo Järvinen wrote: -23668 392 funcs, 104 +, 23772 -, diff: -23668 --- dev_alloc_skb -static inline struct sk_buff *dev_alloc_skb(unsigned int length) -{ - return __dev_alloc_skb(length, GFP_ATOMIC); -} +extern struct sk_buff *dev_alloc_skb(un

Re: 2.6.25-rc2-mm1 - several bugs and a crash

2008-02-20 Thread Patrick McHardy
Tilman Schmidt wrote: Still, X came up fine, I could log in (Gnome feeling subjectively a bit sluggish), call up a web page from the Internet in Firefox, and start perusing the logs, when the whole system froze: neither mouse nor keyboard would react anymore, and only the Wind^Wreset button would

Re: 2.6.25-rc2-mm1 - several bugs and a crash

2008-02-21 Thread Patrick McHardy
736b33102292be0d75be1e950ca9bcd5361db7dd Author: Patrick McHardy <[EMAIL PROTECTED]> Date: Thu Feb 21 12:26:01 2008 +0100 [NETFILTER]: nf_conntrack: fix smp_processor_id() in preemptible code warning Since we're using RCU for the conntrack hash now, we need to avoid

Re: 2.6.25-rc2-mm1 - several bugs and a crash

2008-02-21 Thread Patrick McHardy
Stephen Hemminger wrote: On Thu, 21 Feb 2008 12:28:50 +0100 Patrick McHardy <[EMAIL PROTECTED]> wrote: diff --git a/net/netfilter/nf_conntrack_core.c b/net/netfilter/nf_conntrack_core.c index 327e847..b77eb56 100644 --- a/net/netfilter/nf_conntrack_core.c +++ b/net/net

Re: [announce] Xtables, Xtables-addons 1.5.1 and Writing Xtables Modules

2008-02-21 Thread Patrick McHardy
Jan Engelhardt wrote: I have released “Xtables” 1.5.1, which is a package of my ongoing iptables development¹ that I did lately. Patrick McHardy was not available last week to merge patches due to higher powers, so I branched off the iptables subversion trunk into git since quilting on top of

Re: 2.6.25-rc3: "__divdi3" [drivers/crypto/hifn_795x.ko] undefined!

2008-02-26 Thread Patrick McHardy
Martin Michlmayr wrote: With 2.6.25-rc3 and a config file with CONFIG_CRYPTO_DEV_HIFN_795X=m CONFIG_CRYPTO_DEV_HIFN_795X_RNG=y I get the following build error on at least ARM and MIPS: Building modules, stage 2. MODPOST 759 modules ERROR: "__divdi3" [drivers/crypto/hifn_795x.ko] undefined!

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 anymore

Re: 2.6.12 netfilter: local packets marked as invalid

2005-07-12 Thread Patrick McHardy
[NETFILTER]: 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 PROT

Re: 2.6.12 netfilter: local packets marked as invalid

2005-07-12 Thread Patrick McHardy
onnection-tracking.patch Daniel's 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://vg

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 Check(x-

Re: [PATCH 2.6.13rc3] IPv6: Check interface bindings on IPv6 raw socket reception

2005-07-23 Thread Patrick McHardy
actually delivered to a raw socket to decide whether to send an ICMP unreachable Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> --- commit 0fc074eed6ad201d76392dcabbc156784570ea64 tree 913c87ad866f4abb23e9788c0170b81b564cfe7a parent 36245c9aaddbed7e50d9600c8d1889ebee48a90f author Patrick M

Re: [PATCH 2.6.13rc3] IPv6: Check interface bindings on IPv6 raw socket reception

2005-07-23 Thread Patrick McHardy
Patrick McHardy wrote: Andrew McDonald wrote: Take account of whether a socket is bound to a particular device when selecting an IPv6 raw socket to receive a packet. Also perform this check when receiving IPv6 packets with router alert options. I guess this one makes sense on top. [IPV4/6

Re: Netlink connector

2005-07-25 Thread Patrick McHardy
Evgeniy Polyakov wrote: On Mon, Jul 25, 2005 at 02:02:10AM -0400, James Morris ([EMAIL PROTECTED]) wrote: On Sun, 24 Jul 2005, David S. Miller wrote: From: Evgeniy Polyakov <[EMAIL PROTECTED]> Date: Sat, 23 Jul 2005 13:14:55 +0400 Andrew has no objection against connector and it lives in

Re: Netlink connector

2005-07-25 Thread Patrick McHardy
Evgeniy Polyakov wrote: On Mon, Jul 25, 2005 at 04:32:32PM +0200, Patrick McHardy ([EMAIL PROTECTED]) wrote: If I understand correctly it tries to workaround some netlink limitations (limited number of netlink families and multicast groups) by sending everything to userspace and

Re: Netlink connector

2005-07-25 Thread Patrick McHardy
Thomas Graf wrote: * Patrick McHardy <[EMAIL PROTECTED]> 2005-07-26 01:46 You still have to take care of mixed 64/32 bit environments, u64 fields for example are differently alligned. My solution to this (in the same patchset) is that we never derference u64s but instead copy them. I

Re: [ANNOUNCE 0/7] Open-iSCSI/Linux-iSCSI-5 High-Performance Initiator

2005-08-06 Thread Patrick McHardy
David S. Miller wrote: From: James Bottomley <[EMAIL PROTECTED]> Date: Sat, 30 Jul 2005 15:23:20 -0500 Actually, I saw this and increased MAX_LINKS as well. That does absolutely nothing, you cannot create sockets with protocol numbers larger than NPROTOS which like MAX_LINKS has the value 32.

Re: 2.6.13-rc4-mm1: iptables DROP crashes the computer

2005-08-07 Thread Patrick McHardy
esn't need to drop all references, so flushing is moved to the cleanup path. Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> --- commit d20d18715b8425060334e879ebb4835202457b3e tree 3279243f59162c14f8ac145473e1c3f4c586b2fa parent df2e0392536ecdd6385f4319f746045fd6fae38f author Patr

Re: [PATCH] net/ipv4 debug cleanup, kernel 2.6.13-rc5

2005-08-09 Thread Patrick McHardy
them to pure printk wrappers? [NET]: Make NETDEBUG pure printk wrappers Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> --- commit a2db7bcdba3678fe8f67cd7d631c01a888031753 tree 7201cec98ca35b5854daebc14e4650ff95eb8571 parent db29e85a7ece62de1899917c1ec0ffe55cf1d3a0 author Patrick McHardy

Re: 2.6.13-rc5-mm1: BUG: rwlock recursion on CPU#0

2005-08-14 Thread Patrick McHardy
Rafael J. Wysocki wrote: > I've got the following BUG on Asus L5D (x86-64) with the 2.6.13-rc5-mm1 > kernel: > > BUG: rwlock recursion on CPU#0, nscd/3668, 8817d4a0 > > Call Trace:{add_preempt_count+105} > {rwlock_bug+114} >{_raw_write_lock+62} > {_write_lock_bh+40} >{:

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 a

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: 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 th

Re: BUG: atomic counter underflow at ip_conntrack_event_cache_init+0x91/0xb0 (with patch)

2005-08-01 Thread Patrick McHardy
ch is probably why it makes the underflow go away :) Please try this patch instead. [NETFILTER]: Fix refcnt underflow in ip_conntrack_event_cache_init Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> --- commit 5d55b8c6bfba6e6e2ffe26c2a2e2561e278428b7 tree d43366a793d2fa3058c15a752010ef0fd2289

Re: BUG: atomic counter underflow at ip_conntrack_event_cache_init+0x91/0xb0 (with patch)

2005-08-01 Thread Patrick McHardy
Mattia Dongili wrote: > On Mon, Aug 01, 2005 at 04:27:53PM +0200, Patrick McHardy wrote: > >>>--- include/linux/netfilter_ipv4/ip_conntrack.h.clean2005-08-01 >>>15:09:49.0 +0200 >>>+++ include/linux/netfilter_ipv4/ip_conntrack.h 2005-0

Re: [PATCH] net/sunrpc: fix time conversion error

2005-08-01 Thread Patrick McHardy
Nishanth Aravamudan wrote: > On 01.08.2005 [15:11:48 -0600], Josip Loncaric wrote: > >>Line 589 of linux-2.6.11.10/net/sunrpc/svcsock.c is obviously wrong: >> >>skb->stamp.tv_usec = xtime.tv_nsec * 1000; >> >>To convert nsec to usec, one should divide instead of multiplying: >> >>

Re: BUG: atomic counter underflow at ip_conntrack_event_cache_init+0x91/0xb0 (with patch)

2005-08-01 Thread Patrick McHardy
Patrick McHardy wrote: > Mattia Dongili wrote: > >>On Mon, Aug 01, 2005 at 04:27:53PM +0200, Patrick McHardy wrote: >> >> >>>>--- include/linux/netfilter_ipv4/ip_conntrack.h.clean 2005-08-01 >>>>15:09:49.0 +0200 >>>>++

Re: [netfilter-core] iptables redirect is broken on bridged setup

2005-08-02 Thread Patrick McHardy
Denis Vlasenko wrote: Linux 2.6.12 Was running for months with this simple iptables rule: iptables -t nat -A PREROUTING -s 172.17.6.44 -d 172.16.42.201 -p tcp --dport 9100 -j REDIRECT --to 9123 Chain PREROUTING (policy ACCEPT 0 packets, 0 bytes) 00 REDIRECT tcp -- * *

Re: BUG: atomic counter underflow at ip_conntrack_event_cache_init+0x91/0xb0 (with patch)

2005-08-03 Thread Patrick McHardy
Mattia Dongili wrote: > On Tue, Aug 02, 2005 at 02:47:55AM +0200, Patrick McHardy wrote: > >>This should be a fist step towards fixing it. It's probably incomplete >>(I'm too tired to check it now), but it should fix the problem you're >>seeing. Could y

Re: [PATCH] fix dst_entry leak in icmp_push_reply()

2005-08-17 Thread Patrick McHardy
Ollie Wild wrote: If the ip_append_data() call in icmp_push_reply() fails, ip_flush_pending_frames() needs to be called. Otherwise, ip_rt_put() is never called on inet_sk(icmp_socket->sk)->cork.rt, which prevents the route (and net_device) from ever being freed. I've attached a patch which f

Re: [PATCH] fix dst_entry leak in icmp_push_reply()

2005-08-18 Thread Patrick McHardy
Ollie Wild wrote: > Patrick McHardy wrote: > >> Ollie Wild wrote: >> >>> If the ip_append_data() call in icmp_push_reply() fails, >>> ip_flush_pending_frames() needs to be called. Otherwise, ip_rt_put() >>> is never called on inet_sk(icmp_socket->

Re: [PATCH] fix dst_entry leak in icmp_push_reply()

2005-08-18 Thread Patrick McHardy
aner to me. Patch attached. Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> diff --git a/net/ipv4/icmp.c b/net/ipv4/icmp.c --- a/net/ipv4/icmp.c +++ b/net/ipv4/icmp.c @@ -349,12 +349,12 @@ static void icmp_push_reply(struct icmp_ { struct sk_buff *skb; - ip_append_

Re: 2.6.12 Performance problems

2005-08-21 Thread Patrick McHardy
Danial Thom wrote: --- Patrick McHardy <[EMAIL PROTECTED]> wrote: Do you have netfilter enabled? Briging netfilter was added in 2.6, enabling it will influence performance negatively. Otherwise, is this performance drop visible in other setups besides bridging as well? Yes, bridg

Re: 2.6.12 Performance problems

2005-08-21 Thread Patrick McHardy
Danial Thom wrote: I just started fiddling with 2.6.12, and there seems to be a big drop-off in performance from 2.4.x in terms of networking on a uniprocessor system. Just bridging packets through the machine, 2.6.12 starts dropping packets at ~100Kpps, whereas 2.4.x doesn't start dropping until

Re: tcpdump confused with NAT-T+IPSec Packets

2005-08-22 Thread Patrick McHardy
Gopalakrishnan Raman wrote: Hi I'm using 2.6.11.7 and debugging why my ESP tunnel mode does not work between two 2.6 machines one of which is behind a NAT. I'm using tcpdump to capture NAT-T packets on one of the hosts and using espdecrypt (http://www.cs.rpi.edu/~flemej/freebsd/espdecrypt/) to se

Re: 2.6.12 Performance problems

2005-08-23 Thread Patrick McHardy
Danial Thom wrote: > I think part of the problem is the continued > misuse of the word "latency". Latency, in > language terms, means "unexplained delay". Its > wrong here because for one, its explainable. But > it also depends on your perspective. The > "latency" is increased for kernel tasks, whi

Re: 2.6.12 Performance problems

2005-08-23 Thread Patrick McHardy
Danial Thom wrote: > None of this is helpful, but since no one has > been able to tell me how to tune it to provide > absolute priority to the network stack I'll > assume it can't be done. The network stack already has priority over user processes, except when executed in process context, so preem

Re: Conntrack problem, machines freeze

2005-08-26 Thread Patrick McHardy
Lukasz Spaleniak wrote: > Hello, > > I have simple linux router with three fastethernet cards (intel , e100 > driver). About two months ago it started hanging. It's completly > freezing machine (no ooops. First of all when it's booting few > messages like this appears on screen: > > NF_IP_ASSERT:

Re: oops in 2.6.13-rc6-git12 in tcp/netfilter routines

2005-08-26 Thread Patrick McHardy
Alessandro Suardi wrote: > Stack is hand-copied from the dead box's console. > > [] die+0xe4/0x170 > [] do_trap+0x7f/0xc0 > [] do_invalid_op+0xa3/0xb0 > [] error_code+0x4f/0x54 > [] kfree_skbmem+0xb/0x20 > [] __kfree_skb+0x5f/0xf0 > [] tcp_clean_rtx_queue+0x16a/0x470 > [] tcp_ack+0xf6/0x360 > []

Re: Kernel 2.6.13: TCP (libnet?)

2005-08-30 Thread Patrick McHardy
John McGowan wrote: > Kernel 2.6.13: TCP (libnet?) > > Broken libnet? > > KERNEL: linux-kernel@vger.kernel.org > LIBNET 1.1 (c) 1998 - 2004 Mike D. Schiffman <[EMAIL PROTECTED]> > > I don't like spam. I track spamvertized sites. Many only respond to TCP > packets sent to port 80. I need a TCP tr

Re: ip_queue.c and TCP resets

2005-08-30 Thread Patrick McHardy
Michael Rash wrote: > Attached is a patch against > linux-2.6.11.12/net/ipv4/netfilter/ip_queue.c to put Ethernet MAC > addresses directly into the indev_name and outdev_name portions of the > ipq_packet_msg struct. This is a total kludge and I doubt anyone else > will find this useful, but for li

Re: Kernel 2.6.13 breaks libpcap (and tcpdump).

2005-09-04 Thread Patrick McHardy
Herbert Xu wrote: We aren't handling the reading of specific fields like the IP protocol field correctly. This patch should make it work again. I can't spot the problem, could you give me a hint? I tried to move this logic into the new load_pointer function but it all came out messy so I sim

Re: Kernel 2.6.13 breaks libpcap (and tcpdump).

2005-09-04 Thread Patrick McHardy
Patrick McHardy wrote: Herbert Xu wrote: We aren't handling the reading of specific fields like the IP protocol field correctly. This patch should make it work again. I can't spot the problem, could you give me a hint? Never mind, I got it, we never fall through to the sec

Re: proto_unregister sleeps while atomic

2005-09-06 Thread Patrick McHardy
st-operation which it is supposed to protect. [NET]: proto_unregister: fix sleeping while atomic proto_unregister holds a lock while calling kmem_cache_destroy, which can sleep. Noticed by Daniele Orlandi <[EMAIL PROTECTED]>. Signed-off-by: Patrick McHardy &

Re: proto_unregister sleeps while atomic

2005-09-06 Thread Patrick McHardy
David S. Miller wrote: From: Patrick McHardy <[EMAIL PROTECTED]> Date: Wed, 07 Sep 2005 01:02:01 +0200 You're right, good catch. This patch fixes it by moving the lock down to the list-operation which it is supposed to protect. I think we need to unlink from the list first if y

Re: [PATCH] rmmod notifier chain

2005-09-08 Thread Patrick McHardy
Jan Beulich wrote: > Debugging and maintenance support code occasionally needs to know not > only of module insertions, but also modulke removals. This adds a > notifier > chain for this purpose. > > > diff -Npru 2.6.13/kernel/module.c > 2.6.13-rmmod-notifier/kernel/module.c > --- 2.6.13/kernel/mo

Re: [PATCH] ip[6]_tables.c: remove some inlines

2008-01-08 Thread Patrick McHardy
Denys Vlasenko wrote: On Monday 31 December 2007 17:00, Patrick McHardy wrote: Denys Vlasenko wrote: ip[6]_tables.c seem to abuse inline. This patch removes inlines except those which are used by packet matching code and thus are performance-critical. Some people also consider the ruleset

Re: [RFD] iptables: mangle table obsoletes filter table

2007-10-11 Thread Patrick McHardy
Please send mails discussing netfilter to netfilter-devel. Al Boldi wrote: > With the existence of the mangle table, how useful is the filter table? > > Other than requiring the REJECT target to be ported to the mangle table, is > the filter table faster than the mangle table? There are some mi

Re: [PATCH] net/ipv4/netfilter/ip_tables.c: remove some inlines

2007-12-28 Thread Patrick McHardy
Denys Vlasenko wrote: On Monday 17 December 2007 14:47, Patrick McHardy wrote: This clashes with my pending patches, which I'll push upstream today. I also spent some time resyncing ip_tables and ip6_tables so a diff of both (with some sed'ing) shows only the actual differences,

Re: [PATCH] ip[6]_tables.c: remove some inlines

2007-12-31 Thread Patrick McHardy
Denys Vlasenko wrote: ip[6]_tables.c seem to abuse inline. This patch removes inlines except those which are used by packet matching code and thus are performance-critical. Some people also consider the ruleset replacement path performance critical, but overall I agree with your patch. I'm

Re: constant_tsc and TSC unstable

2007-11-29 Thread Patrick McHardy
H. Peter Anvin wrote: Paul Rolland (ポール・ロラン) wrote: Hello, I've a machine with a Core2Duo CPU. /proc/cpuinfo reports the flag constant_tsc, but at boot time, I have the log : ... Total of 2 processors activated (6919.15 BogoMIPS). ENABLING IO-APIC IRQs ..TIMER: vector=0x31 apic1=0 pin1=2 apic2

Re: [PATCH] Fix forgotten module release in xt_CONNMARK and xt_CONNSECMARK

2007-11-29 Thread Patrick McHardy
Jan Engelhardt wrote: Fix forgotten module release in xt_CONNMARK and xt_CONNSECMARK When xt_CONNMARK is used outside the mangle table and the user specified "--restore-mark", the connmark_tg_check() function will (correctly) error out, but (incorrectly) forgets to release the L3 conntrack modu

Re: namespace support requires network modules to say "GPL"

2007-12-02 Thread Patrick McHardy
Ben Greear wrote: Stephen Hemminger wrote: Naw, enterprise (or any other) distro vendors shouldn't have any issues here, since they can just patch their kernels around any issues. But it looks like Eric has this one thought out well enough. So you are saying all this is not a problem,

Re: namespace support requires network modules to say "GPL"

2007-12-02 Thread Patrick McHardy
Adrian Bunk wrote: On Sun, Dec 02, 2007 at 09:03:56PM +0100, Patrick McHardy wrote: For all I care binary modules can break, but frankly I don't see how encapsulating a couple of structures and pointers in a new structure and adding a new argument to existing functions shifts the dec

Re: [HIFN 00/03]: RNG support v2

2007-11-19 Thread Patrick McHardy
Michael Buesch wrote: On Sunday 18 November 2007 22:32:52 Patrick McHardy wrote: These patches add support for using the HIFN rng. Acked-by: Michael Buesch <[EMAIL PROTECTED]> Patrick, can you send this patchset to Andrew for inclusion into -mm? I think he already pulls from Her

Re: [PATCH 1/4] xfrm_hash: kmalloc + memset conversion to kzalloc

2007-11-26 Thread Patrick McHardy
Joonwoo Park wrote: diff --git a/net/xfrm/xfrm_hash.c b/net/xfrm/xfrm_hash.c index 55ab579..37795bd 100644 --- a/net/xfrm/xfrm_hash.c +++ b/net/xfrm/xfrm_hash.c @@ -17,16 +17,17 @@ struct hlist_head *xfrm_hash_alloc(unsigned int sz) struct hlist_head *n; if (sz <= PAGE_SIZE) -

Re: Linux 2.6.23.9

2007-11-26 Thread Patrick McHardy
Tomasz K wrote: On Mon, 26 Nov 2007, Greg Kroah-Hartman wrote: [..] Still there is no aroud officialy released iptables tarball with support for rules for new xt_{connlimit,time,u32} modules. Anyone know where are patches for manage connlimit, time, u32 rules which will be included in next rel

Re: sparc oops in ip_fast_csum

2008-01-13 Thread Patrick McHardy
Herbert Xu wrote: Patrick, please have a look at the former. In fact it's not just that ihl may be bogus (which might be harmless as long as the REJECT hook only gets called from within the IP stack), I think REJECT would also do the wrong thing if the packet had IP options. So perhaps we shoul

Re: 2.6.24-rc7-git4: Reported regressions from 2.6.23

2008-01-15 Thread Patrick McHardy
ables won't work Submitter : Kristoffer Malmström <[EMAIL PROTECTED]> Date: 2007-12-28 References : http://bugzilla.kernel.org/show_bug.cgi?id=9657 Handled-By : Patrick McHardy <[EMAIL PROTECTED]> The reporter has disappeared, so I guess this wasn't

Re: [PATCH] net/ipv4/netfilter/ip_tables.c: remove some inlines

2007-12-17 Thread Patrick McHardy
Please CC netfilter-devel on netfilter patches. Denys Vlasenko wrote: Hi Patrick, Harald, I was working on unrelated problem and noticed that ip_tables.c seem to abuse inline. I prepared a patch which removes inlines except those which are used by packet matching code (and thus are really perfo

Re: "ip neigh show" not showing arp cache entries?

2007-12-17 Thread Patrick McHardy
Chris Friesen wrote: The original "ip" command and the new one ("/tmp/ip") both give the same results--some of the entries are missing. [EMAIL PROTECTED]:/root> ip neigh show all 172.24.137.0 dev bond0 FAILED 172.24.0.9 dev bond0 lladdr 00:07:e9:41:4b:b4 REACHABLE 10.41.18.101 dev eth6 lladdr

Re: [PATCH] net/netfilter/: Spelling fixes

2007-12-17 Thread Patrick McHardy
Joe Perches wrote: Signed-off-by: Joe Perches <[EMAIL PROTECTED]> --- net/netfilter/nf_conntrack_sip.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) Both your netfilter patches conflict with some pending patches of mine (which also fix those spelling errors). So for now I'm go

Re: "ip neigh show" not showing arp cache entries?

2007-12-17 Thread Patrick McHardy
Chris Friesen wrote: Patrick McHardy wrote: From a kernel perspective there are only complete dumps, the filtering is done by iproute. So the fact that it shows them when querying specifically implies there is a bug in the iproute neighbour filter. Does it work if you omit "all" f

Re: "ip neigh show" not showing arp cache entries?

2007-12-17 Thread Patrick McHardy
Thomas Graf wrote: * Patrick McHardy <[EMAIL PROTECTED]> 2007-12-18 00:51 Chris Friesen wrote: Patrick McHardy wrote: From a kernel perspective there are only complete dumps, the filtering is done by iproute. So the fact that it shows them when querying specifically implies there is a

Re: cups slow on linux-2.6.24

2008-01-30 Thread Patrick McHardy
Jeff Chua wrote: On Jan 29, 2008 6:53 PM, Jozsef Kadlecsik <[EMAIL PROTECTED]> wrote: As the problem can be reproduced so easily, could you capture a full TCP session and send the pcap file? Thus it could be analyzed, replayed, etc. and found the reason why the patch above slows down the printi

Re: [build bug] ip_queue.c:(.init.text+0x322c): undefined reference to `net_ipv4_ctl_path'

2008-01-30 Thread Patrick McHardy
Ingo Molnar wrote: randconfig testing found the following build failure on latest -git: net/built-in.o: In function `ip_queue_init': ip_queue.c:(.init.text+0x322c): undefined reference to `net_ipv4_ctl_path' config attached. Thanks, the reason is CONFIG_SYSCTL=n. I've queued this patch (mig

[RCU]: preemptible RCU: sparse annotations

2008-01-30 Thread Patrick McHardy
Annotate rcu_read_lock/rcu_read_unlock for sparse. [RCU]: preemptible RCU: sparse annotations Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]> diff --git a/include/linux/rcupreempt.h b/include/linux/rcupreempt.h index ece8eb3..60c2a03 100644 --- a/include/linux/rcupreempt.h +++ b/i

Re: [2.6 patch] make nf_ct_path[] static

2008-01-30 Thread Patrick McHardy
Adrian Bunk wrote: This patch makes the needlessly global nf_ct_path[] static. I already have this queued. -- 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

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 modu

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

2005-03-16 Thread Patrick McHardy
s bug, 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 P

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 = --- 1.10/net/i

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 first

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'

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'

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 generated p

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 simil

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 derefe

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: 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 <[EMAI

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 p

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 ide

Re: Last night Linus bk - netfilter busted?

2005-03-11 Thread Patrick McHardy
eper generated diff -Nru style 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

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 va

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 s

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   >