[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v17 00/14] PTimer fixes/features and ARM MPTimer
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v17 00/14] PTimer fixes/features and ARM MPTimer conversion |
Date: |
Mon, 24 Oct 2016 13:23:17 +0100 |
On 2 October 2016 at 16:53, Dmitry Osipenko <address@hidden> wrote:
> Hello,
>
> Currently, QEMU ARM MPTimer device model provides only a certain subset of
> the emulation behavior. This patch series is supposed to add missing parts by
> converting the MPTimer to use generic ptimer helper. It fixes some important
> ptimer bugs and provides new features that are required for the ARM MPTimer.
>
> WARNING! I based V17 on top of the Paolo's patch [0], however I don't see
> the original mail of that patch on the ML nor in patches/patchew.
>
> [0] https://lists.nongnu.org/archive/html/qemu-devel/2016-09/msg06734.html
Looking at the code we end up with in ptimer, we seem to do an
awful lot of adding and subtracting 1 everywhere. That makes me
wonder if we're missing a simplification which would collapse
all of those out (it seems unlikely that hardware would really
ever want some of the policy flags but not all of them, since
I think they boil down to "timer==0 is a real one-timer-cycle
lump of time").
That said, I think the behaviour is right and this patchseries has
been around way too long already, so I've applied it to target-arm.next.
If we can think of a simplification we can always apply it later
as a refactoring with a fair degree of confidence given the tests...
thanks
-- PMM
- [Qemu-devel] [PATCH v17 01/14] hw/ptimer: Add "wraparound after one period" policy, (continued)
- [Qemu-devel] [PATCH v17 01/14] hw/ptimer: Add "wraparound after one period" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-devel] [PATCH v17 03/14] hw/ptimer: Add "continuous trigger" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-devel] [PATCH v17 04/14] tests: ptimer: Add tests for "continuous trigger" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-devel] [PATCH v17 05/14] hw/ptimer: Add "no immediate trigger" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-devel] [PATCH v17 02/14] tests: ptimer: Add tests for "wraparound after one period" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-devel] [PATCH v17 06/14] tests: ptimer: Add tests for "no immediate trigger" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-devel] [PATCH v17 07/14] hw/ptimer: Add "no immediate reload" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-devel] [PATCH v17 08/14] tests: ptimer: Add tests for "no immediate reload" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-devel] [PATCH v17 09/14] hw/ptimer: Add "no counter round down" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-devel] [PATCH v17 10/14] tests: ptimer: Add tests for "no counter round down" policy, Dmitry Osipenko, 2016/10/02
- Re: [Qemu-devel] [PATCH v17 00/14] PTimer fixes/features and ARM MPTimer conversion,
Peter Maydell <=