]> xenbits.xensource.com Git - libvirt.git/commit
storage: Check for valid fc_host parent at startup
authorJohn Ferlan <jferlan@redhat.com>
Thu, 6 Nov 2014 17:32:46 +0000 (12:32 -0500)
committerJohn Ferlan <jferlan@redhat.com>
Wed, 12 Nov 2014 15:18:27 +0000 (10:18 -0500)
commit844c1d7e3283be652bffddf17254c392d68c405d
tree7922e4507a5ecca56147df0e525aa5d93d541a04
parent6c1347ec06a73e328aa3a1e08212d47e5b2d2e5e
storage: Check for valid fc_host parent at startup

https://bugzilla.redhat.com/show_bug.cgi?id=1160565

If a 'parent' attribute is provided for the fchost, then at startup
time check to ensure it is a vport capable scsi_host. If the parent
is not vport capable, then disallow the startup. The following is the
expected results:

error: Failed to start pool fc_pool
error: XML error: parent 'scsi_host2' specified for vHBA is not vport capable

where the XML for the fc_pool is:

    <pool type='scsi'>
      <name>fc_pool</name>
      <source>
        <adapter type='fc_host' parent='scsi_host2' wwnn='5001a4aaf3ca174b' wwpn='5001a4a77192b864'/>
      </source>
...

and 'scsi_host2' is not vport capable.

Providing an incorrect parent and a correct wwnn/wwpn could lead to
failures at shutdown (deleteVport) where the assumption is the parent
is for the fchost.

NOTE: If the provided wwnn/wwpn doesn't resolve to an existing scsi_host,
      then we will be creating one with code (virManageVport) which
      assumes the parent is vport capable.

Signed-off-by: John Ferlan <jferlan@redhat.com>
src/storage/storage_backend_scsi.c