[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH 00/18] Fix exception handling and msr/mrs access
From: |
Peter Maydell |
Subject: |
Re: [Qemu-arm] [PATCH 00/18] Fix exception handling and msr/mrs access |
Date: |
Fri, 20 Nov 2015 13:59:56 +0000 |
On 17 November 2015 at 17:07, Peter Maydell <address@hidden> wrote:
> On 9 November 2015 at 01:11, Michael Davidsaver <address@hidden> wrote:
>> This series grew from a previous incorrect patch attempting to fix
>> some incorrect behavior. After spending some time going through the
>> arch. ref. manual for v7-M I think I understand better how this should
>> work and have made a number of changes which actually improve the situation.
>>
>> These changes have not yet been cross checked against real hardware, and
>> I therefore don't consider them mergeable. It's gotten big enough though
>> that I'd like to get some feedback.
> I'll reply to the various patches individually with comments.
I think I've now done that at least for the earlier patches.
There are probably some other finer details that I'll get to
in a later round of patch review but hopefully you have enough
to do some of the fixes and restructuring of this patchset for v2.
I think the most important thing here is getting the structure
of the changes into patches right so they're easy to review.
The general principles here are:
* each patch should aim to be self-contained and to do one thing,
not several things (for instance, avoid making several bug fixes
in one patch, avoid putting "restructure/refactor code" and
"add new feature" in the same patch, and so on)
* at each point in the patch series QEMU needs to still compile
and run (this is particularly important to allow bisection
of bugs later on where people will want to be able to narrow
down which commit introduced a bug)
If the structure of the patchset is right it should be fairly
easy to review your improvements against the architecture manual.
thanks
-- PMM
- [Qemu-arm] [PATCH 07/18] armv7m: Update NVIC registers, (continued)
- [Qemu-arm] [PATCH 07/18] armv7m: Update NVIC registers, Michael Davidsaver, 2015/11/09
- [Qemu-arm] [PATCH 04/18] armv7m: Explicit error for bad vector table, Michael Davidsaver, 2015/11/09
- [Qemu-arm] [PATCH 12/18] armv7m: simpler/faster exception start, Michael Davidsaver, 2015/11/09
- [Qemu-arm] [PATCH 13/18] armv7m: implement CFSR and HFSR, Michael Davidsaver, 2015/11/09
- [Qemu-arm] [PATCH 01/18] armv7m: MRS/MSR handle unprivileged access, Michael Davidsaver, 2015/11/09
- [Qemu-arm] [PATCH 18/18] armv7m: prevent unprivileged write to STIR, Michael Davidsaver, 2015/11/09
- [Qemu-arm] [PATCH 16/18] armv7m: check exception return consistency, Michael Davidsaver, 2015/11/09
- Re: [Qemu-arm] [PATCH 00/18] Fix exception handling and msr/mrs access, Peter Maydell, 2015/11/17
- Re: [Qemu-arm] [PATCH 00/18] Fix exception handling and msr/mrs access,
Peter Maydell <=