[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 2/3] xlnx-zynqmp-rtc: Add basic time support
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v3 2/3] xlnx-zynqmp-rtc: Add basic time support |
Date: |
Wed, 17 Jan 2018 17:48:04 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2 |
On 01/17/2018 02:27 PM, Alistair Francis wrote:
> Allow the guest to determine the time set from the QEMU command line.
>
> This includes adding a trace event to debug the new time.
>
> The migration logic was copied from the PL031 migration logic.
>
> Signed-off-by: Alistair Francis <address@hidden>
> ---
> V3:
> - Store an offset value
> - Use mktimegm()
> - Log unimplemented writes
> V2:
> - Convert DB_PRINT() macro to trace
>
> hw/timer/trace-events | 3 ++
> hw/timer/xlnx-zynqmp-rtc.c | 64
> ++++++++++++++++++++++++++++++++++++++
> include/hw/timer/xlnx-zynqmp-rtc.h | 4 +++
> 3 files changed, 71 insertions(+)
>
> diff --git a/hw/timer/trace-events b/hw/timer/trace-events
> index 640722b5d1..e6e042fddb 100644
> --- a/hw/timer/trace-events
> +++ b/hw/timer/trace-events
> @@ -60,3 +60,6 @@ systick_write(uint64_t addr, uint32_t value, unsigned size)
> "systick write addr
> cmsdk_apb_timer_read(uint64_t offset, uint64_t data, unsigned size) "CMSDK
> APB timer read: offset 0x%" PRIx64 " data 0x%" PRIx64 " size %u"
> cmsdk_apb_timer_write(uint64_t offset, uint64_t data, unsigned size) "CMSDK
> APB timer write: offset 0x%" PRIx64 " data 0x%" PRIx64 " size %u"
> cmsdk_apb_timer_reset(void) "CMSDK APB timer: reset"
> +
> +# hw/timer/xlnx-zynqmp-rtc.c
> +xlnx_zynqmp_rtc_gettime(int year, int month, int day, int hour, int min, int
> sec) "Get time from host: %d-%d-%d %2d:%02d:%02d"
> diff --git a/hw/timer/xlnx-zynqmp-rtc.c b/hw/timer/xlnx-zynqmp-rtc.c
> index ead40fc42d..76c1af1afe 100644
> --- a/hw/timer/xlnx-zynqmp-rtc.c
> +++ b/hw/timer/xlnx-zynqmp-rtc.c
> @@ -29,6 +29,10 @@
> #include "hw/register.h"
> #include "qemu/bitops.h"
> #include "qemu/log.h"
> +#include "hw/ptimer.h"
> +#include "qemu/cutils.h"
> +#include "sysemu/sysemu.h"
> +#include "trace.h"
> #include "hw/timer/xlnx-zynqmp-rtc.h"
>
> #ifndef XLNX_ZYNQMP_RTC_ERR_DEBUG
> @@ -47,6 +51,19 @@ static void addr_error_int_update_irq(XlnxZynqMPRTC *s)
> qemu_set_irq(s->irq_addr_error_int, pending);
> }
>
> +static uint32_t rtc_get_count(XlnxZynqMPRTC *s)
> +{
> + int64_t now = qemu_clock_get_ns(rtc_clock);
> + return s->tick_offset + now / NANOSECONDS_PER_SECOND;
> +}
> +
> +static uint64_t current_time_postr(RegisterInfo *reg, uint64_t val64)
> +{
> + XlnxZynqMPRTC *s = XLNX_ZYNQMP_RTC(reg->opaque);
> +
> + return rtc_get_count(s);
> +}
> +
> static void rtc_int_status_postw(RegisterInfo *reg, uint64_t val64)
> {
> XlnxZynqMPRTC *s = XLNX_ZYNQMP_RTC(reg->opaque);
> @@ -99,15 +116,25 @@ static uint64_t addr_error_int_dis_prew(RegisterInfo
> *reg, uint64_t val64)
> return 0;
> }
>
> +static void unimp_postw(RegisterInfo *reg, uint64_t val64)
> +{
> + qemu_log_mask(LOG_UNIMP, "The time setting functionality is not "
> + "implemented\n");
> +}
> +
> static const RegisterAccessInfo rtc_regs_info[] = {
> { .name = "SET_TIME_WRITE", .addr = A_SET_TIME_WRITE,
> + .post_write = unimp_postw,
Why bother adding this function? Rather use ".unimp = ~0" which log
register name accessed.
> },{ .name = "SET_TIME_READ", .addr = A_SET_TIME_READ,
> .ro = 0xffffffff,
> + .post_read = current_time_postr,
> },{ .name = "CALIB_WRITE", .addr = A_CALIB_WRITE,
> + .post_write = unimp_postw,
Ditto.
> },{ .name = "CALIB_READ", .addr = A_CALIB_READ,
> .ro = 0x1fffff,
> },{ .name = "CURRENT_TIME", .addr = A_CURRENT_TIME,
> .ro = 0xffffffff,
> + .post_read = current_time_postr,
> },{ .name = "CURRENT_TICK", .addr = A_CURRENT_TICK,
> .ro = 0xffff,
> },{ .name = "ALARM", .addr = A_ALARM,
> @@ -147,6 +174,10 @@ static void rtc_reset(DeviceState *dev)
> register_reset(&s->regs_info[i]);
> }
>
> + trace_xlnx_zynqmp_rtc_gettime(s->current_tm.tm_year,
> s->current_tm.tm_mon,
> + s->current_tm.tm_mday,
> s->current_tm.tm_hour,
> + s->current_tm.tm_min,
> s->current_tm.tm_sec);
> +
> rtc_int_update_irq(s);
> addr_error_int_update_irq(s);
> }
> @@ -182,14 +213,47 @@ static void rtc_init(Object *obj)
> sysbus_init_mmio(sbd, &s->iomem);
> sysbus_init_irq(sbd, &s->irq_rtc_int);
> sysbus_init_irq(sbd, &s->irq_addr_error_int);
> +
> + qemu_get_timedate(&s->current_tm, 0);
> + s->tick_offset = mktimegm(&s->current_tm) -
> + qemu_clock_get_ns(rtc_clock) / NANOSECONDS_PER_SECOND;
> +}
> +
> +static int rtc_pre_save(void *opaque)
> +{
> + XlnxZynqMPRTC *s = opaque;
> +
> + /* tick_offset is base_time - rtc_clock base time. Instead, we want to
> + * store the base time relative to the QEMU_CLOCK_VIRTUAL for
> + * backwards-compatibility.
> + */
> + int64_t delta = qemu_clock_get_ns(rtc_clock) -
> + qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL);
> + s->tick_offset_vmstate = s->tick_offset + delta / NANOSECONDS_PER_SECOND;
> +
> + return 0;
> +}
> +
> +static int rtc_post_load(void *opaque, int version_id)
> +{
> + XlnxZynqMPRTC *s = opaque;
> +
> + int64_t delta = qemu_clock_get_ns(rtc_clock) -
> + qemu_clock_get_ns(QEMU_CLOCK_VIRTUAL);
> + s->tick_offset = s->tick_offset_vmstate - delta / NANOSECONDS_PER_SECOND;
> +
> + return 0;
> }
>
> static const VMStateDescription vmstate_rtc = {
> .name = TYPE_XLNX_ZYNQMP_RTC,
> .version_id = 1,
> .minimum_version_id = 1,
> + .pre_save = rtc_pre_save,
> + .post_load = rtc_post_load,
> .fields = (VMStateField[]) {
> VMSTATE_UINT32_ARRAY(regs, XlnxZynqMPRTC, XLNX_ZYNQMP_RTC_R_MAX),
> + VMSTATE_UINT32(tick_offset_vmstate, XlnxZynqMPRTC),
> VMSTATE_END_OF_LIST(),
> }
> };
> diff --git a/include/hw/timer/xlnx-zynqmp-rtc.h
> b/include/hw/timer/xlnx-zynqmp-rtc.h
> index 87649836cc..b0f54e00c1 100644
> --- a/include/hw/timer/xlnx-zynqmp-rtc.h
> +++ b/include/hw/timer/xlnx-zynqmp-rtc.h
> @@ -79,6 +79,10 @@ typedef struct XlnxZynqMPRTC {
> qemu_irq irq_rtc_int;
> qemu_irq irq_addr_error_int;
>
> + struct tm current_tm;
> + uint32_t tick_offset_vmstate;
> + uint32_t tick_offset;
> +
> uint32_t regs[XLNX_ZYNQMP_RTC_R_MAX];
> RegisterInfo regs_info[XLNX_ZYNQMP_RTC_R_MAX];
> } XlnxZynqMPRTC;
> --
> 2.14.1
removing unimp_postw():
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>