[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/5] pseries: Add spapr_rtc_read() helper functi
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH 2/5] pseries: Add spapr_rtc_read() helper function |
Date: |
Tue, 16 Dec 2014 12:25:06 +1100 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Tue, Dec 16, 2014 at 02:01:29AM +0100, Alexander Graf wrote:
>
>
> On 16.12.14 01:43, David Gibson wrote:
> > The virtual RTC time is used in two places in the pseries machine. First
> > is in the RTAS get-time-of-day function which returns the RTC time to the
> > guest. Second is in the spapr events code which is used to timestamp
> > event messages from the hypervisor to the guest.
> >
> > Currently both call qemu_get_timedate() directly, but we want to change
> > that so we can properly handle the various -rtc options. In preparation,
> > create a helper function to return the virtual RTC time.
> >
> > Signed-off-by: David Gibson <address@hidden>
> > ---
> > hw/ppc/spapr_events.c | 2 +-
> > hw/ppc/spapr_rtc.c | 12 ++++++++++--
> > include/hw/ppc/spapr.h | 1 +
> > 3 files changed, 12 insertions(+), 3 deletions(-)
> >
> > diff --git a/hw/ppc/spapr_events.c b/hw/ppc/spapr_events.c
> > index 1b6157d..80c0266 100644
> > --- a/hw/ppc/spapr_events.c
> > +++ b/hw/ppc/spapr_events.c
> > @@ -246,7 +246,7 @@ static void spapr_powerdown_req(Notifier *n, void
> > *opaque)
> > maina->hdr.section_id = cpu_to_be16(RTAS_LOG_V6_SECTION_ID_MAINA);
> > maina->hdr.section_length = cpu_to_be16(sizeof(*maina));
> > /* FIXME: section version, subtype and creator id? */
> > - qemu_get_timedate(&tm, spapr->rtc_offset);
> > + spapr_rtc_read(spapr, &tm, NULL);
> > year = tm.tm_year + 1900;
> > maina->creation_date = cpu_to_be32((to_bcd(year / 100) << 24)
> > | (to_bcd(year % 100) << 16)
> > diff --git a/hw/ppc/spapr_rtc.c b/hw/ppc/spapr_rtc.c
> > index e290ac0..9ccefbc 100644
> > --- a/hw/ppc/spapr_rtc.c
> > +++ b/hw/ppc/spapr_rtc.c
> > @@ -29,19 +29,27 @@
> > #include "hw/ppc/spapr.h"
> > #include "qapi-event.h"
> >
> > +void spapr_rtc_read(sPAPREnvironment *spapr, struct tm *tm, uint32_t *ns)
> > +{
> > + qemu_get_timedate(tm, spapr->rtc_offset);
> > + if (ns)
>
> checkpatch? ;)
Sod. It's been too long since I was doing much upstream work :).
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
pgpLcCZUdeD8r.pgp
Description: PGP signature
- [Qemu-devel] [PATCH 0/5] pseries: Fix and extend PAPR RTC implementation, David Gibson, 2014/12/15
- [Qemu-devel] [PATCH 2/5] pseries: Add spapr_rtc_read() helper function, David Gibson, 2014/12/15
- [Qemu-devel] [PATCH 5/5] pseries: Make RTAS time of day functions respect -rtc options, David Gibson, 2014/12/15
- [Qemu-devel] [PATCH 1/5] pseries: Move sPAPR RTC code into its own file, David Gibson, 2014/12/15
- Re: [Qemu-devel] [PATCH 1/5] pseries: Move sPAPR RTC code into its own file, Alexander Graf, 2014/12/15
- Re: [Qemu-devel] [PATCH 1/5] pseries: Move sPAPR RTC code into its own file, David Gibson, 2014/12/15
- Re: [Qemu-devel] [PATCH 1/5] pseries: Move sPAPR RTC code into its own file, Alexander Graf, 2014/12/16
- Re: [Qemu-devel] [PATCH 1/5] pseries: Move sPAPR RTC code into its own file, David Gibson, 2014/12/18
- Re: [Qemu-devel] [PATCH 1/5] pseries: Move sPAPR RTC code into its own file, Alexander Graf, 2014/12/18
- Re: [Qemu-devel] [PATCH 1/5] pseries: Move sPAPR RTC code into its own file, David Gibson, 2014/12/19
- Re: [Qemu-devel] [PATCH 1/5] pseries: Move sPAPR RTC code into its own file, Alexander Graf, 2014/12/19
[Qemu-devel] [PATCH 3/5] pseries: Add more parameter validation in RTAS time of day functions, David Gibson, 2014/12/15