qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [Qemu-ppc] [PATCH 3/4] tests/pxe-test: Test net booting


From: Thomas Huth
Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH 3/4] tests/pxe-test: Test net booting over IPv6 in some cases
Date: Fri, 15 Dec 2017 15:15:45 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0

On 15.12.2017 13:56, David Gibson wrote:
> On Fri, Dec 15, 2017 at 12:10:56PM +0100, Thomas Huth wrote:
>> On 15.12.2017 11:16, David Gibson wrote:
>>> This adds IPv6 net boot testing (in addition to IPv4) when in slow test
>>> mode on ppc64 or s390.  IPv6 PXE doesn't seem to work on x86, I'm guessing
>>> out BIOS image doesn't support it.
>>>
>>> Signed-off-by: David Gibson <address@hidden>
>>> ---
>>>  tests/pxe-test.c | 26 ++++++++++++++++++++------
>>>  1 file changed, 20 insertions(+), 6 deletions(-)
>> [...]
>>>      } else if (strcmp(arch, "ppc64") == 0) {
>>> -        test_batch(ppc64_tests);
>>> +        test_batch(ppc64_tests, g_test_slow());
>>>          if (g_test_slow()) {
>>> -            test_batch(ppc64_tests_slow);
>>> +            test_batch(ppc64_tests_slow, true);
>>>          }
>>
>> The NICs from ppc64_tests_slow are now never exercised with IPv4 ...
>> maybe it would be better to do both:
> 
> Yes they are: the ipv6 parameter to test_batch() makes it do IPv6
> tests in addition to IPv4, unlike test_pxe_one() which just does one
> or the other.

Ah, right, my bad, I mixed that up. And since we do not have any machine
that can only do IPv6 but not IPv4, it's fine that test_batch() does
both, so never mind, please ignore my above comment.

 Thomas

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]