[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [PATCH v4] s390x: remove direct reference to mem_path g
From: |
Cornelia Huck |
Subject: |
Re: [qemu-s390x] [PATCH v4] s390x: remove direct reference to mem_path global from s390x code |
Date: |
Wed, 30 Jan 2019 13:21:52 +0100 |
On Wed, 30 Jan 2019 12:18:44 +0100
David Hildenbrand <address@hidden> wrote:
> On 30.01.19 10:41, Cornelia Huck wrote:
> > On Wed, 30 Jan 2019 08:55:06 +0100
> > Igor Mammedov <address@hidden> wrote:
> >
> >> I plan to deprecate -mem-path option and replace it with memory-backend,
> >> for that it's necessary to get rid of mem_path global variable.
> >> Do it for s390x case, replacing it with alternative way to enable
> >> 1Mb hugepages capability.
> >>
> >> Todo that replace qemu_mempath_getpagesize() with qemu_getrampagesize()
> >> which also checks for -mem-path provided RAM.
> >>
> >> Signed-off-by: Igor Mammedov <address@hidden>
> >> ---
> >> v4:
> >> * fix typos in commit message
> >> * add include "exec/ram_addr.h" to make sure it compiles on ppc host
> >> v2:
> >> * s/qemu_mempath_getpagesize/qemu_getrampagesiz/ and drop the rest of
> >> changes
> >> David Hildenbrand <address@hidden>
> >
> > Something odd happened here -- I presume you did not want to drop
> > David's R-b?
> >
> > [I can re-add while applying, no need to resend.]
> >
>
> Indeed, please add
>
> Reviewed-by: David Hildenbrand <address@hidden>
Done, and applied. Thanks!
>
> >> ---
> >> target/s390x/kvm.c | 5 +++--
> >> 1 file changed, 3 insertions(+), 2 deletions(-)
>
>