[PATCH 0/2] replace sysconf(_SC_PAGESIZE) with qemu_real_host_page_size

Wei Yang posted 2 patches 6 years, 3 months ago
Test asan passed
Test checkpatch passed
Test FreeBSD passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu passed
Test docker-quick@centos7 passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20191015031350.4345-1-richardw.yang@linux.intel.com
Maintainers: Max Reitz <mreitz@redhat.com>, "Alex Bennée" <alex.bennee@linaro.org>, Kevin Wolf <kwolf@redhat.com>, Jason Wang <jasowang@redhat.com>
block/file-posix.c              |  2 +-
net/l2tpv3.c                    |  2 +-
tests/tcg/multiarch/test-mmap.c | 67 ++++++++++++++++++---------------
3 files changed, 38 insertions(+), 33 deletions(-)
[PATCH 0/2] replace sysconf(_SC_PAGESIZE) with qemu_real_host_page_size
Posted by Wei Yang 6 years, 3 months ago
This is a following up patch to cleanup page size, suggested by
"Dr. David Alan Gilbert" <dgilbert@redhat.com>.

Patch 2 does the job, while during the cleanup I found test-mmap.c has quite a
lot code style problem. To make the code looks good, patch 1 is introduced to
make checkpatch.pl happy a little.

Wei Yang (2):
  tests/tcg/multiarch: fix code style in function main of test-mmap.c
  core: replace sysconf(_SC_PAGESIZE) with qemu_real_host_page_size

 block/file-posix.c              |  2 +-
 net/l2tpv3.c                    |  2 +-
 tests/tcg/multiarch/test-mmap.c | 67 ++++++++++++++++++---------------
 3 files changed, 38 insertions(+), 33 deletions(-)

-- 
2.17.1


Re: [PATCH 0/2] replace sysconf(_SC_PAGESIZE) with qemu_real_host_page_size
Posted by Wei Yang 6 years, 3 months ago
On Tue, Oct 15, 2019 at 11:13:48AM +0800, Wei Yang wrote:
>This is a following up patch to cleanup page size, suggested by
>"Dr. David Alan Gilbert" <dgilbert@redhat.com>.
>
>Patch 2 does the job, while during the cleanup I found test-mmap.c has quite a
>lot code style problem. To make the code looks good, patch 1 is introduced to
>make checkpatch.pl happy a little.

Does this patch set look good?

>
>Wei Yang (2):
>  tests/tcg/multiarch: fix code style in function main of test-mmap.c
>  core: replace sysconf(_SC_PAGESIZE) with qemu_real_host_page_size
>
> block/file-posix.c              |  2 +-
> net/l2tpv3.c                    |  2 +-
> tests/tcg/multiarch/test-mmap.c | 67 ++++++++++++++++++---------------
> 3 files changed, 38 insertions(+), 33 deletions(-)
>
>-- 
>2.17.1
>

-- 
Wei Yang
Help you, Help me