[PATCH 0/6] Enable Travis builds on arm64, ppc64le and s390x

Thomas Huth posted 6 patches 4 years, 5 months ago
Test asan passed
Test checkpatch passed
Test FreeBSD passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu passed
Test docker-quick@centos7 passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20191119170822.45649-1-thuth@redhat.com
Maintainers: Thomas Huth <thuth@redhat.com>, "Philippe Mathieu-Daudé" <philmd@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>, Fam Zheng <fam@euphon.net>, Laurent Vivier <lvivier@redhat.com>, Max Reitz <mreitz@redhat.com>, "Daniel P. Berrangé" <berrange@redhat.com>, Cornelia Huck <cohuck@redhat.com>, Kevin Wolf <kwolf@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>
There is a newer version of this series
.travis.yml                   | 85 ++++++++++++++++++++++++++++++++++-
tests/hd-geo-test.c           | 12 ++++-
tests/qemu-iotests/060        |  6 +++
tests/qemu-iotests/079        |  6 +++
tests/test-util-filemonitor.c | 11 +++++
5 files changed, 118 insertions(+), 2 deletions(-)
[PATCH 0/6] Enable Travis builds on arm64, ppc64le and s390x
Posted by Thomas Huth 4 years, 5 months ago
Travis recently added build hosts for arm64, ppc64le and s390x, so
this is a welcome addition to our Travis testing matrix.

Unfortunately, the builds are running in quite restricted LXD containers
there, for example it is not possible to create huge files there (even
if they are just sparse), and certain system calls are blocked. So we
have to change some tests first to stop them failing in such environments.

I also included Alex' patch for dropping some targets from the
MAIN_SOFTMMU_TARGETS config variable, since at least on arm64, the
build process seems to be rather slow and sometimes hits the 50 minutes
timeout otherwise.

For the curious, here's a test run (non-x86 at the end of the page):

 https://travis-ci.com/huth/qemu/jobs/258095792

 Thomas


Alex Bennée (1):
  travis.yml: drop 32 bit systems from MAIN_SOFTMMU_TARGETS

Thomas Huth (5):
  iotests: Skip test 060 if it is not possible to create large files
  iotests: Skip test 079 if it is not possible to create large files
  tests/hd-geo-test: Skip test when images can not be created
  tests/test-util-filemonitor: Skip test on non-x86 Travis containers
  travis.yml: Enable builds on arm64, ppc64le and s390x

 .travis.yml                   | 85 ++++++++++++++++++++++++++++++++++-
 tests/hd-geo-test.c           | 12 ++++-
 tests/qemu-iotests/060        |  6 +++
 tests/qemu-iotests/079        |  6 +++
 tests/test-util-filemonitor.c | 11 +++++
 5 files changed, 118 insertions(+), 2 deletions(-)

-- 
2.23.0


Re: [PATCH 0/6] Enable Travis builds on arm64, ppc64le and s390x
Posted by Alex Bennée 4 years, 4 months ago
Thomas Huth <thuth@redhat.com> writes:

> Travis recently added build hosts for arm64, ppc64le and s390x, so
> this is a welcome addition to our Travis testing matrix.
>
> Unfortunately, the builds are running in quite restricted LXD containers
> there, for example it is not possible to create huge files there (even
> if they are just sparse), and certain system calls are blocked. So we
> have to change some tests first to stop them failing in such environments.
<snip>
>   iotests: Skip test 060 if it is not possible to create large files
>   iotests: Skip test 079 if it is not possible to create large files

It seems like 161 is also failing:

  https://travis-ci.org/stsquad/qemu/jobs/615672478


>   tests/hd-geo-test: Skip test when images can not be created
>   tests/test-util-filemonitor: Skip test on non-x86 Travis containers
>   travis.yml: Enable builds on arm64, ppc64le and s390x
>
>  .travis.yml                   | 85 ++++++++++++++++++++++++++++++++++-
>  tests/hd-geo-test.c           | 12 ++++-
>  tests/qemu-iotests/060        |  6 +++
>  tests/qemu-iotests/079        |  6 +++
>  tests/test-util-filemonitor.c | 11 +++++
>  5 files changed, 118 insertions(+), 2 deletions(-)


-- 
Alex Bennée

Re: [PATCH 0/6] Enable Travis builds on arm64, ppc64le and s390x
Posted by Alex Bennée 4 years, 4 months ago
Alex Bennée <alex.bennee@linaro.org> writes:

> Thomas Huth <thuth@redhat.com> writes:
>
>> Travis recently added build hosts for arm64, ppc64le and s390x, so
>> this is a welcome addition to our Travis testing matrix.
>>
>> Unfortunately, the builds are running in quite restricted LXD containers
>> there, for example it is not possible to create huge files there (even
>> if they are just sparse), and certain system calls are blocked. So we
>> have to change some tests first to stop them failing in such environments.
> <snip>
>>   iotests: Skip test 060 if it is not possible to create large files
>>   iotests: Skip test 079 if it is not possible to create large files
>
> It seems like 161 is also failing:
>
>   https://travis-ci.org/stsquad/qemu/jobs/615672478

And sometimes 249

>
>
>>   tests/hd-geo-test: Skip test when images can not be created
>>   tests/test-util-filemonitor: Skip test on non-x86 Travis containers
>>   travis.yml: Enable builds on arm64, ppc64le and s390x
>>
>>  .travis.yml                   | 85 ++++++++++++++++++++++++++++++++++-
>>  tests/hd-geo-test.c           | 12 ++++-
>>  tests/qemu-iotests/060        |  6 +++
>>  tests/qemu-iotests/079        |  6 +++
>>  tests/test-util-filemonitor.c | 11 +++++
>>  5 files changed, 118 insertions(+), 2 deletions(-)


-- 
Alex Bennée

Re: [PATCH 0/6] Enable Travis builds on arm64, ppc64le and s390x
Posted by Thomas Huth 4 years, 4 months ago
On 25/11/2019 11.28, Alex Bennée wrote:
> 
> Alex Bennée <alex.bennee@linaro.org> writes:
> 
>> Thomas Huth <thuth@redhat.com> writes:
>>
>>> Travis recently added build hosts for arm64, ppc64le and s390x, so
>>> this is a welcome addition to our Travis testing matrix.
>>>
>>> Unfortunately, the builds are running in quite restricted LXD containers
>>> there, for example it is not possible to create huge files there (even
>>> if they are just sparse), and certain system calls are blocked. So we
>>> have to change some tests first to stop them failing in such environments.
>> <snip>
>>>    iotests: Skip test 060 if it is not possible to create large files
>>>    iotests: Skip test 079 if it is not possible to create large files
>>
>> It seems like 161 is also failing:
>>
>>    https://travis-ci.org/stsquad/qemu/jobs/615672478
> 
> And sometimes 249

These must be intermittent problems ... I've seen 161 failing once at 
the very beginning of my tests, but then never again, so I assumed that 
it was a quirk with the test system that got fixed later. Seems like 
that was a wrong assumption. I've never seen 249 failing so far... I'll 
try to do some more tests when I've got some spare time...

  Thomas


Re: [PATCH 0/6] Enable Travis builds on arm64, ppc64le and s390x
Posted by Thomas Huth 4 years, 4 months ago
On 27/11/2019 09.50, Thomas Huth wrote:
> On 25/11/2019 11.28, Alex Bennée wrote:
>>
>> Alex Bennée <alex.bennee@linaro.org> writes:
>>
>>> Thomas Huth <thuth@redhat.com> writes:
>>>
>>>> Travis recently added build hosts for arm64, ppc64le and s390x, so
>>>> this is a welcome addition to our Travis testing matrix.
>>>>
>>>> Unfortunately, the builds are running in quite restricted LXD
>>>> containers
>>>> there, for example it is not possible to create huge files there (even
>>>> if they are just sparse), and certain system calls are blocked. So we
>>>> have to change some tests first to stop them failing in such
>>>> environments.
>>> <snip>
>>>>    iotests: Skip test 060 if it is not possible to create large files
>>>>    iotests: Skip test 079 if it is not possible to create large files
>>>
>>> It seems like 161 is also failing:
>>>
>>>    https://travis-ci.org/stsquad/qemu/jobs/615672478
>>
>> And sometimes 249
> 
> These must be intermittent problems ... I've seen 161 failing once at
> the very beginning of my tests, but then never again, so I assumed that
> it was a quirk with the test system that got fixed later. Seems like
> that was a wrong assumption. I've never seen 249 failing so far... I'll
> try to do some more tests when I've got some spare time...

It's not intermittent, it's a problem with "dist: bionic". It seems to
work fine with "dist: xenial", as far as I can tell. So I think we
should simply stick with "dist: xenial" for the ppc64le builder unless
someone has a ppc64le bionic system at hand for debugging.

 Thomas