On Fri, Jul 19, 2019 at 07:06:51PM +0100, Dr. David Alan Gilbert wrote:
>* Paolo Bonzini ([email protected]) 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
Dave,
Any further comment for this patch? or What should I do next?
>>
>> 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 / [email protected] / Manchester, UK
--
Wei Yang
Help you, Help me
