linux-netdev mailing list

FromSubjectsort iconDate
Varun Chandramohan
[RESEND][PATCH 4/4 Rev-3] Initialize and fill IPv6 route age
The age field of the ipv6 route structures are initilized with the current timeval at the time of route creation. When the route dump is called the route age value stored in the structure is subtracted from the present timeval and the difference is passed on as the route age. Signed-off-by: Varun Chandramohan <varunc@linux.vnet.ibm.com> --- include/net/ip6_fib.h | 1 + include/net/ip6_route.h | 3 +++ net/ipv6/addrconf.c | 5 +++++ net/ipv6/route.c | 24 +++++++++++++...
Sep 18, 11:55 pm 2007
Varun Chandramohan
[RESEND][PATCH 3/4 Rev-3] Initilize and populate age field
The age field is filled with the current time at the time of creation of the route. When the routes are dumped then the age value stored in the route structure is subtracted from the current time value and the difference is the age expressed in secs. Signed-off-by: Varun Chandramohan <varunc@linux.vnet.ibm.com> --- net/ipv4/fib_hash.c | 3 +++ net/ipv4/fib_lookup.h | 3 ++- net/ipv4/fib_semantics.c | 20 +++++++++++++++++--- net/ipv4/fib_trie.c | 1 + 4 files changed...
Sep 18, 11:54 pm 2007
Varun Chandramohan
[RESEND][PATCH 2/4 Rev-3] Add new timeval_to_sec function
A new function for converting timeval to time_t is added in netlink.h. Its a common function used in differentplaces. The reason for adding this function in netlink.h is that its used by netlink for stats purpose. Signed-off-by: Varun Chandramohan <varunc@linux.vnet.ibm.com> --- include/net/netlink.h | 13 +++++++++++++ 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/include/net/netlink.h b/include/net/netlink.h index d7b824b..f86cc59 100644 --- a/include/net/netlink.h ++...
Sep 18, 11:54 pm 2007
Varun Chandramohan
[RESEND][PATCH 1/4 Rev-3] New attribute RTA_AGE
A new attribute RTA_AGE is added for the age value to be exported to userlevel using netlink Signed-off-by: Varun Chandramohan <varunc@linux.vnet.ibm.com> --- include/linux/rtnetlink.h | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/linux/rtnetlink.h b/include/linux/rtnetlink.h index c91476c..68046a4 100644 --- a/include/linux/rtnetlink.h +++ b/include/linux/rtnetlink.h @@ -263,6 +263,7 @@ enum rtattr_type_t RTA_SESSION, RTA_MP_ALGO, /* no longer use...
Sep 18, 11:53 pm 2007
Varun Chandramohan
[RESEND][PATCH 0/4 Rev-3] Age Entry For IPv4 & IPv6 Rout...
Hi Dave, Iam resending the patch set again. Let me know if you need anything else. Regards, Varun Original Comment: According to the RFC 4292 (IP Forwarding Table MIB) there is a need for an age entry for all the routes in therouting table. The entry in the RFC is inetCidrRouteAge and oid is inetCidrRouteAge.1.10. Many snmp application require this age entry. So iam adding the age field in the routing table for ipv4 and ipv6 and providing the interface for this value netlink. I made a note of ...
Sep 18, 11:53 pm 2007
David Miller
[PATCH]: Preliminary release of Sun Neptune driver
This is a preliminary posting of my Sun Neptune driver against the net-2.6.24 tree. The TODO list is very long, but it's a lot of small things, most of the basic substance of the driver is all there. Things which are known to be broken: 1) Niagara-2 frontend not written, I have recently received the hardware necessary to code that part up. 2) Jumbo MTU. I have to add some code to adjust the RBR sizes when the MTU goes above 1500, should be about an afternoon of work. 3) Need ...
Sep 18, 6:15 pm 2007
John W. Linville
Please pull 'rt2x00' branch of wireless-2.6
Jeff & Dave, This patch adds the rt2x00 drivers for Ralink wireless hardware. This collection of drivers has seen lots of action in Fedora (both F7 and rawhide) and many people are using them with good results (although some problems do remain). Ivo in particular has been very helpful in responding to bug reports for these drivers, including bugs reported at kernel.org and in distro bugzillas. I extend my thanks to him for both his past and future support of these drivers. I'm sure he and ...
Sep 18, 5:32 pm 2007
David Miller
Re: Please pull 'rt2x00' branch of wireless-2.6
From: "John W. Linville" <linville@tuxdriver.com> Applied to net-2.6.24 and pushed out. Thanks John. -
Sep 18, 6:32 pm 2007
Samo Pogacnik
network functionality - SACIP
Hi, I want to introduce a new network functionality, where established network connections could be preserved after the IP address change of one or both endpoints of communication. A general document (not yet finished) about SACIP (Session Aware Change of IP) and the patch are available at: http://84.255.254.67/sacip-0.1-en.pdf http://84.255.254.67/patch-linux-2.6.19-sacip Short implementation description: --------------------------------- To be able to preserve established network connec...
Sep 18, 6:06 pm 2007
James Chapman
[PATCH net-2.6.24] [L2TP] Fix lockup introduced by recent ch...
[L2TP] Fix double skb free problem introduced by recent changes This patch is to be applied on top of Herbert's recent patch set: "[PPP 3/3] L2TP: Fix skb handling in pppol2tp_xmit" It fixes a double kfree_skb() issue introduced by that patch. Also return 0 on error, allowing PPP to requeue the skb. Signed-off-by: James Chapman <jchapman@katalix.com> Index: net-2.6.24/drivers/net/pppol2tp.c =================================================================== --- net-2.6.24.orig/drivers...
Sep 18, 5:44 pm 2007
David Miller
Re: [PATCH net-2.6.24] [L2TP] Fix lockup introduced by recen...
From: James Chapman <jchapman@katalix.com> Thanks James. I've applied this to net-2.6 Herbert, please take a quick look, thanks. -
Sep 18, 5:51 pm 2007
Herbert Xu
Re: [PATCH net-2.6.24] [L2TP] Fix lockup introduced by recen...
Yep it looks good. I should've read my emails from the bottom up :) Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <herbert@gondor.apana.org.au> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt -
Sep 18, 9:37 pm 2007
Holger Eitzenberger
[FIX NETLINK] properly check arguments to netlink_bind()
Hi, while going through going netlink code I found out that netlink_bind() does not properly check bind parameters. I checked both 2.6.23-rc1 as well as 2.6.16.53, both are affected. With a small test prog I wasn't able to crash my maschine though, but data was accessed out of bounds. Please apply the attached patch. Thanks. /holger -
Sep 18, 5:01 pm 2007
Holger Eitzenberger
Re: [FIX NETLINK] properly check arguments to netlink_bind()
See my attached patch, thanks. /holger
Sep 18, 5:05 pm 2007
David Miller
Re: [FIX NETLINK] properly check arguments to netlink_bind()
From: Holger Eitzenberger <holger@my-eitzenberger.de> Your patch is incorrect and also unnecessary. Firstly, you patch compares the address _pointer_ against the minimum length. That's obviously wrong. And if you check the call sites of the protocol ->bind() methods, they all use on-stack buffer for the address object which is at least MAX_SOCK_ADDR bytes in length so that the bind methods don't have to check the size if they don't want to. -
Sep 18, 5:53 pm 2007
Holger Eitzenberger
Re: [FIX NETLINK] properly check arguments to netlink_bind()
The previous patchlet wasn't correct, please apply this one. Sorry for the noise. /holger
Sep 18, 5:14 pm 2007
John W. Linville
Please pull 'b43' branch of wireless-2.6
Jeff & Dave, This series adds the b43 and b43legacy drivers, as well as the ssb bus infrastructure upon which they depend. These drivers support the BCM43xx wireless hardware from Broadcom. The b43 driver supports modern hardware using the "version 4" firmware, while the b43legacy driver uses the "version 3" firmware to support some older devices which lack the resources to run the newer (and more feature rich) firmware. Both drivers have performed well in -mm and Fedora rawhide, and b43 ha...
Sep 18, 4:28 pm 2007
David Miller
Re: Please pull 'b43' branch of wireless-2.6
From: "John W. Linville" <linville@tuxdriver.com> John, I'm adding this and the iwlwifi bits to my net-2.6.24 tree. Thanks. -
Sep 18, 5:32 pm 2007
James Chapman
Re: [PATCH 3/3] [PPP] L2TP: Fix skb handling in pppol2tp_xmit
This one causes my test system to lock up. I'll investigate. Please don't apply this patch for now. -- James Chapman Katalix Systems Ltd http://www.katalix.com Catalysts for your Embedded Linux software development -
Sep 18, 4:19 pm 2007
Herbert Xu
Re: [PATCH 3/3] [PPP] L2TP: Fix skb handling in pppol2tp_xmit
Sorry, I added a double-free on the skb after ip_queue_xmit. Please try this one instead. [PPP] L2TP: Fix skb handling in pppol2tp_xmit This patch makes pppol2tp_xmit call skb_cow_head so that we don't modify cloned skb data. It also gets rid of skb2 we only need to preserve the original skb for congestion notification, which is only applicable for ppp_async and ppp_sync. The other semantic change made here is the removal of socket accounting for data tranmitted out of pppol2tp_xmit. The o...
Sep 18, 9:25 pm 2007
David Miller
Re: [PATCH 3/3] [PPP] L2TP: Fix skb handling in pppol2tp_xmit
From: James Chapman <jchapman@katalix.com> I'll make sure not to push this until we figure out what's wrong, thanks for checking James. -
Sep 18, 4:32 pm 2007
Herbert Xu
Re: [PATCH 3/3] [PPP] L2TP: Fix skb handling in pppol2tp_xmit
I think it was because of a double-free that I created after transmitting the packet. In fact I had the same bug in PPPOE too. [PPP] pppoe: Fix double-free on skb after transmit failure When I got rid of the second packet in __pppoe_xmit I created a double-free on the skb because of the goto abort on failure. This patch removes that. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <h...
Sep 18, 9:30 pm 2007
James Chapman
Re: [PATCH 2/3] [PPP] L2TP: Fix skb handling in pppol2tp_rec...
Acked-by: James Chapman <jchapman@katalix.com> -- James Chapman Katalix Systems Ltd http://www.katalix.com Catalysts for your Embedded Linux software development -
Sep 18, 4:17 pm 2007
James Chapman
Re: [PATCH 1/3] [PPP] L2TP: Disallow non-UDP datagram sockets
Acked-by: James Chapman <jchapman@katalix.com> -
Sep 18, 4:16 pm 2007
Rick Jones
[PATCH] iproute2: enable setting rto_min, rtt and rttvar in ...
Enable users of ip to specify the times for rtt, rttvar and rto_min in human-friendly terms a la "tc" while maintaining backwards compatability with the previous "raw" mechanism. Builds upon David Miller's uncommited patch to set rto_min. Signed-off-by: Rick Jones <rick.jones2@hp.com> --- include/linux/rtnetlink.h | 2 + include/utils.h | 1 + ip/iproute.c | 30 ++++++++++++++++----- lib/utils.c | 64 +++++++++++++++++++++++++++++++++++++++...
Sep 18, 4:15 pm 2007
Rick Jones
[PATCH] return useful listenq info in tcp_info and INET_DIAG...
Return some useful information such as the maximum listen backlog and the current listen backlog in the tcp_info structure and INET_DIAG_INFO. Signed-off-by: Rick Jones <rick.jones2@hp.com> Signed-off-by: Eric Dumazet <dada1@cosmosbay.com> --- diff -r bdcdd0e1ee9d net/ipv4/tcp.c --- a/net/ipv4/tcp.c Sat Sep 01 07:00:31 2007 +0000 +++ b/net/ipv4/tcp.c Tue Sep 18 11:02:26 2007 -0700 @@ -2030,8 +2030,14 @@ void tcp_get_info(struct sock *sk, struc info->tcpi_snd_mss = tp->mss_ca...
Sep 18, 3:54 pm 2007
David Miller
Re: [PATCH] return useful listenq info in tcp_info and INET_...
From: Rick Jones <rick.jones2@hp.com> Thanks Rick, I'll add this to net-2.6.24 with a minor coding We don't split up closing braces and else clauses like this, it eats up a precious extra screen line :-) } else { is the custom. -
Sep 18, 4:25 pm 2007
John W. Linville
Please pull 'iwlwifi' branch of wireless-2.6
Jeff & Dave, Here it is -- it's big, it's...well...beautiful in its own way...well, at least it seems to work... :-) There are some outstanding issues. The driver does more than it probably should under the covers instead of in the stack, and the issue of including headers with a "../../mac80211/..." path remains. Still, I think it would be better to get this mainlined than to keep it out of stream. Thanks! John --- Patch available here: [ message continues ]
" title="http://www.kernel.org/pub/linux/kernel...">http://www.kernel.org/pub/linux/kernel...
Sep 18, 2:50 pm 2007
David Miller
Re: Please pull 'iwlwifi' branch of wireless-2.6
From: "John W. Linville" <linville@tuxdriver.com> Jeff, if you have no objections I'll pull this into net-2.6.24 Thanks. -
Sep 18, 3:22 pm 2007
Vernon Mauery
NetXen driver causing slab corruption in -RT kernels
In doing some stress testing of the NetXen driver, I found that my machine was dying in all sorts of weird ways. I saw several different crashes, BUG messages in the TCP stack and some assert messages in the TCP stack as well. I really didn't think that there could be six different bugs all at once in the TCP/IP stack, so I started looking at possible memory corruption. I first saw this on 2.6.16-rt22 with a backported netxen driver from 2.6.22. I figured I should try the latest kernel, so I...
Sep 18, 2:17 pm 2007
Michael Chan
Re: bnx2 dirver's firmware images
We can compress all the different sections of the firmware. Currently, we only compress the biggest chunks and the rest are uncompressed. These zeros should compress to almost nothing. But I agree that the Matching the correct version of the firmware with the driver is the main issue. -
Sep 18, 2:45 pm 2007
Denys Vlasenko
Re: bnx2 dirver's firmware images
You don't need to store and fetch zeros at all. You *know* that they are zeros, right? So do this: - REG_WR_IND(bp, offset, fw->bss[j]); + REG_WR_IND(bp, offset, 0); -- vda -
Sep 18, 1:55 pm 2007
Michael Chan
Re: bnx2 dirver's firmware images
Good point. We can do that. Looking at the file, there are other non- zero data that can be compressed to save some more room. -
Sep 18, 3:09 pm 2007
David Miller
Re: bnx2 dirver's firmware images
From: "Michael Chan" <mchan@broadcom.com> I don't like it because it means people have to setup full initrd's in order to do network booting with such network cards. But the days of my opinion mattering on that issue are long gone, the momentum is just too greatly behind using request_firmware() across the board, so there is no reason for bnx2 to be any different. -
Sep 18, 2:23 pm 2007
H. Peter Anvin
Re: bnx2 dirver's firmware images
klibc could help with that, if there is interest in exploring that avenue again. -hpa -
Sep 18, 2:41 pm 2007
David Miller
Re: bnx2 dirver's firmware images
From: "H. Peter Anvin" <hpa@zytor.com> I appreciate the effort you put into klibc and the offer to make initrd's easier to build. But the point is that the initrd shouldn't be necessary in the first place. There becomes zero point in building these drivers statically into the kernel, which many of us do specifically to avoid module loading, initrds, and all that fuss. Because the driver is totally crippled even though it's been fully built into the main kernel image. I mean, it's so in...
Sep 18, 3:20 pm 2007
H. Peter Anvin
Re: bnx2 dirver's firmware images
Well, what I was referring to here, of course, was the initramfs integrated in the kernel image, so it all comes out of the kernel build tree and produces a single bootable image. The fact that part of it contains userspace code is in that way invisible. That was kind of the point here, and the only reason for pushing klibc into the kernel build tree at all. Under the "distros use external initrd anyway" school of thought, whatever libc used for that can be external anyway. -hpa -
Sep 18, 3:27 pm 2007
David Miller
Re: bnx2 dirver's firmware images
From: "H. Peter Anvin" <hpa@zytor.com> Sounds good to me :) -
Sep 18, 4:08 pm 2007
Sam Ravnborg
Re: bnx2 dirver's firmware images
Except there seems to be great resistance to include userland code in the kernel as demonstrated at last KS. Or this is maybe just a single vocal person and the topic were brought up late? Anyway - if we again consider klibc I will do my best to make the build stuff as smooth as possible. Sam -
Sep 18, 4:35 pm 2007
H. Peter Anvin
Re: bnx2 dirver's firmware images
At least as of the last merged tree it was very smooth indeed, thanks to your help. -hpa -
Sep 18, 4:40 pm 2007
Michael Chan
Re: bnx2 dirver's firmware images
The bnx2 firmware changes quite frequently. A new driver quite often requires new firmware to work correctly. Splitting them up makes things difficult for the user. The firmware in tg3 is a lot more mature and I don't expect it to change. I think tg3 is better suited for using request_firmware(). -
Sep 18, 4:05 pm 2007
David Miller
Re: bnx2 dirver's firmware images
From: "Michael Chan" <mchan@broadcom.com> Like I said, I think neither should change and the driver should -
Sep 18, 3:21 pm 2007
Willy Tarreau
Re: bnx2 dirver's firmware images
Michael, doesn't a functional-yet-suboptimal firmware exist ? I mean, just the same principle as we all have kernels, boot CDs, statically built tools, etc... which run everywhere. If you have such a beast, maybe it would be a good start to have it in the kernel, and provide the users with the ability to upgrade the firmware once the system is able to do more complex things. Just a thought... Regards, Willy -
Sep 18, 5:30 pm 2007
David Miller
Re: bnx2 dirver's firmware images
From: Willy Tarreau <w@1wt.eu> So let's save 60K instead of 80K. I mean, the entire discussion is just plain silly :) -
Sep 18, 5:31 pm 2007
Willy Tarreau
Re: bnx2 dirver's firmware images
That's not for this reason I said this. Michael said the firmware needs to be updated somewhat often. What I was wondering was if it was not possible to stick to a stable one (and hopefully small) so that the driver could require less frequent updates. Sorry if it's not the main yes, possibly :-) Cheers, Willy -
Sep 18, 5:37 pm 2007
Michael Chan
Re: bnx2 dirver's firmware images
The bnx2 chip requires a lot of firmware to begin with, so it won't save much space no matter what version is used in the kernel. We update the firmware to fix bugs and sometimes to add new features. New features often require matching changes in the driver. For example, we're planning to add S/G support for jumbo rx frames and this requires changes in both driver and firmware. It's possible to make the driver work with multiple firmware versions, but that adds complexity to enable/disable cer...
Sep 18, 7:14 pm 2007
Chuck Ebbert
SFQ qdisc crashes with limit of 2 packets
Limit of 1 is forbidden, crashes with 2, works with 3: https://bugzilla.redhat.com/show_bug.cgi?id=219895 ========= If the defect is produced at a console (as in ctrl-alt-f<0-6>) a kernel stack trace can be seen the moment "ping" is invoked. Since the stack trace is not written to the /var/log/messages here's part of it (manually copied): syscall_call(() sys_socketcall() sys_sendmsg() sock_sendmsg() inet_sendmsg() raw_sendmsg() ...
Sep 18, 1:18 pm 2007
Patrick McHardy
Re: SFQ qdisc crashes with limit of 2 packets
Is it a NULL pointer or something random? -
Sep 18, 1:31 pm 2007
Patrick McHardy
Re: SFQ qdisc crashes with limit of 2 packets
Never mind, I found the reason. When enqueuing the packet, sfq_enqueue contains an off-by-one in the limit check (which IIRC is there for a reason, but I can't remember right now) and drops the packet again. dev_queue_xmit() calls qdisc_run() anyway and the empty qdisc is dequeued, which is not handled by SFQ. I see three possibilities to fix this (in my preferred order): 1) figure out why the off-by-one is there, if not needed remove 2) don't dequeue qdiscs even once if empty 3) check for N...
Sep 18, 1:57 pm 2007
Patrick McHardy
Re: SFQ qdisc crashes with limit of 2 packets
OK the off-by-one prevents an out-of-bounds array access, which would cause a crash itself. Despite what I said above, sfq does try to handle dequeues while empty, but forgets to update q->tail when dropping the last packet from the only active queue, probably because it wasn't expected that the queue length is too small to queue even a single packet (and that really doesn't make much sense). So one possibility for fixing this is to update q->tail in sfq_drop when dropping the last packet...
Sep 18, 3:15 pm 2007
previous daytodaynext day
September 17, 2007September 18, 2007September 19, 2007