Re: CVS commit: src/sys/dev/scsipi

[Kimmo Suominen <kim%netbsd.org@localhost>]

I don't think this should be reverted, because LUN 0 must exist, but if
there is no device on it, it will report "NOT PRESENT". We do not want
the scan to stop in this case, but it should continue with other LUNs
(such as those found through REPORT LUNS in the future).

Kind regards,
+ Kimmo


On Fri, Sep 18, 2020 at 03:04:25PM +0000, Jonathan A. Kollasch wrote:
> Module Name:	src
> Committed By:	jakllsch
> Date:		Fri Sep 18 15:04:25 UTC 2020
> 
> Modified Files:
> 	src/sys/dev/scsipi: scsiconf.c
> 
> Log Message:
> Revert scsiconf.c 1.288, it only worked for LUN 1.
> 
> vioscsi(4) now sets PQUIRK_FORCELUNS, which fixes the original issue for
> all LUNs.
> 
> To-do: should issue REPORT LUNS and use the information it returns to
> probe LUNs in an optimized way.
> 
> 
> To generate a diff of this commit:
> cvs rdiff -u -r1.289 -r1.290 src/sys/dev/scsipi/scsiconf.c
> 
> Please note that diffs are not public domain; they are subject to the
> copyright notices on the relevant files.
>