[0/3] s390x/cpumodel fixes for 4.1

Message

Christian Borntraeger July 15, 2019, 2:23 p.m. UTC

Some fallout of the gen15 cpu model. As this is new in 4.1
it is still time to fixup some aspects.

Christian Borntraeger (3):
  s390x/cpumodel: remove esort from the default model
  s390x/cpumodel: also change name of vxbeh
  s390x/cpumodel: change internal name of vxp to make description

 target/s390x/cpu_features_def.inc.h | 2 +-
 target/s390x/gen-features.c         | 5 ++---
 2 files changed, 3 insertions(+), 4 deletions(-)

Comments

Cornelia Huck July 16, 2019, 9:36 a.m. UTC | #1

On Mon, 15 Jul 2019 16:23:01 +0200
Christian Borntraeger <[e-mail]> wrote:

> Some fallout of the gen15 cpu model. As this is new in 4.1
> it is still time to fixup some aspects.
> 
> Christian Borntraeger (3):
>   s390x/cpumodel: remove esort from the default model
>   s390x/cpumodel: also change name of vxbeh
>   s390x/cpumodel: change internal name of vxp to make description
> 
>  target/s390x/cpu_features_def.inc.h | 2 +-
>  target/s390x/gen-features.c         | 5 ++---
>  2 files changed, 3 insertions(+), 4 deletions(-)
> 

Thanks, queued to s390-fixes (with s/vxp/vxpdeh/).

[e-mail] July 16, 2019, 10:51 a.m. UTC | #2

Patchew URL: https://patchew.org/QEMU/[e-mail]/



Hi,

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

Message-id: [e-mail]
Type: series
Subject: [Qemu-devel] [PATCH 0/3] s390x/cpumodel fixes for 4.1

=== 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 ===

From https://github.com/patchew-project/qemu
 t [tag update]            patchew/[e-mail] -> patchew/[e-mail]
Switched to a new branch 'test'
7bc436d3ed s390x/cpumodel: change internal name of vxp to make description
6b0474bbb3 s390x/cpumodel: also change name of vxbeh
2a3445555a s390x/cpumodel: remove esort from the default model

=== OUTPUT BEGIN ===
1/3 Checking commit 2a3445555a55 (s390x/cpumodel: remove esort from the default model)
2/3 Checking commit 6b0474bbb3ec (s390x/cpumodel: also change name of vxbeh)
WARNING: line over 80 characters
#29: FILE: target/s390x/cpu_features_def.inc.h:107:
+DEF_FEAT(VECTOR_BCD_ENH, "vxp", STFL, 152, "Vector-Packed-Decimal-Enhancement Facility")

total: 0 errors, 1 warnings, 8 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 7bc436d3eda4 (s390x/cpumodel: change internal name of vxp to make description)
ERROR: line over 90 characters
#22: FILE: target/s390x/cpu_features_def.inc.h:107:
+DEF_FEAT(VECTOR_PACKED_DECIMAL_ENH, "vxp", STFL, 152, "Vector-Packed-Decimal-Enhancement Facility")

total: 1 errors, 0 warnings, 24 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/[e-mail]/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to [e-mail]