storage: Returns earlier if source adapter of the scsi pool is a HBA

It makes no sense to go forward to get the parent host number of a
HBA, and treat the HBA as a vHBA with trying to delete it.

Signed-off-by: Osier Yang <jyang@redhat.com>
This commit is contained in:
Osier Yang 2013-11-20 20:38:17 +08:00
parent c7ccd2c44b
commit b96651dec0

View File

@ -667,6 +667,14 @@ deleteVport(virStoragePoolSourceAdapter adapter)
if (adapter.type != VIR_STORAGE_POOL_SOURCE_ADAPTER_TYPE_FC_HOST)
return 0;
/* It must be a HBA instead of a vHBA as long as "parent"
* is NULL. "createVport" guaranteed "parent" for a vHBA
* cannot be NULL, it's either specified in XML, or detected
* automatically.
*/
if (!adapter.data.fchost.parent)
return 0;
if (!(virGetFCHostNameByWWN(NULL, adapter.data.fchost.wwnn,
adapter.data.fchost.wwpn)))
return -1;