[Xen-devel] Re: [PATCH 2/2] Fix and clean up vscsi_util.py and other files

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Masaki Kanno writes ("[PATCH 2/2] Fix and clean up vscsi_util.py and other 
files"):
> Fix and clean up vscsi_util.py and other files
> Signed-off-by: Masaki Kanno <kanno.masaki@xxxxxxxxxxxxxx>

Thanks.  This looks like a definite improvement.  I think it should
make the vscsi stuff more or less harmless in situations where it's
not being used.

So:
 Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>

However, are you sure that the approach taken in vscsi_util.py is the
right one ?  The set of scsi devices available on a system may change
over time, so it is not correct to try to build a list at startup.
Also, doing so will slow down startup (including startup time of the
xm utility) on systems with many scsi devices even when pvscsi is not
in use, and exposes non-pvscsi users to bugs in the pvscsi system (as
previously discussed).

Surely there must be some way to do this lookup directly in /sys or
whereever at the point of use, when we're instructed to pass through a
specific device ?

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel