Re: [libvirt] [PATCH 1/2] storage: Fix autostart of pool with fc_host type adapter

2014-01-23 Thread Osier Yang
On 22/01/14 21:36, John Ferlan wrote: On 01/07/2014 06:07 AM, Osier Yang wrote: On 07/01/14 01:21, John Ferlan wrote: On 01/06/2014 05:19 AM, Osier Yang wrote: The checkPool is a bit different for pool with fc_host type source adapter, since the vHBA it's based on might be not created yet

Re: [libvirt] [PATCH 1/2] storage: Fix autostart of pool with fc_host type adapter

2014-01-23 Thread John Ferlan
On 01/23/2014 04:45 AM, Osier Yang wrote: On 22/01/14 21:36, John Ferlan wrote: On 01/07/2014 06:07 AM, Osier Yang wrote: On 07/01/14 01:21, John Ferlan wrote: On 01/06/2014 05:19 AM, Osier Yang wrote: The checkPool is a bit different for pool with fc_host type source adapter, since the

Re: [libvirt] [PATCH 1/2] storage: Fix autostart of pool with fc_host type adapter

2014-01-23 Thread Osier Yang
On 23/01/14 21:35, John Ferlan wrote: On 01/23/2014 04:45 AM, Osier Yang wrote: On 22/01/14 21:36, John Ferlan wrote: On 01/07/2014 06:07 AM, Osier Yang wrote: On 07/01/14 01:21, John Ferlan wrote: On 01/06/2014 05:19 AM, Osier Yang wrote: The checkPool is a bit different for pool with

Re: [libvirt] [PATCH 1/2] storage: Fix autostart of pool with fc_host type adapter

2014-01-22 Thread John Ferlan
On 01/07/2014 06:07 AM, Osier Yang wrote: On 07/01/14 01:21, John Ferlan wrote: On 01/06/2014 05:19 AM, Osier Yang wrote: The checkPool is a bit different for pool with fc_host type source adapter, since the vHBA it's based on might be not created yet (it's created by startPool, which is

Re: [libvirt] [PATCH 1/2] storage: Fix autostart of pool with fc_host type adapter

2014-01-22 Thread Osier Yang
On 22/01/14 21:36, John Ferlan wrote: On 01/07/2014 06:07 AM, Osier Yang wrote: On 07/01/14 01:21, John Ferlan wrote: On 01/06/2014 05:19 AM, Osier Yang wrote: The checkPool is a bit different for pool with fc_host type source adapter, since the vHBA it's based on might be not created yet

Re: [libvirt] [PATCH 1/2] storage: Fix autostart of pool with fc_host type adapter

2014-01-07 Thread Osier Yang
On 07/01/14 01:21, John Ferlan wrote: On 01/06/2014 05:19 AM, Osier Yang wrote: The checkPool is a bit different for pool with fc_host type source adapter, since the vHBA it's based on might be not created yet (it's created by startPool, which is involked after checkPool in

[libvirt] [PATCH 1/2] storage: Fix autostart of pool with fc_host type adapter

2014-01-06 Thread Osier Yang
The checkPool is a bit different for pool with fc_host type source adapter, since the vHBA it's based on might be not created yet (it's created by startPool, which is involked after checkPool in storageDriverAutostart). So it should not fail, otherwise the autostart of the pool will fail either.

Re: [libvirt] [PATCH 1/2] storage: Fix autostart of pool with fc_host type adapter

2014-01-06 Thread John Ferlan
On 01/06/2014 05:19 AM, Osier Yang wrote: The checkPool is a bit different for pool with fc_host type source adapter, since the vHBA it's based on might be not created yet (it's created by startPool, which is involked after checkPool in storageDriverAutostart). So it should not fail,