Re: [Qemu-devel] [PATCH v3] pl190: fix read of VECTADDR

[Brendan Fennell <bfennell@skynet.ie>]

On Mon, 20 Aug 2012, Peter Maydell wrote:

> On 20 August 2012 18:59, Brendan Fennell <bfennell@skynet.ie> wrote:
>> Signed-off-by: Brendan Fennell <bfennell@skynet.ie>
>
> Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
>
> The () inside the [] aren't actually necessary, but I'm just
> going to fix those as I take this patch into the arm-devs
> queue, I think. I've also expanded the commit message a little:
>
> =====
> Reading VECTADDR was causing us to set the current priority to
> the wrong value, the most obvious effect of which was that we
> would return the vector for the wrong interrupt as the result
> of the read.
> =====
>
> What guest did you see this problem with? Since the QEMU 1.2
> release is now less than 2 weeks away, and this bug has been
> present since the versatilepb platform was added 6 years ago,
> I'm reluctant to squeeze it into this release unless it's
> going to be hit by a lot of people. (1.3 is only 3 months
> later so it's not a huge deal for a minor fix to miss 1.2.)

I'm working on a custom application, not a standard guest OS. I think it's 
safe to say this can wait for 1.3.

Brendan.

>
> thanks
> -- PMM
>
>