From: no-reply@patchew.org To: jiaxun.yang@flygoat.com Cc: qemu-devel@nongnu.org, kwolf@redhat.com, fam@euphon.net, thuth@redhat.com, kvm@vger.kernel.org, viktor.prutyanov@phystech.edu, lvivier@redhat.com, philmd@redhat.com, alistair@alistair23.me, groug@kaod.org, wainersm@redhat.com, mreitz@redhat.com, qemu-ppc@nongnu.org, pbonzini@redhat.com, qemu-block@nongnu.org, alex.bennee@linaro.org, david@gibson.dropbear.id.au Subject: Re: [PATCH 0/9] Alpine Linux build fix and CI pipeline Date: Sun, 20 Dec 2020 17:06:47 -0800 (PST) [thread overview] Message-ID: <160851280526.21294.6201442635975331015@600e7e483b3a> (raw) In-Reply-To: <20201221005318.11866-1-jiaxun.yang@flygoat.com> Patchew URL: https://patchew.org/QEMU/20201221005318.11866-1-jiaxun.yang@flygoat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20201221005318.11866-1-jiaxun.yang@flygoat.com Subject: [PATCH 0/9] Alpine Linux build fix and CI pipeline === 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/20201221005318.11866-1-jiaxun.yang@flygoat.com -> patchew/20201221005318.11866-1-jiaxun.yang@flygoat.com Switched to a new branch 'test' 10095a9 gitlab-ci: Add alpine to pipeline a177af3 tests: Rename PAGE_SIZE definitions 5fcb0ed accel/kvm: avoid using predefined PAGE_SIZE e7febdf hw/block/nand: Rename PAGE_SIZE to NAND_PAGE_SIZE ba307d5 elf2dmp: Rename PAGE_SIZE to ELF2DMP_PAGE_SIZE 0ccf92b libvhost-user: Include poll.h instead of sys/poll.h 41a10db configure/meson: Only check sys/signal.h on non-Linux 0bcd2f2 configure: Add sys/timex.h to probe clk_adjtime a16c7ff tests/docker: Add dockerfile for Alpine Linux === OUTPUT BEGIN === 1/9 Checking commit a16c7ff7d859 (tests/docker: Add dockerfile for Alpine Linux) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #20: new file mode 100644 total: 0 errors, 1 warnings, 56 lines checked Patch 1/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 2/9 Checking commit 0bcd2f2eae84 (configure: Add sys/timex.h to probe clk_adjtime) 3/9 Checking commit 41a10dbdc8da (configure/meson: Only check sys/signal.h on non-Linux) 4/9 Checking commit 0ccf92b8ec37 (libvhost-user: Include poll.h instead of sys/poll.h) 5/9 Checking commit ba307d5a51aa (elf2dmp: Rename PAGE_SIZE to ELF2DMP_PAGE_SIZE) WARNING: line over 80 characters #69: FILE: contrib/elf2dmp/main.c:284: + h.PhysicalMemoryBlock.NumberOfPages += ps->block[i].size / ELF2DMP_PAGE_SIZE; WARNING: line over 80 characters #79: FILE: contrib/elf2dmp/main.c:291: + h.RequiredDumpSpace += h.PhysicalMemoryBlock.NumberOfPages << ELF2DMP_PAGE_BITS; total: 0 errors, 2 warnings, 70 lines checked Patch 5/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 6/9 Checking commit e7febdf0b056 (hw/block/nand: Rename PAGE_SIZE to NAND_PAGE_SIZE) ERROR: code indent should never use tabs #26: FILE: hw/block/nand.c:117: +# define PAGE_START(page)^I(PAGE(page) * (NAND_PAGE_SIZE + OOB_SIZE))$ ERROR: code indent should never use tabs #46: FILE: hw/block/nand.c:134: +# define NAND_PAGE_SIZE^I^I2048$ WARNING: line over 80 characters #65: FILE: hw/block/nand.c:684: + mem_and(iobuf + (soff | off), s->io, MIN(s->iolen, NAND_PAGE_SIZE - off)); WARNING: line over 80 characters #70: FILE: hw/block/nand.c:687: + mem_and(s->storage + (page << OOB_SHIFT), s->io + NAND_PAGE_SIZE - off, total: 2 errors, 2 warnings, 120 lines checked Patch 6/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 7/9 Checking commit 5fcb0ed1331a (accel/kvm: avoid using predefined PAGE_SIZE) 8/9 Checking commit a177af33938d (tests: Rename PAGE_SIZE definitions) 9/9 Checking commit 10095a92643d (gitlab-ci: Add alpine to pipeline) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20201221005318.11866-1-jiaxun.yang@flygoat.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-devel@redhat.com
WARNING: multiple messages have this Message-ID (diff)
From: no-reply@patchew.org To: jiaxun.yang@flygoat.com Cc: kwolf@redhat.com, fam@euphon.net, thuth@redhat.com, viktor.prutyanov@phystech.edu, kvm@vger.kernel.org, qemu-block@nongnu.org, lvivier@redhat.com, alex.bennee@linaro.org, alistair@alistair23.me, qemu-devel@nongnu.org, wainersm@redhat.com, groug@kaod.org, qemu-ppc@nongnu.org, pbonzini@redhat.com, mreitz@redhat.com, philmd@redhat.com, david@gibson.dropbear.id.au Subject: Re: [PATCH 0/9] Alpine Linux build fix and CI pipeline Date: Sun, 20 Dec 2020 17:06:47 -0800 (PST) [thread overview] Message-ID: <160851280526.21294.6201442635975331015@600e7e483b3a> (raw) In-Reply-To: <20201221005318.11866-1-jiaxun.yang@flygoat.com> Patchew URL: https://patchew.org/QEMU/20201221005318.11866-1-jiaxun.yang@flygoat.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20201221005318.11866-1-jiaxun.yang@flygoat.com Subject: [PATCH 0/9] Alpine Linux build fix and CI pipeline === 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/20201221005318.11866-1-jiaxun.yang@flygoat.com -> patchew/20201221005318.11866-1-jiaxun.yang@flygoat.com Switched to a new branch 'test' 10095a9 gitlab-ci: Add alpine to pipeline a177af3 tests: Rename PAGE_SIZE definitions 5fcb0ed accel/kvm: avoid using predefined PAGE_SIZE e7febdf hw/block/nand: Rename PAGE_SIZE to NAND_PAGE_SIZE ba307d5 elf2dmp: Rename PAGE_SIZE to ELF2DMP_PAGE_SIZE 0ccf92b libvhost-user: Include poll.h instead of sys/poll.h 41a10db configure/meson: Only check sys/signal.h on non-Linux 0bcd2f2 configure: Add sys/timex.h to probe clk_adjtime a16c7ff tests/docker: Add dockerfile for Alpine Linux === OUTPUT BEGIN === 1/9 Checking commit a16c7ff7d859 (tests/docker: Add dockerfile for Alpine Linux) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #20: new file mode 100644 total: 0 errors, 1 warnings, 56 lines checked Patch 1/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 2/9 Checking commit 0bcd2f2eae84 (configure: Add sys/timex.h to probe clk_adjtime) 3/9 Checking commit 41a10dbdc8da (configure/meson: Only check sys/signal.h on non-Linux) 4/9 Checking commit 0ccf92b8ec37 (libvhost-user: Include poll.h instead of sys/poll.h) 5/9 Checking commit ba307d5a51aa (elf2dmp: Rename PAGE_SIZE to ELF2DMP_PAGE_SIZE) WARNING: line over 80 characters #69: FILE: contrib/elf2dmp/main.c:284: + h.PhysicalMemoryBlock.NumberOfPages += ps->block[i].size / ELF2DMP_PAGE_SIZE; WARNING: line over 80 characters #79: FILE: contrib/elf2dmp/main.c:291: + h.RequiredDumpSpace += h.PhysicalMemoryBlock.NumberOfPages << ELF2DMP_PAGE_BITS; total: 0 errors, 2 warnings, 70 lines checked Patch 5/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 6/9 Checking commit e7febdf0b056 (hw/block/nand: Rename PAGE_SIZE to NAND_PAGE_SIZE) ERROR: code indent should never use tabs #26: FILE: hw/block/nand.c:117: +# define PAGE_START(page)^I(PAGE(page) * (NAND_PAGE_SIZE + OOB_SIZE))$ ERROR: code indent should never use tabs #46: FILE: hw/block/nand.c:134: +# define NAND_PAGE_SIZE^I^I2048$ WARNING: line over 80 characters #65: FILE: hw/block/nand.c:684: + mem_and(iobuf + (soff | off), s->io, MIN(s->iolen, NAND_PAGE_SIZE - off)); WARNING: line over 80 characters #70: FILE: hw/block/nand.c:687: + mem_and(s->storage + (page << OOB_SHIFT), s->io + NAND_PAGE_SIZE - off, total: 2 errors, 2 warnings, 120 lines checked Patch 6/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 7/9 Checking commit 5fcb0ed1331a (accel/kvm: avoid using predefined PAGE_SIZE) 8/9 Checking commit a177af33938d (tests: Rename PAGE_SIZE definitions) 9/9 Checking commit 10095a92643d (gitlab-ci: Add alpine to pipeline) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20201221005318.11866-1-jiaxun.yang@flygoat.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-devel@redhat.com
next prev parent reply other threads:[~2020-12-21 1:08 UTC|newest] Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-12-21 0:53 [PATCH 0/9] Alpine Linux build fix and CI pipeline Jiaxun Yang 2020-12-21 0:53 ` Jiaxun Yang 2020-12-21 0:53 ` [PATCH 1/9] tests/docker: Add dockerfile for Alpine Linux Jiaxun Yang 2020-12-21 0:53 ` Jiaxun Yang 2020-12-22 18:37 ` Wainer dos Santos Moschetta 2020-12-22 18:37 ` Wainer dos Santos Moschetta 2020-12-21 0:53 ` [PATCH 2/9] configure: Add sys/timex.h to probe clk_adjtime Jiaxun Yang 2020-12-21 0:53 ` Jiaxun Yang 2021-01-13 6:59 ` Thomas Huth 2021-01-13 6:59 ` Thomas Huth 2020-12-21 0:53 ` [PATCH 3/9] configure/meson: Only check sys/signal.h on non-Linux Jiaxun Yang 2020-12-21 0:53 ` Jiaxun Yang 2021-01-13 7:05 ` Thomas Huth 2021-01-13 7:05 ` Thomas Huth 2021-01-13 11:36 ` Peter Maydell 2021-01-13 11:36 ` Peter Maydell 2020-12-21 0:53 ` [PATCH 4/9] libvhost-user: Include poll.h instead of sys/poll.h Jiaxun Yang 2020-12-21 0:53 ` Jiaxun Yang 2021-01-13 7:08 ` Thomas Huth 2021-01-13 7:08 ` Thomas Huth 2020-12-21 0:53 ` [PATCH 5/9] elf2dmp: Rename PAGE_SIZE to ELF2DMP_PAGE_SIZE Jiaxun Yang 2020-12-21 0:53 ` Jiaxun Yang 2021-01-13 7:14 ` Thomas Huth 2021-01-13 7:14 ` Thomas Huth 2020-12-21 0:53 ` [PATCH 6/9] hw/block/nand: Rename PAGE_SIZE to NAND_PAGE_SIZE Jiaxun Yang 2020-12-21 0:53 ` Jiaxun Yang 2021-01-13 7:16 ` Thomas Huth 2021-01-13 7:16 ` Thomas Huth 2020-12-21 0:53 ` [PATCH 7/9] accel/kvm: avoid using predefined PAGE_SIZE Jiaxun Yang 2020-12-21 0:53 ` Jiaxun Yang 2021-01-13 7:19 ` Thomas Huth 2021-01-13 7:19 ` Thomas Huth 2021-01-13 9:07 ` Jiaxun Yang 2021-01-13 9:07 ` Jiaxun Yang 2020-12-21 0:53 ` [PATCH 8/9] tests: Rename PAGE_SIZE definitions Jiaxun Yang 2020-12-21 0:53 ` Jiaxun Yang 2021-01-13 7:21 ` Thomas Huth 2021-01-13 7:21 ` Thomas Huth 2020-12-21 0:53 ` [PATCH 9/9] gitlab-ci: Add alpine to pipeline Jiaxun Yang 2020-12-21 0:53 ` Jiaxun Yang 2020-12-21 1:06 ` no-reply [this message] 2020-12-21 1:06 ` [PATCH 0/9] Alpine Linux build fix and CI pipeline no-reply 2020-12-21 8:25 ` Jiaxun Yang 2020-12-21 8:25 ` Jiaxun Yang 2020-12-22 18:41 ` Wainer dos Santos Moschetta 2020-12-22 18:41 ` Wainer dos Santos Moschetta 2020-12-23 0:54 ` Jiaxun Yang 2020-12-23 0:54 ` Jiaxun Yang
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=160851280526.21294.6201442635975331015@600e7e483b3a \ --to=no-reply@patchew.org \ --cc=alex.bennee@linaro.org \ --cc=alistair@alistair23.me \ --cc=david@gibson.dropbear.id.au \ --cc=fam@euphon.net \ --cc=groug@kaod.org \ --cc=jiaxun.yang@flygoat.com \ --cc=kvm@vger.kernel.org \ --cc=kwolf@redhat.com \ --cc=lvivier@redhat.com \ --cc=mreitz@redhat.com \ --cc=pbonzini@redhat.com \ --cc=philmd@redhat.com \ --cc=qemu-block@nongnu.org \ --cc=qemu-devel@nongnu.org \ --cc=qemu-ppc@nongnu.org \ --cc=thuth@redhat.com \ --cc=viktor.prutyanov@phystech.edu \ --cc=wainersm@redhat.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.