Patchew URL: 
https://patchew.org/QEMU/20200918204714.27276-1-ehabk...@redhat.com/



Hi,

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

Type: series
Message-id: 20200918204714.27276-1-ehabk...@redhat.com
Subject: [PULL 00/17] QOM queue, 2020-09-18

=== 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
Switched to a new branch 'test'
74e8c80 Use OBJECT_DECLARE_SIMPLE_TYPE when possible
15a16fe Use OBJECT_DECLARE_TYPE when possible
145c405 qom: Remove module_obj_name parameter from OBJECT_DECLARE* macros
580edfa qom: Remove ParentClassType argument from OBJECT_DECLARE_SIMPLE_TYPE
7edd65a scripts/codeconverter: Update to latest version
668fa4d target/s390x: Set instance_align on S390CPU TypeInfo
200d65f target/riscv: Set instance_align on RISCVCPU TypeInfo
0dc3472 target/ppc: Set instance_align on PowerPCCPU TypeInfo
9b1a0bc target/arm: Set instance_align on CPUARM TypeInfo
121003c qom: Allow objects to be allocated with increased alignment
1967925 qom: Correct error values in two contracts
862522d qom: Clean up object_property_get_enum()'s error value
5363814 qom: Correct object_class_dynamic_cast_assert() documentation
c3cbbe1 sifive: Use DECLARE_*CHECKER* macros
bb31b29 sifive: Move QOM typedefs and add missing includes
1bfe9f9 sifive_u: Rename memmap enum constants
20307d4 sifive_e: Rename memmap enum constants

=== OUTPUT BEGIN ===
1/17 Checking commit 20307d4c2db9 (sifive_e: Rename memmap enum constants)
WARNING: line over 80 characters
#86: FILE: hw/riscv/sifive_e.c:114:
+                          memmap[SIFIVE_E_DEV_MROM].base, 
&address_space_memory);

WARNING: line over 80 characters
#131: FILE: hw/riscv/sifive_e.c:229:
+    sysbus_mmio_map(SYS_BUS_DEVICE(&s->gpio), 0, 
memmap[SIFIVE_E_DEV_GPIO0].base);

total: 0 errors, 2 warnings, 186 lines checked

Patch 1/17 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
2/17 Checking commit 1bfe9f9eccb5 (sifive_u: Rename memmap enum constants)
WARNING: line over 80 characters
#303: FILE: hw/riscv/sifive_u.c:531:
+                          memmap[SIFIVE_U_DEV_MROM].base, 
&address_space_memory);

WARNING: line over 80 characters
#367: FILE: hw/riscv/sifive_u.c:734:
+    sysbus_mmio_map(SYS_BUS_DEVICE(&s->prci), 0, 
memmap[SIFIVE_U_DEV_PRCI].base);

WARNING: line over 80 characters
#374: FILE: hw/riscv/sifive_u.c:740:
+    sysbus_mmio_map(SYS_BUS_DEVICE(&s->gpio), 0, 
memmap[SIFIVE_U_DEV_GPIO].base);

total: 0 errors, 3 warnings, 412 lines checked

Patch 2/17 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
3/17 Checking commit bb31b295487b (sifive: Move QOM typedefs and add missing 
includes)
4/17 Checking commit c3cbbe15fbe1 (sifive: Use DECLARE_*CHECKER* macros)
5/17 Checking commit 5363814b4203 (qom: Correct 
object_class_dynamic_cast_assert() documentation)
6/17 Checking commit 862522d76ecb (qom: Clean up object_property_get_enum()'s 
error value)
7/17 Checking commit 1967925fb215 (qom: Correct error values in two contracts)
8/17 Checking commit 121003cf6782 (qom: Allow objects to be allocated with 
increased alignment)
WARNING: architecture specific defines should be avoided
#74: FILE: qom/object.c:694:
+#if __STDC_VERSION__ >= 201112L

total: 0 errors, 1 warnings, 84 lines checked

Patch 8/17 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
9/17 Checking commit 9b1a0bce035d (target/arm: Set instance_align on CPUARM 
TypeInfo)
10/17 Checking commit 0dc3472929b0 (target/ppc: Set instance_align on 
PowerPCCPU TypeInfo)
11/17 Checking commit 200d65fb6653 (target/riscv: Set instance_align on 
RISCVCPU TypeInfo)
12/17 Checking commit 668fa4dcfc21 (target/s390x: Set instance_align on S390CPU 
TypeInfo)
13/17 Checking commit 7edd65a82b36 (scripts/codeconverter: Update to latest 
version)
ERROR: line over 90 characters
#26: FILE: scripts/codeconverter/codeconverter/patching.py:8:
+from typing import IO, Match, NamedTuple, Optional, Literal, Iterable, Type, 
Dict, List, Any, TypeVar, NewType, Tuple, Union

ERROR: line over 90 characters
#78: FILE: scripts/codeconverter/codeconverter/patching.py:184:
+    def group_finditer(self, klass: Type['FileMatch'], group: Union[str, int]) 
-> Iterable['FileMatch']:

ERROR: line over 90 characters
#85: FILE: scripts/codeconverter/codeconverter/patching.py:191:
+    def try_group_match(self, klass: Type['FileMatch'], group: Union[str, 
int]) -> Optional['FileMatch']:

WARNING: line over 80 characters
#161: FILE: scripts/codeconverter/codeconverter/patching.py:289:
+    def find_match(self, t: Type[T], name: str, group: str='name') -> 
Optional[T]:

WARNING: line over 80 characters
#194: FILE: scripts/codeconverter/codeconverter/patching.py:331:
+        matches = [klass(self, m) for m in 
klass.finditer(self.original_content)]

ERROR: line over 90 characters
#221: FILE: scripts/codeconverter/codeconverter/patching.py:370:
+    def scan_for_matches(self, class_names: Optional[List[str]]=None) -> 
Iterable[FileMatch]:

WARNING: line over 80 characters
#337: FILE: scripts/codeconverter/codeconverter/qom_macros.py:28:
+    regexp = S(r'^[ \t]*#[ \t]*define', CPP_SPACE, NAMED('name', 
RE_IDENTIFIER), r'\b')

ERROR: line over 90 characters
#394: FILE: scripts/codeconverter/codeconverter/qom_macros.py:444:
+               r'\s*\(\s*', OR(NAMED('instancetype', RE_IDENTIFIER), RE_TYPE, 
name='c_type'),

ERROR: line over 90 characters
#457: FILE: scripts/codeconverter/codeconverter/qom_macros.py:513:
+               OR(RE_IDENTIFIER, RE_STRING, RE_MACRO_CONCAT, RE_FUN_CALL, 
name='typename'), SP,

WARNING: line over 80 characters
#521: FILE: scripts/codeconverter/codeconverter/qom_macros.py:575:
+    """Common base class for code that will look at a set of type 
declarations"""

ERROR: line over 90 characters
#531: FILE: scripts/codeconverter/codeconverter/qom_macros.py:583:
+        decl_types: List[Type[TypeDeclaration]] = [DeclareInstanceChecker, 
DeclareInstanceType,

WARNING: line over 80 characters
#532: FILE: scripts/codeconverter/codeconverter/qom_macros.py:584:
+                                                   DeclareClassCheckers, 
DeclareClassType,

WARNING: line over 80 characters
#550: FILE: scripts/codeconverter/codeconverter/qom_macros.py:602:
+            field_dict = dict((f, v.pop() if v else None) for f,v in 
fvalues.items())

WARNING: line over 80 characters
#551: FILE: scripts/codeconverter/codeconverter/qom_macros.py:603:
+            yield from self.gen_patches_for_type(uppercase, checkers, 
field_dict)

WARNING: line over 80 characters
#553: FILE: scripts/codeconverter/codeconverter/qom_macros.py:605:
+    def find_conflicts(self, uppercase: str, checkers: List[TypeDeclaration]) 
-> bool:

WARNING: line over 80 characters
#554: FILE: scripts/codeconverter/codeconverter/qom_macros.py:606:
+        """Look for conflicting declarations that would make it unsafe to add 
new ones"""

WARNING: line over 80 characters
#557: FILE: scripts/codeconverter/codeconverter/qom_macros.py:609:
+        conflicting.extend(chain(self.file.find_matches(DefineDirective, 
uppercase),

ERROR: line over 90 characters
#558: FILE: scripts/codeconverter/codeconverter/qom_macros.py:610:
+                                 
self.file.find_matches(DeclareInterfaceChecker, uppercase, 'uppercase'),

ERROR: line over 90 characters
#559: FILE: scripts/codeconverter/codeconverter/qom_macros.py:611:
+                                 self.file.find_matches(DeclareClassType, 
uppercase, 'uppercase'),

ERROR: line over 90 characters
#560: FILE: scripts/codeconverter/codeconverter/qom_macros.py:612:
+                                 self.file.find_matches(DeclareInstanceType, 
uppercase, 'uppercase')))

ERROR: line over 90 characters
#563: FILE: scripts/codeconverter/codeconverter/qom_macros.py:615:
+        conflicting.extend(o for o in 
chain(self.allfiles.find_matches(DeclareInstanceChecker, uppercase, 
'uppercase'),

ERROR: line over 90 characters
#564: FILE: scripts/codeconverter/codeconverter/qom_macros.py:616:
+                                            
self.allfiles.find_matches(DeclareClassCheckers, uppercase, 'uppercase'),

ERROR: line over 90 characters
#565: FILE: scripts/codeconverter/codeconverter/qom_macros.py:617:
+                                            
self.allfiles.find_matches(DeclareInterfaceChecker, uppercase, 'uppercase'),

ERROR: line over 90 characters
#566: FILE: scripts/codeconverter/codeconverter/qom_macros.py:618:
+                                            
self.allfiles.find_matches(DefineDirective, uppercase))

WARNING: line over 80 characters
#568: FILE: scripts/codeconverter/codeconverter/qom_macros.py:620:
+                               # if both are .c files, there's no conflict at 
all:

WARNING: line over 80 characters
#585: FILE: scripts/codeconverter/codeconverter/qom_macros.py:637:
+                             fields: Dict[str, Optional[str]]) -> 
Iterable[Patch]:

WARNING: line over 80 characters
#595: FILE: scripts/codeconverter/codeconverter/qom_macros.py:647:
+                             fields: Dict[str, Optional[str]]) -> 
Iterable[Patch]:

WARNING: line over 80 characters
#625: FILE: scripts/codeconverter/codeconverter/qom_macros.py:655:
+            yield first_checker.prepend(f'DECLARE_INSTANCE_TYPE({uppercase}, 
void)\n')

WARNING: line over 80 characters
#627: FILE: scripts/codeconverter/codeconverter/qom_macros.py:657:
+            yield first_checker.prepend(f'DECLARE_CLASS_TYPE({uppercase}, 
void)\n')

ERROR: line over 90 characters
#634: FILE: scripts/codeconverter/codeconverter/qom_macros.py:664:
+        #s = (f"DECLARE_OBJ_CHECKERS({final_values['instancetype']}, 
{final_values['classtype']},\n"+

ERROR: line over 90 characters
#635: FILE: scripts/codeconverter/codeconverter/qom_macros.py:665:
+        #        f"                     {final_values['uppercase']}, 
{final_values['typename']})\n")

WARNING: line over 80 characters
#645: FILE: scripts/codeconverter/codeconverter/qom_macros.py:675:
+                             fields: Dict[str, Optional[str]]) -> 
Iterable[Patch]:

ERROR: line over 90 characters
#719: FILE: scripts/codeconverter/codeconverter/qom_macros.py:754:
+def find_type_checkers(files:FileList, name:str, group:str='uppercase') -> 
Iterable[TypeCheckerDeclaration]:

ERROR: line over 90 characters
#722: FILE: scripts/codeconverter/codeconverter/qom_macros.py:757:
+    for c in (DeclareInstanceChecker, DeclareClassCheckers, 
DeclareObjCheckers, ObjectDeclareType, ObjectDeclareSimpleType):

ERROR: line over 90 characters
#738: FILE: scripts/codeconverter/codeconverter/qom_macros.py:794:
+        user_classes = [TypeCheckMacro, DeclareObjCheckers, 
DeclareInstanceChecker, DeclareClassCheckers, InterfaceCheckMacro]

ERROR: line over 90 characters
#796: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:43:
+        return [m.group('arrayitem') for m in self.group_finditer(ArrayItem, 
'arrayitems')]

WARNING: line over 80 characters
#820: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:66:
+    Common base class for type definitions (TypeInfo variables or 
OBJECT_DEFINE* macros)

WARNING: line over 80 characters
#987: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:245:
+            self.warn("%s class size type (%s) is not *Class?", self.name, 
classtype)

ERROR: line over 90 characters
#992: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:250:
+        self.debug("searching for simpletype declaration using %s as 
InstanceType", instancetype)

ERROR: line over 90 characters
#1005: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:263:
+        c = (f'OBJECT_DECLARE_SIMPLE_TYPE({self.group("instancetype")}, 
{self.group("lowercase")},\n'

WARNING: line over 80 characters
#1012: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:270:
+        # this will just ensure the caches for find_match() and 
matches_for_type()

ERROR: line over 90 characters
#1047: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:321:
+        #            self.warn("macro/constant type name is required for 
instance/class type")

WARNING: line over 80 characters
#1066: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:333:
+                self.warn("This will make type validation difficult in the 
future")

WARNING: line over 80 characters
#1069: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:336:
+        parent_uppercase = find_typename_uppercase(self.allfiles, 
parent_typename)

ERROR: line over 90 characters
#1071: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:338:
+            self.info("Can't find right uppercase name for parent type (%s)", 
parent_typename)

ERROR: line over 90 characters
#1073: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:340:
+                self.warn("Can't find right uppercase name for parent type 
(%s)", parent_typename)

WARNING: line over 80 characters
#1074: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:341:
+                self.warn("This will make type validation difficult in the 
future")

ERROR: line over 90 characters
#1096: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:346:
+        #checkers: List[TypeCheckerDeclaration] = 
list(find_type_checkers(self.allfiles, uppercase))

ERROR: line over 90 characters
#1098: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:348:
+        #    c.info("instance type checker declaration (%s) is here", 
c.group('uppercase'))

ERROR: line over 90 characters
#1102: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:352:
+        #        self.warn("Can't find where type checkers for %s (%s) are 
declared.  We will need them to validate sizes of %s",

ERROR: line over 90 characters
#1123: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:371:
+        #        self.warn("type at instance_size is %r.  Should instance_size 
be set to sizeof(%s) ?",

ERROR: line over 90 characters
#1128: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:376:
+        #        self.warn("instance type checker for %s (%s) not found", 
typename, instancetype)

ERROR: line over 90 characters
#1142: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:390:
+        #        self.warn("type at class_size is %r.  Should class_size be 
set to sizeof(%s) ?",

ERROR: line over 90 characters
#1147: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:395:
+        #        self.warn("class type checker for %s (%s) not found", 
typename, classtype)

WARNING: line over 80 characters
#1170: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:416:
+        newbegin += f'                            {instancetype}, 
{classtype},\n'

WARNING: line over 80 characters
#1171: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:417:
+        newbegin += f'                            {uppercase}, 
{parent_uppercase}'

WARNING: line over 80 characters
#1204: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:450:
+def find_type_definitions(files: FileList, uppercase: str) -> 
Iterable[TypeDefinition]:

ERROR: line over 90 characters
#1205: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:451:
+    types: List[Type[TypeDefinition]] = [TypeInfoVar, ObjectDefineType, 
ObjectDefineTypeExtended]

WARNING: line over 80 characters
#1217: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:463:
+                             fields: Dict[str, Optional[str]]) -> 
Iterable[Patch]:

WARNING: line over 80 characters
#1265: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:511:
+        instance_checkers = [c for c in find_type_checkers(self.allfiles, 
uppercase)

WARNING: line over 80 characters
#1297: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:543:
+                    self.warn("typedef mismatch: %s is defined as %s", t, 
td_type)

ERROR: line over 90 characters
#1302: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:548:
+        other_instance_checkers = [c for c in 
find_type_checkers(self.allfiles, instancetype, 'instancetype')

ERROR: line over 90 characters
#1311: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:557:
+        decl_types: List[Type[TypeDeclaration]] = [DeclareClassCheckers, 
DeclareObjCheckers]

WARNING: line over 80 characters
#1313: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:559:
+                       for m in self.allfiles.find_matches(t, uppercase, 
'uppercase')]

ERROR: line over 90 characters
#1329: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:575:
+                self.warn("mismatching instance type for %s (%s)", uppercase, 
instancetype)

WARNING: line over 80 characters
#1334: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:580:
+                self.warn("mismatching class type for %s (%s)", uppercase, 
classtype)

ERROR: line over 90 characters
#1351: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:597:
+                elif not re.search(r'\b'+t+r'\b', 
self.file.original_content[td.end():self.start()]):

WARNING: line over 80 characters
#1377: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:623:
+                    self.warn("typedef mismatch: %s is defined as %s", t, 
td_type)

ERROR: line over 90 characters
#1382: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:628:
+        other_instance_checkers = [c for c in 
find_type_checkers(self.allfiles, instancetype, 'instancetype')

ERROR: line over 90 characters
#1391: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:637:
+        decl_types: List[Type[TypeDeclaration]] = [DeclareClassCheckers, 
DeclareObjCheckers]

WARNING: line over 80 characters
#1393: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:639:
+                       for m in self.allfiles.find_matches(t, uppercase, 
'uppercase')]

ERROR: line over 90 characters
#1414: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:660:
+                self.warn("mismatching instance type for %s (%s)", uppercase, 
instancetype)

ERROR: line over 90 characters
#1436: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:682:
+                elif not re.search(r'\b'+t+r'\b', 
self.file.original_content[td.end():self.start()]):

WARNING: line over 80 characters
#1457: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:703:
+        #all_defines = [m for m in 
self.allfiles.matches_of_type(ExpressionDefine)]

ERROR: line over 90 characters
#1487: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:744:
+        if 'instance_size' in values and 
parent.get_raw_initializer_value('instance_size') != 
values['instance_size'].raw:

ERROR: line over 90 characters
#1493: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:749:
+        if 'class_size' in values and 
parent.get_raw_initializer_value('class_size') != values['class_size'].raw:

ERROR: line over 90 characters
#1564: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:871:
+            self.warn("can't identify the function where 
type_register_static(&%s) is called", self.name)

WARNING: line over 80 characters
#1613: FILE: scripts/codeconverter/codeconverter/qom_type_info.py:912:
+            yield 
self.group_match('func_name').make_patch('type_register_static')

total: 44 errors, 34 warnings, 1701 lines checked

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

14/17 Checking commit 580edfa51e34 (qom: Remove ParentClassType argument from 
OBJECT_DECLARE_SIMPLE_TYPE)
WARNING: line over 80 characters
#228: FILE: include/qom/object.h:649:
+#define OBJECT_DECLARE_SIMPLE_TYPE(InstanceType, module_obj_name, 
MODULE_OBJ_NAME) \

WARNING: line over 80 characters
#233: FILE: include/qom/object.h:654:
+    DECLARE_INSTANCE_CHECKER(InstanceType, MODULE_OBJ_NAME, 
TYPE_##MODULE_OBJ_NAME)

total: 0 errors, 2 warnings, 192 lines checked

Patch 14/17 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
15/17 Checking commit 145c40532e4a (qom: Remove module_obj_name parameter from 
OBJECT_DECLARE* macros)
16/17 Checking commit 15a16fe73496 (Use OBJECT_DECLARE_TYPE when possible)
WARNING: line over 80 characters
#153: FILE: hw/arm/vexpress.c:184:
+OBJECT_DECLARE_TYPE(VexpressMachineState, VexpressMachineClass, 
VEXPRESS_MACHINE)

WARNING: line over 80 characters
#970: FILE: include/hw/s390x/3270-ccw.h:34:
+OBJECT_DECLARE_TYPE(EmulatedCcw3270Device, EmulatedCcw3270Class, 
EMULATED_CCW_3270)

total: 0 errors, 2 warnings, 902 lines checked

Patch 16/17 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
17/17 Checking commit 74e8c804ff31 (Use OBJECT_DECLARE_SIMPLE_TYPE when 
possible)
WARNING: line over 80 characters
#46: FILE: backends/cryptodev-vhost-user.c:42:
+OBJECT_DECLARE_SIMPLE_TYPE(CryptoDevBackendVhostUser, 
CRYPTODEV_BACKEND_VHOST_USER)

total: 0 errors, 1 warnings, 6146 lines checked

Patch 17/17 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/20200918204714.27276-1-ehabk...@redhat.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-de...@redhat.com

Reply via email to