login
Header Space

 
 

linux-netdev mailing list

FromSubjectsort iconDate
Andy Fleming
[PATCH v2.6.26] gianfar: Fix skb allocation strategy
gianfar was unable to handle failed skb allocation for rx buffers, so we were spinning until it succeeded. Actually, it was worse--we were spinning for a long time, and then silently failing. Instead, we take Stephen Hemminger's suggestion to try the allocation earlier, and drop the packet if it failed. We also make a couple of tweaks to how buffer descriptors are set up. Signed-off-by: Andy Fleming <afleming@freescale.com> --- drivers/net/gianfar.c | 100 ++++++++++++++++++++++++++++++...
Apr 22, 6:18 pm 2008
Adrian Bunk
drivers/net/ucc_geth.c compile error
Commit 9d9326d3bc0ea9a8bbe40bf3e5e66c7b9858caa0 (phy: Change mii_bus id field to a string) causes the following compile error: <-- snip --> ... CC drivers/net/ucc_geth.o /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/net/ucc_geth.c: In function 'ucc_geth_startup': /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/net/ucc_geth.c:2605: warning: assignment makes integer from pointer without a cast /home/bunk/linux/kernel-2.6/git/linux-2.6/drivers/net/ucc_geth.c:2642: warning: assi...
Apr 22, 2:49 pm 2008
Harvey Harrison
[PATCH] netxen: reduce stack usage of netxen_nic_flash_print
Don't need to keep a struct netxen_new_user_info on the stack when we only are interested in printing the serial_num. Change to only reading the serial_num. Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com> --- drivers/net/netxen/netxen_nic_hw.c | 15 ++++++++------- 1 files changed, 8 insertions(+), 7 deletions(-) diff --git a/drivers/net/netxen/netxen_nic_hw.c b/drivers/net/netxen/netxen_nic_hw.c index 05748ca..af73564 100644 --- a/drivers/net/netxen/netxen_nic_hw.c +++ b/...
Apr 22, 2:48 pm 2008
Rui T. Matos
[PATCH] libnl: Add tcindex classifier
This is a request for comments patch to add support for the tcindex classifier to libnl. It lacks support for the police and action attributes and the dump callbacks aren't implementend since I don't know what to output exactly on each one. Please comment. Thanks, Rui --
Apr 22, 11:32 am 2008
Rui T. Matos
[PATCH] Add tcindex classifier
From: Rui Tiago Ca
Apr 22, 11:32 am 2008
Rui Tiago Cação Matos
Re: [PATCH] Add tcindex classifier
[ That email should have had this text ] This is a request for comments patch to add support for the tcindex classifier to libnl. It lacks support for the police and action attributes and the dump callbacks aren't implementend since I don't know what to output exactly on each one. Please comment. Thanks, Rui --
Apr 22, 11:42 am 2008
David Woodhouse
Re: physmap and "request_module: runaway loop modprobe net-p...
Better still, just initialise af_unix earlier so that it's there before any normal drivers which happen to call request_module(). Since core network stuff is initialised with subsys_initcall() which is really __define_initcall("4", ...), we can initialise af_unix with fs_initcall() which is really __define_initcall("5", ...). --- [AF_UNIX] Initialise UNIX sockets before general device initcalls When drivers call request_module(), it tries to do something with UNIX sockets and triggers a 'r...
Apr 22, 8:37 am 2008
Badalian Vyacheslav
miss section in menuconfig generator?
Hello all. Sorry if its not bug. look to section "IP: Netfilter Configuration" in .config i see values like this: CONFIG_NF_NAT_PROTO_GRE=m but in menuconfig i can find it in this section ("network->Network packet filtering framework->IP: Netfilter Configuration"). its blackhole? =) I attach my config. Thanks all
Apr 22, 5:30 am 2008
Badalian Vyacheslav
Re: miss section in menuconfig generator?
I found. its becouse in file ./net/ipv4/netfilter/Kconfig all this options have NULL "tristate" like this: config NF_NAT_PROTO_GRE tristate depends on NF_NAT && NF_CT_PROTO_GRE Is that normal? How to compile that parts in kernel (not modules)? --
Apr 22, 6:12 am 2008
Badalian Vyacheslav
Re: miss section in menuconfig generator?
I understand. This values are hidden. And get values from "default NF_NAT && NF_CONNTRACK_PPTP" if "user not set this values". But my config have this values from older config version (make menuconfig) and not set it again that i change NF_CONNTRACK_PPTP to Y/N. Also if i one time save config - it write value of hidden param (i say about CONFIG_NF_NAT_PPTP now). Have i chance to reconfig this value if menuconfig set it 1 time for me? --
Apr 22, 6:47 am 2008
Herbert Xu
[IPSEC]: Fix catch-22 with algorithm IDs above 31
Hi: [IPSEC]: Fix catch-22 with algorithm IDs above 31 As it stands it's impossible to use any authentication algorithms with an ID above 31 portably. It just happens to work on x86 but fails miserably on ppc64. The reason is that we're using a bit mask to check the algorithm ID but the mask is only 32 bits wide. After looking at how this is used in the field, I have concluded that in the long term we should phase out state matching by IDs because this is made superfluous by the reqid featu...
Apr 22, 3:43 am 2008
David Miller
Re: [IPSEC]: Fix catch-22 with algorithm IDs above 31
From: Herbert Xu <herbert@gondor.apana.org.au> Please update your GIT index :-) --
Apr 22, 3:47 am 2008
Benjamin Herrenschmidt
[PATCH 6/6] ibm_newemac: PowerPC 440EP/440GR EMAC PHY clock ...
From: Valentine Barshak <vbarshak@ru.mvista.com> This patch adds ibm_newemac PHY clock workaround for 440EP/440GR EMAC attached to a PHY which doesn't generate RX clock if there is no link. The code is based on the previous ibm_emac driver stuff. The 440EP/440GR allows controlling each EMAC clock separately as opposed to global clock selection for 440GX. BenH: Made that #ifdef CONFIG_PPC_DCR_NATIVE for now as dcri_* stuff doesn't exist for MMIO type DCRs like Cell. Some future rework & ...
Apr 21, 8:46 pm 2008
Benjamin Herrenschmidt
[PATCH 5/6] ibm_newemac: PowerPC 440GX EMAC PHY clock workar...
From: Valentine Barshak <vbarshak@ru.mvista.com> The PowerPC 440GX Taishan board fails to reset EMAC3 (reset timeout error) if there's no link. Because of that it fails to find PHY chip. The older ibm_emac driver had a workaround for that: the EMAC_CLK_INTERNAL/EMAC_CLK_EXTERNAL macros, which toggle the Ethernet Clock Select bit in the SDR0_MFR register. This patch does the same for "ibm,emac-440gx" compatible chips. The workaround forces clock on -all- EMACs, so we select clock under global e...
Apr 21, 8:46 pm 2008
Benjamin Herrenschmidt
[PATCH 3/6] ibm_newemac: Fix section mismatch warnings
From: Josh Boyer <jwboyer@gmail.com> This patch fixes several section mismatch warnings in the ibm_newemac driver similar to: WARNING: vmlinux.o(.devinit.text+0x3a04): Section mismatch in reference from the function emac_probe() to the function .devexit.text:tah_detach() The function __devinit emac_probe() references a function __devexit tah_detach(). Signed-off-by: Josh Boyer <jwboyer@linux.vnet.ibm.com> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> --- ...
Apr 21, 8:46 pm 2008
Benjamin Herrenschmidt
[PATCH 4/6] ibm_newemac Use status property for unused/unwir...
From: Josh Boyer <jwboyer@linux.vnet.ibm.com> Convert ibm_newemac to use the of_device_is_available function when checking for unused/unwired EMACs. We leave the current check for an "unused" property to maintain backwards compatibility for older device trees. Newer device trees should simply use the standard "status" property in the EMAC node. Signed-off-by: Josh Boyer <jwboyer@linux.vnet.ibm.com> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> --- driver...
Apr 21, 8:46 pm 2008
Benjamin Herrenschmidt
[PATCH 1/6] ibm_newemac: Fix problem with jumbo frame suppor...
From: Stefan Roese <sr@denx.de> This fixes the jumbo frame support on EMAC V4 systems. Now the correct bit is set depending on the EMAC version configured. Tested on Kilauea (405EX) and Canyonlands (460EX). Signed-off-by: Stefan Roese <sr@denx.de> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> --- drivers/net/ibm_newemac/core.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/net/ibm_newemac/core.c b/drivers/net/ibm_new...
Apr 21, 8:46 pm 2008
Benjamin Herrenschmidt
[PATCH 2/6] ibm_newemac: Add support for 460EX/GT-type MAL r...
From: Stefan Roese <sr@denx.de> On some 4xx PPC's (e.g. 460EX/GT), the rx channel number is a multiple of 8 (e.g. 8 for EMAC1, 16 for EMAC2), but enabling in MAL_RXCASR needs the divided by 8 value for the bitmask. Signed-off-by: Stefan Roese <sr@denx.de> Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> --- drivers/net/ibm_newemac/mal.c | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/drivers/net/ibm_newemac/mal.c b/drivers/net/ibm_ne...
Apr 21, 8:46 pm 2008
David Dillow
Re: [PATCH 2/2] drivers/net: Fix synchronize_irq() bugs, red...
Should probably kill the comment as well, since it was referring to the synchronize_irq() call. Acked-by: Dave Dillow <dave@thedillows.org> --
Apr 21, 9:04 pm 2008
Michael Chan
Re: [ PATCH 1/1 2.6.25-rc9] bnx2: Add EEH support in bnx2x d...
I think it is better to add rtnl_lock() here to prevent racing with other functions such as ethtool and close that can be resetting the Shouldn't we attach first before netif_start? Thanks. --
Apr 21, 8:14 pm 2008
wendy xiong
Re: [ PATCH 1/1 2.6.25-rc9] bnx2: Add EEH support in bnx2x d...
Hi Michael Chan, Yes. Most of network interface functions called by net/core/dev.c layer which hold rtnl_lock()/rtcl_unlock(). We may need to add device driver's open function to initialize internal data structures and enable interrupt, then driver starts the receive queue by calling netif_start_queue. Looks bnx2_netif_start() enables hardware interrupt and receive polling interrupt. So I think we can do netif_device_attach after bnx2_netif_start() according to dev->open route concept. T...
Apr 22, 3:44 pm 2008
Michael Chan
Re: [ PATCH 1/1 2.6.25-rc9] bnx2: Add EEH support in bnx2x d...
I think it doesn't make a whole lot of difference either way, especially if you hold the rtnl_lock(). But in the normal course of loading the driver, the LINK_STATE_PRESENT bit gets set during ->probe() when the driver calls register_netdev(). So it is attached before ->open(). --
Apr 22, 5:46 pm 2008
Rusty Russell
Re: [6/6] [VIRTIO] net: Allow receiving SG packets
Right, and so that's +2 for virtio_net because we have an extra header as Herbert points out. But I was curious as to why the +2 in the MAX_SKB_FRAGS definition? Thanks, Rusty. --
Apr 21, 10:50 pm 2008
David Miller
Re: [6/6] [VIRTIO] net: Allow receiving SG packets
From: Rusty Russell <rusty@rustcorp.com.au> To be honest I have no idea. When Alexey added the TSO changeset way back then, it had the "+2", from the history-2.6 tree: commit 80223d5186f73bf42a7e260c66c9cb9f7d8ec9cf Author: Alexey Kuznetsov <kuznet@ms2.inr.ac.ru> Date: Wed Aug 28 11:52:03 2002 -0700 [NET]: Add TCP segmentation offload core infrastructure. ... diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index a812681..9b6e6ad 100644 --- a/include/linux...
Apr 21, 10:55 pm 2008
Herbert Xu
Re: [6/6] [VIRTIO] net: Allow receiving SG packets
Theoretically yes :) But for virtualisation the underlying transport may present meta-physically contiguous memory that is physically discrete. So we may actually need to have multiple SG entries for skb->data. However, no current code path should generate packets with both long skb->data areas *and* skb page frags so we could just drop them if they show up. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <herbert@gondor.apana.org.au> Home...
Apr 21, 9:13 pm 2008
Herbert Xu
Re: [bug] build failure in net/netfilter/nf_conntrack_sip.c,...
It is human nature to blame someone/something when things go wrong. It just happens that the lack of a merge request was the first you tacked onto in this case :) I don't think we can seriously argue that had there been a merge request on lkml that these build issues would have somehow gone away or resolved any sooner than they have been. The fact is with or without the merge request you would have realised that Unfortunately the email medium tends to make what one thinks of as a neutral and ...
Apr 21, 10:20 pm 2008
YOSHIFUJI Hideaki /
Re: [PATCH] [DCCP]: Convert do_gettimeofday() to getnstimeof...
Sorry I have just realized set_normalized_timespec() (used in timespec_sub()) is not exported, and link will fail because of it... What should I/we do? Send the patch to LKML as well? --- Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org> --- diff --git a/kernel/time.c b/kernel/time.c index a5ec013..35d373a 100644 --- a/kernel/time.c +++ b/kernel/time.c @@ -379,6 +379,7 @@ void set_normalized_timespec(struct time ts->tv_sec = sec; ts->tv_nsec = nsec; } +EXPORT...
Apr 21, 10:40 pm 2008
David Miller
Re: [PATCH] [DCCP]: Convert do_gettimeofday() to getnstimeof...
From: YOSHIFUJI Hideaki / 吉藤英明 <yoshfuji@linux-ipv6.org> I'll apply and push this change, thank you. --
Apr 21, 10:45 pm 2008
Bodo Eggert
Re: [PATCH] Re: Bad network performance over 2Gbps
No, not a per-device-exemption. My reasoning was: If the IRQ balancer bounces the IRQ too often, doing it less often seems to be the correct solution. One cache miss each ten seconds sounds like it should be OK. As said before, I can't verify this theory. --
Apr 22, 3:36 am 2008
Kok, Auke
Re: [PATCH] Re: Bad network performance over 2Gbps
this is exaclty what the userspace irqbalance does and it's even optimized to not do those migrations once every 10 seconds if things look OK. from that perspective, it's definately more mature and it's maintained as well. Auke --
Apr 22, 1:46 pm 2008
Nelson A. de Oliveira
Re: [Bugme-new] [Bug 10473] New: Infinite loop "b44: eth0: p...
Hi! Maybe this can help: Using a new .config, I started to enable/disable options and test. What I found is that if I enable "3G/1G user/kernel split", the kernel works (it boots normally, the network interface works, etc). If I select "3G/1G user/kernel split (for full 1G low memory)" I get the infinite loop of "b44: eth0: powering down PHY". Working config file (on 2.6.25) is attached. Diff to the non-working is below: --- working_config 2008-04-21 23:42:40.000000000 -0300 +++ not_working_...
Apr 21, 11:01 pm 2008
Michael Buesch
Re: [Bugme-new] [Bug 10473] New: Infinite loop "b44: eth0: p...
Ah, so this bug isn't actually caused by a patch but rather by a different config option. I think we can't do much about it, currently. The device has strange memory requirements and changing the split does actually break it. This cannot be fixed until andi kleen's mask-allocator is merged. -- Greetings Michael. --
Apr 22, 9:34 am 2008
Paulius Zaleckas
Re: [RFC] updated Patch to option HSO driver to the kernel
Incremental patch. - use net_device_stats from dev->stats
Apr 22, 4:33 am 2008
Paulius Zaleckas
Re: [RFC] updated Patch to option HSO driver to the kernel
Incremental patch. - remove not needed variable initializations - remove hso_net_get_stats, because by default identical internal_stats function from net/core/dev.c is used. - remove not needed memset(hso_net, 0, sizeof(*hso_net)), because memory is allocated with kzalloc
Apr 22, 9:50 am 2008
Oliver Neukum
Re: [RFC] updated Patch to option HSO driver to the kernel
But what in this setup would prevent the other end of the connection from sending packets to a multicast address down the link? If that can happen, the questions remains, can the device filter these packets in hardware? Regards Oliver --
Apr 22, 4:52 am 2008
Filip Aben
Re: [RFC] updated Patch to option HSO driver to the kernel
Correct. And we don't support DHCP. Filip- --
Apr 22, 2:29 am 2008
Laurent Pinchart
Re: [RFC fs_enet: Convert MII bitbang driver to use GPIO lib
I would be fine with that if the glue code wasn't 90% of the whole driver.= =20 There is really little (not to say nothing) that can be shared between the= =20 two drivers. =2D-=20 Laurent Pinchart CSE Semaphore Belgium Chaussee de Bruxelles, 732A B-1410 Waterloo Belgium T +32 (2) 387 42 59 =46 +32 (2) 387 42 75
Apr 22, 4:55 am 2008
Scott Wood
Re: [RFC fs_enet: Convert MII bitbang driver to use GPIO lib
I think we're thinking of a different pair of drivers... I thought you were talking about duplicating drivers/net/phy/mdio-bitbang.c, not drivers/net/fs_enet/mii-bitbang.c. -Scott --
Apr 22, 11:08 am 2008
Laurent Pinchart
Re: [RFC fs_enet: Convert MII bitbang driver to use GPIO lib
I don't plan to touch drivers/net/phy/mdio-bitbang.c at all, it does its jo= b=20 well enough. As the openfirmware + gpio + mdio driver might benefit non-powerpc platform= s,=20 I plan to create a new driver (probably drivers/net/phy/mdio-ofgpio.c) that= =20 mostly ports drivers/net/fs_enet/mii-bitbang.c to the gpiolib (this replace= s=20 around 90% of the code). If both drivers/net/fs_enet/mii-bitbang.c and drivers/net/phy/mdio-ofgpio.c= =20 must live together, I'll have a problem in=20 ...
Apr 22, 11:21 am 2008
Scott Wood
Re: [RFC fs_enet: Convert MII bitbang driver to use GPIO lib
The only reason I can see for that would be if the gpiolib version doesn't work on arch/ppc -- but that's not going to be a problem for very If mii-bitbang is only used on arch/ppc, then #ifdef CONFIG_PPC_MERGE should do it. It'd be nice to be able to give phylib an OF node rather than a string, though... -Scott --
Apr 22, 11:36 am 2008
Laurent Pinchart
Re: [RFC fs_enet: Convert MII bitbang driver to use GPIO lib
Didn't you mention platforms such as ep8248e as well, where mdiobb would be= =20 Doesn't phylib have non-OF users ? =2D-=20 Laurent Pinchart CSE Semaphore Belgium Chaussee de Bruxelles, 732A B-1410 Waterloo Belgium T +32 (2) 387 42 59 =46 +32 (2) 387 42 75
Apr 22, 11:55 am 2008
Scott Wood
Re: [RFC fs_enet: Convert MII bitbang driver to use GPIO lib
That was when I thought you were talking about changing drivers/net/phy/mdio-bitbang.c. ep8248e doesn't use I meant as an option, not the only way to specify it. -Scott --
Apr 22, 12:04 pm 2008
Anton Vorontsov
Re: [RFC fs_enet: Convert MII bitbang driver to use GPIO lib
Hm... PHYs are having bus_id equal to MDIOBUS:PHYID, for example e0024520:02, see fs_enet-main.c find_phy() and fs_init_phy(): data = of_get_property(np, "phy-handle", &len); phynode = of_find_node_by_phandle(*data); mdionode = of_get_parent(phynode); ret = of_address_to_resource(mdionode, 0, &res); data = of_get_property(phynode, "reg", &len); snprintf(fpi->bus_id, 16, PHY_ID_FMT, res.start, *data); ... phydev = phy_connect(dev, f...
Apr 22, 11:36 am 2008
Bill Fink
Re: [PATCH] Re: Bad network performance over 2Gbps
Since you're changing the default setting, shouldn't the above be changed to: Saying yes will allow the kernel to do irq load balancing. -Bill --
Apr 22, 1:07 am 2008
Pavel Emelyanov
Re: [RFC PATCH net-2.6.26 (Plan B)] [TCP]: Lower stack usage...
Plan C: we can follow the Al Viro's proposal about %n like in this patch. The same applies to udp, fib (the /proc/net/route file), rt_cache and sctp debug. This is minus ~150-200 bytes for each. How do you think? Signed-off-by: Pavel Emelyanov <xemul@openvz.org> diff --git a/net/ipv4/fib_hash.c b/net/ipv4/fib_hash.c index 02088de..2e2fc33 100644 --- a/net/ipv4/fib_hash.c +++ b/net/ipv4/fib_hash.c @@ -1003,7 +1003,7 @@ static unsigned fib_flag_trans(int type, __be32 mask, struct fib_i...
Apr 22, 10:35 am 2008
Jay Cliburn
Re: atl1 64-bit => 32-bit DMA borkage (reproducible, bisected)
On Mon, 21 Apr 2008 00:55:00 +0400 Alexey, can you please try this (very minimally tested) patch? diff --git a/drivers/net/atlx/atl1.c b/drivers/net/atlx/atl1.c index 5586fc6..07fe5c0 100644 --- a/drivers/net/atlx/atl1.c +++ b/drivers/net/atlx/atl1.c @@ -1115,9 +1115,6 @@ static void atl1_free_ring_resources(struct atl1_adapter *adapter) struct atl1_rrd_ring *rrd_ring = &adapter->rrd_ring; struct atl1_ring_header *ring_header = &adapter->ring_header; - atl1_clean_tx_ring(ad...
Apr 21, 10:08 pm 2008
Alexey Dobriyan
Re: atl1 64-bit => 32-bit DMA borkage (reproducible, bisected)
Patch doesn't help unfortunately. BTW, below is clean corruption trace: atl1 0000:03:00.0: eth0 link is up 1000 Mbps full duplex ============================================================================= BUG kmalloc-2048: Poison overwritten ----------------------------------------------------------------------------- INFO: 0xffff81017ed7a97a-0xffff81017ed7af71. First byte 0x0 instead of 0x6b INFO: Allocated in dev_alloc_skb+0x18/0x30 age=23894 cpu=1 pid=30255 INFO: Freed in skb_release_...
Apr 22, 3:02 pm 2008
Waskiewicz Jr, Peter P
RE: [PATCH 3/5] e1000e: Allow TSO to trickle down to VLAN de...
This was a conversation I had late last year on the list regarding TSO and Tx checksum offload not getting turned on for VLAN devices, since the VLAN device creation code doesn't copy those device feature flags. I had proposed sending a patch, and the statement was made that some devices might not support both of those hardware features at the same I hear you loud and clear. I wanted to do this the "right" way in the drivers if I couldn't do it in the kernel, and your suggestions sound like the ...
Apr 22, 3:46 am 2008
Gerrit Renker
Re: [DCCP] [RFC] [Patchv2 1/1]: Queuing policies -- reworked...
| If inet{,6}_skb_parm is used only outside DCCP code then why at all should it | be placed in struct dccp_skb_cb taking up quite a lot of valuable space? Why | not put it directly in struct sk_buff? Especially that it is present in | struct udp_skb_cb, struct tcp_skb_cb as well. | With or without inet{,6}_sbk_parm there are a further unused 32 bits in the cb -- by restricting the 48-bit sequence/ack numbers from their current u64 type to 48 bit only. This requires to change the parts which rely...
Apr 22, 1:30 pm 2008
Tomasz Grobelny
Re: [DCCP] [RFC] [Patchv2 1/1]: Queuing policies -- reworked...
I mean that what we are currently discussing (only priorities) is in itself useful ("works half"). Adding more features (expiry times) would be a nice but non essential feature ("work full"). That is an important question. But I'm afraid the answer is no (even though I thought otherwise when writing previous mail). When we want to add packet expiry times we will need a field for timestamp. Which is quite big (64 bits?). But note that it is only needed for in-kernel implemetation. Userspace ca...
Apr 22, 4:30 pm 2008
previous daytodaynext day
April 21, 2008April 22, 2008April 23, 2008
speck-geostationary