- Add type attribute to ctc and lcs driver.
- Add user attribute to netiucv driver.
- Remove initialization of device.name from ctc, lcs and qeth.
- Some qeth bug fixes:
+ Call qeth_free_card on removal.
+ Remove async hsi.
+ Remove contig memusage.
+ Add check for -EFAULT to copy_from_user/copy_to_user.
+ Some inlining.
+ vlan header fixes.
+ Replace atomic_return_sub with atomic_add_return.
diffstat:
drivers/s390/net/ctcmain.c | 40 ++-
drivers/s390/net/cu3088.c | 3
drivers/s390/net/iucv.c | 9
drivers/s390/net/lcs.c | 25 +-
drivers/s390/net/netiucv.c | 20 +
drivers/s390/net/qeth.c | 537 +++++++++++++++++----------------------------
drivers/s390/net/qeth.h | 46 ---
7 files changed, 279 insertions(+), 401 deletions(-)
diff -urN linux-2.6/drivers/s390/net/ctcmain.c linux-2.6-s390/drivers/s390/net/ctcmain.c
--- linux-2.6/drivers/s390/net/ctcmain.c Mon Sep 8 21:50:02 2003
+++ linux-2.6-s390/drivers/s390/net/ctcmain.c Thu Sep 11 19:21:27 2003
@@ -1,5 +1,5 @@
/*
- * $Id: ctcmain.c,v 1.43 2003/05/27 11:34:23 mschwide Exp $
+ * $Id: ctcmain.c,v 1.46 2003/09/01 09:03:23 cohuck Exp $
*
* CTC / ESCON network driver
*
@@ -36,7 +36,7 @@
* along with this program; if not, write to the Free Software
* Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
*
- * RELEASE-TAG: CTC/ESCON network driver $Revision: 1.43 $
+ * RELEASE-TAG: CTC/ESCON network driver $Revision: 1.46 $
*
*/
@@ -102,7 +102,7 @@
#define READ 0
#define WRITE 1
-#define CTC_ID_SIZE DEVICE_ID_SIZE+3
+#define CTC_ID_SIZE BUS_ID_SIZE+3
struct ctc_profile {
@@ -272,7 +272,7 @@
print_banner(void)
{
static int printed = 0;
- char vbuf[] = "$Revision: 1.43 $";
+ char vbuf[] = "$Revision: 1.46 $";
char *version = vbuf;
if (printed)
@@ -1791,7 +1791,7 @@
ch->ccw[7].cda = 0;
ch->cdev = cdev;
- snprintf(ch->id, DEVICE_ID_SIZE, "ch-%s", cdev->dev.bus_id);
+ snprintf(ch->id, CTC_ID_SIZE, "ch-%s", cdev->dev.bus_id);
ch->type = type;
ch->fsm = init_fsm(ch->id, ch_state_names,
ch_event_names, NR_CH_STATES, NR_CH_EVENTS,
@@ -2786,16 +2786,38 @@
static DEVICE_ATTR(protocol, 0644, ctc_proto_show, ctc_proto_store);
+static ssize_t
+ctc_type_show(struct device *dev, char *buf)
+{
+ struct ccwgroup_device *cgdev;
+
+ cgdev = to_ccwgroupdev(dev);
+ if (!cgdev)
+ return -ENODEV;
+
+ return sprintf(buf, "%s\n", cu3088_type[cgdev->cdev[0]->id.driver_info]);
+}
+
+static DEVICE_ATTR(type, 0444, ctc_type_show, NULL);
+
static int
ctc_add_files(struct device *dev)
{
- return device_create_file(dev, &dev_attr_protocol);
+ int rc;
+ rc = device_create_file(dev, &dev_attr_protocol);
+ if (rc)
+ return rc;
+ rc = device_create_file(dev, &dev_attr_type);
+ if (rc)
+ device_remove_file(dev, &dev_attr_protocol);
+ return rc;
}
static void
ctc_remove_files(struct device *dev)
{
+ device_remove_file(dev, &dev_attr_type);
device_remove_file(dev, &dev_attr_protocol);
}
@@ -2838,8 +2860,6 @@
cgdev->dev.driver_data = priv;
cgdev->cdev[0]->dev.driver_data = priv;
cgdev->cdev[1]->dev.driver_data = priv;
- snprintf(cgdev->dev.name, DEVICE_NAME_SIZE, "%s",
- cu3088_type[cgdev->cdev[0]->id.driver_info]);
return 0;
}
@@ -2868,8 +2888,8 @@
type = get_channel_type(&cgdev->cdev[0]->id);
- snprintf(read_id, DEVICE_ID_SIZE, "ch-%s", cgdev->cdev[0]->dev.bus_id);
- snprintf(write_id, DEVICE_ID_SIZE, "ch-%s", cgdev->cdev[1]->dev.bus_id);
+ snprintf(read_id, CTC_ID_SIZE, "ch-%s", cgdev->cdev[0]->dev.bus_id);
+ snprintf(write_id, CTC_ID_SIZE, "ch-%s", cgdev->cdev[1]->dev.bus_id);
if (add_channel(cgdev->cdev[0], type))
return -ENOMEM;
diff -urN linux-2.6/drivers/s390/net/cu3088.c linux-2.6-s390/drivers/s390/net/cu3088.c
--- linux-2.6/drivers/s390/net/cu3088.c Thu Sep 11 19:21:26 2003
+++ linux-2.6-s390/drivers/s390/net/cu3088.c Thu Sep 11 19:21:27 2003
@@ -1,5 +1,5 @@
/*
- * $Id: cu3088.c,v 1.26 2003/01/17 13:46:13 cohuck Exp $
+ * $Id: cu3088.c,v 1.30 2003/08/28 11:14:11 cohuck Exp $
*
* CTC / LCS ccw_device driver
*
@@ -56,7 +56,6 @@
static struct ccw_driver cu3088_driver;
struct device cu3088_root_dev = {
- .name = "CU3088 Devices",
.bus_id = "cu3088",
};
diff -urN linux-2.6/drivers/s390/net/iucv.c linux-2.6-s390/drivers/s390/net/iucv.c
--- linux-2.6/drivers/s390/net/iucv.c Mon Sep 8 21:49:52 2003
+++ linux-2.6-s390/drivers/s390/net/iucv.c Thu Sep 11 19:21:27 2003
@@ -1,5 +1,5 @@
/*
- * $Id: iucv.c,v 1.11 2003/04/15 16:45:37 aberg Exp $
+ * $Id: iucv.c,v 1.12 2003/07/31 15:11:13 cohuck Exp $
*
* IUCV network driver
*
@@ -29,11 +29,12 @@
* along with this program; if not, write to the Free Software
* Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
*
- * RELEASE-TAG: IUCV lowlevel driver $Revision: 1.11 $
+ * RELEASE-TAG: IUCV lowlevel driver $Revision: 1.12 $
*
*/
#include <linux/module.h>
+#include <linux/moduleparam.h>
#include <linux/config.h>
#include <linux/spinlock.h>
@@ -283,7 +284,7 @@
#ifdef DEBUG
static int debuglevel = 0;
-MODULE_PARM(debuglevel, "i");
+module_param(debuglevel, int, 0);
MODULE_PARM_DESC(debuglevel,
"Specifies the debug level (0=off ... 3=all)");
@@ -332,7 +333,7 @@
static void
iucv_banner(void)
{
- char vbuf[] = "$Revision: 1.11 $";
+ char vbuf[] = "$Revision: 1.12 $";
char *version = vbuf;
if ((version = strchr(version, ':'))) {
diff -urN linux-2.6/drivers/s390/net/lcs.c linux-2.6-s390/drivers/s390/net/lcs.c
--- linux-2.6/drivers/s390/net/lcs.c Mon Sep 8 21:49:53 2003
+++ linux-2.6-s390/drivers/s390/net/lcs.c Thu Sep 11 19:21:27 2003
@@ -11,7 +11,7 @@
* Frank Pavlic ([email protected]) and
* Martin Schwidefsky <[email protected]>
*
- * $Revision: 1.53 $ $Date: 2003/06/17 11:36:45 $
+ * $Revision: 1.55 $ $Date: 2003/08/28 11:14:11 $
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
@@ -58,7 +58,7 @@
/**
* initialization string for output
*/
-#define VERSION_LCS_C "$Revision: 1.53 $"
+#define VERSION_LCS_C "$Revision: 1.55 $"
static char version[] __initdata = "LCS driver ("VERSION_LCS_C "/" VERSION_LCS_H ")";
@@ -1675,6 +1675,20 @@
static DEVICE_ATTR(portno, 0644, lcs_portno_show, lcs_portno_store);
+static ssize_t
+lcs_type_show(struct device *dev, char *buf)
+{
+ struct ccwgroup_device *cgdev;
+
+ cgdev = to_ccwgroupdev(dev);
+ if (!cgdev)
+ return -ENODEV;
+
+ return sprintf(buf, "%s\n", cu3088_type[cgdev->cdev[0]->id.driver_info]);
+}
+
+static DEVICE_ATTR(type, 0444, lcs_type_show, NULL);
+
/**
* lcs_probe_device is called on establishing a new ccwgroup_device.
*/
@@ -1695,15 +1709,16 @@
return -ENOMEM;
}
ret = device_create_file(&ccwgdev->dev, &dev_attr_portno);
+ if (!ret)
+ ret = device_create_file(&ccwgdev->dev, &dev_attr_type);
if (ret) {
PRINT_ERR("Creating attributes failed");
+ device_remove_file(&ccwgdev->dev, &dev_attr_portno);
lcs_free_card(card);
put_device(&ccwgdev->dev);
return ret;
}
ccwgdev->dev.driver_data = card;
- snprintf(ccwgdev->dev.name, DEVICE_NAME_SIZE, "%s",
- cu3088_type[ccwgdev->cdev[0]->id.driver_info]);
ccwgdev->cdev[0]->dev.driver_data = card;
ccwgdev->cdev[0]->handler = lcs_irq;
ccwgdev->cdev[1]->dev.driver_data = card;
@@ -1826,6 +1841,8 @@
card = (struct lcs_card *)ccwgdev->dev.driver_data;
if (!card)
return 0;
+ device_remove_file(&ccwgdev->dev, &dev_attr_type);
+ device_remove_file(&ccwgdev->dev, &dev_attr_portno);
lcs_cleanup_card(card);
lcs_free_card(card);
put_device(&ccwgdev->dev);
diff -urN linux-2.6/drivers/s390/net/netiucv.c linux-2.6-s390/drivers/s390/net/netiucv.c
--- linux-2.6/drivers/s390/net/netiucv.c Mon Sep 8 21:50:07 2003
+++ linux-2.6-s390/drivers/s390/net/netiucv.c Thu Sep 11 19:21:27 2003
@@ -1,5 +1,5 @@
/*
- * $Id: netiucv.c,v 1.20 2003/05/27 11:34:24 mschwide Exp $
+ * $Id: netiucv.c,v 1.22 2003/08/28 16:22:40 mschwide Exp $
*
* IUCV network driver
*
@@ -30,7 +30,7 @@
* along with this program; if not, write to the Free Software
* Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
*
- * RELEASE-TAG: IUCV network driver $Revision: 1.20 $
+ * RELEASE-TAG: IUCV network driver $Revision: 1.22 $
*
*/
@@ -128,7 +128,6 @@
};
static struct device iucv_root = {
- .name = "IUCV",
.bus_id = "iucv",
};
@@ -1255,6 +1254,16 @@
#define CTRL_BUFSIZE 40
static ssize_t
+user_show (struct device *dev, char *buf)
+{
+ struct netiucv_priv *priv = dev->driver_data;
+
+ return sprintf(buf, "%s\n", netiucv_printname(priv->conn->userid));
+}
+
+static DEVICE_ATTR(user, 0444, user_show, NULL);
+
+static ssize_t
buffer_show (struct device *dev, char *buf)
{
struct netiucv_priv *priv = dev->driver_data;
@@ -1441,6 +1450,7 @@
int ret = 0;
if ((ret = device_create_file(dev, &dev_attr_buffer)) ||
+ (ret = device_create_file(dev, &dev_attr_user)) ||
(ret = device_create_file(dev, &dev_attr_device_fsm_state)) ||
(ret = device_create_file(dev, &dev_attr_connection_fsm_state)) ||
(ret = device_create_file(dev, &dev_attr_max_tx_buffer_used)) ||
@@ -1456,6 +1466,7 @@
device_remove_file(dev, &dev_attr_max_tx_buffer_used);
device_remove_file(dev, &dev_attr_connection_fsm_state);
device_remove_file(dev, &dev_attr_device_fsm_state);
+ device_remove_file(dev, &dev_attr_user);
device_remove_file(dev, &dev_attr_buffer);
}
return ret;
@@ -1469,7 +1480,6 @@
int ret;
char *str = "netiucv";
- snprintf(dev->name, DEVICE_NAME_SIZE, "%s", priv->conn->userid);
snprintf(dev->bus_id, BUS_ID_SIZE, "%s%x", str, ifno);
dev->bus = &iucv_bus;
dev->parent = &iucv_root;
@@ -1717,7 +1727,7 @@
static void
netiucv_banner(void)
{
- char vbuf[] = "$Revision: 1.20 $";
+ char vbuf[] = "$Revision: 1.22 $";
char *version = vbuf;
if ((version = strchr(version, ':'))) {
diff -urN linux-2.6/drivers/s390/net/qeth.c linux-2.6-s390/drivers/s390/net/qeth.c
--- linux-2.6/drivers/s390/net/qeth.c Mon Sep 8 21:50:58 2003
+++ linux-2.6-s390/drivers/s390/net/qeth.c Thu Sep 11 19:21:27 2003
@@ -1,6 +1,6 @@
/*
*
- * linux/drivers/s390/net/qeth.c ($Revision: 1.126 $)
+ * linux/drivers/s390/net/qeth.c ($Revision: 1.147 $)
*
* Linux on zSeries OSA Express and HiperSockets support
*
@@ -106,6 +106,7 @@
#include <linux/config.h>
#include <linux/module.h>
+#include <linux/moduleparam.h>
#include <linux/string.h>
#include <linux/errno.h>
@@ -160,12 +161,12 @@
/****************** MODULE PARAMETER VARIABLES ********************/
static int qeth_sparebufs = 0;
-MODULE_PARM(qeth_sparebufs, "i");
+module_param(qeth_sparebufs, int, 0);
MODULE_PARM_DESC(qeth_sparebufs, "the number of pre-allocated spare buffers "
"reserved for low memory situations");
/****************** MODULE STUFF **********************************/
-#define VERSION_QETH_C "$Revision: 1.126 $"
+#define VERSION_QETH_C "$Revision: 1.147 $"
static const char *version = "qeth S/390 OSA-Express driver ("
VERSION_QETH_C "/" VERSION_QETH_H "/" VERSION_QETH_MPC_H
QETH_VERSION_IPV6 QETH_VERSION_VLAN ")";
@@ -218,9 +219,9 @@
/* thought I could get along without forward declarations...
* just lazyness here */
static int qeth_reinit_thread(void *);
-static void qeth_schedule_recovery(struct qeth_card *card);
+static inline void qeth_schedule_recovery(struct qeth_card *card);
-inline static int
+static inline int
QETH_IP_VERSION(struct sk_buff *skb)
{
switch (skb->protocol) {
@@ -648,10 +649,6 @@
case 1:
return 0;
case 4:
- if ((card->can_do_async_iqd) &&
- (card->options.async_iqd == ASYNC_IQD)) {
- return card->no_queues - 1;
- }
if (card->is_multicast_different) {
if (multicast) {
return card->is_multicast_different &
@@ -706,11 +703,7 @@
QETH_DBF_TEXT5(0, trace, card->rdev->dev.bus_id);
atomic_set(&card->data_has_arrived, 1);
- spin_lock(&card->wait_q_lock);
- if (atomic_read(&card->wait_q_active)) {
- wake_up(&card->wait_q);
- }
- spin_unlock(&card->wait_q_lock);
+ wake_up(&card->wait_q);
}
static int
@@ -1273,9 +1266,6 @@
goto nomem;
}
- if (card->easy_copy_cap)
- memcpy(skb_put(skb, length), data_ptr, length);
-
QETH_DBF_HEX6(0, trace, &data_ptr, sizeof (void *));
QETH_DBF_HEX6(0, trace, &skb, sizeof (void *));
@@ -1302,8 +1292,7 @@
dev_kfree_skb_irq(skb);
return NULL;
}
- if (!card->easy_copy_cap)
- memcpy(skb_put(skb, step), data_ptr, step);
+ memcpy(skb_put(skb, step), data_ptr, step);
len_togo -= step;
if (len_togo) {
pos_in_el = 0;
@@ -1603,11 +1592,14 @@
#ifdef QETH_VLAN
struct qeth_card *card;
- /* before we're going to overwrite this location with next hop ip */
+ /*
+ * before we're going to overwrite this location with next hop ip.
+ * v6 uses passthrough, v4 sets the tag in the QDIO header.
+ */
card = (struct qeth_card *) skb->dev->priv;
- if ((card->vlangrp != NULL) &&
- vlan_tx_tag_present(skb) && (version == 4)) {
- hdr->ext_flags = QETH_EXT_HEADER_VLAN_FRAME;
+ if ((card->vlangrp != NULL) && vlan_tx_tag_present(skb)) {
+ hdr->ext_flags = (version == 4) ? QETH_EXT_HEADER_VLAN_FRAME :
+ QETH_EXT_HEADER_INCLUDE_VLAN_TAG;
hdr->vlan_id = vlan_tx_tag_get(skb);
}
#endif
@@ -1684,7 +1676,9 @@
skb->dev->broadcast, 6)) { /* broadcast? */
hdr->flags = QETH_CAST_BROADCAST | QETH_HEADER_PASSTHRU;
} else {
- hdr->flags = QETH_CAST_UNICAST | QETH_HEADER_PASSTHRU;
+ hdr->flags = (multicast == RTN_MULTICAST) ?
+ QETH_CAST_MULTICAST | QETH_HEADER_PASSTHRU :
+ QETH_CAST_UNICAST | QETH_HEADER_PASSTHRU;
}
}
sprintf(dbf_text, "filhdr%2x", version);
@@ -2356,14 +2350,6 @@
}
}
-static __inline__ int
-atomic_return_sub(int i, atomic_t * v)
-{
- int old_val, new_val;
- __CS_LOOP(old_val, new_val, v, i, "sr");
- return old_val;
-}
-
static inline void
__qeth_dump_packet_info(struct qeth_card *card, int version, int multicast,
int queue)
@@ -2580,50 +2566,21 @@
static int
qeth_sleepon(struct qeth_card *card, int timeout)
{
- unsigned long flags;
- unsigned long start;
- int retval;
char dbf_text[15];
- DECLARE_WAITQUEUE(current_wait_q, current);
-
QETH_DBF_TEXT5(0, trace, "slpn");
QETH_DBF_TEXT5(0, trace, card->rdev->dev.bus_id);
sprintf(dbf_text, "%08x", timeout);
QETH_DBF_TEXT5(0, trace, dbf_text);
- add_wait_queue(&card->wait_q, ¤t_wait_q);
- atomic_set(&card->wait_q_active, 1);
- start = qeth_get_millis();
- for (;;) {
- set_task_state(current, TASK_INTERRUPTIBLE);
- if (atomic_read(&card->data_has_arrived)) {
- atomic_set(&card->data_has_arrived, 0);
- retval = 0;
- goto out;
- }
- if (qeth_get_millis() - start > timeout) {
- retval = -ETIME;
- goto out;
- }
- schedule_timeout(((start + timeout -
- qeth_get_millis()) >> 10) * HZ);
- }
-out:
- spin_lock_irqsave(&card->wait_q_lock, flags);
- atomic_set(&card->wait_q_active, 0);
- spin_unlock_irqrestore(&card->wait_q_lock, flags);
-
- /* we've got to check once again to close the window */
+ wait_event_interruptible_timeout(card->wait_q,
+ atomic_read(&card->data_has_arrived),
+ timeout * HZ);
if (atomic_read(&card->data_has_arrived)) {
atomic_set(&card->data_has_arrived, 0);
- retval = 0;
+ return 0;
}
-
- set_task_state(current, TASK_RUNNING);
- remove_wait_queue(&card->wait_q, ¤t_wait_q);
-
- return retval;
+ return -ETIME;
}
static void
@@ -2634,60 +2591,28 @@
QETH_DBF_TEXT5(0, trace, card->rdev->dev.bus_id);
atomic_set(&card->ioctl_data_has_arrived, 1);
- spin_lock(&card->ioctl_wait_q_lock);
- if (atomic_read(&card->ioctl_wait_q_active)) {
- wake_up(&card->ioctl_wait_q);
- }
- spin_unlock(&card->ioctl_wait_q_lock);
+ wake_up(&card->ioctl_wait_q);
}
static int
qeth_sleepon_ioctl(struct qeth_card *card, int timeout)
{
- unsigned long flags;
- unsigned long start;
- int retval;
char dbf_text[15];
- DECLARE_WAITQUEUE(current_wait_q, current);
-
QETH_DBF_TEXT5(0, trace, "ioctlslpn");
QETH_DBF_TEXT5(0, trace, card->rdev->dev.bus_id);
sprintf(dbf_text, "%08x", timeout);
QETH_DBF_TEXT5(0, trace, dbf_text);
- add_wait_queue(&card->ioctl_wait_q, ¤t_wait_q);
- atomic_set(&card->ioctl_wait_q_active, 1);
- start = qeth_get_millis();
- for (;;) {
- set_task_state(current, TASK_INTERRUPTIBLE);
- if (atomic_read(&card->ioctl_data_has_arrived)) {
- atomic_set(&card->ioctl_data_has_arrived, 0);
- retval = 0;
- goto out;
- }
- if (qeth_get_millis() - start > timeout) {
- retval = -ETIME;
- goto out;
- }
- schedule_timeout(((start + timeout -
- qeth_get_millis()) >> 10) * HZ);
- }
-out:
- spin_lock_irqsave(&card->ioctl_wait_q_lock, flags);
- atomic_set(&card->ioctl_wait_q_active, 0);
- spin_unlock_irqrestore(&card->ioctl_wait_q_lock, flags);
-
- /* we've got to check once again to close the window */
+ wait_event_interruptible_timeout(card->ioctl_wait_q,
+ atomic_read(&card->
+ ioctl_data_has_arrived),
+ timeout * HZ);
if (atomic_read(&card->ioctl_data_has_arrived)) {
atomic_set(&card->ioctl_data_has_arrived, 0);
- retval = 0;
+ return 0;
}
-
- set_task_state(current, TASK_RUNNING);
- remove_wait_queue(&card->ioctl_wait_q, ¤t_wait_q);
-
- return retval;
+ return -ETIME;
}
/*SNMP IOCTL on Procfile */
@@ -3227,8 +3152,9 @@
result = IPA_REPLY_SUCCESS;
memcpy(((char *) (card->ioctl_data_buffer)) + sizeof (__u16),
&(card->number_of_entries), sizeof (int));
- copy_to_user(req->ifr_ifru.ifru_data,
- card->ioctl_data_buffer, data_size);
+ if (copy_to_user(req->ifr_ifru.ifru_data,
+ card->ioctl_data_buffer, data_size))
+ result = -EFAULT;
}
card->ioctl_buffer_pointer = NULL;
vfree(card->ioctl_data_buffer);
@@ -3296,13 +3222,17 @@
goto snmp_out;
}
if (result == ARP_RETURNCODE_ERROR) {
- copy_to_user(req->ifr_ifru.ifru_data + SNMP_REQUEST_DATA_OFFSET,
- card->ioctl_data_buffer, card->ioctl_buffersize);
result = IPA_REPLY_FAILED;
+ if (copy_to_user(req->ifr_ifru.ifru_data +
+ SNMP_REQUEST_DATA_OFFSET, card->ioctl_data_buffer,
+ card->ioctl_buffersize))
+ result = -EFAULT;
} else {
- copy_to_user(req->ifr_ifru.ifru_data + SNMP_REQUEST_DATA_OFFSET,
- card->ioctl_data_buffer, card->ioctl_buffersize);
result = IPA_REPLY_SUCCESS;
+ if (copy_to_user(req->ifr_ifru.ifru_data +
+ SNMP_REQUEST_DATA_OFFSET, card->ioctl_data_buffer,
+ card->ioctl_buffersize))
+ result = -EFAULT;
}
snmp_out:
card->number_of_entries = 0;
@@ -4387,7 +4317,8 @@
#define QETH_STANDARD_RETVALS \
ret_val=-EIO; \
- if (result==IPA_REPLY_SUCCESS) ret_val=0; \
+ if (result == -EFAULT) ret_val = -EFAULT; \
+ if (result==IPA_REPLY_SUCCESS) ret_val=0; \
if (result==IPA_REPLY_FAILED) ret_val=-EIO; \
if (result==IPA_REPLY_OPNOTSUPP) ret_val=-EOPNOTSUPP
@@ -4413,7 +4344,8 @@
if ((cmd < SIOCDEVPRIVATE) || (cmd > SIOCDEVPRIVATE + 5))
return -EOPNOTSUPP;
- copy_from_user(buff, rq->ifr_ifru.ifru_data, sizeof (buff));
+ if (copy_from_user(buff, rq->ifr_ifru.ifru_data, sizeof (buff)))
+ return -EFAULT;
data = buff;
if ((!atomic_read(&card->is_registered)) ||
@@ -5957,7 +5889,7 @@
}
}
-static void
+static inline void
qeth_schedule_recovery(struct qeth_card *card)
{
if (card) {
@@ -6172,8 +6104,9 @@
}
}
- buffers_used = atomic_return_sub(count,
- &card->outbound_used_buffers[queue]);
+ buffers_used = atomic_add_return(-count,
+ &card->outbound_used_buffers[queue])
+ + count;
switch (card->send_state[queue]) {
case SEND_STATE_PACK:
@@ -6205,7 +6138,7 @@
PRINT_WARN("timeout on device %s\n", cdev->dev.bus_id);
break;
default:
- PRINT_WARN("unknown error %d on device %s\n", PTR_ERR(irb),
+ PRINT_WARN("unknown error %ld on device %s\n", PTR_ERR(irb),
cdev->dev.bus_id);
}
return PTR_ERR(irb);
@@ -6504,7 +6437,7 @@
QETH_DBF_HEX0(0, sense, irb, QETH_DBF_SENSE_LEN);
}
- if ((rqparam == READ_CONF_DATA_STATE) || (rqparam == NOP_STATE)) {
+ if (rqparam == NOP_STATE) {
qeth_wakeup(card);
return;
}
@@ -6645,18 +6578,17 @@
}
static void
-qeth_free_card(struct qeth_card *card)
+qeth_free_card_stuff(struct qeth_card *card)
{
int i, j;
- int element_count;
struct qeth_vipa_entry *e, *e2;
if (!card)
return;
- QETH_DBF_TEXT3(0, trace, "free");
+ QETH_DBF_TEXT3(0, trace, "freest");
QETH_DBF_TEXT3(0, trace, card->rdev->dev.bus_id);
- QETH_DBF_TEXT1(0, setup, "free");
+ QETH_DBF_TEXT1(0, setup, "freest");
QETH_DBF_TEXT1(0, setup, card->rdev->dev.bus_id);
write_lock(&card->vipa_list_lock);
@@ -6668,10 +6600,8 @@
}
write_unlock(&card->vipa_list_lock);
- element_count = (card->options.memusage == MEMUSAGE_DISCONTIG) ?
- BUFFER_MAX_ELEMENTS : 1;
for (i = 0; i < card->options.inbound_buffer_count; i++) {
- for (j = 0; j < element_count; j++) {
+ for (j = 0; j < BUFFER_MAX_ELEMENTS; j++) {
if (card->inbound_buffer_pool_entry[i][j]) {
kfree(card->inbound_buffer_pool_entry[i][j]);
card->inbound_buffer_pool_entry[i][j] = NULL;
@@ -6687,7 +6617,22 @@
if (card->dma_stuff)
kfree(card->dma_stuff);
if (card->dev)
- kfree(card->dev);
+ free_netdev(card->dev);
+
+}
+
+static void
+qeth_free_card(struct qeth_card *card)
+{
+
+ if (!card)
+ return;
+
+ QETH_DBF_TEXT3(0, trace, "free");
+ QETH_DBF_TEXT3(0, trace, card->rdev->dev.bus_id);
+ QETH_DBF_TEXT1(0, setup, "free");
+ QETH_DBF_TEXT1(0, setup, card->rdev->dev.bus_id);
+
vfree(card); /* we checked against NULL already */
}
@@ -6903,6 +6848,8 @@
}
/* returns last four digits of bus_id */
+/* FIXME: device driver shouldn't be aware of bus_id format - but don't know
+ what else to use... (CH) */
static inline __u16
__raw_devno_from_bus_id(char *id)
{
@@ -7076,7 +7023,7 @@
memcpy(QETH_IDX_ACT_FUNC_LEVEL(card->dma_stuff->sendbuf),
&card->func_level, 2);
- temp = _ccw_device_get_device_number(card->ddev);
+ temp = __raw_devno_from_bus_id(card->ddev->dev.bus_id);
memcpy(QETH_IDX_ACT_QDIO_DEV_CUA(card->dma_stuff->sendbuf), &temp, 2);
temp = (card->cula << 8) + card->unit_addr2;
memcpy(QETH_IDX_ACT_QDIO_DEV_REALADDR(card->dma_stuff->sendbuf),
@@ -7345,7 +7292,7 @@
memcpy(QETH_ULP_SETUP_FILTER_TOKEN(card->send_buf),
&card->token.ulp_filter_r, QETH_MPC_TOKEN_LENGTH);
- temp = _ccw_device_get_device_number(card->ddev);
+ temp = __raw_devno_from_bus_id(card->ddev->dev.bus_id);
memcpy(QETH_ULP_SETUP_CUA(card->send_buf), &temp, 2);
temp = (card->cula << 8) + card->unit_addr2;
memcpy(QETH_ULP_SETUP_REAL_DEVADDR(card->send_buf), &temp, 2);
@@ -8224,11 +8171,6 @@
card->dev->init = qeth_init_dev;
- if (card->options.memusage == MEMUSAGE_CONTIG) {
- card->easy_copy_cap =
- qeth_determine_easy_copy_cap(card->type);
- } else
- card->easy_copy_cap = 0;
card->ipa_timeout = qeth_get_ipa_timeout(card->type);
}
@@ -8492,30 +8434,21 @@
card->options.default_queue = QETH_DEFAULT_QUEUE;
card->options.inbound_buffer_count = DEFAULT_BUFFER_COUNT;
card->options.polltime = QETH_MAX_INPUT_THRESHOLD;
- card->options.memusage = MEMUSAGE_DISCONTIG;
card->options.macaddr_mode = MACADDR_NONCANONICAL;
card->options.broadcast_mode = BROADCAST_ALLRINGS;
card->options.fake_broadcast = DONT_FAKE_BROADCAST;
card->options.ena_ipat = ENABLE_TAKEOVER;
card->options.add_hhlen = DEFAULT_ADD_HHLEN;
card->options.fake_ll = DONT_FAKE_LL;
- card->options.async_iqd = SYNC_IQD;
}
-static struct qeth_card *
-qeth_alloc_card(void)
+static int
+qeth_alloc_card_stuff(struct qeth_card *card)
{
- struct qeth_card *card;
-
- QETH_DBF_TEXT3(0, trace, "alloccrd");
- card = (struct qeth_card *) vmalloc(sizeof (struct qeth_card));
if (!card)
- goto exit_card;
- memset(card, 0, sizeof (struct qeth_card));
- init_waitqueue_head(&card->wait_q);
- init_waitqueue_head(&card->ioctl_wait_q);
+ return -EINVAL;
- qeth_fill_qeth_card_options(card);
+ QETH_DBF_TEXT3(0, trace, "alccrdst");
card->dma_stuff =
(struct qeth_dma_stuff *) kmalloc(sizeof (struct qeth_dma_stuff),
@@ -8549,7 +8482,44 @@
goto exit_stats;
memset(card->stats, 0, sizeof (struct net_device_stats));
- spin_lock_init(&card->wait_q_lock);
+ /* setup net_device stuff */
+ card->dev->priv = card;
+
+ strncpy(card->dev->name, card->dev_name, IFNAMSIZ);
+
+ /* setup net_device_stats stuff */
+ /* =nothing yet */
+
+ return 0;
+
+ /* these are quick exits in case of failures of the kmallocs */
+exit_stats:
+ free_netdev(card->dev);
+exit_dev:
+ kfree(card->dma_stuff->sendbuf);
+exit_dma2:
+ kfree(card->dma_stuff->recbuf);
+exit_dma1:
+ kfree(card->dma_stuff);
+exit_dma:
+ return -ENOMEM;
+}
+
+static struct qeth_card *
+qeth_alloc_card(void)
+{
+ struct qeth_card *card;
+
+ QETH_DBF_TEXT3(0, trace, "alloccrd");
+ card = (struct qeth_card *) vmalloc(sizeof (struct qeth_card));
+ if (!card)
+ return NULL;
+ memset(card, 0, sizeof (struct qeth_card));
+ init_waitqueue_head(&card->wait_q);
+ init_waitqueue_head(&card->ioctl_wait_q);
+
+ qeth_fill_qeth_card_options(card);
+
spin_lock_init(&card->softsetup_lock);
spin_lock_init(&card->hardsetup_lock);
spin_lock_init(&card->ioctl_lock);
@@ -8576,30 +8546,9 @@
card->csum_enable_mask = IPA_CHECKSUM_DEFAULT_ENABLE_MASK;
- /* setup net_device stuff */
- card->dev->priv = card;
-
- strncpy(card->dev->name, card->dev_name, IFNAMSIZ);
-
- /* setup net_device_stats stuff */
- /* =nothing yet */
-
/* and return to the sender */
return card;
- /* these are quick exits in case of failures of the kmallocs */
-exit_stats:
- kfree(card->dev);
-exit_dev:
- kfree(card->dma_stuff->sendbuf);
-exit_dma2:
- kfree(card->dma_stuff->recbuf);
-exit_dma1:
- kfree(card->dma_stuff);
-exit_dma:
- kfree(card);
-exit_card:
- return NULL;
}
static int
@@ -8634,66 +8583,41 @@
qeth_init_ringbuffers2(struct qeth_card *card)
{
int i, j;
- int failed = 0;
- int discont_mem, element_count;
- long alloc_size;
QETH_DBF_TEXT3(0, trace, "irb2");
QETH_DBF_TEXT3(0, trace, card->rdev->dev.bus_id);
- discont_mem = (card->options.memusage == MEMUSAGE_DISCONTIG);
- element_count = (discont_mem) ? BUFFER_MAX_ELEMENTS : 1;
- alloc_size = (discont_mem) ? PAGE_SIZE : BUFFER_SIZE;
- if (discont_mem) {
- for (i = 0; i < card->options.inbound_buffer_count; i++) {
- for (j = 0; j < element_count; j++) {
- card->inbound_buffer_pool_entry[i][j] =
- kmalloc(alloc_size, GFP_KERNEL);
- if (!card->inbound_buffer_pool_entry[i][j]) {
- failed = 1;
- goto out;
- }
+ for (i = 0; i < card->options.inbound_buffer_count; i++) {
+ for (j = 0; j < BUFFER_MAX_ELEMENTS; j++) {
+ card->inbound_buffer_pool_entry[i][j] =
+ kmalloc(PAGE_SIZE, GFP_KERNEL);
+ if (!card->inbound_buffer_pool_entry[i][j]) {
+ goto out;
}
- card->inbound_buffer_pool_entry_used[i] = BUFFER_UNUSED;
- }
- } else {
- for (i = 0; i < card->options.inbound_buffer_count; i++) {
- card->inbound_buffer_pool_entry[i][0] =
- kmalloc(alloc_size, GFP_KERNEL);
- if (!card->inbound_buffer_pool_entry[i][0])
- failed = 1;
- for (j = 1; j < element_count; j++)
- card->inbound_buffer_pool_entry[i][j] =
- card->inbound_buffer_pool_entry[i][0] +
- PAGE_SIZE * j;
- card->inbound_buffer_pool_entry_used[i] = BUFFER_UNUSED;
}
+ card->inbound_buffer_pool_entry_used[i] = BUFFER_UNUSED;
}
+ spin_lock_init(&card->requeue_input_lock);
+
+ return 0;
out:
- if (failed) {
- for (i = 0; i < card->options.inbound_buffer_count; i++) {
- for (j = 0; j < QDIO_MAX_ELEMENTS_PER_BUFFER; j++) {
- if (card->inbound_buffer_pool_entry[i][j]) {
- if (j < element_count)
- kfree(card->
- inbound_buffer_pool_entry
- [i][j]);
- card->inbound_buffer_pool_entry
- [i][j] = NULL;
- }
+ for (i = 0; i < card->options.inbound_buffer_count; i++) {
+ for (j = 0; j < QDIO_MAX_ELEMENTS_PER_BUFFER; j++) {
+ if (card->inbound_buffer_pool_entry[i][j]) {
+ if (j < BUFFER_MAX_ELEMENTS)
+ kfree(card->
+ inbound_buffer_pool_entry[i][j]);
+ card->inbound_buffer_pool_entry[i][j] = NULL;
}
}
- for (i = 0; i < card->no_queues; i++) {
- vfree(card->outbound_ringbuffer[i]);
- card->outbound_ringbuffer[i] = NULL;
- }
- return -ENOMEM;
}
+ for (i = 0; i < card->no_queues; i++) {
+ vfree(card->outbound_ringbuffer[i]);
+ card->outbound_ringbuffer[i] = NULL;
+ }
+ return -ENOMEM;
- spin_lock_init(&card->requeue_input_lock);
-
- return 0;
}
/* also locked from outside (setup_lock) */
@@ -9092,11 +9016,11 @@
length += sprintf(buffer + length,
"devices CHPID "
"device cardtype port chksum prio-q'ing "
- "rtr fsz C cnt\n");
+ "rtr fsz cnt\n");
length += sprintf(buffer + length,
"-------------------- --- ----"
"------ -------------- -- -- ---------- "
- "--- --- - ---\n");
+ "--- --- ---\n");
card = firstcard;
while (card) {
strcpy(checksum_str,
@@ -9212,7 +9136,7 @@
} else {
length += sprintf(buffer + length,
"%s/%s/%s x%02X %10s %14s %2i"
- " %2s %10s %3s %3s %c %3i\n",
+ " %2s %10s %3s %3s %3i\n",
card->rdev->dev.bus_id,
card->wdev->dev.bus_id,
card->ddev->dev.bus_id,
@@ -9222,8 +9146,6 @@
card->is_guest_lan),
card->options.portno, checksum_str,
queueing_str, router_str, bufsize_str,
- (card->options.memusage ==
- MEMUSAGE_CONTIG) ? 'c' : ' ',
card->options.inbound_buffer_count);
}
card = card->next;
@@ -9662,7 +9584,8 @@
qeth_version = 0;
number_of_devices = 0;
- copy_from_user((void *) parms, (void *) arg, sizeof (parms));
+ if (copy_from_user((void *) parms, (void *) arg, sizeof (parms)))
+ return -EFAULT;
memcpy(&data_size, parms, sizeof (__u32));
if (!(data_size > 0))
@@ -9725,7 +9648,8 @@
sizeof (__u32));
memcpy(((char *) buffer_pointer) + (3 * sizeof (__u32)),
&number_of_devices, sizeof (__u32));
- copy_to_user((char *) arg, buffer, data_len);
+ if (copy_to_user((char *) arg, buffer, data_len))
+ result = -EFAULT;
vfree(buffer);
out:
read_unlock(&list_lock);
@@ -10017,7 +9941,6 @@
};
static struct device qeth_root_dev = {
- .name = "QETH Devices",
.bus_id = "qeth",
};
@@ -10467,44 +10390,6 @@
static DEVICE_ATTR(portno, 0644, qeth_portno_show, qeth_portno_store);
static ssize_t
-qeth_contig_show(struct device *dev, char *buf)
-{
- struct qeth_card *card = dev->driver_data;
-
- if (!card)
- return -EINVAL;
-
- return sprintf(buf, "%s\n",
- (card->options.memusage == MEMUSAGE_CONTIG)?"yes":"no");
-}
-
-static ssize_t
-qeth_contig_store(struct device *dev, const char *buf, size_t count)
-{
- struct qeth_card *card = dev->driver_data;
- int i;
- char *tmp;
-
- if (!card)
- return count;
-
- if (atomic_read(&card->is_hardsetup))
- return -EPERM;
-
- i = simple_strtoul(buf, &tmp, 16);
- if (i == 0)
- card->options.memusage = MEMUSAGE_DISCONTIG;
- else if (i == 1)
- card->options.memusage = MEMUSAGE_CONTIG;
- else
- return -EINVAL;
-
- return count;
-}
-
-static DEVICE_ATTR(contig, 0644, qeth_contig_show, qeth_contig_store);
-
-static ssize_t
qeth_polltime_show(struct device *dev, char *buf)
{
struct qeth_card *card = dev->driver_data;
@@ -10585,7 +10470,7 @@
return -EINVAL;
return sprintf(buf, "%s\n",
- (card->options.ena_ipat == ENABLE_TAKEOVER)?"yes":"no");
+ (card->options.ena_ipat == ENABLE_TAKEOVER)?"1":"0");
}
static ssize_t
@@ -10602,9 +10487,9 @@
return -EPERM;
i = simple_strtoul(buf, &tmp, 16);
- if (i == 0)
+ if (i == 1)
card->options.ena_ipat = ENABLE_TAKEOVER;
- else if (i == 1)
+ else if (i == 0)
card->options.ena_ipat = DISABLE_TAKEOVER;
else
return -EINVAL;
@@ -10623,7 +10508,7 @@
return -EINVAL;
return sprintf(buf, "%s\n",
- (card->options.macaddr_mode == MACADDR_CANONICAL)?"yes":"no");
+ (card->options.macaddr_mode == MACADDR_CANONICAL)?"1":"0");
}
static ssize_t
@@ -10661,7 +10546,7 @@
return -EINVAL;
return sprintf(buf, "%s\n",
- (card->options.fake_broadcast == FAKE_BROADCAST)?"yes":"no");
+ (card->options.fake_broadcast == FAKE_BROADCAST)?"1":"0");
}
static ssize_t
@@ -10699,7 +10584,7 @@
return -EINVAL;
return sprintf(buf, "%s\n",
- (card->options.fake_ll == FAKE_LL)?"yes":"no");
+ (card->options.fake_ll == FAKE_LL)?"1":"0");
}
static ssize_t
@@ -10729,44 +10614,6 @@
static DEVICE_ATTR(fake_ll, 0644, qeth_fakell_show, qeth_fakell_store);
static ssize_t
-qeth_hsi_show(struct device *dev, char *buf)
-{
- struct qeth_card *card = dev->driver_data;
-
- if (!card)
- return -EINVAL;
-
- return sprintf(buf, "%s\n",
- (card->options.async_iqd == ASYNC_IQD)?"async":"sync");
-}
-
-static ssize_t
-qeth_hsi_store(struct device *dev, const char *buf, size_t count)
-{
- struct qeth_card *card = dev->driver_data;
- int i;
- char *tmp;
-
- if (!card)
- return count;
-
- if (atomic_read(&card->is_hardsetup))
- return -EPERM;
-
- i = simple_strtoul(buf, &tmp, 16);
- if (i == 0)
- card->options.async_iqd = SYNC_IQD;
- else if (i == 1)
- card->options.async_iqd = ASYNC_IQD;
- else
- return -EINVAL;
-
- return count;
-}
-
-static DEVICE_ATTR(async_hsi, 0644, qeth_hsi_show, qeth_hsi_store);
-
-static ssize_t
qeth_broadcast_show(struct device *dev, char *buf)
{
struct qeth_card *card = dev->driver_data;
@@ -10830,6 +10677,24 @@
static DEVICE_ATTR(recover, 0200, 0, qeth_recover_store);
+static ssize_t
+qeth_card_type_show(struct device *dev, char *buf)
+{
+ struct qeth_card *card = dev->driver_data;
+
+ if (!card)
+ return -EINVAL;
+
+ if (!atomic_read(&card->is_softsetup))
+ return sprintf(buf, "n/a\n");
+
+ return sprintf(buf, "%s\n",
+ qeth_get_cardname_short(card->type, card->link_type,
+ card->is_guest_lan));
+}
+
+static DEVICE_ATTR(card_type, 0444, qeth_card_type_show, NULL);
+
static inline int
__qeth_create_attributes(struct device *dev)
{
@@ -10863,10 +10728,6 @@
if (ret != 0)
goto out_noportno;
- ret = device_create_file(dev, &dev_attr_contig);
- if (ret != 0)
- goto out_nocontig;
-
ret = device_create_file(dev, &dev_attr_polltime);
if (ret != 0)
goto out_nopolltime;
@@ -10891,10 +10752,6 @@
if (ret != 0)
goto out_nofakell;
- ret = device_create_file(dev, &dev_attr_async_hsi);
- if (ret != 0)
- goto out_nohsi;
-
ret = device_create_file(dev, &dev_attr_broadcast_mode);
if (ret != 0)
goto out_nobrmode;
@@ -10903,13 +10760,16 @@
if (ret != 0)
goto out_norecover;
- return 0;
+ ret = device_create_file(dev, &dev_attr_card_type);
+ if (ret != 0)
+ goto out_nocardtype;
+ return 0;
+out_nocardtype:
+ device_remove_file(dev, &dev_attr_recover);
out_norecover:
device_remove_file(dev, &dev_attr_broadcast_mode);
out_nobrmode:
- device_remove_file(dev, &dev_attr_async_hsi);
-out_nohsi:
device_remove_file(dev, &dev_attr_fake_ll);
out_nofakell:
device_remove_file(dev, &dev_attr_fake_broadcast);
@@ -10922,8 +10782,6 @@
out_nohhlen:
device_remove_file(dev, &dev_attr_polltime);
out_nopolltime:
- device_remove_file(dev, &dev_attr_contig);
-out_nocontig:
device_remove_file(dev, &dev_attr_portno);
out_noportno:
device_remove_file(dev, &dev_attr_priority_queueing);
@@ -10941,6 +10799,27 @@
return ret;
}
+static inline void
+__qeth_remove_attributes(struct device *dev)
+{
+ device_remove_file(dev, &dev_attr_card_type);
+ device_remove_file(dev, &dev_attr_recover);
+ device_remove_file(dev, &dev_attr_broadcast_mode);
+ device_remove_file(dev, &dev_attr_fake_ll);
+ device_remove_file(dev, &dev_attr_fake_broadcast);
+ device_remove_file(dev, &dev_attr_canonical_macaddr);
+ device_remove_file(dev, &dev_attr_enable_takeover);
+ device_remove_file(dev, &dev_attr_add_hhlen);
+ device_remove_file(dev, &dev_attr_polltime);
+ device_remove_file(dev, &dev_attr_portno);
+ device_remove_file(dev, &dev_attr_priority_queueing);
+ device_remove_file(dev, &dev_attr_checksumming);
+ device_remove_file(dev, &dev_attr_route6);
+ device_remove_file(dev, &dev_attr_route4);
+ device_remove_file(dev, &dev_attr_portname);
+ device_remove_file(dev, &dev_attr_bufcnt);
+}
+
static int
qeth_probe_device(struct ccwgroup_device *gdev)
{
@@ -10974,7 +10853,6 @@
if (ret != 0)
goto out;
- snprintf(gdev->dev.name, DEVICE_NAME_SIZE, "qeth device");
return 0;
out:
put_device(&gdev->dev);
@@ -11060,9 +10938,10 @@
{
struct qeth_card *card = gdev->dev.driver_data;
+ __qeth_remove_attributes(&gdev->dev);
gdev->dev.driver_data = NULL;
if (card)
- kfree(card);
+ qeth_free_card(card);
put_device(&gdev->dev);
return 0;
}
@@ -11070,17 +10949,15 @@
static int
qeth_set_online(struct ccwgroup_device *gdev)
{
+ int rc;
struct qeth_card *card = gdev->dev.driver_data;
- int ret;
BUG_ON(!card);
- ret = qeth_activate(card);
- if (ret == 0)
- snprintf(gdev->dev.name, DEVICE_NAME_SIZE, "%s",
- qeth_get_cardname_short(card->type, card->link_type,
- card->is_guest_lan));
- return ret;
+ rc = qeth_alloc_card_stuff(card);
+
+ return rc ? rc : qeth_activate(card);
+
}
static int
@@ -11096,14 +10973,12 @@
QETH_DBF_TEXT4(0, trace, "freecard");
- memset(card->dev, 0, sizeof (struct net_device));
- card->dev->priv = card;
- strncpy(card->dev->name, card->dev_name, IFNAMSIZ);
-
ccw_device_set_offline(card->ddev);
ccw_device_set_offline(card->wdev);
ccw_device_set_offline(card->rdev);
+ qeth_free_card_stuff(card);
+
return 0;
}
diff -urN linux-2.6/drivers/s390/net/qeth.h linux-2.6-s390/drivers/s390/net/qeth.h
--- linux-2.6/drivers/s390/net/qeth.h Mon Sep 8 21:49:51 2003
+++ linux-2.6-s390/drivers/s390/net/qeth.h Thu Sep 11 19:21:27 2003
@@ -14,7 +14,7 @@
#define QETH_NAME " qeth"
-#define VERSION_QETH_H "$Revision: 1.49 $"
+#define VERSION_QETH_H "$Revision: 1.55 $"
/******************** CONFIG STUFF ***********************/
//#define QETH_DBF_LIKE_HELL
@@ -567,13 +567,6 @@
#define QETH_LOCK_NORMAL 1
#define QETH_LOCK_FLUSH 2
-#define QETH_MAX_DEVICES 16
- /* DEPENDENCY ON QETH_MAX_DEVICES.
- *__MOUDLE_STRING expects simple literals */
-#define QETH_MAX_DEVICES_TIMES_4 64
-#define QETH_MAX_DEVNAMES 16
-#define QETH_DEVNAME "eth"
-
#define QETH_TX_TIMEOUT 100*HZ /* 100 seconds */
#define QETH_REMOVE_WAIT_TIME 200
@@ -581,8 +574,6 @@
#define QETH_IDLE_WAIT_TIME 10
#define QETH_WAIT_BEFORE_2ND_DOIO 1000
-#define QETH_MAX_PARM_LEN 128
-
#define QETH_FAKE_LL_LEN ETH_HLEN /* 14 */
#define QETH_FAKE_LL_PROT_LEN 2
#define QETH_FAKE_LL_ADDR_LEN ETH_ALEN /* 6 */
@@ -609,16 +600,12 @@
IPA_PDU_HEADER_SIZE+sizeof(struct ipa_cmd)), \
QETH_RCD_LENGTH)
-#define QETH_FINAL_STATUS_TIMEOUT 1500
-#define QETH_CLEAR_TIMEOUT 1500
-#define QETH_RCD_TIMEOUT 1500
#define QETH_NOP_TIMEOUT 1500
#define QETH_QUIESCE_NETDEV_TIME 300
#define QETH_QUIESCE_WAIT_BEFORE_CLEAR 4000
#define QETH_QUIESCE_WAIT_AFTER_CLEAR 4000
#define NOP_STATE 0x1001
-#define READ_CONF_DATA_STATE 0x1002
#define IDX_ACTIVATE_READ_STATE 0x1003
#define IDX_ACTIVATE_WRITE_STATE 0x1004
#define MPC_SETUP_STATE 0x1005
@@ -647,8 +634,6 @@
#define BROADCAST_LOCAL 1
#define MACADDR_NONCANONICAL 0
#define MACADDR_CANONICAL 1
-#define MEMUSAGE_DISCONTIG 0
-#define MEMUSAGE_CONTIG 1
#define ENABLE_TAKEOVER 0
#define DISABLE_TAKEOVER 1
#define FAKE_BROADCAST 0
@@ -656,8 +641,6 @@
#define FAKE_LL 0
#define DONT_FAKE_LL 1
-#define SYNC_IQD 0
-#define ASYNC_IQD 1
#define QETH_BREAKOUT_LEAVE 1
#define QETH_BREAKOUT_AGAIN 2
@@ -684,9 +667,6 @@
#define SENSE_RESETTING_EVENT_BYTE 1
#define SENSE_RESETTING_EVENT_FLAG 0x80
-#define DEFAULT_RCD_CMD 0x72
-#define DEFAULT_RCD_COUNT 0x80
-
#define BUFFER_USED 1
#define BUFFER_UNUSED -1
@@ -744,14 +724,12 @@
int polltime;
char portname[9];
int portno;
- int memusage;
int broadcast_mode;
int macaddr_mode;
int ena_ipat;
int fake_broadcast;
int add_hhlen;
int fake_ll;
- int async_iqd;
};
struct qeth_hdr {
@@ -811,7 +789,6 @@
/* ugly. I know. */
struct qeth_card { /* pointed to by dev->priv */
- int easy_copy_cap;
/* pointer to options (defaults + parameters) */
struct qeth_card_options options;
@@ -930,8 +907,6 @@
int is_multicast_different; /* if multicast traffic is to be sent
on a different queue, this is the
queue+no_queues */
- int can_do_async_iqd; /* 1 only on IQD that provides async
- unicast sigas */
__u32 ipa_supported;
__u32 ipa_enabled;
__u32 ipa6_supported;
@@ -969,8 +944,6 @@
atomic_t ioctl_data_has_arrived;
wait_queue_head_t ioctl_wait_q;
- atomic_t ioctl_wait_q_active;
- spinlock_t ioctl_wait_q_lock;
/* stuff under 2 gb */
struct qeth_dma_stuff *dma_stuff;
@@ -987,8 +960,6 @@
atomic_t shutdown_phase;
atomic_t data_has_arrived;
wait_queue_head_t wait_q;
- atomic_t wait_q_active;
- spinlock_t wait_q_lock; /* for wait_q_active and wait_q */
atomic_t clear_succeeded0;
atomic_t clear_succeeded1;
@@ -1034,21 +1005,6 @@
}
}
-inline static int
-qeth_determine_easy_copy_cap(int cardtype)
-{
- switch (cardtype) {
- case QETH_CARD_TYPE_UNKNOWN:
- return 0; /* better be cautious */
- case QETH_CARD_TYPE_OSAE:
- return 1;
- case QETH_CARD_TYPE_IQD:
- return 0;
- default:
- return 0; /* ?? */
- }
-}
-
inline static __u8
qeth_get_adapter_type_for_ipa(int link_type)
{
Martin Schwidefsky wrote:
> - spin_lock_init(&card->wait_q_lock);
> + /* setup net_device stuff */
> + card->dev->priv = card;
> +
> + strncpy(card->dev->name, card->dev_name, IFNAMSIZ);
what's this about? Why avoid the net stack's dev->name assignment?
> + QETH_DBF_TEXT3(0, trace, "alloccrd");
> + card = (struct qeth_card *) vmalloc(sizeof (struct qeth_card));
> + if (!card)
> + return NULL;
Is the card's private info really so large that you need vmalloc() ?
Most of the patch looks ok to me, except for these minor niggles.
Jeff
Hi Jeff,
> > - spin_lock_init(&card->wait_q_lock);
> > + /* setup net_device stuff */
> > + card->dev->priv = card;
> > +
> > + strncpy(card->dev->name, card->dev_name, IFNAMSIZ);
>
> what's this about? Why avoid the net stack's dev->name assignment?
This is indeed strange, because in qeth_init_netdev the name is copied
from the net_device to the card structure. Seems like the card->dev_name
is superflous. I'll ask our networking guys.
> > + QETH_DBF_TEXT3(0, trace, "alloccrd");
> > + card = (struct qeth_card *) vmalloc(sizeof (struct qeth_card));
> > + if (!card)
> > + return NULL;
>
> Is the card's private info really so large that you need vmalloc() ?
For 31 bit struct qeth_card has 55808 bytes for 64 bit 64256. We could
allocate it with kmalloc but overall the qeth driver allocates a LOT of
memory with vmalloc. The inbound/outbound queues are quite big and the
qeth_card structure is almost neglectable...
blue skies,
Martin