[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 0/9] remove useless muldiv64()
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-devel] [PATCH v3 0/9] remove useless muldiv64() |
Date: |
Thu, 24 Sep 2015 18:21:24 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 |
Hi,
all the patches of the series have been reviewed, except:
[PATCH v4 1/9] i6300esb: remove muldiv64(),
which is a rebased version.
[PATCH v3 5/9] openrisc: remove muldiv64()
Any volunteers ?
Laurent
On 27/08/2015 21:32, Laurent Vivier wrote:
> Originally, timers were ticks based, and it made sense to
> add ticks to current time to know when to trigger an alarm.
>
> But since commit:
>
> 7447545 change all other clock references to use nanosecond resolution
> accessors
>
> All timers use nanoseconds and we need to convert ticks to nanoseconds, by
> doing something like:
>
> y = muldiv64(x, get_ticks_per_sec(), TIMER_FREQ)
>
> where x is the number of device ticks and y the number of system ticks.
>
> y is used as nanoseconds in timer functions,
> it works because 1 tick is 1 nanosecond.
> (get_ticks_per_sec() is 10^9)
>
> But if get_ticks_per_sec() / TIMER_FREQ is an integer, we can do:
>
> y = x * TIMER_PERIOD;
>
> v3:
> Split "PCI: remove muldiv64()" in 3 patches:
> i6300esb: remove muldiv64()
> rtl8139: remove muldiv64()
> pcnet: remove muldiv64()
> v2:
> 4/4 For target-arm, don't remove muldiv64() but clarify
> the use of the values.
> 7/7 Replace qemu_clock_get_ns()/1000 by qemu_clock_get_us()
>
> Laurent Vivier (9):
> i6300esb: remove muldiv64()
> rtl8139: remove muldiv64()
> pcnet: remove muldiv64()
> mips: remove muldiv64()
> openrisc: remove muldiv64()
> arm: clarify the use of muldiv64()
> hpet: remove muldiv64()
> bt: remove muldiv64()
> net: remove muldiv64()
>
> hw/bt/hci.c | 4 ++--
> hw/mips/cputimer.c | 19 ++++++++-----------
> hw/net/pcnet.c | 3 +--
> hw/net/rtl8139.c | 14 ++++++--------
> hw/openrisc/cputimer.c | 7 +++----
> hw/timer/hpet.c | 6 +++---
> hw/watchdog/wdt_i6300esb.c | 11 +++--------
> include/hw/timer/hpet.h | 4 ++--
> net/dump.c | 2 +-
> target-arm/helper.c | 14 ++++++++------
> tests/rtl8139-test.c | 2 +-
> 11 files changed, 38 insertions(+), 48 deletions(-)
>
- Re: [Qemu-devel] [PATCH v3 0/9] remove useless muldiv64(),
Laurent Vivier <=