All of lore.kernel.org
 help / color / mirror / Atom feed
From: Avi Kivity <avi@qumranet.com>
To: Dan Kenigsberg <danken@qumranet.com>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [Patch] [kinda-resend] persistent real-time-clock
Date: Mon, 28 Apr 2008 10:11:12 +0300	[thread overview]
Message-ID: <48157890.7070101@qumranet.com> (raw)
In-Reply-To: <20080428065438.GA6184@karma.qumranet.com>

Dan Kenigsberg wrote:
> On Sun, Apr 27, 2008 at 07:39:42PM +0300, Avi Kivity wrote:
>   
>> Dan Kenigsberg wrote:
>>     
>>> Real PC lets its user set the real-time-clock and store it on CMOS,
>>> which advances the clock even when the PC is offline.
>>>
>>> These patches will allow doing the same with VM:
>>> - Before shutting a VM down, use the monitor's info timeoffset to see
>>>   how much (in seconds) does the rtc differ from the host clock.
>>> - Store this offset somewhere.
>>> - Use it next time with -startdate now+offset.
>>>
>>> Please tell me what do you think of it.
>>>   
>>>       
>> How about (additionally) modifying -startdate to accept an offset directly?
>>
>>   qemu -startdate +0300 ...
>>
>> Useful for those Windows VMs which have localtime in cmos.
>>     
>
> I'm missing somthing: in what way is it different than the suggested
>
>     qemu -startdate now+offset ?
>
> Only in allowing to specify the offset in hours/minutes?
>
>   

The difference is that I didn't read your patch, only the description.

I assumed you meant the user calculates now + offset and enters the 
result as the argument to -startdate, but you're not doing that.  Sorry 
for the noise.

-- 
Any sufficiently difficult bug is indistinguishable from a feature.

  reply	other threads:[~2008-04-28  7:12 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-27 15:45 [Qemu-devel] [Patch] [kinda-resend] persistent real-time-clock Dan Kenigsberg
2008-04-27 16:39 ` Avi Kivity
2008-04-28  6:54   ` Dan Kenigsberg
2008-04-28  7:11     ` Avi Kivity [this message]
2008-04-30 12:22   ` Sergey Bychkov
2008-04-27 16:58 ` Anthony Liguori
2008-04-28  8:00   ` Dan Kenigsberg
2008-05-01  8:48   ` Dan Kenigsberg
2008-05-05 20:23     ` Anthony Liguori
2008-05-05 21:13       ` Avi Kivity
2008-05-05 22:38         ` Anthony Liguori
2008-05-06  8:26           ` Tristan Gingold
2008-05-06  8:53             ` Laurent Vivier
2008-04-27 21:28 ` Jamie Lokier
2008-04-28  7:27   ` Dan Kenigsberg
2008-04-30 12:21   ` Sergey Bychkov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=48157890.7070101@qumranet.com \
    --to=avi@qumranet.com \
    --cc=danken@qumranet.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.