Re: [libvirt] [PATCH 0/3] patches necessary to make the parse vfio in commandline work
On 06/06/2014 05:32 PM, Eric Blake wrote: On 06/06/2014 07:54 AM, Laine Stump wrote: Patches 1/3 and 2/3 are prerequisites the the patch that started this thread. Patch 3/3 should be squashed into the original patch. I also noticed that the original patch causes all unrecognized -device options to now be ignored rather than being added to the qemu namespace (with a warning). This needs to be fixed before resubmitting that patch too, but I didn't have the time/interest to do it. (If needed/desired, all three of these new patches can be pushed separately before the patch at the top of this thread). So for 3/3, which is it? Push now, or squash into the respin of the patch that started this thread? I think it's too small, and not of any use by itself, so I recommended squashing it into the new version of the original patch that adds vfio-pci parsing. -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/3] patches necessary to make the parse vfio in commandline work
On 06/06/2014 04:54 PM, Laine Stump wrote: Patches 1/3 and 2/3 are prerequisites the the patch that started this thread. Patch 3/3 should be squashed into the original patch. I also noticed that the original patch causes all unrecognized -device options to now be ignored rather than being added to the qemu namespace (with a warning). This needs to be fixed before resubmitting that patch too, but I didn't have the time/interest to do it. (If needed/desired, all three of these new patches can be pushed separately before the patch at the top of this thread). Laine Stump (3): test: display qemuParseCommandline warnings when VIR_TEST_DEBUG 0 qemu: ignore -nodefconfig and -nodefaults in qemuParseCommandLineString test: make hostdev-vfio test able to pass qemuargv2xmltest src/qemu/qemu_command.c| 4 +- tests/qemuargv2xmltest.c | 45 -- .../qemuxml2argvdata/qemuxml2argv-hostdev-vfio.xml | 1 + 3 files changed, 38 insertions(+), 12 deletions(-) I pushed 1/3 and 2/3, and have sent another patch to add parsing for memballoon (which is also required for all the tests to patch once the original is done correctly). I left 3/3 to be squashed into the original patch of the thread. Thanks for the reviews! -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
[libvirt] [PATCH 0/3] patches necessary to make the parse vfio in commandline work
Patches 1/3 and 2/3 are prerequisites the the patch that started this thread. Patch 3/3 should be squashed into the original patch. I also noticed that the original patch causes all unrecognized -device options to now be ignored rather than being added to the qemu namespace (with a warning). This needs to be fixed before resubmitting that patch too, but I didn't have the time/interest to do it. (If needed/desired, all three of these new patches can be pushed separately before the patch at the top of this thread). Laine Stump (3): test: display qemuParseCommandline warnings when VIR_TEST_DEBUG 0 qemu: ignore -nodefconfig and -nodefaults in qemuParseCommandLineString test: make hostdev-vfio test able to pass qemuargv2xmltest src/qemu/qemu_command.c| 4 +- tests/qemuargv2xmltest.c | 45 -- .../qemuxml2argvdata/qemuxml2argv-hostdev-vfio.xml | 1 + 3 files changed, 38 insertions(+), 12 deletions(-) -- 1.9.3 -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH 0/3] patches necessary to make the parse vfio in commandline work
On 06/06/2014 07:54 AM, Laine Stump wrote: Patches 1/3 and 2/3 are prerequisites the the patch that started this thread. Patch 3/3 should be squashed into the original patch. I also noticed that the original patch causes all unrecognized -device options to now be ignored rather than being added to the qemu namespace (with a warning). This needs to be fixed before resubmitting that patch too, but I didn't have the time/interest to do it. (If needed/desired, all three of these new patches can be pushed separately before the patch at the top of this thread). So for 3/3, which is it? Push now, or squash into the respin of the patch that started this thread? Laine Stump (3): test: display qemuParseCommandline warnings when VIR_TEST_DEBUG 0 qemu: ignore -nodefconfig and -nodefaults in qemuParseCommandLineString test: make hostdev-vfio test able to pass qemuargv2xmltest src/qemu/qemu_command.c| 4 +- tests/qemuargv2xmltest.c | 45 -- .../qemuxml2argvdata/qemuxml2argv-hostdev-vfio.xml | 1 + 3 files changed, 38 insertions(+), 12 deletions(-) -- Eric Blake eblake redhat com+1-919-301-3266 Libvirt virtualization library http://libvirt.org signature.asc Description: OpenPGP digital signature -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list
