Re: [libvirt] [PATCH 1/3] qemu: Need to assign PCI address to vhost-scsi

2017-12-14 Thread Laine Stump
On 12/12/2017 09:22 AM, John Ferlan wrote: > Commit id '70249927b' neglected to cover this case because the test > had taken the "shortcut" to already add the , so I altered > that too just to make sure it doesn't happen again. > > Previously the qemuxml2xmloutdata was a softlink to the source > qe

Re: [libvirt] [PATCH 1/3] qemu: Need to assign PCI address to vhost-scsi

2017-12-13 Thread Eric Farman
On 12/13/2017 09:05 AM, John Ferlan wrote: On 12/13/2017 08:28 AM, Eric Farman wrote: On 12/12/2017 09:22 AM, John Ferlan wrote: Commit id '70249927b' neglected to cover this case because the test had taken the "shortcut" to already add the , so I altered that too just to make sure it doe

Re: [libvirt] [PATCH 1/3] qemu: Need to assign PCI address to vhost-scsi

2017-12-13 Thread John Ferlan
On 12/13/2017 08:28 AM, Eric Farman wrote: > > > On 12/12/2017 09:22 AM, John Ferlan wrote: >> Commit id '70249927b' neglected to cover this case because the test >> had taken the "shortcut" to already add the , so I altered >> that too just to make sure it doesn't happen again. > > My bad.  :

Re: [libvirt] [PATCH 1/3] qemu: Need to assign PCI address to vhost-scsi

2017-12-13 Thread Eric Farman
On 12/12/2017 09:22 AM, John Ferlan wrote: Commit id '70249927b' neglected to cover this case because the test had taken the "shortcut" to already add the , so I altered that too just to make sure it doesn't happen again. My bad. :( Previously the qemuxml2xmloutdata was a softlink to the

[libvirt] [PATCH 1/3] qemu: Need to assign PCI address to vhost-scsi

2017-12-12 Thread John Ferlan
Commit id '70249927b' neglected to cover this case because the test had taken the "shortcut" to already add the , so I altered that too just to make sure it doesn't happen again. Previously the qemuxml2xmloutdata was a softlink to the source qemuxml2argvdata, so I unlinked and recreated the output