[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH v17 00/14] PTimer fixes/features and ARM MPTimer c
From: |
Dmitry Osipenko |
Subject: |
Re: [Qemu-arm] [PATCH v17 00/14] PTimer fixes/features and ARM MPTimer conversion |
Date: |
Mon, 24 Oct 2016 21:19:18 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.4.0 |
On 24.10.2016 15:23, Peter Maydell wrote:
> 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").
>
The "timer==0 is a real one-timer-cycle lump of time" is handled by the
"wraparound after one period" policy. "no counter round down" policy, probably,
could be used by default, but we wanted to keep the old ptimer behaviour
untouched. The rest of the policies handle running with/setting counter to 0
cases, I'm not sure that all timers share same behaviour for those cases. There
is always a room for improvement :)
> 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...
>
Yay! That took a while. Now we can move to the next ptimer issue, like
period/freq change glitch :)
--
Dmitry
- [Qemu-arm] [PATCH v17 01/14] hw/ptimer: Add "wraparound after one period" policy, (continued)
- [Qemu-arm] [PATCH v17 01/14] hw/ptimer: Add "wraparound after one period" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-arm] [PATCH v17 04/14] tests: ptimer: Add tests for "continuous trigger" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-arm] [PATCH v17 03/14] hw/ptimer: Add "continuous trigger" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-arm] [PATCH v17 05/14] hw/ptimer: Add "no immediate trigger" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-arm] [PATCH v17 08/14] tests: ptimer: Add tests for "no immediate reload" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-arm] [PATCH v17 10/14] tests: ptimer: Add tests for "no counter round down" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-arm] [PATCH v17 06/14] tests: ptimer: Add tests for "no immediate trigger" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-arm] [PATCH v17 02/14] tests: ptimer: Add tests for "wraparound after one period" policy, Dmitry Osipenko, 2016/10/02
- [Qemu-arm] [PATCH v17 09/14] hw/ptimer: Add "no counter round down" policy, Dmitry Osipenko, 2016/10/02
- Re: [Qemu-arm] [PATCH v17 00/14] PTimer fixes/features and ARM MPTimer conversion, Peter Maydell, 2016/10/24
- Re: [Qemu-arm] [PATCH v17 00/14] PTimer fixes/features and ARM MPTimer conversion,
Dmitry Osipenko <=