mmap-alloc: fix hugetlbfs misaligned length in ppc64

[Qemu-devel] [PATCH 0/2] mmap-alloc: fix hugetlbfs misaligned length in ppc64

Murilo Opsfelder Araujo posted 2 patches 5 years, 2 months ago

Download series mbox

Test docker-clang@ubuntu passed

Test docker-mingw@fedora passed

Test asan passed

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20190130233605.22163-1-muriloo@linux.ibm.com

Maintainers: Peter Crosthwaite <crosthwaite.peter@gmail.com>, Richard Henderson <rth@twiddle.net>, Paolo Bonzini <pbonzini@redhat.com>

exec.c                    |  4 +--
include/qemu/mmap-alloc.h |  2 +-
util/mmap-alloc.c         | 73 ++++++++++++++++++++++++++-------------
util/oslib-posix.c        |  2 +-
4 files changed, 53 insertions(+), 28 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH 0/2] mmap-alloc: fix hugetlbfs misaligned length in ppc64

Posted by Murilo Opsfelder Araujo 5 years, 2 months ago

The first patch unfolds parts of qemu_ram_mmap() to make it clearer.
No changes in the function behaviour.

The second one fixes the alignment of the length given to munmap().

I am pretty sure there is room for improvement, so I would love to
hear your feedback.

Thank you!

Murilo Opsfelder Araujo (2):
  mmap-alloc: unfold qemu_ram_mmap()
  mmap-alloc: fix hugetlbfs misaligned length in ppc64

 exec.c                    |  4 +--
 include/qemu/mmap-alloc.h |  2 +-
 util/mmap-alloc.c         | 73 ++++++++++++++++++++++++++-------------
 util/oslib-posix.c        |  2 +-
 4 files changed, 53 insertions(+), 28 deletions(-)

--
2.20.1

Re: [Qemu-devel] [PATCH 0/2] mmap-alloc: fix hugetlbfs misaligned length in ppc64

Posted by David Gibson 5 years, 2 months ago

On Wed, Jan 30, 2019 at 09:36:03PM -0200, Murilo Opsfelder Araujo wrote:
> The first patch unfolds parts of qemu_ram_mmap() to make it clearer.
> No changes in the function behaviour.
> 
> The second one fixes the alignment of the length given to munmap().
> 
> I am pretty sure there is room for improvement, so I would love to
> hear your feedback.
> 
> Thank you!

Applied to ppc-for-4.0.

> 
> Murilo Opsfelder Araujo (2):
>   mmap-alloc: unfold qemu_ram_mmap()
>   mmap-alloc: fix hugetlbfs misaligned length in ppc64
> 
>  exec.c                    |  4 +--
>  include/qemu/mmap-alloc.h |  2 +-
>  util/mmap-alloc.c         | 73 ++++++++++++++++++++++++++-------------
>  util/oslib-posix.c        |  2 +-
>  4 files changed, 53 insertions(+), 28 deletions(-)
> 

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

Re: [Qemu-devel] [PATCH 0/2] mmap-alloc: fix hugetlbfs misaligned length in ppc64

Posted by Murilo Opsfelder Araujo 5 years, 2 months ago

On Mon, Feb 04, 2019 at 11:08:05AM +1100, David Gibson wrote:
> On Wed, Jan 30, 2019 at 09:36:03PM -0200, Murilo Opsfelder Araujo wrote:
> > The first patch unfolds parts of qemu_ram_mmap() to make it clearer.
> > No changes in the function behaviour.
> >
> > The second one fixes the alignment of the length given to munmap().
> >
> > I am pretty sure there is room for improvement, so I would love to
> > hear your feedback.
> >
> > Thank you!
>
> Applied to ppc-for-4.0.

Thank you all for reviewing and testing it.

I did appreciate!

--
Murilo