[Qemu-devel] [PULL 0/6] Fixes 31 20181127 patches

Gerd Hoffmann posted 6 patches 6 years, 11 months ago
Test asan passed
Test checkpatch failed
Test docker-quick@centos7 passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20181127064932.7299-1-kraxel@redhat.com
hw/audio/fmopl.h        |  4 ++--
hw/audio/intel-hda.c    |  6 ++++++
hw/display/cirrus_vga.c |  7 ++++---
hw/input/ps2.c          |  2 +-
hw/usb/host-libusb.c    |  3 +++
vl.c                    |  6 ++++++
qapi/ui.json            | 13 +++++++++++++
7 files changed, 35 insertions(+), 6 deletions(-)
[Qemu-devel] [PULL 0/6] Fixes 31 20181127 patches
Posted by Gerd Hoffmann 6 years, 11 months ago
The following changes since commit 5298f4d67a911dd9cefa4c4185eed242074d64c2:

  Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging (2018-11-23 08:54:52 +0000)

are available in the git repository at:

  git://git.kraxel.org/qemu tags/fixes-31-20181127-pull-request

for you to fetch changes up to e1ca8f7e1915496148f6e0ce1f7c2309af013312:

  qapi: add query-display-options command (2018-11-27 07:47:57 +0100)

----------------------------------------------------------------
various bugfixes for 3.1: fmops, ps2, cirrus, hda, usb-host, qapi

----------------------------------------------------------------

Gerd Hoffmann (3):
  fmops: fix off-by-one in AR_TABLE and DR_TABLE array size
  audio/hda: fix guest triggerable assert
  qapi: add query-display-options command

Hervé Poussineau (1):
  ps2kbd: default to scan enabled after reset

Wang Xin (1):
  cirrus_vga/migration: update the bank offset before use

linzhecheng (1):
  usb-host: set ifs.detached as true if kernel driver is not active

 hw/audio/fmopl.h        |  4 ++--
 hw/audio/intel-hda.c    |  6 ++++++
 hw/display/cirrus_vga.c |  7 ++++---
 hw/input/ps2.c          |  2 +-
 hw/usb/host-libusb.c    |  3 +++
 vl.c                    |  6 ++++++
 qapi/ui.json            | 13 +++++++++++++
 7 files changed, 35 insertions(+), 6 deletions(-)

-- 
2.9.3


Re: [Qemu-devel] [PULL 0/6] Fixes 31 20181127 patches
Posted by Peter Maydell 6 years, 11 months ago
On Tue, 27 Nov 2018 at 06:51, Gerd Hoffmann <kraxel@redhat.com> wrote:
>
> The following changes since commit 5298f4d67a911dd9cefa4c4185eed242074d64c2:
>
>   Merge remote-tracking branch 'remotes/kevin/tags/for-upstream' into staging (2018-11-23 08:54:52 +0000)
>
> are available in the git repository at:
>
>   git://git.kraxel.org/qemu tags/fixes-31-20181127-pull-request
>
> for you to fetch changes up to e1ca8f7e1915496148f6e0ce1f7c2309af013312:
>
>   qapi: add query-display-options command (2018-11-27 07:47:57 +0100)
>
> ----------------------------------------------------------------
> various bugfixes for 3.1: fmops, ps2, cirrus, hda, usb-host, qapi
>
> ----------------------------------------------------------------

Applied, thanks.

-- PMM

Re: [Qemu-devel] [PULL 0/6] Fixes 31 20181127 patches
Posted by no-reply@patchew.org 6 years, 11 months ago
Hi,

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

Type: series
Subject: [Qemu-devel] [PULL 0/6] Fixes 31 20181127 patches
Message-id: 20181127064932.7299-1-kraxel@redhat.com

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

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

git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram

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
Switched to a new branch 'test'
47e8b7b qapi: add query-display-options command
b38268d usb-host: set ifs.detached as true if kernel driver is not active
36275b8 audio/hda: fix guest triggerable assert
a20816e cirrus_vga/migration: update the bank offset before use
93d3ea2 ps2kbd: default to scan enabled after reset
0d552dd fmops: fix off-by-one in AR_TABLE and DR_TABLE array size

=== OUTPUT BEGIN ===
Checking PATCH 1/6: fmops: fix off-by-one in AR_TABLE and DR_TABLE array size...
ERROR: code indent should never use tabs
#28: FILE: hw/audio/fmopl.h:75:
+^Iint32_t AR_TABLE[76];^I/* attack rate tables  */$

ERROR: code indent should never use tabs
#29: FILE: hw/audio/fmopl.h:76:
+^Iint32_t DR_TABLE[76];^I/* decay rate tables   */$

total: 2 errors, 0 warnings, 10 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/6: ps2kbd: default to scan enabled after reset...
Checking PATCH 3/6: cirrus_vga/migration: update the bank offset before use...
Checking PATCH 4/6: audio/hda: fix guest triggerable assert...
Checking PATCH 5/6: usb-host: set ifs.detached as true if kernel driver is not active...
Checking PATCH 6/6: qapi: add query-display-options command...
=== OUTPUT END ===

Test command exited with code: 1


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