Re: [PATCH] Re: SCSI scanning

From: David S. Miller (davem@redhat.com)
Date: Mon Sep 18 2000 - 20:34:58 EST


   Date: Mon, 18 Sep 2000 17:51:43 -0700 (PDT)
   From: Linus Torvalds <torvalds@transmeta.com>

   Umm, reading the code it looks more like the proper test would be

           if (!tpnt->present)
                   return;

   because if "present == 0", then the host not only won't have had the proc
   entry added, it also won't have been added to the "scsi_hosts" list - so
   the "remove the adapter from the linked list" step is the wrong way around
   too. No?

The problem is that regardless of the tpnt->present setting, the
MOD_DEC_USE_COUNT must occur.

   And again, why did this not show up with modules?

I have no idea, I'm just the messenger in this case :-)
My situation was:

1) esp.c sym53c8xx.c statically built into kernel
2) I have 2 esp interfaces present in this machine,
   none for the sym53c8xx driver.

The boot logs looked something like (note the request_module messages,
wheee...):

request_module[scsi_hostadapter]: Root fs not mounted
esp0: IRQ 3,7e0 SCSI ID 7 Clk 40MHz CCYC=25000 CCF=8 TOut 167 NCR53C9XF(espfast)
esp1: IRQ 7,7db SCSI ID 7 Clk 40MHz CCYC=25000 CCF=8 TOut 167 NCR53C9XF(espfast)
ESP: Total of 2 ESP hosts found, 2 actually in use.
scsi0 : Sparc ESP366-HME
scsi1 : Sparc ESP366-HME
scsi : 2 hosts.
...
Partition check:
 sda: sda1 sda2 sda3
esp0: target 1 [period 100ns offset 15 20.00MHz FAST-WIDE SCSI-II]
SCSI device sdb: hdwr sector= 512 bytes. Sectors= 4194995 [2048 MB] [2.0 GB]
 sdb: sdb1 sdb2 sdb3
...
scsi : 2 hosts.

And precisely here is where the OOPS was seen.

Later,
David S. Miller
davem@redhat.com
-
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/



This archive was generated by hypermail 2b29 : Sat Sep 23 2000 - 21:00:19 EST