On 06/17/2010 03:15 PM, Matthias Bolte wrote: > Also don't abuse the disk driver name to specify the SCSI controller > model anymore: > > <driver name='buslogic'/> > > Use the newly added model attribute of the controller element for this: > > <controller type='scsi' index='0' model='buslogic'/>
I don't see any change to docs/schemas/domain.rng for this new XML
attribute. Am I missing something, or how do you write an xml file that
uses this attribute and still passes validation?
>
> The disk driver name approach is deprecated now, but still works for
> backward compatibility reasons.
>
> Update the documentation and tests accordingly.
>
> Fix usage of the words controller and id in the VMX handling code. Use
> controller, bus and unit properly.
> ---
> docs/drvesx.html.in | 25 +-
Is the xml addition ESX-specific, or should the new controller attribute
also be documented in docs/formatdomain.html.in? Is anything other than
model='lsilogic' supported?
>
> @@ -1014,18 +1184,18 @@ esxVMX_ParseConfig(esxVI_Context *ctx, const char
> *vmx,
> continue;
> }
>
> - for (id = 0; id < 16; ++id) {
> - if (id == 7) {
> + for (unit = 0; unit < 16; ++unit) {
> + if (unit == 7) {
> /*
> - * SCSI ID 7 is assigned to the SCSI controller and cannot be
> + * SCSI unit 7 is assigned to the SCSI controller and cannot
> be
> * used for disk devices.
> */
> continue;
I've run out of time for reviewing the rest of the patch today (didn't
spot anything else obvious above this point), but you may still want to
answer my questions and/or respin accordingly.
--
Eric Blake ebl...@redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
