[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 02/10] linux-user: Rename validate_guest_space =
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 02/10] linux-user: Rename validate_guest_space => init_guest_commpage |
Date: |
Fri, 2 Mar 2018 13:18:53 +0000 |
On 28 December 2017 at 18:08, Luke Shumaker <address@hidden> wrote:
> From: Luke Shumaker <address@hidden>
>
> init_guest_commpage is a much more honest description of what the function
> does. validate_guest_space not only suggests that the function has no
> side-effects, but also introduces confusion as to why it is only needed on
> 32-bit ARM targets.
>
> Signed-off-by: Luke Shumaker <address@hidden>
> ---
> linux-user/elfload.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/linux-user/elfload.c b/linux-user/elfload.c
> index cac991159c..453394239c 100644
> --- a/linux-user/elfload.c
> +++ b/linux-user/elfload.c
> @@ -362,8 +362,8 @@ enum {
> * The guest code may leave a page mapped and populate it if the
> * address is suitable.
> */
> -static int validate_guest_space(unsigned long guest_base,
> - unsigned long guest_size)
> +static int init_guest_commpage(unsigned long guest_base,
> + unsigned long guest_size)
> {
> unsigned long real_start, test_page_addr;
>
> @@ -1836,7 +1836,7 @@ unsigned long init_guest_space(unsigned long host_start,
> * address. */
> if (host_start && !host_size) {
> #if defined(TARGET_ARM) && !defined(TARGET_AARCH64)
> - if (validate_guest_space(host_start, host_size) == 1) {
> + if (init_guest_commpage(host_start, host_size) != 1) {
this is the bit that includes the '==' to '!=' change that should
be in patch 1.
Otherwise
Reviewed-by: Peter Maydell <address@hidden>
thanks
-- PMM
- Re: [Qemu-devel] [PATCH 02/10] linux-user: Rename validate_guest_space => init_guest_commpage,
Peter Maydell <=