On Mon, 18 Sep 2000, David S. Miller wrote:
>
> Did you try to boot these kernels containing scsi devices you
> don't have? I don't see how it could work (actually I do, see
> below).
Hmm..
Why would this not have happened for a module?
I agree that the thing looks fishy. But this is not new code, and it has
worked previously. What changed?
> Anyways, the code at the end of scsi_unregister_host should rather
> be something like:
>
> if (tpnt->present) {
> remove_proc_entry(tpnt->proc_name, proc_scsi);
> return;
> }
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?
(That linked list implementation, btw, is the worst possible list
implementation I have ever seen. Ugh).
And again, why did this not show up with modules?
Curious.
Linus
-
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