[v1] Fixing several GCC 11 warnings

[RHEL7 qemu-kvm PATCH 0/3] Fixing several GCC 11 warnings

mrezanin@redhat.com posted 3 patches 3 years, 4 months ago

Diff against v2 v3
Download series mbox

Test checkpatch failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/cover.1610364304.git.mrezanin@redhat.com

Maintainers: Cornelia Huck <cohuck@redhat.com>, Thomas Huth <thuth@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Dmitry Fleytman <dmitry.fleytman@gmail.com>, Richard Henderson <richard.henderson@linaro.org>, David Hildenbrand <david@redhat.com>, Jason Wang <jasowang@redhat.com>, Halil Pasic <pasic@linux.ibm.com>, Christian Borntraeger <borntraeger@de.ibm.com>

There is a newer version of this series

net/eth.c                    | 3 +++
target/s390x/kvm.c           | 2 +-
target/s390x/misc_helper.c   | 4 +++-
tcg/aarch64/tcg-target.c.inc | 3 +--
tcg/sparc/tcg-target.c.inc   | 3 +--
5 files changed, 9 insertions(+), 6 deletions(-)

Expand all Fold all

[RHEL7 qemu-kvm PATCH 0/3] Fixing several GCC 11 warnings

Posted by mrezanin@redhat.com 3 years, 4 months ago

From: Miroslav Rezanina <mrezanin@redhat.com>

Compiling qemu using GCC 11 we got several new warnings. To allow
build with --enable-werror, we need to solve issues generating these
warnings.

Signed-of-by: Miroslav Rezanina <mrezanin@redhat.com>

Miroslav Rezanina (3):
  Fix net.c warning on GCC 11
  s390x: Fix vm name copy length
  Fix tcg_out_op argument mismatch warning

 net/eth.c                    | 3 +++
 target/s390x/kvm.c           | 2 +-
 target/s390x/misc_helper.c   | 4 +++-
 tcg/aarch64/tcg-target.c.inc | 3 +--
 tcg/sparc/tcg-target.c.inc   | 3 +--
 5 files changed, 9 insertions(+), 6 deletions(-)

-- 
2.18.4

Re: [RHEL7 qemu-kvm PATCH 0/3] Fixing several GCC 11 warnings

Posted by no-reply@patchew.org 3 years, 4 months ago

Patchew URL: https://patchew.org/QEMU/cover.1610364304.git.mrezanin@redhat.com/



Hi,

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

Type: series
Message-id: cover.1610364304.git.mrezanin@redhat.com
Subject: [RHEL7 qemu-kvm PATCH 0/3] Fixing several GCC 11 warnings

=== TEST SCRIPT BEGIN ===
#!/bin/bash
git rev-parse base > /dev/null || exit 0
git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram
./scripts/checkpatch.pl --mailback base..
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
 * [new tag]         patchew/cover.1610364304.git.mrezanin@redhat.com -> patchew/cover.1610364304.git.mrezanin@redhat.com
Switched to a new branch 'test'
c05a0c4 Fix tcg_out_op argument mismatch warning
a8b8ace s390x: Fix vm name copy length
b64e754 Fix net.c warning on GCC 11

=== OUTPUT BEGIN ===
1/3 Checking commit b64e754d193f (Fix net.c warning on GCC 11)
2/3 Checking commit a8b8ace484f8 (s390x: Fix vm name copy length)
ERROR: trailing whitespace
#36: FILE: target/s390x/misc_helper.c:372:
+^I^Imemset((char *)sysib.sysib_322.ext_names[0], 0, $

ERROR: code indent should never use tabs
#36: FILE: target/s390x/misc_helper.c:372:
+^I^Imemset((char *)sysib.sysib_322.ext_names[0], 0, $

ERROR: code indent should never use tabs
#37: FILE: target/s390x/misc_helper.c:373:
+^I^I       sizeof(sysib.sysib_322.ext_names[0]));$

total: 3 errors, 0 warnings, 19 lines checked

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

3/3 Checking commit c05a0c4ed447 (Fix tcg_out_op argument mismatch warning)
ERROR: Missing Signed-off-by: line(s)

total: 1 errors, 0 warnings, 18 lines checked

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

=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/cover.1610364304.git.mrezanin@redhat.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com