From: Andrew Morton <andrewm@uow.edu.au>
To: Linus Torvalds <torvalds@transmeta.com>
Cc: "shuu@wondernetworkresources.com"
<shuu@wondernetworkresources.com>,
Jeff Garzik <jgarzik@mandrakesoft.com>,
lkml <linux-kernel@vger.kernel.org>
Subject: [patch] 8139too.c
Date: Tue, 19 Dec 2000 02:32:29 +1100 [thread overview]
Message-ID: <3A3E2E0D.648B04E0@uow.edu.au> (raw)
- Clear current->blocked in the kernel thread. We shouldn't
be inheriting this from the process which opens the interface.
- Fixed a few printk warnings which are coming out
when RTL8139_DEBUG is defined.
- Killed the undefined and unused module parm `debug' (finally!)
- Fixed a potential buffer overrun when setting current->comm[].
Currently, if the user renames "eth0" to "my-nifty-realtek-nic"
with SIOCSIFNAME and then tries to open it, the kernel thread
will scrog its own task_struct.
The kernel thread is now simply called "[eth0]".
I think it would be better to use boring old waitqueues for this
stuff, rather than signals....
--- linux-2.4.0-test13-pre3/drivers/net/8139too.c Tue Dec 12 19:24:18 2000
+++ linux-akpm/drivers/net/8139too.c Tue Dec 19 02:15:23 2000
@@ -74,6 +74,9 @@
Tobias Ringström - Rx interrupt status checking suggestion
+ Andrew Morton - (v0.9.13): clear blocked signals, avoid
+ buffer overrun setting current->comm.
+
Submitting bug reports:
"rtl8139-diag -mmmaaavvveefN" output
@@ -147,7 +150,7 @@
#include <asm/io.h>
-#define RTL8139_VERSION "0.9.12"
+#define RTL8139_VERSION "0.9.13"
#define MODNAME "8139too"
#define RTL8139_DRIVER_NAME MODNAME " Fast Ethernet driver " RTL8139_VERSION
#define PFX MODNAME ": "
@@ -536,7 +539,6 @@
MODULE_DESCRIPTION ("RealTek RTL-8139 Fast Ethernet driver");
MODULE_PARM (multicast_filter_limit, "i");
MODULE_PARM (max_interrupt_work, "i");
-MODULE_PARM (debug, "i");
MODULE_PARM (media, "1-" __MODULE_STRING(8) "i");
static int read_eeprom (void *ioaddr, int location, int addr_len);
@@ -1461,7 +1463,7 @@
DPRINTK ("%s: Media selection tick, Link partner %4.4x.\n",
dev->name, RTL_R16 (NWayLPAR));
DPRINTK ("%s: Other registers are IntMask %4.4x IntStatus %4.4x"
- " RxStatus %4.4x.\n", dev->name,
+ " RxStatus %4.4lx.\n", dev->name,
RTL_R16 (IntrMask),
RTL_R16 (IntrStatus),
RTL_R32 (RxEarlyStatus));
@@ -1478,7 +1480,13 @@
unsigned long timeout;
daemonize ();
- sprintf (current->comm, "k8139d-%s", dev->name);
+ spin_lock_irq(¤t->sigmask_lock);
+ sigemptyset(¤t->blocked);
+ recalc_sigpending(current);
+ spin_unlock_irq(¤t->sigmask_lock);
+
+ strncpy (current->comm, dev->name, sizeof(current->comm) - 1);
+ current->comm[sizeof(current->comm) - 1] = '\0';
while (1) {
timeout = next_tick;
@@ -2136,7 +2144,7 @@
DPRINTK ("ENTER\n");
- DPRINTK ("%s: rtl8139_set_rx_mode(%4.4x) done -- Rx config %8.8x.\n",
+ DPRINTK ("%s: rtl8139_set_rx_mode(%4.4x) done -- Rx config %8.8lx.\n",
dev->name, dev->flags, RTL_R32 (RxConfig));
/* Note: do not reorder, GCC is clever about common statements. */
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/
next reply other threads:[~2000-12-18 15:58 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2000-12-18 15:32 Andrew Morton [this message]
2001-11-30 11:23 [PATCH] 8139too.c kumon
2001-11-30 16:57 ` Andreas Dilger
2001-11-30 19:07 ` Jeff Garzik
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=3A3E2E0D.648B04E0@uow.edu.au \
--to=andrewm@uow.edu.au \
--cc=jgarzik@mandrakesoft.com \
--cc=linux-kernel@vger.kernel.org \
--cc=shuu@wondernetworkresources.com \
--cc=torvalds@transmeta.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).