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

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

<Prev in Thread]

Current Thread

[Next in Thread>

Previous by Date:

[Xen-devel] [PATCH] Use hvm_dirq_assist() to replace {vmx, svm}_dirq_assist()., Cui, Dexuan

Next by Date:

[Xen-devel] Linux options pci-mem-align vs. reassigndev=, Jan Beulich

Previous by Thread:

[Xen-devel] [PATCH 2/2] Fix and clean up vscsi_util.py and other files, Masaki Kanno

Next by Thread:

Re: [Xen-devel] Re: [PATCH 2/2] Fix and clean up vscsi_util.py andother files, Masaki Kanno

Indexes:

[Date] [Thread] [Top] [All Lists]