Re: [libvirt] [PATCH v2 12/12] conf: Check for hostdev conflicts when assign default disk address

2015-08-12 Thread John Ferlan
... >> After looking through the code and thinking more about this - using >> virDomainDefPostParse won't work for the hotplug case since it's only >> going through virDomainDeviceDefParse and virDomainDeviceDefPostParse >> code in order to validate whether the address (whether provided or >> gen

Re: [libvirt] [PATCH v2 12/12] conf: Check for hostdev conflicts when assign default disk address

2015-08-12 Thread Ján Tomko
On Tue, Aug 04, 2015 at 10:21:29AM -0400, John Ferlan wrote: > > > On 08/03/2015 09:57 AM, Ján Tomko wrote: > > On Wed, Jul 22, 2015 at 10:54:34AM -0400, John Ferlan wrote: > >> https://bugzilla.redhat.com/show_bug.cgi?id=1210587 (completed) > >> > >> When generating the default drive address fo

Re: [libvirt] [PATCH v2 12/12] conf: Check for hostdev conflicts when assign default disk address

2015-08-04 Thread John Ferlan
On 08/03/2015 09:57 AM, Ján Tomko wrote: > On Wed, Jul 22, 2015 at 10:54:34AM -0400, John Ferlan wrote: >> https://bugzilla.redhat.com/show_bug.cgi?id=1210587 (completed) >> >> When generating the default drive address for a SCSI device, >> check the generated address to ensure it doesn't confl

Re: [libvirt] [PATCH v2 12/12] conf: Check for hostdev conflicts when assign default disk address

2015-08-03 Thread Ján Tomko
On Wed, Jul 22, 2015 at 10:54:34AM -0400, John Ferlan wrote: > https://bugzilla.redhat.com/show_bug.cgi?id=1210587 (completed) > > When generating the default drive address for a SCSI device, > check the generated address to ensure it doesn't conflict with a SCSI > address. The address generat

[libvirt] [PATCH v2 12/12] conf: Check for hostdev conflicts when assign default disk address

2015-07-22 Thread John Ferlan
https://bugzilla.redhat.com/show_bug.cgi?id=1210587 (completed) When generating the default drive address for a SCSI device, check the generated address to ensure it doesn't conflict with a SCSI address. The address generation algorithm uses the "dev" name in order to determine which controll