[Qemu-devel] [PULL v2 00/14] Linux user for upstream patches

Laurent Vivier posted 14 patches 7 years, 2 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170216143816.2384-1-laurent@vivier.eu
Test checkpatch failed
Test docker failed
Test s390x failed
linux-user/elfload.c                     | 22 +++++++++++++++-------
linux-user/host/s390x/safe-syscall.inc.S |  2 +-
linux-user/ioctls.h                      |  5 +++++
linux-user/m68k/syscall_nr.h             |  3 +++
linux-user/mmap.c                        |  9 +++------
linux-user/sh4/syscall_nr.h              | 14 ++++++++++++++
linux-user/signal.c                      |  5 +++++
linux-user/strace.c                      |  2 --
linux-user/syscall.c                     | 26 +++++++++++++++++++-------
linux-user/syscall_defs.h                |  4 ++++
linux-user/syscall_types.h               |  6 ++++++
11 files changed, 75 insertions(+), 23 deletions(-)
[Qemu-devel] [PULL v2 00/14] Linux user for upstream patches
Posted by Laurent Vivier 7 years, 2 months ago
The following changes since commit 5dae13cd71f0755a1395b5a4cde635b8a6ee3f58:

  Merge remote-tracking branch 'remotes/rth/tags/pull-or-20170214' into staging (2017-02-14 09:55:48 +0000)

are available in the git repository at:

  git://github.com/vivier/qemu.git tags/linux-user-for-upstream-pull-request

for you to fetch changes up to 21992cb6794a5f8edb0cee01bdfe0b03d0438dac:

  linux-user: Add FICLONE and FICLONERANGE ioctls (2017-02-16 15:29:30 +0100)

----------------------------------------------------------------
To help Riku in his maintainer role of linux-user, I've proposed
to help him to manage this pull request.

This series is based on his branch linux-user-for-upstream at:

    https://git.linaro.org/people/riku.voipio/qemu.git

I have added some reviewed patches from the mailing list since
the last pull request.

I have tested this series by creating some containers:

ppc64   fedora-21
ppc     debian-8
aarch64 centos-7.2

All these containers can boot and run as expected
(create an user, log in as this user, install packages with yum/dnf/apt-get
using sudo)

I have tested the LTP-20170116 in the ppc container (big-endian 32bit)
on an x86_64 host (little-endian 64bit). The result is: 

Total Tests: 1111
Total Skipped Tests: 174
Total Failures: 68
Kernel Version: 4.9.5-200.fc25.x86_64
Machine Architecture: ppc
Hostname: debian-8-ppc

v2:
    Fix John Paul Adrian Glaubitz author name
----------------------------------------------------------------

Helge Deller (1):
  linux-user: Add FICLONE and FICLONERANGE ioctls

John Paul Adrian Glaubitz (2):
  linux-user: Update sh4 syscall definitions to match Linux 4.8
  linux-user: Update m68k syscall definitions to match Linux 4.6

Jose Ricardo Ziviani (1):
  linux-user: fill target sigcontext struct accordingly

Laurent Vivier (1):
  linux-user: manage two new IFLA host message types

Lena Djokic (3):
  linux-user: Fix inotify_init1 support
  linux-user: Fix readahead
  linux-user: Fix mq_open

Marc-André Lureau (2):
  linux-user: fix settime old value location
  linux-user: fix tcg/mmap test

Peter Maydell (1):
  linux-user: Use correct types in load_symbols()

Richard Henderson (1):
  linux-user: Fix s390x safe-syscall for z900

Riku Voipio (2):
  linux-user: remove ifdef __USER_MISC
  linux-user: drop __cygwin__ ifdef

 linux-user/elfload.c                     | 22 +++++++++++++++-------
 linux-user/host/s390x/safe-syscall.inc.S |  2 +-
 linux-user/ioctls.h                      |  5 +++++
 linux-user/m68k/syscall_nr.h             |  3 +++
 linux-user/mmap.c                        |  9 +++------
 linux-user/sh4/syscall_nr.h              | 14 ++++++++++++++
 linux-user/signal.c                      |  5 +++++
 linux-user/strace.c                      |  2 --
 linux-user/syscall.c                     | 26 +++++++++++++++++++-------
 linux-user/syscall_defs.h                |  4 ++++
 linux-user/syscall_types.h               |  6 ++++++
 11 files changed, 75 insertions(+), 23 deletions(-)

-- 
2.9.3


Re: [Qemu-devel] [PULL v2 00/14] Linux user for upstream patches
Posted by Peter Maydell 7 years, 2 months ago
On 16 February 2017 at 14:38, Laurent Vivier <laurent@vivier.eu> wrote:
> The following changes since commit 5dae13cd71f0755a1395b5a4cde635b8a6ee3f58:
>
>   Merge remote-tracking branch 'remotes/rth/tags/pull-or-20170214' into staging (2017-02-14 09:55:48 +0000)
>
> are available in the git repository at:
>
>   git://github.com/vivier/qemu.git tags/linux-user-for-upstream-pull-request
>
> for you to fetch changes up to 21992cb6794a5f8edb0cee01bdfe0b03d0438dac:
>
>   linux-user: Add FICLONE and FICLONERANGE ioctls (2017-02-16 15:29:30 +0100)
>
> ----------------------------------------------------------------
> To help Riku in his maintainer role of linux-user, I've proposed
> to help him to manage this pull request.
>
> This series is based on his branch linux-user-for-upstream at:
>
>     https://git.linaro.org/people/riku.voipio/qemu.git
>
> I have added some reviewed patches from the mailing list since
> the last pull request.
>
> I have tested this series by creating some containers:
>
> ppc64   fedora-21
> ppc     debian-8
> aarch64 centos-7.2
>
> All these containers can boot and run as expected
> (create an user, log in as this user, install packages with yum/dnf/apt-get
> using sudo)
>
> I have tested the LTP-20170116 in the ppc container (big-endian 32bit)
> on an x86_64 host (little-endian 64bit). The result is:
>
> Total Tests: 1111
> Total Skipped Tests: 174
> Total Failures: 68
> Kernel Version: 4.9.5-200.fc25.x86_64
> Machine Architecture: ppc
> Hostname: debian-8-ppc
>
> v2:
>     Fix John Paul Adrian Glaubitz author name
> ----------------------------------------------------------------

Sorry, I applied v1 before I noticed you'd done this respin :-(

thanks
-- PMM

Re: [Qemu-devel] [PULL v2 00/14] Linux user for upstream patches
Posted by Laurent Vivier 7 years, 2 months ago
Le 16/02/2017 à 17:06, Peter Maydell a écrit :
> On 16 February 2017 at 14:38, Laurent Vivier <laurent@vivier.eu> wrote:
>> The following changes since commit 5dae13cd71f0755a1395b5a4cde635b8a6ee3f58:
>>
>>   Merge remote-tracking branch 'remotes/rth/tags/pull-or-20170214' into staging (2017-02-14 09:55:48 +0000)
>>
>> are available in the git repository at:
>>
>>   git://github.com/vivier/qemu.git tags/linux-user-for-upstream-pull-request
>>
>> for you to fetch changes up to 21992cb6794a5f8edb0cee01bdfe0b03d0438dac:
>>
>>   linux-user: Add FICLONE and FICLONERANGE ioctls (2017-02-16 15:29:30 +0100)
>>
>> ----------------------------------------------------------------
>> To help Riku in his maintainer role of linux-user, I've proposed
>> to help him to manage this pull request.
>>
>> This series is based on his branch linux-user-for-upstream at:
>>
>>     https://git.linaro.org/people/riku.voipio/qemu.git
>>
>> I have added some reviewed patches from the mailing list since
>> the last pull request.
>>
>> I have tested this series by creating some containers:
>>
>> ppc64   fedora-21
>> ppc     debian-8
>> aarch64 centos-7.2
>>
>> All these containers can boot and run as expected
>> (create an user, log in as this user, install packages with yum/dnf/apt-get
>> using sudo)
>>
>> I have tested the LTP-20170116 in the ppc container (big-endian 32bit)
>> on an x86_64 host (little-endian 64bit). The result is:
>>
>> Total Tests: 1111
>> Total Skipped Tests: 174
>> Total Failures: 68
>> Kernel Version: 4.9.5-200.fc25.x86_64
>> Machine Architecture: ppc
>> Hostname: debian-8-ppc
>>
>> v2:
>>     Fix John Paul Adrian Glaubitz author name
>> ----------------------------------------------------------------
> 
> Sorry, I applied v1 before I noticed you'd done this respin :-(

It's fine: the tag was the same, and it seems you've applied it after
I've pushed the v2 to the repo.

The author name is correct:

    Author: John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de>

Thanks,
Laurent



Re: [Qemu-devel] [PULL v2 00/14] Linux user for upstream patches
Posted by no-reply@patchew.org 7 years, 2 months ago
Hi,

This series seems to have some coding style problems. See output below for
more information:

Type: series
Subject: [Qemu-devel] [PULL v2 00/14] Linux user for upstream patches
Message-id: 20170216143816.2384-1-laurent@vivier.eu

=== TEST SCRIPT BEGIN ===
#!/bin/bash

BASE=base
n=1
total=$(git log --oneline $BASE.. | wc -l)
failed=0

# Useful git options
git config --local diff.renamelimit 0
git config --local diff.renames True

commits="$(git log --format=%H --reverse $BASE..)"
for c in $commits; do
    echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..."
    if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then
        failed=1
        echo
    fi
    n=$((n+1))
done

exit $failed
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
 * [new tag]         patchew/20170216143816.2384-1-laurent@vivier.eu -> patchew/20170216143816.2384-1-laurent@vivier.eu
Switched to a new branch 'test'
fatal: Not a valid branch point: '7a37b59f1d28e4b4c0f99f49aa305a59986f6643'.
Traceback (most recent call last):
  File "/usr/bin/patchew", line 410, in test_one
    cwd=clone, stdout=logf, stderr=logf)
  File "/usr/lib64/python3.4/subprocess.py", line 561, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['git', 'branch', 'base', '7a37b59f1d28e4b4c0f99f49aa305a59986f6643']' returned non-zero exit status 128



---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-devel@freelists.org
Re: [Qemu-devel] [PULL v2 00/14] Linux user for upstream patches
Posted by no-reply@patchew.org 7 years, 2 months ago
Hi,

This series failed automatic build test. Please find the testing commands and
their output below. If you have docker installed, you can probably reproduce it
locally.

Type: series
Subject: [Qemu-devel] [PULL v2 00/14] Linux user for upstream patches
Message-id: 20170216143816.2384-1-laurent@vivier.eu

=== TEST SCRIPT BEGIN ===
#!/bin/bash
set -e
git submodule update --init dtc
# Let docker tests dump environment info
export SHOW_ENV=1
export J=16
make docker-test-quick@centos6
make docker-test-mingw@fedora
make docker-test-build@min-glib
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
Switched to a new branch 'test'
fatal: Not a valid branch point: '7a37b59f1d28e4b4c0f99f49aa305a59986f6643'.
Traceback (most recent call last):
  File "/usr/bin/patchew", line 410, in test_one
    cwd=clone, stdout=logf, stderr=logf)
  File "/usr/lib64/python3.4/subprocess.py", line 561, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['git', 'branch', 'base', '7a37b59f1d28e4b4c0f99f49aa305a59986f6643']' returned non-zero exit status 128



---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-devel@freelists.org