Re: [patch 4/5] bluetooth: small cleanups

2006-09-19 Thread Marcel Holtmann
Hi Andrew, > From: Pavel Machek <[EMAIL PROTECTED]> > > This cleans up bluetooth a bit, no code changes. NAK on this one. These cleanups will come with my other pending cleanups. Regards Marcel - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [E

Re: [PATCH] make some netfilter globals __read_mostly

2006-09-19 Thread Patrick McHardy
Brian Haley wrote: > Make some netfilter globals __read_mostly at the request of Patrick > McHardy. > > Signed-off-by: Brian Haley <[EMAIL PROTECTED]> Applied, thanks Brian. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More maj

Re: [RFC 3/3] cfg80211 thoughts on configuration

2006-09-19 Thread Thomas Graf
* Johannes Berg <[EMAIL PROTECTED]> 2006-09-14 12:53 > This is some preliminary code how I'm currently thinking (and that might > change radically :) ) configuration might look like. > > It uses the patch I previously posted to make genetlink attributes > custom-definable. > > --- wireless-dev.or

Re: [RFC] genetlink custom attribute type

2006-09-19 Thread Thomas Graf
* Johannes Berg <[EMAIL PROTECTED]> 2006-09-14 11:21 > On Thu, 2006-09-14 at 10:14 +0200, Thomas Graf wrote: > > > Looks good, we have to watch the size of struct nla_policy though. > > This bumps the size from 4 bytes to 16 bytes on 64bit architectures > > which might become a problem since we al

Re: [patch 3/5] ATM: he: fix section mismatch

2006-09-19 Thread Roland Dreier
> WARNING: drivers/atm/he.o - Section mismatch: reference to .init.text: from > .text between 'he_start' (at offset 0x218a) and 'he_service_tbrq' > -static int __init > +static int __devinit > he_init_group(struct he_dev *he_dev, int group) There are a ton of other __init functions (he_ini

Re: [GIT PATCH] IPV6: Updates for net-2.6.19

2006-09-19 Thread YOSHIFUJI Hideaki / 吉藤英明
In article <[EMAIL PROTECTED]> (at Tue, 19 Sep 2006 18:26:29 +0200), Nicolas DICHTEL <[EMAIL PROTECTED]> says: > > + if (pneigh) > > + is_router = pneigh->flags & NTF_ROUTER; > > > In function ndisc_send_na(), is_router must be 1 or 0, doesn't it ? Because, > icmph.icmp6_router is

Re: 2.6.18-rc7-mm1: networking breakage on HPC nx6325 + SUSE 10.1

2006-09-19 Thread Greg KH
On Wed, Sep 20, 2006 at 12:56:57AM +0200, Rafael J. Wysocki wrote: > On Wednesday, 20 September 2006 00:30, Greg KH wrote: > > On Tue, Sep 19, 2006 at 03:06:29PM -0700, David Miller wrote: > > > From: "Rafael J. Wysocki" <[EMAIL PROTECTED]> > > > Date: Wed, 20 Sep 2006 00:06:52 +0200 > > > > > > >

Re: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Segher Boessenkool
Well, I'm having trouble thinking of other busses that have as strong a sense of the "address-data" style I/O as PCI. Busses like scsi and ide are primarily "command-data" or "data-data" in style. Only the address-data style busses need readl/writel-style routines. SBUS, JBUS, VMEbus, NuBus, Ra

Re: 2.6.18-rc7-mm1: networking breakage on HPC nx6325 + SUSE 10.1

2006-09-19 Thread Dmitry Torokhov
On Tuesday 19 September 2006 18:30, Greg KH wrote: > On Tue, Sep 19, 2006 at 03:06:29PM -0700, David Miller wrote: > > From: "Rafael J. Wysocki" <[EMAIL PROTECTED]> > > Date: Wed, 20 Sep 2006 00:06:52 +0200 > > > > > I _guess_ the problem is caused by > > > gregkh-driver-network-class_device-to-de

Re: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Jeff Garzik
Segher Boessenkool wrote: Sure, PCI busses are little-endian. But is readX()/writeX() for PCI only? Yes. For other buses, use foo_writel(), etc. Can this please be documented then? Never heard this before... You have come late to the party. This has been the case for many, many years.

Re: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Linas Vepstas
Hi, I am alarmed and embarassed that sloppy comments on my part has turned onto a long conversation. On Wed, Sep 20, 2006 at 02:58:39AM +0200, Segher Boessenkool wrote: > >> Sure, PCI busses are little-endian. But is readX()/writeX() for PCI > >> only? I sure hope not. > > > > It's defined for

[patch 1/5] net/ipv6: bh_lock_sock_nested on tcp_v6_rcv

2006-09-19 Thread akpm
From: Fabio Olive Leite <[EMAIL PROTECTED]> A while ago Ingo patched tcp_v4_rcv on net/ipv4/tcp_ipv4.c to use bh_lock_sock_nested and silence a lock validator warning. This fixed it for IPv4, but recently I saw a report of the same warning on IPv6. Cc: Herbert Xu <[EMAIL PROTECTED]> Cc: "David S.

[patch 3/5] ATM: he: fix section mismatch

2006-09-19 Thread akpm
From: Randy Dunlap <[EMAIL PROTECTED]> WARNING: drivers/atm/he.o - Section mismatch: reference to .init.text: from .text between 'he_start' (at offset 0x218a) and 'he_service_tbrq' Signed-off-by: Randy Dunlap <[EMAIL PROTECTED]> Signed-off-by: Andrew Morton <[EMAIL PROTECTED]> --- drivers/atm/

[patch 2/5] via-ircc: fix memory leak

2006-09-19 Thread akpm
From: Chuck Short <[EMAIL PROTECTED]> Fix memory leak. Coverity id# 653 patch location: http://www.kernel.org/git/?p=linux/kernel/git/bcollins/ubuntu-dapper.git;a=commitdiff;h=a1f34cb68b16807ed9d5ebb0f6a6ec5ff8a5fc78 Signed-off-by: Chuck Short <[EMAIL PROTECTED]> Signed-off-by: Ben Collins <[EM

[patch 5/5] Add netpoll/netconsole support to vlan devices

2006-09-19 Thread akpm
From: Jesper Juhl <[EMAIL PROTECTED]> I recently tried running netconsole via a vlan interface without luck, and decided to do something about it (see patch below). My interfaces look like this: eth0 Link encap:Ethernet HWaddr 00:14:5E:28:3C:2E UP BROADCAST RUNNING MULTICAST MTU

[patch 4/5] bluetooth: small cleanups

2006-09-19 Thread akpm
From: Pavel Machek <[EMAIL PROTECTED]> This cleans up bluetooth a bit, no code changes. Signed-off-by: Pavel Machek <[EMAIL PROTECTED]> Cc: Marcel Holtmann <[EMAIL PROTECTED]> Signed-off-by: Andrew Morton <[EMAIL PROTECTED]> --- drivers/bluetooth/hci_ldisc.c | 13 + 1 file changed

Re: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Segher Boessenkool
Sure, PCI busses are little-endian. But is readX()/writeX() for PCI only? Yes. For other buses, use foo_writel(), etc. Can this please be documented then? Never heard this before... Segher - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EM

Re: 2.6.18-rc7-mm1: networking breakage on HPC nx6325 + SUSE 10.1

2006-09-19 Thread Valdis . Kletnieks
On Tue, 19 Sep 2006 23:30:34 +0200, "Rafael J. Wysocki" said: > Well, I can configure the interfaces manually, with ifconfig, but the SUSE's > configuration tools don't work. For example, "ifup eth0" tells me that > "No configuration found for eth0" and that's all. I'm seeing issues on a Dell La

Re: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Segher Boessenkool
Sure, PCI busses are little-endian. But is readX()/writeX() for PCI only? I sure hope not. It's defined for PCI and possibly ISA memory. You can use it for other things if you whish to, but "other things" are arch specific in any case. Huh? You're saying that only PCI and ISA are standardis

Re: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Jeff Garzik
Segher Boessenkool wrote: Sure, PCI busses are little-endian. But is readX()/writeX() for PCI only? Yes. For other buses, use foo_writel(), etc. Jeff - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo in

Re: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Benjamin Herrenschmidt
On Wed, 2006-09-20 at 02:21 +0200, Segher Boessenkool wrote: > > Nah. We have the basic rule that readl/writel are little endian. > > PowerPC > > additionally provides arch specific low level in_{be,le}32 type > > accessors with explicit endianness. Or you can also use > > cpu_to_le32/le32_to_cpu

Re: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Segher Boessenkool
Nah. We have the basic rule that readl/writel are little endian. PowerPC additionally provides arch specific low level in_{be,le}32 type accessors with explicit endianness. Or you can also use cpu_to_le32/le32_to_cpu kind of macros to convert between native and explicit endianness. Sure, PCI b

Re: [PATCH 12/23] e1000: Maybe stop TX if not enough free descriptors

2006-09-19 Thread Stephen Hemminger
On Tue, 19 Sep 2006 16:26:12 -0700 Auke Kok <[EMAIL PROTECTED]> wrote: > Stephen Hemminger wrote: > > On Tue, 19 Sep 2006 16:45:06 -0400 > > Jeff Garzik <[EMAIL PROTECTED]> wrote: > > > >> looks OK except for the tasklet, which may starve if the lock is being > >> held upon entry > >> > >> > >

Re: Fw: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Paul Mackerras
Linas Vepstas writes: > The rest of this patch might indeed be correct, but the above comment > bothers me. The "ns" versions of routines are supposed to be > non-byte-swapped versions of the insl/outsl routines (which would > byte-swap on big-endian archs such as powerpc.) If it were true that

Re: [PATCH 12/23] e1000: Maybe stop TX if not enough free descriptors

2006-09-19 Thread Auke Kok
Stephen Hemminger wrote: On Tue, 19 Sep 2006 16:45:06 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: looks OK except for the tasklet, which may starve if the lock is being held upon entry Why would the tasklet starve anymore than NAPI? Worst case, the transmitters fill the ring completely an

Re: Fw: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Benjamin Herrenschmidt
On Tue, 2006-09-19 at 20:52 +0200, Matt Sealey wrote: > Some northbridges and PCI bridges have "clever" byteswapping in > hardware, maybe this is just an effect of that. In theory depending on > the host bridge, you should pass in big endian data and have it swap or > not swap, not pick that way

Re: [PATCH] tcp: simpler bic default

2006-09-19 Thread Stephen Hemminger
On Tue, 19 Sep 2006 16:04:41 -0700 (PDT) David Miller <[EMAIL PROTECTED]> wrote: > From: Stephen Hemminger <[EMAIL PROTECTED]> > Date: Tue, 19 Sep 2006 16:02:02 -0700 > > > > > > When TCP_CONG_ADVANCED is disabled, TCP_CONG_BIC should always be set > > > and be the default, but that's not what t

Re: Fw: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Benjamin Herrenschmidt
> However, I presume someone added the __powerpc__ define here > because they picked up a 3c509 at a garage sale, stuck it in > a powerpc, found out it didn't work due to a byte-swapping bug, > and then patched it as above. I'm disturbed that somehow > outsl_ns() became identical to outsl() at

Re: [PATCH] tcp: simpler bic default

2006-09-19 Thread David Miller
From: Stephen Hemminger <[EMAIL PROTECTED]> Date: Tue, 19 Sep 2006 16:02:02 -0700 > > > When TCP_CONG_ADVANCED is disabled, TCP_CONG_BIC should always be set > > and be the default, but that's not what the new code is doing. > > > > I'm reverting Stephen's patch (again) until this is all sorted

Re: [PATCH] tcp: simpler bic default

2006-09-19 Thread Stephen Hemminger
> When TCP_CONG_ADVANCED is disabled, TCP_CONG_BIC should always be set > and be the default, but that's not what the new code is doing. > > I'm reverting Stephen's patch (again) until this is all sorted > out as things are still broken in various ways even with the > followon patch. I don't see

Re: 2.6.18-rc7-mm1: networking breakage on HPC nx6325 + SUSE 10.1

2006-09-19 Thread Rafael J. Wysocki
On Wednesday, 20 September 2006 00:30, Greg KH wrote: > On Tue, Sep 19, 2006 at 03:06:29PM -0700, David Miller wrote: > > From: "Rafael J. Wysocki" <[EMAIL PROTECTED]> > > Date: Wed, 20 Sep 2006 00:06:52 +0200 > > > > > I _guess_ the problem is caused by > > > gregkh-driver-network-class_device-to

Re: [PATCH] tcp: simpler bic default

2006-09-19 Thread David Miller
From: bert hubert <[EMAIL PROTECTED]> Date: Tue, 19 Sep 2006 23:43:05 +0200 > On Tue, Sep 19, 2006 at 02:32:09PM -0700, Stephen Hemminger wrote: > > Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> > > # CONFIG_TCP_CONG_ADVANCED is not set > # CONFIG_DEFAULT_BIC is not set > # CONFIG_DEFAULT_

Re: [PATCH 1/3] sky2: tx pause bug fix

2006-09-19 Thread Andrew Hall
Not that I've been able to uncover. Thanks for fixing this.. - Original Message - From: "Stephen Hemminger" <[EMAIL PROTECTED]> To: "Andrew Hall" <[EMAIL PROTECTED]> Cc: Sent: Wednesday, September 20, 2006 6:31 AM Subject: Re: [PATCH 1/3] sky2: tx pause bug fix On Mon, 11 Sep 2006

Re: 2.6.18-rc7-mm1: networking breakage on HPC nx6325 + SUSE 10.1

2006-09-19 Thread Greg KH
On Tue, Sep 19, 2006 at 03:06:29PM -0700, David Miller wrote: > From: "Rafael J. Wysocki" <[EMAIL PROTECTED]> > Date: Wed, 20 Sep 2006 00:06:52 +0200 > > > I _guess_ the problem is caused by > > gregkh-driver-network-class_device-to-device.patch, but I can't verify this, > > because the kernel (ob

Re: [PATCH 03/23] e100: Add debugging code for cb cleaning and csum failures.

2006-09-19 Thread Auke Kok
Jeff Garzik wrote: Kok, Auke wrote: Refine cb cleaning debug printout and print out all cleaned cbs' status. Add debug flag for EEPROM csum failures that were overridden by the user. Signed-off-by: Jesse Brandeburg <[EMAIL PROTECTED]> Signed-off-by: Auke Kok <[EMAIL PROTECTED]> ACK patch, NA

Re: [PATCH 03/23] e100: Add debugging code for cb cleaning and csum failures.

2006-09-19 Thread Auke Kok
Dave Jones wrote: On Tue, Sep 19, 2006 at 05:40:34PM -0400, Jeff Garzik wrote: > Dave Jones wrote: > > On Tue, Sep 19, 2006 at 10:28:38AM -0700, Kok, Auke wrote: > > > + add_taint(TAINT_MACHINE_CHECK); > > > > I object to this flag being abused this way. > > A corrup

Re: 2.6.18-rc7-mm1: networking breakage on HPC nx6325 + SUSE 10.1

2006-09-19 Thread David Miller
From: "Rafael J. Wysocki" <[EMAIL PROTECTED]> Date: Wed, 20 Sep 2006 00:06:52 +0200 > I _guess_ the problem is caused by > gregkh-driver-network-class_device-to-device.patch, but I can't verify this, > because the kernel (obviously) doesn't compile if I revert it. Indeed. I thought we threw this

Re: 2.6.18-rc7-mm1: networking breakage on HPC nx6325 + SUSE 10.1

2006-09-19 Thread Rafael J. Wysocki
On Tuesday, 19 September 2006 23:30, Rafael J. Wysocki wrote: > On Tuesday, 19 September 2006 22:36, Andrew Morton wrote: > > On Tue, 19 Sep 2006 22:25:21 +0200 > > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > > > > > - It took maybe ten hours solid work to get this dogpile vaguely > > > >

RE: [redhat-lspp] ipsec acquire has security context although I a m not using it.

2006-09-19 Thread Venkat Yekkirala
This doesn't look right since kzalloc would already have zeroed the structure out. Are you sure you are getting garbage in the acquire from the kernel? If you are, I strongly doubt that this would be the one causing it (unless kzalloc on this arch misbehaved). Or is this a racoon bug? > -Origi

[PATCH 3/3] [IrDA] stir4200: removing undocumented bits handling

2006-09-19 Thread Samuel Ortiz
FIFOCTL_RXERR and FIFOCTL_TXERR are undocumented bits, according to the Sigmatel datasheet. We should thus not take any assumption on their values and semantics. Problem spotted by andrzej zaborowski <[EMAIL PROTECTED]> Signed-off-by: Samuel Ortiz <[EMAIL PROTECTED]> --- drivers/net/irda/stir420

[PATCH 2/3] [IrDA] smsc-ircc: More laptops detected

2006-09-19 Thread Samuel Ortiz
This patch detects the smsc-ircc chipset on the nx1000 (including nx7000 and nx7010) and the nx5000 HP/Compaq laptop series. Patch from "Linus Walleij (LD/EAB)" <[EMAIL PROTECTED]> Signed-off-by: Samuel Ortiz <[EMAIL PROTECTED]> --- drivers/net/irda/smsc-ircc2.c | 38 +

[PATCH 1/3] [IrDA] nsc-ircc: Configuration base address for PC87383

2006-09-19 Thread Samuel Ortiz
Hi Dave, According to NatSemi datasheet, the configuration base address for the PC8738x family is 0x2e or 0x164. 0x0 doesn't appear in any datasheet. Patch from Lamarque Vieira Souza <[EMAIL PROTECTED]> Signed-off-by: Samuel Ortiz <[EMAIL PROTECTED]> --- drivers/net/irda/nsc-ircc.c |2 +- 1

Re: [PATCH 05/23] e1000: rename flow control symbols

2006-09-19 Thread Auke Kok
Jeff Garzik wrote: Kok, Auke wrote: --- a/drivers/net/e1000/e1000_hw.h +++ b/drivers/net/e1000/e1000_hw.h @@ -90,15 +90,6 @@ typedef enum { e1000_100_full = 3 } e1000_speed_duplex_type; -/* Flow Control Settings */ -typedef enum { -e1000_fc_none = 0, -e1000_fc_rx_pause = 1, -

Re: [PATCH 03/23] e100: Add debugging code for cb cleaning and csum failures.

2006-09-19 Thread Dave Jones
On Tue, Sep 19, 2006 at 05:40:34PM -0400, Jeff Garzik wrote: > Dave Jones wrote: > > On Tue, Sep 19, 2006 at 10:28:38AM -0700, Kok, Auke wrote: > > > + add_taint(TAINT_MACHINE_CHECK); > > > > I object to this flag being abused this way. > > A corrupt EEPROM on a networ

Re: [PATCH] tcp: simpler bic default

2006-09-19 Thread bert hubert
On Tue, Sep 19, 2006 at 02:32:09PM -0700, Stephen Hemminger wrote: > Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> # CONFIG_TCP_CONG_ADVANCED is not set # CONFIG_DEFAULT_BIC is not set # CONFIG_DEFAULT_CUBIC is not set # CONFIG_DEFAULT_HTCP is not set # CONFIG_DEFAULT_VEGAS is not set # CON

Re: [PATCH 04/23] e100: rework WoL and shutdown handling

2006-09-19 Thread Auke Kok
Jeff Garzik wrote: Kok, Auke wrote: Unify our shutdown/suspend/resume code and make it similar to e1000: e1000_shutdown now calls suspend which does the exact same thing on shutdown except saving PCI config state on suspend. WoL setup code is now also more simple and works even when CONFIG_PM is

Re: [PATCH 03/23] e100: Add debugging code for cb cleaning and csum failures.

2006-09-19 Thread Jeff Garzik
Dave Jones wrote: On Tue, Sep 19, 2006 at 10:28:38AM -0700, Kok, Auke wrote: > > Refine cb cleaning debug printout and print out all cleaned cbs' status. Add > debug flag for EEPROM csum failures that were overridden by the user. > > Signed-off-by: Jesse Brandeburg <[EMAIL PROTECTED]> >

Re: [PATCH] tcp: set congestion default through Kconfig (v2)

2006-09-19 Thread bert hubert
On Tue, Sep 19, 2006 at 02:20:07PM -0700, David Miller wrote: > > Bert's attempt was noble > > It showed your desire for the truth It was also crap :-) > Applied, but... > net/ipv4/Kconfig:607:warning: defaults for choice values not supported It does appear to do the right thing in all cases I

Re: [PATCH 03/23] e100: Add debugging code for cb cleaning and csum failures.

2006-09-19 Thread Dave Jones
On Tue, Sep 19, 2006 at 10:28:38AM -0700, Kok, Auke wrote: > > Refine cb cleaning debug printout and print out all cleaned cbs' status. Add > debug flag for EEPROM csum failures that were overridden by the user. > > Signed-off-by: Jesse Brandeburg <[EMAIL PROTECTED]> > Signed-off-by: Auke K

[PATCH] tcp: simpler bic default

2006-09-19 Thread Stephen Hemminger
New tcp default config means we can eliminate the hack that was used previous Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> --- net-2.6.19.orig/net/ipv4/Kconfig2006-09-19 14:27:58.0 -0700 +++ net-2.6.19/net/ipv4/Kconfig 2006-09-19 14:28:17.0 -0700 @@ -602,10 +602,6 @@

ipsec acquire has security context although I am not using it.

2006-09-19 Thread Joy Latten
When using ipsec while selinux is enabled in my kernel, my racoon daemon fails to establish an SA. I believe the ACQUIRE sent from kernel has a security context although I am not using this feature with ipsec. As a result, racoon fails to establish the SA, because it is looking for a policy with

Re: 2.6.18-rc7-mm1: networking breakage on HPC nx6325 + SUSE 10.1

2006-09-19 Thread Rafael J. Wysocki
On Tuesday, 19 September 2006 22:36, Andrew Morton wrote: > On Tue, 19 Sep 2006 22:25:21 +0200 > "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > > > - It took maybe ten hours solid work to get this dogpile vaguely > > > compiling and limping to a login prompt on x86, x86_64 and powerpc. > >

Re: [PATCH] tcp: set congestion default through Kconfig (v2)

2006-09-19 Thread David Miller
From: Stephen Hemminger <[EMAIL PROTECTED]> Date: Tue, 19 Sep 2006 13:41:32 -0700 > Bert's attempt was noble > It showed your desire for the truth > A simple path existed > > Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> Applied, but... net/ipv4/Kconfig:607:warning: defaults for choice v

[PATCH] sky2: process tx pause frames.

2006-09-19 Thread Stephen Hemminger
This patch already is in 2.6.17 stable, but the bigger version was pushed off till 2.6.19. Here is a less intrusive version that needs to go into 2.6.18 (or I'll end up sending it for 2.6.18.1). The driver was telling the GMAC to flush (not process) pause frames. Manually disabling pause wasn't wor

Re: [PATCH 00/23] e100, e1000, ixgb updates

2006-09-19 Thread Auke Kok
Jeff Garzik wrote: FYI, note that I killed some OS wrappers from osdep.h in e1000 and ixgb excellent, I am ashamed I didn't get to that myself yet :) Thanks! Auke - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo i

Re: [PATCH 12/23] e1000: Maybe stop TX if not enough free descriptors

2006-09-19 Thread Stephen Hemminger
On Tue, 19 Sep 2006 16:45:06 -0400 Jeff Garzik <[EMAIL PROTECTED]> wrote: > looks OK except for the tasklet, which may starve if the lock is being > held upon entry > > Why would the tasklet starve anymore than NAPI? Worst case, the transmitters fill the ring completely and have to wait for th

Re: [PATCH 12/23] e1000: Maybe stop TX if not enough free descriptors

2006-09-19 Thread Jeff Garzik
looks OK except for the tasklet, which may starve if the lock is being held upon entry - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 18/23] e1000: revert 'e1000: Remove 0x1000 as supported device'

2006-09-19 Thread Jeff Garzik
Auke Kok wrote: Jeff Garzik wrote: Kok, Auke wrote: The commit 'e1000: Remove 0x1000 as supported device' (Jeff Kirsher, 673a052fde79ab5e9dce569b0336358812ddba2d) Removes PIC device ID 8086:1000 from the list of supported devices. A fix was submitted for the original issue (commit 6a9516989f9

Re: Network performance degradation from 2.6.11.12 to 2.6.16.20

2006-09-19 Thread Andi Kleen
> It seems only natural to me that the real problem is the slow > clock source which needs to be resolved regardless of the > outcome of this discussion. I believe that updating the stamp > at socket enqueue time is the right thing to do but it shouldn't > be considered as a solution to the perfor

Re: [PATCH 18/23] e1000: revert 'e1000: Remove 0x1000 as supported device'

2006-09-19 Thread Auke Kok
Jeff Garzik wrote: Kok, Auke wrote: The commit 'e1000: Remove 0x1000 as supported device' (Jeff Kirsher, 673a052fde79ab5e9dce569b0336358812ddba2d) Removes PIC device ID 8086:1000 from the list of supported devices. A fix was submitted for the original issue (commit 6a9516989f94df10d9a27ba543c6b5

[PATCH] tcp: set congestion default through Kconfig (v2)

2006-09-19 Thread Stephen Hemminger
Bert's attempt was noble It showed your desire for the truth A simple path existed Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> --- net/ipv4/Kconfig | 39 +-- net/ipv4/sysctl_net_ipv4.c |6 ++ net/ipv4/tcp_cong.c|2 +- 3

Re: 2.6.18-rc7-mm1

2006-09-19 Thread Andrew Morton
On Tue, 19 Sep 2006 22:25:21 +0200 "Rafael J. Wysocki" <[EMAIL PROTECTED]> wrote: > > - It took maybe ten hours solid work to get this dogpile vaguely > > compiling and limping to a login prompt on x86, x86_64 and powerpc. > > I guess it's worth briefly testing if you're keen. > > It's not t

Re: [PATCH 1/3] sky2: tx pause bug fix

2006-09-19 Thread Stephen Hemminger
On Mon, 11 Sep 2006 11:35:11 +1000 "Andrew Hall" <[EMAIL PROTECTED]> wrote: > Stephen, > > After some serious testing, this patch seems to fix the lockup issue > completely. I manually applied these changes against the 2.6.17.13 release. Any more problems? - To unsubscribe from this list: send

Re: [PATCH 04/23] e100: rework WoL and shutdown handling

2006-09-19 Thread Jeff Garzik
Auke Kok wrote: Jeff Garzik wrote: Kok, Auke wrote: Unify our shutdown/suspend/resume code and make it similar to e1000: e1000_shutdown now calls suspend which does the exact same thing on shutdown except saving PCI config state on suspend. WoL setup code is now also more simple and works even

Re: Network performance degradation from 2.6.11.12 to 2.6.16.20

2006-09-19 Thread Thomas Graf
* David Miller <[EMAIL PROTECTED]> 2006-09-18 14:22 > From: Alexey Kuznetsov <[EMAIL PROTECTED]> > Date: Tue, 19 Sep 2006 01:03:21 +0400 > > > 1. It even does not disable possibility to record timestamp inside > >driver, which Alan was afraid of. The sequence is: > > > > if (!skb->tstamp.

Re: [PATCH 04/23] e100: rework WoL and shutdown handling

2006-09-19 Thread Auke Kok
Jeff Garzik wrote: Kok, Auke wrote: Unify our shutdown/suspend/resume code and make it similar to e1000: e1000_shutdown now calls suspend which does the exact same thing on shutdown except saving PCI config state on suspend. WoL setup code is now also more simple and works even when CONFIG_PM is

Re: [PATCH 3/3][ATM]: potential NULL pointer dereference in clip_mkip()

2006-09-19 Thread David Miller
From: "chas williams - CONTRACTOR" <[EMAIL PROTECTED]> Date: Sat, 16 Sep 2006 20:19:21 -0400 > please consider for 2.6.18 -- thanks! > > [ATM]: potential NULL pointer dereference in clip_mkip() > > http://article.gmane.org/gmane.linux.kernel/445717 > > When re-processing received data, a struct

Re: [XFRM]: Fix wildcard as tunnel source

2006-09-19 Thread David Miller
From: Patrick McHardy <[EMAIL PROTECTED]> Date: Mon, 18 Sep 2006 11:43:49 +0200 > D'oh. Fixed patch attached. Looks good Patrick, applied. Thanks. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger

Re: [PATCH 2/3][ATM]: [he] don't hold the device lock when upcalling

2006-09-19 Thread David Miller
From: "chas williams - CONTRACTOR" <[EMAIL PROTECTED]> Date: Sat, 16 Sep 2006 20:19:02 -0400 > please consider for 2.6.18 -- thanks! > > [ATM]: [he] don't hold the device lock when upcalling > > This can create a deadlock/lock ordering problem with other layers that > want to use the transmit (o

Re: [PATCH 12/23] e1000: Maybe stop TX if not enough free descriptors

2006-09-19 Thread Stephen Hemminger
My preference would be for drivers not to use LLTX (except loopback) and just use the dev->xmit_lock via netif_tx_lock if possible. Something like this for e1000. Subject: [PATCH] e1000 no lltx Get rid of lockless transmit for e1000. Use netif_tx_lock instead of having to do locking in device.

Re: Network performance degradation from 2.6.11.12 to 2.6.16.20

2006-09-19 Thread David Miller
From: Alexey Kuznetsov <[EMAIL PROTECTED]> Date: Tue, 19 Sep 2006 02:00:38 +0400 > * I do not undestand what the hell dhcp needs timestamps for. I can't even find a reference to SIOCGSTAMP in the dhcp-2.0pl5 or dhcp3-3.0.3 sources shipped in Ubuntu. But I will note that tpacket_rcv() expects to

Re: Network performance degradation from 2.6.11.12 to 2.6.16.20

2006-09-19 Thread Stephen Hemminger
The sky2 hardware (and others) can timestamp in hardware, but trying to keep device ticks and system clock in sync looked too nasty to contemplate actually using it. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info

Re: [PATCH 23/23] e100, e1000, ixgb: increment version numbers

2006-09-19 Thread Jeff Garzik
ACK patches 21-23 - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] tcp: set congestion default through Kconfig

2006-09-19 Thread Stephen Hemminger
It built (and ran) for me. But that was on a different machine. Perhaps there was some git confusion, I'll go back to using quilt it gives me less grief. -- Stephen Hemminger <[EMAIL PROTECTED]> - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL

Re: Fw: [PATCH] Remove powerpc specific parts of 3c509 driver

2006-09-19 Thread Linas Vepstas
On Tue, Sep 19, 2006 at 08:52:18PM +0200, Matt Sealey wrote: [...] > > Linas Vepstas wrote: > >On Tue, Sep 19, 2006 at 02:54:33PM +1000, Stephen Rothwell wrote: > >>On powerpc and ppc, insl_ns and insl are identical as are outsl_ns and > >>outsl, so remove the conditional use of insl_ns and outsl_

Re: Network performance degradation from 2.6.11.12 to 2.6.16.20

2006-09-19 Thread David Miller
From: David Lang <[EMAIL PROTECTED]> Date: Mon, 18 Sep 2006 14:57:04 -0700 (PDT) > yes tcpdump may be wrong in requesting timestamps (in most cases it > probably is, but in some cases it's doing exactly what the sysadmin > wants it to do), but I don't think that many sysadmins would expect > this

Re: [PATCH 12/23] e1000: Maybe stop TX if not enough free descriptors

2006-09-19 Thread Jeff Garzik
ACK - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: Network performance degradation from 2.6.11.12 to 2.6.16.20

2006-09-19 Thread David Miller
From: "Vladimir B. Savkin" <[EMAIL PROTECTED]> Date: Tue, 19 Sep 2006 02:03:31 +0400 > On Tue, Sep 19, 2006 at 02:00:38AM +0400, Alexey Kuznetsov wrote: > > * I do see get_offset_pmtmr() in top lines of profile. That's scary enough. > > I had it at the very top line. That is just rediculious. Y

Re: [PATCH 12/23] e1000: Maybe stop TX if not enough free descriptors

2006-09-19 Thread Jeff Garzik
Actually, I rescind the ACK. The code should be inside a spinlock, and therefore not need this additional check. If this check were truly needed, then SMP code all over the kernel would be broken. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message

Re: [PATCH 00/23] e100, e1000, ixgb updates

2006-09-19 Thread Jeff Garzik
FYI, note that I killed some OS wrappers from osdep.h in e1000 and ixgb - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 14/23] e1000: handle manageability for pci-e adapters at PHY powerdown

2006-09-19 Thread Jeff Garzik
Kok, Auke wrote: When powering down the PHY (if WoL is disabled) we should only check copper PHY's and handle PCI-E adapters differently. Signed-off-by: Bruce Allan <[EMAIL PROTECTED]> Signed-off-by: Auke Kok <[EMAIL PROTECTED]> --- drivers/net/e1000/e1000_main.c | 41 +++

Re: [PATCH 16/23] e1000: reduce RAR entries available for ICH8

2006-09-19 Thread Jeff Garzik
ACK patches 15-16 - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 18/23] e1000: revert 'e1000: Remove 0x1000 as supported device'

2006-09-19 Thread Jeff Garzik
Kok, Auke wrote: The commit 'e1000: Remove 0x1000 as supported device' (Jeff Kirsher, 673a052fde79ab5e9dce569b0336358812ddba2d) Removes PIC device ID 8086:1000 from the list of supported devices. A fix was submitted for the original issue (commit 6a9516989f94df10d9a27ba543c6b53b3e69c84a). This c

Re: [PATCH 17/23] e1000: driver state fixes (race fix)

2006-09-19 Thread Jeff Garzik
ACK - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 20/23] e1000: don't strip vlan ID if 8021q claims it

2006-09-19 Thread Jeff Garzik
ACK 19-20 - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] make some netfilter globals __read_mostly

2006-09-19 Thread James Morris
On Tue, 19 Sep 2006, Brian Haley wrote: > Make some netfilter globals __read_mostly at the request of Patrick McHardy. > > Signed-off-by: Brian Haley <[EMAIL PROTECTED]> These look good to me. -- James Morris <[EMAIL PROTECTED]> - To unsubscribe from this list: send the line "unsubscribe netd

Re: [PATCH 13/23] e1000: gather hardware bit tweaks.

2006-09-19 Thread Jeff Garzik
Kok, Auke wrote: Several hardware bits were set all over the driver and have been consolidated into a single function. diff --git a/drivers/net/e1000/e1000_hw.c b/drivers/net/e1000/e1000_hw.c index 9422864..a143b49 100644 --- a/drivers/net/e1000/e1000_hw.c +++ b/drivers/net/e1000/e1000_hw.c @@

Re: [PATCH 11/23] e1000: Jumbo frames fixes for 82573

2006-09-19 Thread Jeff Garzik
Kok, Auke wrote: Disable jumbo frames for 82573L alltogether and when ASPM is enabled since the hardware has problems with it. For the NICs that do support this in the 82573 series we set ERT_2048 to attempt to receive as much traffic as early as we can. Signed-off-by: Bruce Allan <[EMAIL PROTEC

Re: [PATCH] tcp: set congestion default through Kconfig

2006-09-19 Thread David Miller
Ok, I really give up, after adding the missing semicolon I now get this: net/ipv4/sysctl_net_ipv4.c:136: warning: initialization from incompatible pointer type This is getting rediculious. Folks, don't post patches which are not "request for comments" explicitly in the subject line without tes

[PATCH] make some netfilter globals __read_mostly

2006-09-19 Thread Brian Haley
Make some netfilter globals __read_mostly at the request of Patrick McHardy. Signed-off-by: Brian Haley <[EMAIL PROTECTED]> diff --git a/net/ipv4/netfilter/ip_conntrack_core.c b/net/ipv4/netfilter/ip_conntrack_core.c index aa45917..2370245 100644 --- a/net/ipv4/netfilter/ip_conntrack_core.c +++ b

Re: [PATCH 07/23] e1000: remove unused code and make symbols static

2006-09-19 Thread Jeff Garzik
ACK patches 6-7 - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH 10/23] e1000: Fix MANC detection for PCIE adapters

2006-09-19 Thread Jeff Garzik
Kok, Auke wrote: Several manageability capability detection parts hinted towards our code being incomplete for PCI-E. According to spec, we do not want to poke any MANC bits at all. Signed-off-by: Jeff Kirsher <[EMAIL PROTECTED]> Signed-off-by: Auke Kok <[EMAIL PROTECTED]> --- drivers/net/e100

Re: [PATCH] tcp: set congestion default through Kconfig

2006-09-19 Thread David Miller
From: Stephen Hemminger <[EMAIL PROTECTED]> Date: Mon, 18 Sep 2006 21:41:04 -0700 > Bert's attempt was noble > It showed your desire for the truth > A simple path exists > > Signed-off-by: Stephen Hemminger <[EMAIL PROTECTED]> Looks good, applied. - To unsubscribe from this list: send the line "

Re: [PATCH] tcp: set congestion default through Kconfig

2006-09-19 Thread David Miller
Actually, I take that back: net/ipv4/sysctl_net_ipv4.c: In function $,1rx(Btcp_congestion_default$,1ry(B: net/ipv4/sysctl_net_ipv4.c:134: error: syntax error before $,1rx(B}$,1ry(B token net/ipv4/sysctl_net_ipv4.c: At top level: net/ipv4/sysctl_net_ipv4.c:136: warning: initialization fr

Re: [PATCH 08/23] e1000: add multicast stats counters

2006-09-19 Thread Jeff Garzik
Kok, Auke wrote: Signed-off-by: Mitch Williams <[EMAIL PROTECTED]> Signed-off-by: Auke Kok <[EMAIL PROTECTED]> --- drivers/net/e1000/e1000_ethtool.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/drivers/net/e1000/e1000_ethtool.c b/drivers/net/e1000/e1000_ethtool.c

Re: [PATCH 05/23] e1000: rename flow control symbols

2006-09-19 Thread Jeff Garzik
Kok, Auke wrote: --- a/drivers/net/e1000/e1000_hw.h +++ b/drivers/net/e1000/e1000_hw.h @@ -90,15 +90,6 @@ typedef enum { e1000_100_full = 3 } e1000_speed_duplex_type; -/* Flow Control Settings */ -typedef enum { -e1000_fc_none = 0, -e1000_fc_rx_pause = 1, -e1000_fc_tx_pause =

Re: [PATCH 03/23] e100: Add debugging code for cb cleaning and csum failures.

2006-09-19 Thread Jeff Garzik
Kok, Auke wrote: Refine cb cleaning debug printout and print out all cleaned cbs' status. Add debug flag for EEPROM csum failures that were overridden by the user. Signed-off-by: Jesse Brandeburg <[EMAIL PROTECTED]> Signed-off-by: Auke Kok <[EMAIL PROTECTED]> ACK patch, NAK description: taint

Re: [PATCH 04/23] e100: rework WoL and shutdown handling

2006-09-19 Thread Jeff Garzik
Kok, Auke wrote: Unify our shutdown/suspend/resume code and make it similar to e1000: e1000_shutdown now calls suspend which does the exact same thing on shutdown except saving PCI config state on suspend. WoL setup code is now also more simple and works even when CONFIG_PM is not set, which was

Re: [PATCH] [PATCH] ethtool: add ixgb register dump support

2006-09-19 Thread Jeff Garzik
applied - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [REVISED] [PATCH] ethtool v4: add autoneg advertise feature

2006-09-19 Thread Jeff Garzik
Jeff Kirsher wrote: On 9/5/06, Jeff Kirsher <[EMAIL PROTECTED]> wrote: adds the ability to change the advertised speed and duplex for a network interface. Previously, a network interface was only able to advertise all supported speed's and duplex's, or one individual speed and duplex. The

  1   2   >