From: Ben Greear <greearb@candelatech.com>
To: "'netdev@oss.sgi.com'" <netdev@oss.sgi.com>
Subject: [PATCH 3/3] rx_all e100 patch
Date: Mon, 24 Nov 2003 23:55:26 -0800 [thread overview]
Message-ID: <3FC30AEE.7000005@candelatech.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 96 bytes --]
--
Ben Greear <greearb@candelatech.com>
Candela Technologies Inc http://www.candelatech.com
[-- Attachment #2: rx_all_e100.patch --]
[-- Type: text/plain, Size: 11631 bytes --]
--- linux-2.4.22/drivers/net/e100/e100_main.c 2003-08-25 04:44:42.000000000 -0700
+++ linux-2.4.22.p4s/drivers/net/e100/e100_main.c 2003-11-24 21:50:49.000000000 -0800
@@ -1,4 +1,4 @@
-/*******************************************************************************
+/** -*-linux-c-*- ************************************************************
Copyright(c) 1999 - 2003 Intel Corporation. All rights reserved.
@@ -175,7 +175,7 @@
MODULE_PARM_DESC(X, S);
/* ====================================================================== */
-static u8 e100_D101M_checksum(struct e100_private *, struct sk_buff *);
+static u8 e100_D101M_checksum(struct e100_private *, struct sk_buff *, int crc_there);
static u8 e100_D102_check_checksum(rfd_t *);
static int e100_ioctl(struct net_device *, struct ifreq *, int);
static int e100_change_mtu(struct net_device *, int);
@@ -644,8 +644,9 @@
dev->do_ioctl = &e100_ioctl;
if (bdp->flags & USE_IPCB)
- dev->features = NETIF_F_SG | NETIF_F_HW_CSUM |
- NETIF_F_HW_VLAN_TX | NETIF_F_HW_VLAN_RX;
+ dev->features = NETIF_F_SG | NETIF_F_HW_CSUM |
+ NETIF_F_HW_VLAN_TX | NETIF_F_HW_VLAN_RX |
+ NETIF_F_RX_ALL | NETIF_F_SAVE_CRC;
if ((rc = register_netdev(dev)) != 0) {
goto err_pci;
@@ -1198,11 +1199,17 @@
struct e100_private *bdp = dev->priv;
unsigned char promisc_enbl;
unsigned char mulcast_enbl;
+ unsigned char enable_rx_all;
promisc_enbl = ((dev->flags & IFF_PROMISC) == IFF_PROMISC);
mulcast_enbl = ((dev->flags & IFF_ALLMULTI) ||
(dev->mc_count > MAX_MULTICAST_ADDRS));
+ enable_rx_all = ((dev->priv_flags & IFF_ACCEPT_ALL_FRAMES) == IFF_ACCEPT_ALL_FRAMES);
+ printk("e100_set_rx_multi (%s), promisc: %d mcast: %d rxall: %d\n",
+ dev->name, promisc_enbl, mulcast_enbl, enable_rx_all);
+ /* NOTE: rx_long is unconditionally set to TRUE if the chipset supports it. */
+ e100_config_rx_all(bdp, enable_rx_all);
e100_config_promisc(bdp, promisc_enbl);
e100_config_mulcast_enbl(bdp, mulcast_enbl);
@@ -2016,8 +2023,14 @@
/* do not free & unmap badly received packet.
* move it to the end of skb list for reuse */
if (!(rfd_status & RFD_STATUS_OK)) {
- e100_add_skb_to_end(bdp, rx_struct);
- continue;
+ if (unlikely(dev->priv_flags & IFF_ACCEPT_ALL_FRAMES)) {
+ /* printk("%s: Accepting a bogon, rfd_status: 0x%x\n",
+ dev->name, rfd_status); */
+ }
+ else {
+ e100_add_skb_to_end(bdp, rx_struct);
+ continue;
+ }
}
data_sz = min_t(u16, (le16_to_cpu(rfd->rfd_act_cnt) & 0x3fff),
@@ -2052,13 +2065,26 @@
if (bdp->rev_id >= D102_REV_ID) {
skb->ip_summed = e100_D102_check_checksum(rfd);
} else {
- skb->ip_summed = e100_D101M_checksum(bdp, skb);
+ skb->ip_summed = e100_D101M_checksum(bdp, skb, !!(dev->priv_flags & IFF_ACCEPT_ALL_FRAMES));
}
} else {
skb->ip_summed = CHECKSUM_NONE;
}
+ /* Show the FCS? */
+ if (unlikely(dev->priv_flags & IFF_SAVE_FCS)) {
+ if (bdp->rev_id < D102_REV_ID) {
+ /* Have to over-write the two IP checksum bytes
+ * TODO: Will this break vlan_hwaccel_rx???
+ */
+ skb->tail[-4] = skb->tail[-2];
+ skb->tail[-3] = skb->tail[-1];
+ skb->tail[-2] = skb->tail[0];
+ skb->tail[-1] = skb->tail[1];
+ }
+ }
+
bdp->drv_stats.net_stats.rx_bytes += skb->len;
if(bdp->vlgrp && (rfd_status & CB_STATUS_VLAN)) {
@@ -2934,13 +2972,16 @@
* assign this value to skb->csum.
*/
static unsigned char
-e100_D101M_checksum(struct e100_private *bdp, struct sk_buff *skb)
+e100_D101M_checksum(struct e100_private *bdp, struct sk_buff *skb, int crc_there)
{
unsigned short proto = (skb->protocol);
-
+ int offset = 0;
+ if (unlikely(crc_there)) {
+ offset = -4;
+ }
if (proto == __constant_htons(ETH_P_IP)) {
- skb->csum = get_unaligned((u16 *) (skb->tail));
+ skb->csum = get_unaligned((u16 *) (skb->tail - offset));
return CHECKSUM_HW;
}
return CHECKSUM_NONE;
@@ -3143,6 +3184,27 @@
}
}
+static int e100_ethtool_setrxall(struct net_device *netdev, uint32_t val) {
+ unsigned short old_flags = netdev->priv_flags;
+ if (val) {
+ netdev->priv_flags |= IFF_ACCEPT_ALL_FRAMES;
+ }
+ else {
+ netdev->priv_flags &= ~(IFF_ACCEPT_ALL_FRAMES);
+ }
+
+ /* printk("e100_ethtool_setrxall (%s) val: %d\n",
+ netdev->name, val); */
+ if (old_flags != netdev->priv_flags) {
+ /* Kick the driver to flush the values...
+ * TODO: Needs review of driver folks to make sure locking is sane, etc
+ */
+ /*printk("Kicking e100_set_multi..\n");*/
+ e100_set_multi(netdev);
+ }
+ return 0;
+}
+
static int
e100_do_ethtool_ioctl(struct net_device *dev, struct ifreq *ifr)
{
@@ -3342,7 +3404,43 @@
return 0;
}
#endif
+ case ETHTOOL_SETRXALL: {
+ struct ethtool_value id;
+ if (copy_from_user(&id, ifr->ifr_data, sizeof(id)))
+ return -EFAULT;
+ spin_lock_bh(&dev->xmit_lock);
+ e100_ethtool_setrxall(dev, id.data);
+ spin_unlock_bh(&dev->xmit_lock);
+ return 0;
+ }
+ case ETHTOOL_GETRXALL: {
+ struct ethtool_value edata = { ETHTOOL_GSG };
+ edata.data = !!(dev->priv_flags & IFF_ACCEPT_ALL_FRAMES);
+ /*printk("GETRXALL, data: %d priv_flags: %hx\n",
+ edata.data, netdev->priv_flags);*/
+ if (copy_to_user(ifr->ifr_data, &edata, sizeof(edata)))
+ return -EFAULT;
+ return 0;
+ }
+ case ETHTOOL_SETRXFCS: {
+ struct ethtool_value id;
+ if (copy_from_user(&id, ifr->ifr_data, sizeof(id)))
+ return -EFAULT;
+ spin_lock_bh(&dev->xmit_lock);
+ dev->priv_flags |= IFF_SAVE_FCS;
+ spin_unlock_bh(&dev->xmit_lock);
+ return 0;
+ }
+ case ETHTOOL_GETRXFCS: {
+ struct ethtool_value edata = { ETHTOOL_GSG };
+ edata.data = !!(dev->priv_flags & IFF_SAVE_FCS);
+ /*printk("GETRXFCS, data: %d priv_flags: %hx\n",
+ edata.data, netdev->priv_flags);*/
+ if (copy_to_user(ifr->ifr_data, &edata, sizeof(edata)))
+ return -EFAULT;
+ return 0;
+ }
default:
break;
} //switch
--- linux-2.4.22/drivers/net/e100/e100_config.c 2003-06-13 07:51:34.000000000 -0700
+++ linux-2.4.22.p4s/drivers/net/e100/e100_config.c 2003-11-24 14:56:14.000000000 -0800
@@ -1,4 +1,4 @@
-/*******************************************************************************
+/**** -*-linux-c-*- ***********************************************************
Copyright(c) 1999 - 2003 Intel Corporation. All rights reserved.
@@ -326,42 +326,92 @@
{
spin_lock_bh(&(bdp->config_lock));
- /* if in promiscuous mode, save bad frames */
+ /* Promiscuity */
if (enable) {
+ if (!(bdp->config[15] & CB_CFIG_PROMISCUOUS)) {
+ bdp->config[15] |= CB_CFIG_PROMISCUOUS;
+ E100_CONFIG(bdp, 15);
+ }
+
+ } else { /* not in promiscuous mode */
+
+ if (bdp->config[15] & CB_CFIG_PROMISCUOUS) {
+ bdp->config[15] &= ~CB_CFIG_PROMISCUOUS;
+ E100_CONFIG(bdp, 15);
+ }
+ }
+
+ spin_unlock_bh(&(bdp->config_lock));
+}
+
+
+/**
+ * e100_config_promisc - configure promiscuous mode
+ * @bdp: atapter's private data struct
+ * @enable: should we enable this option or not
+ *
+ * This routine will enable or disable receiving all frames to
+ * memory, including bad ones, short ones, and long ones. It also
+ * causes the Frame Check Sum (FCS) to be transferred to memory.
+ */
+void
+e100_config_rx_all(struct e100_private *bdp, unsigned char enable)
+{
+ spin_lock_bh(&(bdp->config_lock));
+
+ /* Should we save bad frames? */
+ if (enable) {
if (!(bdp->config[6] & CB_CFIG_SAVE_BAD_FRAMES)) {
bdp->config[6] |= CB_CFIG_SAVE_BAD_FRAMES;
E100_CONFIG(bdp, 6);
}
- if (bdp->config[7] & (u8) BIT_0) {
- bdp->config[7] &= (u8) (~BIT_0);
+ /* Don't discard short-receive */
+ if (bdp->config[7] & (u8) CB_CFIG_DISC_SHORT_FRAMES) {
+ bdp->config[7] &= (u8) (~CB_CFIG_DISC_SHORT_FRAMES);
E100_CONFIG(bdp, 7);
}
- if (!(bdp->config[15] & CB_CFIG_PROMISCUOUS)) {
- bdp->config[15] |= CB_CFIG_PROMISCUOUS;
- E100_CONFIG(bdp, 15);
+ /* Save over-runs */
+ if (!(bdp->config[6] & CB_CFIG_SAVE_OVERRUNS)) {
+ bdp->config[6] |= CB_CFIG_SAVE_OVERRUNS;
+ E100_CONFIG(bdp, 6);
}
- } else { /* not in promiscuous mode */
-
+ /* Transfer the etherne CRC to memory too */
+ if (!(bdp->config[18] & CB_CFIG_CRC_IN_MEM)) {
+ bdp->config[18] |= CB_CFIG_CRC_IN_MEM;
+ E100_CONFIG(bdp, 18);
+ }
+
+ }
+ else {
+ /* Don't discard short frames */
if (bdp->config[6] & CB_CFIG_SAVE_BAD_FRAMES) {
bdp->config[6] &= ~CB_CFIG_SAVE_BAD_FRAMES;
E100_CONFIG(bdp, 6);
}
- if (!(bdp->config[7] & (u8) BIT_0)) {
- bdp->config[7] |= (u8) (BIT_0);
+ /* Discard short-receive */
+ if (!(bdp->config[7] & (u8) CB_CFIG_DISC_SHORT_FRAMES)) {
+ bdp->config[7] |= (u8) (CB_CFIG_DISC_SHORT_FRAMES);
E100_CONFIG(bdp, 7);
}
- if (bdp->config[15] & CB_CFIG_PROMISCUOUS) {
- bdp->config[15] &= ~CB_CFIG_PROMISCUOUS;
- E100_CONFIG(bdp, 15);
+ /* Discard over-runs */
+ if (bdp->config[6] & CB_CFIG_SAVE_OVERRUNS) {
+ bdp->config[6] &= !CB_CFIG_SAVE_OVERRUNS;
+ E100_CONFIG(bdp, 6);
}
- }
+
+ /* Don't send CRC (FCS) to memory */
+ if (bdp->config[18] & CB_CFIG_CRC_IN_MEM) {
+ bdp->config[18] &= !CB_CFIG_CRC_IN_MEM;
+ E100_CONFIG(bdp, 18);
+ }
+ }
spin_unlock_bh(&(bdp->config_lock));
}
--- linux-2.4.22/drivers/net/e100/e100_config.h 2003-06-13 07:51:34.000000000 -0700
+++ linux-2.4.22.p4s/drivers/net/e100/e100_config.h 2003-11-24 21:15:03.000000000 -0800
@@ -67,6 +67,7 @@
#define CB_CFIG_CI_INT BIT_3 /* Command Complete Interrupt */
#define CB_CFIG_EXT_TCB_DIS BIT_4 /* Extended TCB */
#define CB_CFIG_EXT_STAT_DIS BIT_5 /* Extended Stats */
+#define CB_CFIG_SAVE_OVERRUNS BIT_6 /* Save over-run frames if != 0 */
#define CB_CFIG_SAVE_BAD_FRAMES BIT_7 /* Save Bad Frames Enabled */
/* byte 7 bit definitions*/
@@ -117,6 +118,8 @@
#define CB_CFIG_STRIPPING BIT_0 /* Padding Disabled */
#define CB_CFIG_PADDING BIT_1 /* Padding Disabled */
#define CB_CFIG_CRC_IN_MEM BIT_2 /* Transfer CRC To Memory */
+/* Only valid for 82558 and 82559. Must be zero for 82557 */
+#define CB_CFIG_LONG_RX_OK BIT_3 /* OK to receive Long frames */
/* byte 19 bit definitions*/
#define CB_CFIG_TX_ADDR_WAKE BIT_0 /* Address Wakeup */
@@ -142,8 +145,7 @@
/* byte 22 bit defines */
#define CB_CFIG_RECEIVE_GAMLA_MODE BIT_0 /* D102 receive mode */
#define CB_CFIG_VLAN_DROP_ENABLE BIT_1 /* vlan stripping */
-
-#define CB_CFIG_LONG_RX_OK BIT_3
+/* LONG-RX OK (needed for VLAN) is in byte 18, bit 3, see above */
#define NO_LOOPBACK 0
#define MAC_LOOPBACK 0x01
@@ -155,7 +157,8 @@
extern unsigned char e100_config(struct e100_private *bdp);
extern void e100_config_fc(struct e100_private *bdp);
extern void e100_config_promisc(struct e100_private *bdp, unsigned char enable);
+extern void e100_config_rx_all(struct e100_private *bdp, unsigned char enable);
extern void e100_config_brdcast_dsbl(struct e100_private *bdp);
extern void e100_config_mulcast_enbl(struct e100_private *bdp,
unsigned char enable);
--- linux-2.4.22/drivers/net/e100/e100.h 2003-08-25 04:44:42.000000000 -0700
+++ linux-2.4.22.p4s/drivers/net/e100/e100.h 2003-11-24 21:15:03.000000000 -0800
@@ -362,6 +362,7 @@
#define CB_EL_BIT BIT_15 /* CB EL Bit */
#define CB_S_BIT BIT_14 /* CB Suspend Bit */
#define CB_I_BIT BIT_13 /* CB Interrupt Bit */
+#define CB_TX_NC_BIT BIT_4 /* If true, do not calculate FCS */
#define CB_TX_SF_BIT BIT_3 /* TX CB Flexible Mode */
#define CB_CMD_MASK BIT_0_3 /* CB 4-bit CMD Mask */
#define CB_CID_DEFAULT (0x1f << 8) /* CB 5-bit CID (max value) */
next reply other threads:[~2003-11-25 7:55 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-11-25 7:55 Ben Greear [this message]
2003-11-25 15:21 ` [PATCH 3/3] rx_all e100 patch Rask Ingemann Lambertsen
2003-11-25 18:03 ` Ben Greear
2003-11-25 23:41 ` Rask Ingemann Lambertsen
2003-11-25 16:56 ` Rask Ingemann Lambertsen
-- strict thread matches above, loose matches on Subject: below --
2003-11-25 17:39 Feldman, Scott
2003-11-25 18:05 ` Ben Greear
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3FC30AEE.7000005@candelatech.com \
--to=greearb@candelatech.com \
--cc=netdev@oss.sgi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.