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 From https://github.com/patchew-project/qemu - [tag update] patchew/20200918204714.27276-1-ehabk...@redhat.com -> patchew/20200918204714.27276-1-ehabk...@redhat.com Switched to a new branch 'test' f6c74dd Use OBJECT_DECLARE_SIMPLE_TYPE when possible 8272134 Use OBJECT_DECLARE_TYPE when possible 5185bc1 qom: Remove module_obj_name parameter from OBJECT_DECLARE* macros 43d39ef qom: Remove ParentClassType argument from OBJECT_DECLARE_SIMPLE_TYPE dd1224a scripts/codeconverter: Update to latest version f874a90 target/s390x: Set instance_align on S390CPU TypeInfo 28a1f3a target/riscv: Set instance_align on RISCVCPU TypeInfo 2bea693 target/ppc: Set instance_align on PowerPCCPU TypeInfo 21154be target/arm: Set instance_align on CPUARM TypeInfo a7c4b0a qom: Allow objects to be allocated with increased alignment 8f20dae qom: Correct error values in two contracts e12e839 qom: Clean up object_property_get_enum()'s error value c57d948 qom: Correct object_class_dynamic_cast_assert() documentation ceb22c8 sifive: Use DECLARE_*CHECKER* macros 4f27d5d sifive: Move QOM typedefs and add missing includes 5bafd89 sifive_u: Rename memmap enum constants 87c152d sifive_e: Rename memmap enum constants === OUTPUT BEGIN === 1/17 Checking commit 87c152dcf03d (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 5bafd89b66d5 (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 4f27d5da8e4b (sifive: Move QOM typedefs and add missing includes) 4/17 Checking commit ceb22c81179f (sifive: Use DECLARE_*CHECKER* macros) 5/17 Checking commit c57d9480b4b6 (qom: Correct object_class_dynamic_cast_assert() documentation) 6/17 Checking commit e12e839c4ad7 (qom: Clean up object_property_get_enum()'s error value) 7/17 Checking commit 8f20daefe95f (qom: Correct error values in two contracts) 8/17 Checking commit a7c4b0a56a69 (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 21154bec67a2 (target/arm: Set instance_align on CPUARM TypeInfo) 10/17 Checking commit 2bea6934f453 (target/ppc: Set instance_align on PowerPCCPU TypeInfo) 11/17 Checking commit 28a1f3afc08d (target/riscv: Set instance_align on RISCVCPU TypeInfo) 12/17 Checking commit f874a90dd3a9 (target/s390x: Set instance_align on S390CPU TypeInfo) 13/17 Checking commit dd1224ac349f (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 43d39efe7f8b (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 5185bc16076a (qom: Remove module_obj_name parameter from OBJECT_DECLARE* macros) 16/17 Checking commit 827213434dea (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 f6c74ddf0414 (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