[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] migration: check length directly to make sure t
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [Qemu-devel] [PATCH] migration: check length directly to make sure the range is aligned |
Date: |
Fri, 19 Jul 2019 19:06:51 +0100 |
User-agent: |
Mutt/1.12.0 (2019-05-25) |
* Paolo Bonzini (address@hidden) wrote:
> On 19/07/19 19:54, Dr. David Alan Gilbert wrote:
> >> - if ((uintptr_t)host_endaddr & (rb->page_size - 1)) {
> >> - error_report("ram_block_discard_range: Unaligned end address:
> >> %p",
> >> - host_endaddr);
> >> + if (length & (rb->page_size - 1)) {
> >> + error_report("ram_block_discard_range: Unaligned length: %lx",
> >> + length);
> > Yes, I *think* this is safe, we'll need to watch out for any warnings;
>
> Do you mean compiler or QEMU warning?
No, I mean lots of these error reports being printed out in some common
case.
Dave
The patch is safe since there's an
>
> if ((uintptr_t)host_startaddr & (rb->page_size - 1)) {
> error_report("ram_block_discard_range: Unaligned start address: %p",
> host_startaddr);
> goto err;
> }
>
> just before this context.
>
> Paolo
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK