Fwd: Patch - SCSI host numbers - please apply

From: Itai Nahshon (nahshon@actcom.co.il)
Date: Thu Jun 06 2002 - 05:23:50 EST


This was sent on Apr 22 and probably lost.
This patch still applies cleanly to 2.4.19-pre10.

Can this get in the kernel before 2.4.19 final ?

-- Itai

---------- Forwarded Message ----------

Subject: Patch - SCSI host numbers - please apply
Date: Mon, 22 Apr 2002 09:19:56 +0300
From: Itai Nahshon <nahshon@actcom.co.il>
To: Marcelo Tosatti <marcelo@conectiva.com.br>, Jens Axboe <axboe@suse.de>

Hello,
I think this has gone through enough testing...

History:
  A similar patch was submitted to to lkml and to linux-scsi
  two months ago. Verified by Aron Zeh <ARZEH@de.ibm.com> on Apr 9.
  The attached patch is a cleaned-up version
  (The original had #ifdef'ed the code that this one removes).

Kernel version info:
  2.4.x: Patch applies cleanly (and tested with) all 2.4.19-preX
        kernels.
  2.5.x: Patch applies with 8 line offset to 2.5.8.
        Bug verified only by "visual inspection". (but really nothing
        in the host_no allocation algorithm has changed since 2.4).

Bug info:
  Actually fixing two _different_ scenarios that look almost the same.
  Need two or more) scsi host adapters that are not required for
  normal system usage. I uses ide-scsi and usb-storage fr my
  tests. Start with a "clean system" (before any of these drivers
  is loaded). Suppose scsi host adapter drivers are named A and B.

  scenario 1:
        insmod A
        rmmod A
        insmod A
        insmod B

  scenario 2:
        insmod A
        rmmod A
        insmod B
        insmod A

  Without the patch, both scenarios end with both host adapters having
  the same host_no.

More tests:
  I tested also with param scsihosts=<host0>:<host1>:....
  (when loading scsi_mod)

Comment/motivation:
  This keeps max_scsi_host coherent with the length of the list
  scsi_host_no_list. Since we never shorten the list
  we should never decrement max_scsi_host.

Sincerely,
-- Itai

-------------------------------------------------------



-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Fri Jun 07 2002 - 22:00:27 EST