[lkml]   [2000]   [Sep]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] Re: SCSI scanning

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).


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)

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?



To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
Please read the FAQ at

 \ /
  Last update: 2005-03-22 12:38    [W:0.228 / U:2.672 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site