Re: [PATHCH 1/16] ServerEngines 10Gb NIC driver

2008-02-19 Thread Jay Cliburn
On Mon, 18 Feb 2008 10:00:51 -0800 "Subbu Seetharaman" <[EMAIL PROTECTED]> wrote: > I have one question about bit fields. Several of > headers in the common code are generated by > srcgen from f/w source files. Some of the structures > in these headers have bit fields (with separate defin

[PATCH 00/10] atl1: move to atlx and update for 2.6.25

2008-02-02 Thread Jay Cliburn
In preparation for a future atl2 driver for the Atheros L2 10/100 chip, we propose to move the existing atl1 driver to a new directory (drivers/net/atlx), then split out functions and definitions that both atl1 and atl2 can share. The final structure will look like this: drivers/net/atl1

[PATCH 09/10] atl1: make functions static

2008-02-02 Thread Jay Cliburn
Make needlessly global functions static. In a couple of cases this requires removing forward declarations and reordering functions. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> Acked-by: Chris Snook <[EMAIL PROTECTED]> --- drivers/net/atlx/a

[PATCH 10/10] atl1: reduce forward declarations

2008-02-02 Thread Jay Cliburn
Rearrange functions to allow removal of some forward declarations. Make certain global functions static along the way. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> Acked-by: Chris Snook <[EMAIL PROTECTED]> --- drivers/net/atlx/a

[PATCH 07/10] atl1: use netif_msg

2008-02-02 Thread Jay Cliburn
Use netif_msg_* for console messages emitted by the driver. Add a parameter to allow control of messaging at driver startup, and also add the ability to control it with ethtool. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> Acked-by: Chris Snook <[EMAIL PROTECTED]> --- drivers/net

[PATCH 08/10] atl1: print debug info if rrd error

2008-02-02 Thread Jay Cliburn
Add some debug printks if we encounter a potentially bad receive return descriptor. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> Acked-by: Chris Snook <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 26 ++ 1 files changed, 22 insertions(+), 4 deleti

[PATCH 06/10] atl1: use csum_start

2008-02-02 Thread Jay Cliburn
Use skb->csum_start for tx checksum offload preparation. Also swap the variables css and cso so they hold the intended values of csum start and offset, respectively. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> Acked-by: Chris Snook <[EMAIL PROTECTED]> --- drivers/net/atl

[PATCH 05/10] atl1: simplify tx packet descriptor

2008-02-02 Thread Jay Cliburn
. Finally, shorten some variable names in the transmit processing path to reduce line lengths, rename some variables to better describe their purpose (e.g., nseg versus m), and add a comment or two to better describe what the code is doing. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> Acked-by:

[PATCH 04/10] atl1: add ethtool register dump

2008-02-02 Thread Jay Cliburn
Add the ethtool register dump option to the atl1 driver. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> Acked-by: Chris Snook <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 53 +++ drivers/net/atlx/atl1.h |1 + 2 files changed, 5

[PATCH 01/10] atl1: relocate atl1 driver to /drivers/net/atlx

2008-02-02 Thread Jay Cliburn
In preparation for a future Atheros L2 NIC driver (called atl2), relocate the atl1 driver into a new /drivers/net/atlx directory that will ultimately be shared with the future atl2 driver. Signed-off-by: Chris Snook <[EMAIL PROTECTED]> Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]>

[PATCH 03/10] atl1: fix broken TSO

2008-02-02 Thread Jay Cliburn
tpd. Also, some of our bit assignments are made to the wrong tpd words. Change those to the correct words. Finally, since all this fixes TSO, enable TSO by default. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> Acked-by: Chris Snook <[EMAIL PROTECTED]> --- drivers/net/atlx/a

Re: [PATCH 09/26] atl1: refactor tx processing

2008-01-24 Thread Jay Cliburn
On Tue, 22 Jan 2008 18:31:09 -0600 Jay Cliburn <[EMAIL PROTECTED]> wrote: > On Tue, 22 Jan 2008 04:58:17 -0500 > Jeff Garzik <[EMAIL PROTECTED]> wrote: > [...] > > for such a huge patch, this description is very tiny. [describe] > > what is refactored, and why.

Re: [PATCH 06/26] atl1: update initialization parameters

2008-01-22 Thread Jay Cliburn
On Tue, 22 Jan 2008 04:56:11 -0500 Jeff Garzik <[EMAIL PROTECTED]> wrote: > [EMAIL PROTECTED] wrote: > > From: Jay Cliburn <[EMAIL PROTECTED]> > > > > Update initialization parameters to match the current vendor driver > > version 1.2.40.2. [...] > A

Re: [PATCH 09/26] atl1: refactor tx processing

2008-01-22 Thread Jay Cliburn
On Tue, 22 Jan 2008 04:58:17 -0500 Jeff Garzik <[EMAIL PROTECTED]> wrote: > [EMAIL PROTECTED] wrote: > > From: Jay Cliburn <[EMAIL PROTECTED]> > > > > Refactor tx processing to use a less convoluted tx packet > > descriptor and to conform generally with th

Re: [PATCH] atl1: fix frame length bug

2008-01-18 Thread Jay Cliburn
On Mon, 14 Jan 2008 20:04:28 -0600 Jay Cliburn <[EMAIL PROTECTED]> wrote: > On Mon, 14 Jan 2008 19:56:41 -0600 > Jay Cliburn <[EMAIL PROTECTED]> wrote: > > > The driver sets up the hardware to accept a frame with max length > > equal to MTU + Ethernet header +

Re: [PATCH] atl1: fix frame length bug

2008-01-14 Thread Jay Cliburn
On Mon, 14 Jan 2008 19:56:41 -0600 Jay Cliburn <[EMAIL PROTECTED]> wrote: > The driver sets up the hardware to accept a frame with max length > equal to MTU + Ethernet header + FCS + VLAN tag, but we neglect to > add the VLAN tag size to the ingress buffer. When a VLAN-tagged

[PATCH] atl1: fix frame length bug

2008-01-14 Thread Jay Cliburn
patch fixes that. Thanks to David Harris for reporting the bug and testing the fix. Tested-by: David Harris <[EMAIL PROTECTED]> Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --g

Re: my NAPI patches

2008-01-10 Thread Jay Cliburn
On Tue, 08 Jan 2008 15:08:48 -0800 (PST) David Miller <[EMAIL PROTECTED]> wrote: > > Can people do me a favor and do more constructive things like test > that my patches really do fix the "device down during packet flood" > bug Hi Dave, Your patches seem to work for the atl1 NAPI implementatio

Re: [PATCH 25/26] [REVISED] atl1: add NAPI support

2008-01-01 Thread Jay Cliburn
Thanks for your comments Stephen and Joonwoo. Here's the revised version of the atl1 NAPI patch. >From 9c3a8944220287671f983557099bc329f02fda9b Mon Sep 17 00:00:00 2001 From: Jay Cliburn <[EMAIL PROTECTED]> Date: Tue, 1 Jan 2008 11:55:24 -0600 Subject: [PATCH 25/26] atl1: add NAP

Re: [PATCH 00/26] atl1: divide and modernize

2008-01-01 Thread Jay Cliburn
On Tue, 1 Jan 2008 17:58:40 + Christoph Hellwig <[EMAIL PROTECTED]> wrote: > On Mon, Dec 31, 2007 at 08:22:27PM -0600, Jay Cliburn wrote: > > The first two patches submitted in this patchset accomplish the > > relocation by movng the atl1 driver -- lock, stock, and

[PATCH 11/26] atl1: refactor initialization and startup

2007-12-31 Thread Jay Cliburn
Refactor atl1 initialization and startup to conform with the current vendor driver version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 507 --- drivers/net/atlx/atl1.h | 10 +- drivers/net/atlx/

[PATCH 18/26] atl1: make function static

2007-12-31 Thread Jay Cliburn
Make atl1_reset() a static function. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/net/atlx/atl1.c b/drivers/net/atlx/atl1.c index 6432956..7697e80 100644 --- a/drive

[PATCH 22/26] atl1: update netpoll

2007-12-31 Thread Jay Cliburn
Rename atl1_poll_controller() to atl1_netpoll() and update to conform with the current vendor driver version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 11 +++ 1 files changed, 7 insertions(+), 4 deletions(-) diff --git a/drivers/ne

[PATCH 21/26] atl1: update atl1_close

2007-12-31 Thread Jay Cliburn
Update atl1_close() to conform with current vendor driver version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 14 ++ 1 files changed, 14 insertions(+), 0 deletions(-) diff --git a/drivers/net/atlx/atl1.c b/drivers/net/atlx/atl1.c

[PATCH 15/26] atl1: tidy up ring management

2007-12-31 Thread Jay Cliburn
Check for null pointers and such in ring handling functions. Make needlessly global functions static. Clean up some comments and indentation. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 20 +++- 1 files changed, 11 insertions(+), 9 del

[PATCH 24/26] atl1: update wake-on-lan

2007-12-31 Thread Jay Cliburn
Update wake-on-lan to conform with the current vendor driver version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 140 --- 1 files changed, 84 insertions(+), 56 deletions(-) diff --git a/drivers/net/atl

[PATCH 26/26] atl1: remove experimental tag and clean up comments

2007-12-31 Thread Jay Cliburn
All major netdevice functionality is now present in the atl1 driver. As a result, remove the EXPERIMENTAL Kconfig tag in the main driver, but leave it in the NAPI driver. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/Kconfig |4 ++-- drivers/net/atlx/atl1.c

[PATCH 12/26] atl1: refactor atl1_probe

2007-12-31 Thread Jay Cliburn
Refactor atl1_probe to conform with current vendor driver version 1.2.40.2. Also reorder functions to minimize the need for forward declarations. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 1397 +++ drivers/ne

[PATCH 20/26] atl1: update change mtu

2007-12-31 Thread Jay Cliburn
Update atl1_change_mtu() to conform with the current vendor driver version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 39 ++- 1 files changed, 22 insertions(+), 17 deletions(-) diff --git a/drivers/net/atlx/

[PATCH 05/26] atl1: print debug info if rrd error

2007-12-31 Thread Jay Cliburn
Add some debug dev_printks if we encounter a bad receive return descriptor. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 21 - 1 files changed, 20 insertions(+), 1 deletions(-) diff --git a/drivers/net/atlx/atl1.c b/drivers/net/atlx/

[PATCH 23/26] atl1: update shutdown and remove functions

2007-12-31 Thread Jay Cliburn
Update shutdown and remove functions to conform with the current vendor driver version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 31 +++ 1 files changed, 23 insertions(+), 8 deletions(-) diff --git a/drivers/ne

[PATCH 25/26] atl1: add NAPI support

2007-12-31 Thread Jay Cliburn
Add support for NAPI, styled after the e1000 NAPI implementation. That we follow the e1000 for NAPI shouldn't come as much of a surprise, since the entire atl1 driver is based heavily upon it. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/Kconfig | 14

[PATCH 16/26] atl1: modernize check link function

2007-12-31 Thread Jay Cliburn
Update atl1_check_link() to conform with the current vendor driver version 1.2.40.2. Clean up vertical spacing, indentation, and remove dead code. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 65 +++--- drivers/ne

[PATCH 17/26] atl1: update phy config function

2007-12-31 Thread Jay Cliburn
Update atl1_phy_config() to conform with current vendor driver version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/atlx/atl1.c b/drivers/net/atlx/atl1.c index a

[PATCH 13/26] atl1: refactor interrupt handling

2007-12-31 Thread Jay Cliburn
Refactor interrupt handling to conform with the current vendor driver version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 196 ++- drivers/net/atlx/atl1.h | 25 +- drivers/net/atlx/atlx.c |2

[PATCH 08/26] atl1: additional DMA engine configuration

2007-12-31 Thread Jay Cliburn
Add DMA engine configuration tweaks per current vendor atl1 driver version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 10 ++ drivers/net/atlx/atlx.h |6 ++ 2 files changed, 16 insertions(+), 0 deletions(-) diff --git a/drive

[PATCH 04/26] atl1: add ethtool register dump

2007-12-31 Thread Jay Cliburn
Add the ethtool register dump option to the atl1 driver. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 53 +++ drivers/net/atlx/atl1.h |1 + 2 files changed, 54 insertions(+), 0 deletions(-) diff --git a/d

[PATCH 19/26] atl1: modernize down/up functions

2007-12-31 Thread Jay Cliburn
Update atl1_down() and atl1_up() to conform with the current vendor driver version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 21 - drivers/net/atlx/atl1.h |1 - drivers/net/atlx/atlx.c |2 +- 3 files changed, 9 inse

[PATCH 10/26] atl1: use csum_start

2007-12-31 Thread Jay Cliburn
Use skb->csum_start for tx checksum offload preparation. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/net/atlx/atl1.c b/drivers/net/atlx/atl1.c index b0c3273..31aad

[PATCH 06/26] atl1: update initialization parameters

2007-12-31 Thread Jay Cliburn
Update initialization parameters to match the current vendor driver version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 14 +++--- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/drivers/net/atlx/atl1.c b/drivers/net/atlx/

[PATCH 07/26] atl1: clarify max rx frame size

2007-12-31 Thread Jay Cliburn
When we initially set max rx frame size, we don't explicitly allow room for the VLAN header; it's done later in a somewhat obscure fashion. Let's make it clear from the top that we've allowed enough room for the VLAN header. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]

[PATCH 00/26] atl1: divide and modernize

2007-12-31 Thread Jay Cliburn
mode 100644 drivers/net/atlx/Makefile create mode 100644 drivers/net/atlx/atl1.c create mode 100644 drivers/net/atlx/atl1.h create mode 100644 drivers/net/atlx/atlx.c create mode 100644 drivers/net/atlx/atlx.h Best regards, Jay Cliburn -- To unsubscribe from this list: send the line "u

[PATCH 03/26] atl1: fix broken TSO

2007-12-31 Thread Jay Cliburn
tpd. Also, some of our bit assignments are made to the wrong tpd words. Change those to the correct words. Finally, since all this fixes TSO, enable TSO by default. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 29 +++-- 1 files chang

[PATCH 09/26] atl1: refactor tx processing

2007-12-31 Thread Jay Cliburn
Refactor tx processing to use a less convoluted tx packet descriptor and to conform generally with the vendor's current version 1.2.40.2. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 265 +-- drivers/net/

[PATCH 01/26] atl1: relocate atl1 driver to /drivers/net/atlx

2007-12-31 Thread Jay Cliburn
In preparation for a future Atheros L2 NIC driver (called atl2), relocate the atl1 driver into a new /drivers/net/atlx directory that will ultimately be shared with the future atl2 driver. Signed-off-by: Chris Snook <[EMAIL PROTECTED]> Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]>

[PATCH 14/26] atl1: move stray defines to header file

2007-12-31 Thread Jay Cliburn
Move some stray defines out to a header file. Improve indentation from ghastly to horrid. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atlx/atl1.c | 29 - drivers/net/atlx/atl1.h |6 ++ 2 files changed, 18 insertions(+), 17 del

Re: netif_rx will not free skb when I use ftp in kernel 2.6.22/2.6.21

2007-09-20 Thread Jay Cliburn
Chris Snook wrote: wrote: in function at_alloc_rx_buffers(), pci_unmap_page() and netif_rx() in function at_clean_rx_irq(), Okay, I didn't know you were talking about the atl1 driver. Are you using the in-tree driver in 2.6.22, or the pre-merge driver on sourceforge, or the vendor d

Re: [Bugme-new] [Bug 8996] New: atl1 driver cause kernel oops IF ram > 4Gyte and a lot of network load

2007-09-08 Thread Jay Cliburn
On Sat, 8 Sep 2007 17:36:21 -0700 Andrew Morton <[EMAIL PROTECTED]> wrote: > On Sat, 8 Sep 2007 13:08:37 -0700 (PDT) > [EMAIL PROTECTED] wrote: > > > http://bugzilla.kernel.org/show_bug.cgi?id=8996 > > > >Summary: atl1 driver cause kernel oops IF ram > 4Gyte > > and a lot of network

[PATCH] atl1: use spin_trylock_irqsave()

2007-07-31 Thread Jay Cliburn
From: Ingo Molnar <[EMAIL PROTECTED]> use the simpler spin_trylock_irqsave() API to get the adapter lock. [ this is also a fix for -rt where adapter->lock is a sleeping lock. ] Signed-off-by: Ingo Molnar <[EMAIL PROTECTED]> Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]>

Re: atl1 driver corrupting memory?

2007-07-25 Thread Jay Cliburn
On Wed, 25 Jul 2007 17:31:02 -0400 Chuck Ebbert <[EMAIL PROTECTED]> wrote: > This is what was committed. > > http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=3f516c00d416bd39aab6cfb348b68919e295fe23 > http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.

[PATCH 5/5] atl1: use kernel provided ethernet length constants

2007-07-19 Thread Jay Cliburn
Use constants already provided by the kernel for ethernet related lengths. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_hw.h |5 - drivers/net/atl1/atl1_main.c | 13 +++-- 2 files changed, 7 insertions(+), 11 deletions(-) diff --git a/drive

[PATCH 2/5] atl1: fix typo in DMA engine setup

2007-07-19 Thread Jay Cliburn
The DMA engine setup contains a typo that can result in an incorrect dmaw_block setting. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/atl1/atl1_main.c b/drivers/ne

[PATCH 4/5] atl1: fix typo in dma_req_block

2007-07-19 Thread Jay Cliburn
s/dam/dma Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_hw.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/atl1/atl1_hw.h b/drivers/net/atl1/atl1_hw.h index 100c09c..f0d7e82 100644 --- a/drivers/net/atl1/atl1_hw.h +++ b/d

[PATCH 3/5] atl1: change cmb write threshold

2007-07-19 Thread Jay Cliburn
: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/drivers/net/atl1/atl1_main.c b/drivers/net/atl1/atl1_main.c index 3b8f633..6aa2dc3 100644 --- a/drivers/net/atl1/atl1_main.c +++ b/drivers/ne

[PATCH 0/5] atl1: additional patches for 2.6.23

2007-07-19 Thread Jay Cliburn
Please accept the following patches for the atl1 driver. Thanks. atl1: use kernel provided ethernet length constants atl1: fix typo in dma_req_block atl1: change cmb write threshold atl1: fix typo in DMA engine setup atl1: change tpd_avail function name drivers/net/atl1/atl1_hw.h |9 ++---

[PATCH 1/5] atl1: change tpd_avail function name

2007-07-19 Thread Jay Cliburn
Change tpd_avail() to atl1_tpd_avail(). Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/atl1/atl1_main.c b/drivers/net/atl1/atl1_main.c index fd1e156..79d60e1 100644

[PATCH 3/5] atl1: cleanup atl1_main

2007-07-15 Thread Jay Cliburn
Fix indentation, remove dead code, improve some comments, change dev_dbg to dev_printk. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c | 278 +- 1 files changed, 137 insertions(+), 141 deletions(-) diff --git a/d

[PATCH 2/5] atl1: header file cleanup

2007-07-15 Thread Jay Cliburn
Remove unused structure members, improve comments, break long comment lines, rename a constant to be consistent with others in the file. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1.h | 155 ++ drivers/net/atl1/atl1_

[PATCH 1/5] atl1: remove irq_sem

2007-07-15 Thread Jay Cliburn
Remove unnecessary irq_sem code. Signed-off-by: Chris Snook <[EMAIL PROTECTED]> Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1.h |1 - drivers/net/atl1/atl1_main.c |6 ++ 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/driv

[PATCH 4/5] atl1: fix excessively indented code

2007-07-15 Thread Jay Cliburn
Move excessively indented code to separate functions. Also move ring pointer initialization to its own function. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c | 86 - 1 files changed, 50 insertions(+), 36 del

[PATCH 0/5] atl1: miscellaneous cleanup and code reorganization

2007-07-15 Thread Jay Cliburn
Please accept the following patches for the atl1 driver. atl1: remove irq_sem atl1: header file cleanup atl1: cleanup atl1_main atl1: fix excessively indented code atl1: reorder atl1_main functions drivers/net/atl1/atl1.h | 156 ++-- drivers/net/atl1/atl1_main.c | 2176

Re: [PATCH] atl1: remove write-only var in tx handler

2007-07-06 Thread Jay Cliburn
update = 1; > + > tpd = ATL1_TPD_DESC(tpd_ring, sw_tpd_next_to_clean); > buffer_info = > &tpd_ring->buffer_info[sw_tpd_next_to_clean]; if (buffer_info->dma) { > Acked-by: Jay Cliburn <[EMAIL PROTECTED]> - To unsubscribe from this list: send the line "u

Re: [PATCH] atl1: disable 64bit DMA

2007-06-26 Thread Jay Cliburn
On Mon, 25 Jun 2007 23:18:55 +0200 Luca Tettamanti <[EMAIL PROTECTED]> wrote: > Il Mon, Jun 25, 2007 at 07:42:44AM -0500, Jay Cliburn ha scritto: > > Jay L. T. Cornwall wrote: > > >Jay Cliburn wrote: > > > > > >>For reasons not yet clear to me, it

Re: [PATCH] atl1: disable 64bit DMA

2007-06-25 Thread Jay Cliburn
On Mon, 25 Jun 2007 17:57:20 -0400 Chris Snook <[EMAIL PROTECTED]> wrote: > Jay L. T. Cornwall wrote: > > Chris Snook wrote: > > > >> What boards have we seen this on? It's quite possible this is: > > > > I can reproduce on an Asus P5K with a Core 2 Duo E6600. > > > > lspci identifies the cont

Re: Problem with atl1 and msi in kernel 2.6.22-rc3

2007-05-30 Thread Jay Cliburn
On Wed, 30 May 2007 02:00:25 +0200 Jose Alberto Reguero <[EMAIL PROTECTED]> wrote: > I have problems with a M2V motherboard and atl1 driver with msi and > kernel 2.6.22-rc3. With kernel 2.6.21 I had no problems. > I must add in drivers/pci/quirks.c (line 1723): > > DECLARE_PCI_FIXUP_FINAL(PCI_VE

Re: APIC error on 32-bit kernel

2007-05-12 Thread Jay Cliburn
Thank you very much for looking at this, Len. On Fri, 11 May 2007 23:28:58 -0400 Len Brown <[EMAIL PROTECTED]> wrote: > > > [ 94.754852] APIC error on CPU0: 08(40) > > > [ 94.806045] APIC error on CPU0: 40(08) > > /* Here is what the APIC error bits mean: >0: Send CS err

[PATCH 1/2] atl1: use dev_printk macros

2007-04-29 Thread Jay Cliburn
Use dev_printk macros for PCI related errors, warnings, debug and info console messages. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_ethtool.c | 19 +++-- drivers/net/atl1/atl1_hw.c | 22 +-- drivers/net/atl1/atl1_main.c

[PATCH 2/2] atl1: fix whitespace damage

2007-04-29 Thread Jay Cliburn
Remove trailing whitespace and spaces preceding tabs. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_hw.c | 22 +++--- 1 files changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/net/atl1/atl1_hw.c b/drivers/net/atl1/atl1_hw.c

[PATCH 0/2] atl1: minor cleanup

2007-04-29 Thread Jay Cliburn
Please accept the following trivial patches to the atl1 driver. - use dev_printk macros - fix whitespace damage drivers/net/atl1/atl1_ethtool.c | 19 +++-- drivers/net/atl1/atl1_hw.c | 44 ++--- drivers/net/atl1/atl1_main.c| 83 +---

Re: APIC error on 32-bit kernel

2007-04-09 Thread Jay Cliburn
Chuck Ebbert wrote: Where is the text of the oops? In one of the files on the website I referenced. Here's the text... [ 173.584000] APIC error on CPU1: 08(08) [ 173.665000] APIC error on CPU0: 08(08) [ 173.665000] APIC error on CPU1: 08(08) [ 173.746000] APIC error on CPU0: 08(08) [ 17

Re: APIC error on 32-bit kernel

2007-04-08 Thread Jay Cliburn
[Adding linux-kernel to the cc list, hoping for wider exposure.] On Fri, 23 Mar 2007 20:08:17 -0500 Jay Cliburn <[EMAIL PROTECTED]> wrote: > We're trying to track down the source of a problem that occurs > whenever the atl1 network driver is activated on a 32-bit 2.6.21-rc

Re: r8169 null pointer dereference

2007-04-01 Thread Jay Cliburn
On Sun, 1 Apr 2007 12:00:00 +0200 Francois Romieu <[EMAIL PROTECTED]> wrote: > Jay Cliburn <[EMAIL PROTECTED]> : > > Francois, > > > > I get this on 2.6.21-rc5 and earlier kernels going back to at least > > 2.6.20-git14, both i386 and x86_64, dual-core AM2

r8169 null pointer dereference

2007-03-31 Thread Jay Cliburn
Francois, I get this on 2.6.21-rc5 and earlier kernels going back to at least 2.6.20-git14, both i386 and x86_64, dual-core AM2 and LGA775 motherboards, using two different RTL8169 PCI add-in cards. Has anyone else reported it? Unable to handle kernel NULL pointer dereference at [] :r8169:rtl

Re: 2.6.21-rc5: known regressions with patches

2007-03-27 Thread Jay Cliburn
On Wed, 28 Mar 2007 01:00:24 +0200 Adrian Bunk <[EMAIL PROTECTED]> wrote: > Subject: atl1 net driver: problem with sockets > References : http://lkml.org/lkml/2007/3/21/248 > Submitter : Jose Alberto Reguero <[EMAIL PROTECTED]> > Patch : http://marc.info/?l=linux-netdev&m=117502041808665

[PATCH] atl1: remove unnecessary crc inversion

2007-03-27 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> The original vendor driver contained a private ether_crc_le() function that produced an inverted crc. When we changed to the kernel version of ether_crc_le(), we neglected to undo the inversion. Let's do it now. Discovered by and patch proffe

Re: APIC error on 32-bit kernel

2007-03-27 Thread Jay Cliburn
On Tue, 27 Mar 2007 14:42:20 -0600 [EMAIL PROTECTED] (Eric W. Biederman) wrote: Thanks for replying, Eric. I've added atl1-devel to the cc list. > Do you have msi's working in 2.6.21-rc4 in the x86_64 kernel? I can't personally verify it anymore because I removed x86_64 to duplicate the MSI pro

PCIe NICs that correctly suspend/resume

2007-02-28 Thread Jay Cliburn
We're trying to make the Attansic L1 NIC correctly suspend, resume, and wake-on-lan. Can someone point me to a PCIe-based NIC driver in the kernel tree that correctly does these things? I'd like to see how it's *supposed* to be done. Thanks, Jay - To unsubscribe from this list: send the line "un

[PATCH 6/6] atl1: bump version number

2007-02-14 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> Bump the version number. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/net/atl1/atl1_main.c b/drivers/net/atl1/atl1_main.c i

[PATCH 5/6] atl1: add L1 device id to pci_ids, then use it

2007-02-14 Thread Jay Cliburn
From: Chris Snook <[EMAIL PROTECTED]> Add device id for the Attansic L1 chip to pci_ids.h, then use it. Signed-off-by: Chris Snook <[EMAIL PROTECTED]> Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c |2 +- include/linux/pci_ids.h

[PATCH 4/6] atl1: remove unused define

2007-02-14 Thread Jay Cliburn
From: Chris Snook <[EMAIL PROTECTED]> Remove unused define from atl1_main.c. Signed-off-by: Chris Snook <[EMAIL PROTECTED]> Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c |1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git

[PATCH 3/6] atl1: read MAC address from register

2007-02-14 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> On some Asus motherboards containing the L1 NIC, the MAC address is written by the BIOS directly to the MAC register during POST, and is not stored in eeprom. If we don't succeed in fetching the MAC address from eeprom or spi, try reading it di

[PATCH 2/6] atl1: fix bad ioread address

2007-02-14 Thread Jay Cliburn
From: Al Viro <[EMAIL PROTECTED]> An ioread32 statement reads the wrong address. Fix it. Signed-off-by: Al Viro <[EMAIL PROTECTED]> Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> Signed-off-by: Chris Snook <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_hw.c |

[PATCH 1/6] atl1: drop NET_PCI from Kconfig

2007-02-14 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> The atl1 driver doesn't need NET_PCI. Remove it from Kconfig. Noticed by Chad Sprouse. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> Signed-off-by: Chris Snook <[EMAIL PROTECTED]> --- drivers/net/Kconfig |2 +- 1 files chan

[PATCH 0/6] atl1: bugfix, cleanup, enhancement

2007-02-14 Thread Jay Cliburn
Jeff, Please accept the following patchset for the atl1 network device driver. * Drop unnecessary NET_PCI config * Fix incorrect hash table address * Read MAC address from register * Remove unused define * Add Attansic L1 device id to pci_ids * Bump version number This patchset contains changes

[PATCH netdev#atl1 1/1] atl1: clean up extern, global scope declarations

2007-02-06 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> atl1: move extern to header file; make some global code static Move an extern declaration to a header file. Make needlessly global functions static. Noticed by Adrian Bunk. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net

Re: [RFC: -mm patch] drivers/net/atl1/: possible cleanups

2007-02-06 Thread Jay Cliburn
On Tue, 6 Feb 2007 23:12:29 +0100 Adrian Bunk <[EMAIL PROTECTED]> wrote: > On Mon, Jan 29, 2007 at 08:45:28PM -0800, Andrew Morton wrote: > >... > > Changes since 2.6.20-rc6-mm2: > >... > > git-netdev-all.patch > >... > > git trees > >... > > > This patch contains the following possible cleanu

[PATCH netdev#atl1 3/3] pci_ids: add Attansic vendor id

2007-02-03 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> pci_ids: add Attansic vendor id Add Attansic to pci_ids and use the ID in the driver. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c |2 +- include/linux/pci_ids.h |2 ++ 2 files changed, 3 inse

[PATCH netdev#atl1 2/3] maintainers: add atl1 maintainers

2007-02-03 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> MAINTAINERS: add atl1 maintainers Add a maintainers entry for atl1. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- MAINTAINERS | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS i

[PATCH netdev#atl1 1/3] atl1: remove whitespace damage

2007-02-03 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> atl1: fix whitespace damage Remove trailing whitespace and spaces preceding tabs. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1.h |8 +- drivers/net/atl1/atl1_ethtool.c | 42 ++-- drivers/net/at

[PATCH netdev#atl1 0/3] atl1: fix whitespace, add maintainers and pci_ids

2007-02-03 Thread Jay Cliburn
This set of patches completes the final bit of tidying up in the Attansic ethernet driver. Sorry for nickel-and-diming you, Jeff, but this really should be the end of it for awhile. Summary: 1. Clean up whitespace damage. 2. Add a maintainers entry. 3. Add a pci_ids entry. --- MAINTAINERS

[PATCH 2.6.20-rc7 2/2] atl1: add ethtool set ring params functionality

2007-01-31 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> Fix up some constants relating to max and min ring descriptor counts. Also add functionality to enable ethtool to set tx and rx ring parameters. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1.h |4 +- driv

[PATCH 2.6.20-rc7 1/2] atl1: get rid of pci_module_init

2007-01-31 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> pci_module_init is deprecated. Use pci_register_driver instead. Discovered by and modification suggested by Alan Cox. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c |5 + 1 files changed, 1 inse

Re: [PATCH 2.6.20-rc5 1/4] atl1: unconditionally enable MSI

2007-01-27 Thread Jay Cliburn
The subject line on all four of the current crop of atl1 patches is incorrect; they were generated against *2.6.20-rc6*, not rc5. I apologize for the error. Jay - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info

[PATCH 2.6.20-rc5 4/4] atl1: incorporate reviewer comments

2007-01-27 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> Incorporate reviewer comments from: Randy Dunlap, http://lkml.org/lkml/2007/1/21/157 Arjan van de Ven, http://lkml.org/lkml/2007/1/22/21 Francois Romieu, http://lkml.org/lkml/2007/1/22/49 Fixup to follow coding standards, remove MII defines already fo

[PATCH 2.6.20-rc5 3/4] atl1: properly use CONFIG_PM

2007-01-27 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> Fix power management by properly using ifdef CONFIG_PM. Discovered by and modification suggested by Andrew Morton. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c |7 +-- 1 files changed, 5 inse

[PATCH 2.6.20-rc5 2/4] atl1: add missing include dma-mapping.h

2007-01-27 Thread Jay Cliburn
From: Jay Cliburn <[EMAIL PROTECTED]> Include dma-mapphing.h to provide DMA_32BIT_MASK and DMA_64BIT_MASK. Discovered by and modification suggested by Andrew Morton. Signed-off-by: Jay Cliburn <[EMAIL PROTECTED]> --- drivers/net/atl1/atl1_main.c |1 + 1 files changed, 1 inse

[PATCH 2.6.20-rc5 1/4] atl1: unconditionally enable MSI

2007-01-27 Thread Jay Cliburn
From: Luca Tettamanti <[EMAIL PROTECTED]> Unconditionally enable MSI in atl1 driver. Also remove some useless #ifdef since pci_{en,dis}able_msi() are no-op when MSI support is not configured in. Signed-off-by: Luca Tettamanti <[EMAIL PROTECTED]> Signed-off-by: Jay Cliburn <[

Re: [PATCH 4/4] atl1: Ancillary C files for Attansic L1 driver

2007-01-27 Thread Jay Cliburn
Jeff Garzik wrote: As a driver maintainer, you need to patch sets, and submit them in a timely fashion to me. Note I said patch set, not patch, in following with Rule #3 from Documentation/SubmittingPatches. Also make sure to review http://linux.yyz.us/patch-format.html Understood. Both re

Re: [PATCH 4/4] atl1: Ancillary C files for Attansic L1 driver

2007-01-27 Thread Jay Cliburn
Luca Tettamanti wrote: [snip] Anyway... Unconditionally enable MSI in atl1 driver. Also remove some useless #ifdef since pci_{en,dis}able_msi() are no-op when MSI support is not configured in. Signed-off-by: Luca Tettamanti <[EMAIL PROTECTED]> Acked-by: Jay Cliburn <[EMAIL PROTE

Re: [PATCH 0/4] atl1: Attansic L1 ethernet driver

2007-01-23 Thread Jay Cliburn
Jeff Garzik wrote: OK, I have merged the monolithic patch into jgarzik/netdev-2.6.git#atl1. Once I'm done merging patches tonight, I will merge this new 'atl1' branch into the 'ALL' meta-branch, which will auto-propagate this driver into Andrew Morton's -mm for testing. For future driver up

  1   2   >