Oops, there are 3 actual mistakes I missed among the false positives, below: On Sun, Jan 08, 2017 at 11:47:21AM -0800, no-reply@patchew.org wrote: [...]

=== OUTPUT BEGIN === Checking PATCH 1/5: i386: Add explicit array size to x86_cpu_vendor_words2str()... ERROR: line over 90 characters #27: FILE: target/i386/cpu.c:172: +static void x86_cpu_vendor_words2str(char dst[static (CPUID_VENDOR_SZ + 1)], uint32_t vendor1,

False positive.

ERROR: space prohibited between function name and open parenthesis '(' #27: FILE: target/i386/cpu.c:172: +static void x86_cpu_vendor_words2str(char dst[static (CPUID_VENDOR_SZ + 1)], uint32_t vendor1,

False positive.

total: 2 errors, 0 warnings, 8 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/5: i386: host_vendor_fms() helper function... ERROR: line over 90 characters #20: FILE: target/i386/cpu.c:685: +void host_vendor_fms(char vendor[static (CPUID_VENDOR_SZ + 1)], int *family, int *model, int *stepping)

Oops, will fix it.

ERROR: space prohibited between function name and open parenthesis '(' #20: FILE: target/i386/cpu.c:685: +void host_vendor_fms(char vendor[static (CPUID_VENDOR_SZ + 1)], int *family, int *model, int *stepping)

False positive.

ERROR: storage class should be at the beginning of the declaration #20: FILE: target/i386/cpu.c:685: +void host_vendor_fms(char vendor[static (CPUID_VENDOR_SZ + 1)], int *family, int *model, int *stepping)

False positive.

ERROR: line over 90 characters #57: FILE: target/i386/cpu.c:1599: + host_vendor_fms(host_cpudef.vendor, &host_cpudef.family, &host_cpudef.model, &host_cpudef.stepping);

Will fix it.

ERROR: line over 90 characters #69: FILE: target/i386/cpu.h:1426: +void host_vendor_fms(char vendor[static (CPUID_VENDOR_SZ + 1)], int *family, int *model, int *stepping);

Will fix it.

ERROR: space prohibited between function name and open parenthesis '(' #69: FILE: target/i386/cpu.h:1426: +void host_vendor_fms(char vendor[static (CPUID_VENDOR_SZ + 1)], int *family, int *model, int *stepping);

False positive.

ERROR: storage class should be at the beginning of the declaration #69: FILE: target/i386/cpu.h:1426: +void host_vendor_fms(char vendor[static (CPUID_VENDOR_SZ + 1)], int *family, int *model, int *stepping);

False positive.

total: 7 errors, 0 warnings, 50 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 3/5: i386/kvm: Blacklist TSX on known broken hosts... Checking PATCH 4/5: pc: Add 2.9 machine-types... Checking PATCH 5/5: i386: Change stepping of Haswell to non-blacklisted value... === OUTPUT END ===

Test command exited with code: 1

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

-- Eduardo