* [PATCH] NCR53c406a.c warning
@ 2003-09-11 6:37 Geert Uytterhoeven
2003-09-12 13:41 ` James Bottomley
0 siblings, 1 reply; 4+ messages in thread
From: Geert Uytterhoeven @ 2003-09-11 6:37 UTC (permalink / raw)
To: James E.J. Bottomley; +Cc: linux-scsi, Linux Kernel Development
NCR53c406a: Apparently wait_intr() is unused, so remove it.
--- linux-2.6.0-test5/drivers/scsi/NCR53c406a.c.orig Sun Aug 24 09:49:34 2003
+++ linux-2.6.0-test5/drivers/scsi/NCR53c406a.c Tue Sep 9 15:01:48 2003
@@ -170,7 +170,6 @@
/* Static function prototypes */
static void NCR53c406a_intr(int, void *, struct pt_regs *);
static irqreturn_t do_NCR53c406a_intr(int, void *, struct pt_regs *);
-static void wait_intr(void);
static void chip_init(void);
static void calc_port_addr(void);
#ifndef IRQ_LEV
@@ -663,26 +662,6 @@
{
DEB(printk("NCR53c406a_info called\n"));
return (info_msg);
-}
-
-static void wait_intr(void)
-{
- unsigned long i = jiffies + WATCHDOG;
-
- while (time_after(i, jiffies) && !(inb(STAT_REG) & 0xe0)) { /* wait for a pseudo-interrupt */
- cpu_relax();
- barrier();
- }
-
- if (time_before_eq(i, jiffies)) { /* Timed out */
- rtrc(0);
- current_SC->result = DID_TIME_OUT << 16;
- current_SC->SCp.phase = idle;
- current_SC->scsi_done(current_SC);
- return;
- }
-
- NCR53c406a_intr(0, NULL, NULL);
}
static int NCR53c406a_queue(Scsi_Cmnd * SCpnt, void (*done) (Scsi_Cmnd *))
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------------------- Sint-Stevens-Woluwestraat 55
Voice +32-2-2908453 Fax +32-2-7262686 ---------------- B-1130 Brussels, Belgium
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] NCR53c406a.c warning
2003-09-11 6:37 [PATCH] NCR53c406a.c warning Geert Uytterhoeven
@ 2003-09-12 13:41 ` James Bottomley
2003-09-12 14:16 ` Maciej W. Rozycki
0 siblings, 1 reply; 4+ messages in thread
From: James Bottomley @ 2003-09-12 13:41 UTC (permalink / raw)
To: Geert Uytterhoeven; +Cc: SCSI Mailing List, Linux Kernel Development
On Thu, 2003-09-11 at 02:37, Geert Uytterhoeven wrote:
> NCR53c406a: Apparently wait_intr() is unused, so remove it.
It is currently unused. However, the reason is that we removed the scsi
command method that allows polled operation in a driver (this routine is
actually polling the interrupt port on the chip).
I'd like to wait a while to see if anyone still needs this mode when 2.6
gets a wider test audience. If you wish, you can surround the routine
with #if 0 and a comment saying we can junk it later if it really is
unnecessary.
Thanks,
James
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] NCR53c406a.c warning
2003-09-12 13:41 ` James Bottomley
@ 2003-09-12 14:16 ` Maciej W. Rozycki
2003-09-12 14:47 ` Alan Cox
0 siblings, 1 reply; 4+ messages in thread
From: Maciej W. Rozycki @ 2003-09-12 14:16 UTC (permalink / raw)
To: James Bottomley
Cc: Geert Uytterhoeven, SCSI Mailing List, Linux Kernel Development
On 12 Sep 2003, James Bottomley wrote:
> > NCR53c406a: Apparently wait_intr() is unused, so remove it.
>
> It is currently unused. However, the reason is that we removed the scsi
> command method that allows polled operation in a driver (this routine is
> actually polling the interrupt port on the chip).
>
> I'd like to wait a while to see if anyone still needs this mode when 2.6
> gets a wider test audience. If you wish, you can surround the routine
> with #if 0 and a comment saying we can junk it later if it really is
> unnecessary.
I've encountered an ISA adapter using this chip in polled mode (no ISA
IRQ line routed to the chip) quite recently. But I can't say if the guy
using it won't throw it away before final 2.6. ;-)
--
+ Maciej W. Rozycki, Technical University of Gdansk, Poland +
+--------------------------------------------------------------+
+ e-mail: macro@ds2.pg.gda.pl, PGP key available +
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] NCR53c406a.c warning
2003-09-12 14:16 ` Maciej W. Rozycki
@ 2003-09-12 14:47 ` Alan Cox
0 siblings, 0 replies; 4+ messages in thread
From: Alan Cox @ 2003-09-12 14:47 UTC (permalink / raw)
To: Maciej W. Rozycki
Cc: James Bottomley, Geert Uytterhoeven, SCSI Mailing List,
Linux Kernel Development
On Gwe, 2003-09-12 at 15:16, Maciej W. Rozycki wrote:
> I've encountered an ISA adapter using this chip in polled mode (no ISA
> IRQ line routed to the chip) quite recently. But I can't say if the guy
> using it won't throw it away before final 2.6. ;-)
For a lot of these dumb controllers polled I/O seems to materially
outperform interrupt driven I/O on SMP boxes - its true of the NCR5380
for example. We trade a ton of locking jamming the box up for a CPU
thats spending some of its time pretending to be half a scsi chip.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2003-09-12 14:49 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-09-11 6:37 [PATCH] NCR53c406a.c warning Geert Uytterhoeven
2003-09-12 13:41 ` James Bottomley
2003-09-12 14:16 ` Maciej W. Rozycki
2003-09-12 14:47 ` Alan Cox
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).