[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] pl190: fix read of VECTADDR
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] pl190: fix read of VECTADDR |
Date: |
Sat, 18 Aug 2012 11:00:07 +0100 |
On 18 August 2012 03:55, Brendan Fennell <address@hidden> wrote:
> Signed-off-by: Brendan Fennell <address@hidden>
> ---
> hw/pl190.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/hw/pl190.c b/hw/pl190.c
> index cb50afb..d69d5be 100644
> --- a/hw/pl190.c
> +++ b/hw/pl190.c
> @@ -133,7 +133,7 @@ static uint64_t pl190_read(void *opaque,
> target_phys_addr_t offset,
> s->priority = i;
> pl190_update(s);
> }
> - return s->vect_addr[s->priority];
> + return s->vect_addr[s->priority - 1];
> case 13: /* DEFVECTADDR */
> return s->vect_addr[16];
> default:
This doesn't look right -- if s->priority is zero then we'll read off
the beginning of the array.
What's the actual bug you're trying to fix here?
-- PMM