[libvirt] [PATCH 2/2] V2 Modify generic ethernet interface so it will work when sVirt is enabled with qemu
Tyler Coumbes
coumbes at gmail.com
Sat Oct 29 02:59:51 UTC 2011
On Thu, Oct 27, 2011 at 10:25 PM, Eric Blake <eblake at redhat.com> wrote:
> On 10/24/2011 05:44 PM, Tyler Coumbes wrote:
>>
>> This patch makes the changes to the generic ethernet interface for
>> QEMU. Allowing it to be used with sVirt enabled.
>>
>> src/qemu/qemu_command.c | 79
>> ++++++++++++++++++++++++++++++++++++++++++++--
>> src/qemu/qemu_command.h | 4 ++
>> src/qemu/qemu_hotplug.c | 15 +++++++++
>> src/qemu/qemu_process.c | 13 ++++++++
>> 4 files changed, 107 insertions(+), 4 deletions(-)
>
> I haven't reviewed this one closely yet, but it would help if you added a
> test in tests/qemuxml2argvtest.c and a new pair of files in
> tests/qemuxml2argvdata/* that exercise the code to prove the new command
> line options look sane.
>
Two questions about creating a test for this code. How do I get the
actual args so I can compare them to my expected args to see what
isn't matching up? VIR_TEST_DEBUG=2 shows me what tests fail but not
any detail about why. I think it is because one of the parameters to
qemu is a file descriptor which could change for each run. Is there
some type of wildcard I can use like fd=%. I don't see any other tests
using file descriptors to compare to and am not finding anything in
any documentation I can find.
Thanks,
Tyler
More information about the libvir-list
mailing list