All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Lieven <lieven-lists@dlhnet.de>
To: Juan Quintela <quintela@redhat.com>, qemu-devel@nongnu.org
Cc: Isaku Yamahata <yamahata@private.email.ne.jp>, anthony@codemonkey.ws
Subject: Re: [Qemu-devel] [PATCH 8/9] arch_init: make is_zero_page accept size
Date: Mon, 07 Oct 2013 11:15:22 +0200	[thread overview]
Message-ID: <52527BAA.3000302@dlhnet.de> (raw)
In-Reply-To: <1380024203-25897-9-git-send-email-quintela@redhat.com>

On 24.09.2013 14:03, Juan Quintela wrote:
> From: Isaku Yamahata <yamahata@private.email.ne.jp>
>
> Later is_zero_page will be used for non TARGET_PAGE_SIZE
> range.
> And rename it to is_zero_range as it isn't page size any more.
>
> Signed-off-by: Isaku Yamahata <yamahata@private.email.ne.jp>
> Signed-off-by: Juan Quintela <quintela@redhat.com>
> ---
>   arch_init.c | 9 ++++-----
>   1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/arch_init.c b/arch_init.c
> index 18cd9a1..c72790f 100644
> --- a/arch_init.c
> +++ b/arch_init.c
> @@ -150,10 +150,9 @@ int qemu_read_default_config_files(bool userconfig)
>       return 0;
>   }
>
> -static inline bool is_zero_page(uint8_t *p)
> +static inline bool is_zero_range(uint8_t *p, uint64_t size)
>   {
> -    return buffer_find_nonzero_offset(p, TARGET_PAGE_SIZE) ==
> -        TARGET_PAGE_SIZE;
> +    return buffer_find_nonzero_offset(p, size) == size;
>   }

this is buffer_is_zero(p, size); maybe this function should be used.

Peter

  reply	other threads:[~2013-10-07  9:15 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-24 12:03 [Qemu-devel] [PULL 0/9] migration queue Juan Quintela
2013-09-24 12:03 ` [Qemu-devel] [PATCH 1/9] savevm: add comments for qemu_file_get_error() Juan Quintela
2013-09-24 12:03 ` [Qemu-devel] [PATCH 2/9] savevm: fix wrong initialization by ram_control_load_hook Juan Quintela
2013-09-24 12:03 ` [Qemu-devel] [PATCH 3/9] arch_init: right return for ram_save_iterate Juan Quintela
2013-09-24 12:03 ` [Qemu-devel] [PATCH 4/9] rdma: clean up of qemu_rdma_cleanup() Juan Quintela
2013-09-24 12:03 ` [Qemu-devel] [PATCH 5/9] rdma: constify ram_chunk_{index, start, end} Juan Quintela
2013-09-24 12:03 ` [Qemu-devel] [PATCH 6/9] migration: add version supporting macros for struct pointer Juan Quintela
2013-09-24 12:03 ` [Qemu-devel] [PATCH 7/9] migration: Fix debug print type Juan Quintela
2013-09-24 12:03 ` [Qemu-devel] [PATCH 8/9] arch_init: make is_zero_page accept size Juan Quintela
2013-10-07  9:15   ` Peter Lieven [this message]
2013-09-24 12:03 ` [Qemu-devel] [PATCH 9/9] migration: ram_handle_compressed Juan Quintela

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=52527BAA.3000302@dlhnet.de \
    --to=lieven-lists@dlhnet.de \
    --cc=anthony@codemonkey.ws \
    --cc=qemu-devel@nongnu.org \
    --cc=quintela@redhat.com \
    --cc=yamahata@private.email.ne.jp \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.