[Qemu-devel] [PULL 0/5] target/hppa patch queue

Richard Henderson posted 5 patches 7 years, 1 month ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20170207023229.29431-1-rth@twiddle.net
Test checkpatch failed
Test docker passed
Test s390x passed
There is a newer version of this series
linux-user/hppa/target_syscall.h |  2 +-
linux-user/syscall.c             |  1 -
scripts/qemu-binfmt-conf.sh      |  9 +++++++--
target/hppa/gdbstub.c            |  1 +
target/hppa/translate.c          | 17 +++++------------
5 files changed, 14 insertions(+), 16 deletions(-)
[Qemu-devel] [PULL 0/5] target/hppa patch queue
Posted by Richard Henderson 7 years, 1 month ago
Nothing complicated here.  Two bugs from Laurent, two noticed
by Coverity (through multiple channels).


r~



The following changes since commit 7d2c6c95511e42dffe2b263275e09957723d0ff4:

  Merge remote-tracking branch 'remotes/kraxel/tags/pull-usb-20170206-1' into staging (2017-02-06 11:44:08 +0000)

are available in the git repository at:

  git://github.com/rth7680/qemu.git tags/pull-hppa-20170206

for you to fetch changes up to 6836a8fb96963a29f309d49873062bbca9ab0e28:

  target/hppa: Fix gdb_write_register (2017-02-06 18:25:31 -0800)

----------------------------------------------------------------
Misc hppa fixes.

----------------------------------------------------------------
Laurent Vivier (3):
      linux-user: add hppa magic numbers in qemu-binfmt-conf.sh
      linux-user: fix "apt-get update" on linux-user hppa
      linux-user: define correct UTS machine name for hppa

Richard Henderson (2):
      target/hppa: Tidy do_cbranch
      target/hppa: Fix gdb_write_register

 linux-user/hppa/target_syscall.h |  2 +-
 linux-user/syscall.c             |  1 -
 scripts/qemu-binfmt-conf.sh      |  9 +++++++--
 target/hppa/gdbstub.c            |  1 +
 target/hppa/translate.c          | 17 +++++------------
 5 files changed, 14 insertions(+), 16 deletions(-)

Re: [Qemu-devel] [PULL 0/5] target/hppa patch queue
Posted by Peter Maydell 7 years, 1 month ago
On 7 February 2017 at 02:32, Richard Henderson <rth@twiddle.net> wrote:
> Nothing complicated here.  Two bugs from Laurent, two noticed
> by Coverity (through multiple channels).
>
>
> r~
>
>
>
> The following changes since commit 7d2c6c95511e42dffe2b263275e09957723d0ff4:
>
>   Merge remote-tracking branch 'remotes/kraxel/tags/pull-usb-20170206-1' into staging (2017-02-06 11:44:08 +0000)
>
> are available in the git repository at:
>
>   git://github.com/rth7680/qemu.git tags/pull-hppa-20170206
>
> for you to fetch changes up to 6836a8fb96963a29f309d49873062bbca9ab0e28:
>
>   target/hppa: Fix gdb_write_register (2017-02-06 18:25:31 -0800)
>
> ----------------------------------------------------------------
> Misc hppa fixes.
>
> ----------------------------------------------------------------

Applied, thanks.

-- PMM

Re: [Qemu-devel] [PULL 0/5] target/hppa patch queue
Posted by no-reply@patchew.org 7 years, 1 month ago
Hi,

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

Type: series
Subject: [Qemu-devel] [PULL 0/5] target/hppa patch queue
Message-id: 20170207023229.29431-1-rth@twiddle.net

=== 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
 - [tag update]      patchew/1486110164-13797-1-git-send-email-peterx@redhat.com -> patchew/1486110164-13797-1-git-send-email-peterx@redhat.com
 * [new tag]         patchew/20170207023229.29431-1-rth@twiddle.net -> patchew/20170207023229.29431-1-rth@twiddle.net
Switched to a new branch 'test'
ce2db6b target/hppa: Fix gdb_write_register
dfb4b9c target/hppa: Tidy do_cbranch
b17ea53 linux-user: define correct UTS machine name for hppa
84e4ae2 linux-user: fix "apt-get update" on linux-user hppa
a2cb156 linux-user: add hppa magic numbers in qemu-binfmt-conf.sh

=== OUTPUT BEGIN ===
Checking PATCH 1/5: linux-user: add hppa magic numbers in qemu-binfmt-conf.sh...
ERROR: line over 90 characters
#36: FILE: scripts/qemu-binfmt-conf.sh:95:
+hppa_magic='\x7f\x45\x4c\x46\x01\x02\x01\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x02\x00\x0f'

ERROR: line over 90 characters
#37: FILE: scripts/qemu-binfmt-conf.sh:96:
+hppa_mask='\xff\xff\xff\xff\xff\xff\xff\x00\xff\xff\xff\xff\xff\xff\xff\xff\xff\xfe\xff\xff'

total: 2 errors, 0 warnings, 22 lines checked

Your patch has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

Checking PATCH 2/5: linux-user: fix "apt-get update" on linux-user hppa...
Checking PATCH 3/5: linux-user: define correct UTS machine name for hppa...
Checking PATCH 4/5: target/hppa: Tidy do_cbranch...
Checking PATCH 5/5: target/hppa: Fix gdb_write_register...
=== OUTPUT END ===

Test command exited with code: 1


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