forked from Minki/linux
8199d3a79c
A Linux driver for the Chelsio 10Gb Ethernet Network Controller by Chelsio (http://www.chelsio.com). This driver supports the Chelsio N210 NIC and is backward compatible with the Chelsio N110 model 10Gb NICs. It supports AMD64, EM64T and x86 systems. Signed-off-by: Tina Yang <tinay@chelsio.com> Signed-off-by: Scott Bardone <sbardone@chelsio.com> Signed-off-by: Christoph Lameter <christoph@lameter.com> Adrian said: - my3126.c is unused (because t1_my3126_ops isn't used anywhere) - what are the EXTRA_CFLAGS in drivers/net/chelsio/Makefile for? - $(cxgb-y) in drivers/net/chelsio/Makefile seems to be unneeded - completely unused global functions: - espi.c: t1_espi_get_intr_counts - sge.c: t1_sge_get_intr_counts - the following functions can be made static: - sge.c: t1_espi_workaround - sge.c: t1_sge_tx - subr.c: __t1_tpi_read - subr.c: __t1_tpi_write - subr.c: t1_wait_op_done shemminger said: The performance recommendations in cxgb.txt are common to all fast devices, and should be in one file rather than just for this device. I would rather see ip-sysctl.txt updated or a new file on tuning recommendations started. Some of them have consequences that aren't documented well. For example, turning off TCP timestamps risks data corruption from sequence wrap. A new driver shouldn't need so may #ifdef's unless you want to putit on older vendor versions of 2.4 Some accessor and wrapper functions like: t1_pci_read_config_4 adapter_name t1_malloc are just annoying noise. Why have useless dead code like: /* Interrupt handler */ +static int pm3393_interrupt_handler(struct cmac *cmac) +{ + u32 master_intr_status; +/* + 1. Read master interrupt register. + 2. Read BLOCK's interrupt status registers. + 3. Handle BLOCK interrupts. +*/ Jeff said: step 1: kill all the OS wrappers. And do you really need hooks for multiple MACs, when only one MAC is really supported? Typically these hooks are at a higher level anyway -- struct net_device. From: Christoph Lameter <christoph@lameter Driver modified as suggested by Pekka Enberg, Stephen Hemminger and Andrian Bunk. Reduces the size of the driver to ~260k. - clean up tabs - removed my3126.c - removed 85% of suni1x10gexp_regs.h - removed 80% of regs.h - removed various calls, renamed variables/functions. - removed system specific and other wrappers (usleep, msleep) - removed dead code - dropped redundant casts in osdep.h - dropped redundant check of kfree - dropped weird code (MODVERSIONS stuff) - reduced number of #ifdefs - use kcalloc now instead of kmalloc - Add information about known issues with the driver - Add information about authors Signed-off-by: Scott Bardone <sbardone@chelsio.com> Signed-off-by: Christoph Lameter <christoph@lameter.com> Signed-off-by: Andrew Morton <akpm@osdl.org> diff -puN /dev/null Documentation/networking/cxgb.txt
123 lines
4.5 KiB
C
123 lines
4.5 KiB
C
/*****************************************************************************
|
|
* *
|
|
* File: cxgb2.h *
|
|
* $Revision: 1.8 $ *
|
|
* $Date: 2005/03/23 07:41:27 $ *
|
|
* Description: *
|
|
* part of the Chelsio 10Gb Ethernet Driver. *
|
|
* *
|
|
* This program is free software; you can redistribute it and/or modify *
|
|
* it under the terms of the GNU General Public License, version 2, as *
|
|
* published by the Free Software Foundation. *
|
|
* *
|
|
* You should have received a copy of the GNU General Public License along *
|
|
* with this program; if not, write to the Free Software Foundation, Inc., *
|
|
* 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. *
|
|
* *
|
|
* THIS SOFTWARE IS PROVIDED ``AS IS'' AND WITHOUT ANY EXPRESS OR IMPLIED *
|
|
* WARRANTIES, INCLUDING, WITHOUT LIMITATION, THE IMPLIED WARRANTIES OF *
|
|
* MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE. *
|
|
* *
|
|
* http://www.chelsio.com *
|
|
* *
|
|
* Copyright (c) 2003 - 2005 Chelsio Communications, Inc. *
|
|
* All rights reserved. *
|
|
* *
|
|
* Maintainers: maintainers@chelsio.com *
|
|
* *
|
|
* Authors: Dimitrios Michailidis <dm@chelsio.com> *
|
|
* Tina Yang <tainay@chelsio.com> *
|
|
* Felix Marti <felix@chelsio.com> *
|
|
* Scott Bardone <sbardone@chelsio.com> *
|
|
* Kurt Ottaway <kottaway@chelsio.com> *
|
|
* Frank DiMambro <frank@chelsio.com> *
|
|
* *
|
|
* History: *
|
|
* *
|
|
****************************************************************************/
|
|
|
|
#ifndef __CXGB_LINUX_H__
|
|
#define __CXGB_LINUX_H__
|
|
|
|
#include <linux/netdevice.h>
|
|
#include <linux/skbuff.h>
|
|
#include <linux/version.h>
|
|
#include <asm/semaphore.h>
|
|
#include <asm/bitops.h>
|
|
|
|
/* This belongs in if_ether.h */
|
|
#define ETH_P_CPL5 0xf
|
|
|
|
struct cmac;
|
|
struct cphy;
|
|
|
|
struct port_info {
|
|
struct net_device *dev;
|
|
struct cmac *mac;
|
|
struct cphy *phy;
|
|
struct link_config link_config;
|
|
struct net_device_stats netstats;
|
|
};
|
|
|
|
struct cxgbdev;
|
|
struct t1_sge;
|
|
struct pemc3;
|
|
struct pemc4;
|
|
struct pemc5;
|
|
struct peulp;
|
|
struct petp;
|
|
struct pecspi;
|
|
struct peespi;
|
|
struct work_struct;
|
|
struct vlan_group;
|
|
|
|
enum { /* adapter flags */
|
|
FULL_INIT_DONE = 0x1,
|
|
USING_MSI = 0x2,
|
|
TSO_CAPABLE = 0x4,
|
|
TCP_CSUM_CAPABLE = 0x8,
|
|
UDP_CSUM_CAPABLE = 0x10,
|
|
VLAN_ACCEL_CAPABLE = 0x20,
|
|
RX_CSUM_ENABLED = 0x40,
|
|
};
|
|
|
|
struct adapter {
|
|
u8 *regs;
|
|
struct pci_dev *pdev;
|
|
unsigned long registered_device_map;
|
|
unsigned long open_device_map;
|
|
unsigned int flags;
|
|
|
|
const char *name;
|
|
int msg_enable;
|
|
u32 mmio_len;
|
|
|
|
struct work_struct ext_intr_handler_task;
|
|
struct adapter_params params;
|
|
|
|
struct vlan_group *vlan_grp;
|
|
|
|
/* Terminator modules. */
|
|
struct sge *sge;
|
|
struct pemc3 *mc3;
|
|
struct pemc4 *mc4;
|
|
struct pemc5 *mc5;
|
|
struct petp *tp;
|
|
struct pecspi *cspi;
|
|
struct peespi *espi;
|
|
struct peulp *ulp;
|
|
|
|
struct port_info port[MAX_NPORTS];
|
|
struct work_struct stats_update_task;
|
|
struct timer_list stats_update_timer;
|
|
|
|
struct semaphore mib_mutex;
|
|
spinlock_t tpi_lock;
|
|
spinlock_t work_lock;
|
|
|
|
spinlock_t async_lock ____cacheline_aligned; /* guards async operations */
|
|
u32 slow_intr_mask;
|
|
};
|
|
|
|
#endif
|