[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 01/13] cg3: remove TARGET_PAGE_SIZE rounding
From: |
Mark Cave-Ayland |
Subject: |
Re: [Qemu-devel] [PATCH v2 01/13] cg3: remove TARGET_PAGE_SIZE rounding on dirty page detection |
Date: |
Mon, 1 May 2017 07:12:11 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
On 25/04/17 15:57, Peter Maydell wrote:
> On 21 April 2017 at 09:28, Mark Cave-Ayland
> <address@hidden> wrote:
>> This was an artifact from very early versions of the code from before the
>> memory API and is no longer needed.
>>
>> Signed-off-by: Mark Cave-Ayland <address@hidden>
>> Reviewed-by: Gerd Hoffmann <address@hidden>
>> ---
>> hw/display/cg3.c | 6 ++----
>> 1 file changed, 2 insertions(+), 4 deletions(-)
>>
>> diff --git a/hw/display/cg3.c b/hw/display/cg3.c
>> index 1174220..7d43694 100644
>> --- a/hw/display/cg3.c
>> +++ b/hw/display/cg3.c
>> @@ -26,7 +26,6 @@
>> #include "qemu/osdep.h"
>> #include "qapi/error.h"
>> #include "qemu-common.h"
>> -#include "cpu.h"
>> #include "qemu/error-report.h"
>> #include "ui/console.h"
>> #include "hw/sysbus.h"
>> @@ -114,7 +113,7 @@ static void cg3_update_display(void *opaque)
>> for (y = 0; y < height; y++) {
>> int update = s->full_update;
>>
>> - page = (y * width) & TARGET_PAGE_MASK;
>> + page = y * width;
>
> Coverity warns that this is multiplying two 32 bit quantities
> and giving a 32 bit result that's then assigned to a 64 bit variable.
> Casting like:
> page = (ram_addr_t)y * width;
>
> is the usual way to placate it.
Ah okay. I've done a quick test here and the above change seems fine so
I'll send along a patch.
ATB,
Mark.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v2 01/13] cg3: remove TARGET_PAGE_SIZE rounding on dirty page detection,
Mark Cave-Ayland <=