Re: [PATCH v4 0/7] block: Add retry for werror=/rerror= mechanism

2020-12-20 Thread Jiahui Cen
Kindly ping... On 2020/12/15 20:30, Jiahui Cen wrote: > A VM in the cloud environment may use a virutal disk as the backend storage, > and there are usually filesystems on the virtual block device. When backend > storage is temporarily down, any I/O issued to the virtual block device > will cause

[PATCH 6/9] hw/block/nand: Rename PAGE_SIZE to NAND_PAGE_SIZE

2020-12-20 Thread Jiaxun Yang
As per POSIX specification of limits.h [1], OS libc may define PAGE_SIZE in limits.h. To prevent collosion of definition, we rename PAGE_SIZE here. [1]: https://pubs.opengroup.org/onlinepubs/7908799/xsh/limits.h.html Signed-off-by: Jiaxun Yang --- hw/block/nand.c | 40 -

[PATCH 4/9] libvhost-user: Include poll.h instead of sys/poll.h

2020-12-20 Thread Jiaxun Yang
Musl libc complains about it's wrong usage. In file included from ../subprojects/libvhost-user/libvhost-user.h:20, from ../subprojects/libvhost-user/libvhost-user-glib.h:19, from ../subprojects/libvhost-user/libvhost-user-glib.c:15: /usr/include/sys/poll.h:1:2: er

[PATCH 7/9] accel/kvm: avoid using predefined PAGE_SIZE

2020-12-20 Thread Jiaxun Yang
As per POSIX specification of limits.h [1], OS libc may define PAGE_SIZE in limits.h. To prevent collosion of definition, we discard PAGE_SIZE from defined by libc and take QEMU's variable. [1]: https://pubs.opengroup.org/onlinepubs/7908799/xsh/limits.h.html Signed-off-by: Jiaxun Yang --- acce

[PATCH 9/9] gitlab-ci: Add alpine to pipeline

2020-12-20 Thread Jiaxun Yang
We only run build test and check-acceptance as their are too many failures in checks due to minor string mismatch. Signed-off-by: Jiaxun Yang --- .gitlab-ci.d/containers.yml | 5 + .gitlab-ci.yml | 23 +++ 2 files changed, 28 insertions(+) diff --git a/.git

[PATCH 5/9] elf2dmp: Rename PAGE_SIZE to ELF2DMP_PAGE_SIZE

2020-12-20 Thread Jiaxun Yang
As per POSIX specification of limits.h [1], OS libc may define PAGE_SIZE in limits.h. To prevent collosion of definition, we rename PAGE_SIZE here. [1]: https://pubs.opengroup.org/onlinepubs/7908799/xsh/limits.h.html Signed-off-by: Jiaxun Yang --- contrib/elf2dmp/addrspace.c | 4 ++-- contrib

[PATCH 2/9] configure: Add sys/timex.h to probe clk_adjtime

2020-12-20 Thread Jiaxun Yang
It is not a part of standard time.h. Glibc put it under time.h however musl treat it as a sys timex extension. Signed-off-by: Jiaxun Yang --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index c228f7c21e..990f37e123 100755 --- a/configure +++ b/configure @@

[PATCH 8/9] tests: Rename PAGE_SIZE definitions

2020-12-20 Thread Jiaxun Yang
As per POSIX specification of limits.h [1], OS libc may define PAGE_SIZE in limits.h. Self defined PAGE_SIZE is frequently used in tests, to prevent collosion of definition, we give PAGE_SIZE definitons reasonable prefixs. [1]: https://pubs.opengroup.org/onlinepubs/7908799/xsh/limits.h.html Sign

[PATCH 1/9] tests/docker: Add dockerfile for Alpine Linux

2020-12-20 Thread Jiaxun Yang
Alpine Linux[1] is a security-oriented, lightweight Linux distribution based on musl libc and busybox. It it popular among Docker guests and embedded applications. Adding it to test against different libc. [1]: https://alpinelinux.org/ Signed-off-by: Jiaxun Yang --- tests/docker/dockerfiles/a

[PATCH 0/9] Alpine Linux build fix and CI pipeline

2020-12-20 Thread Jiaxun Yang
Alpine Linux[1] is a security-oriented, lightweight Linux distribution based on musl libc and busybox. It it popular among Docker guests and embedded applications. Adding it to test against different libc. [1]: https://alpinelinux.org/ Tree avilable at: https://gitlab.com/FlyGoat/qemu/-/tree/al

[PATCH 3/9] configure/meson: Only check sys/signal.h on non-Linux

2020-12-20 Thread Jiaxun Yang
signal.h is equlevant of sys/signal.h on Linux, musl would complain wrong usage of sys/signal.h. In file included from /builds/FlyGoat/qemu/include/qemu/osdep.h:108, from ../tests/qemu-iotests/socket_scm_helper.c:13: /usr/include/sys/signal.h:1:2: error: #warning redirecting incor

Re: [PATCH 0/9] Alpine Linux build fix and CI pipeline

2020-12-20 Thread no-reply
Patchew URL: https://patchew.org/QEMU/20201221005318.11866-1-jiaxun.y...@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.y...@flygoat.com Subject: [PATCH 0/9] Alpine Linux buil