qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: qemu-devel@email.fries.net
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 1/1] Make vmport an optional feature at run time.
Date: Sat, 6 Sep 2008 22:48:22 -0500	[thread overview]
Message-ID: <20080907034822.GA31050@fries.net> (raw)
In-Reply-To: <48C33B6D.5080606@codemonkey.ws>

Maybe I'm missing the point of this exercise.  I can use the rtc and other
mechanisms to get the guests version of system time.

vmport I understand is to help the guest stay in sync with real time (i.e.
the hosts idea of time).

Am I missing something?
-- 
Todd Fries .. todd@fries.net

 _____________________________________________
|                                             \  1.636.410.0632 (voice)
| Free Daemon Consulting, LLC                 \  1.405.227.9094 (voice)
| http://FreeDaemonConsulting.com             \  1.866.792.3418 (FAX)
| "..in support of free software solutions."  \          250797 (FWD)
|                                             \
 \\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\
                                                 
              37E7 D3EB 74D0 8D66 A68D  B866 0326 204E 3F42 004A
                        http://todd.fries.net/pgp.txt

Penned by Anthony Liguori on 20080906 21:24.45, we have:
> Todd T. Fries wrote:
>> Just incase a 'more complete' vmport codeset is useful, here is my
>> stab at extending it.  I haven't extended it far enough yet that
>> OpenBSD's vmt(4) interface works, but would be more than happy
>> if someone else were to carry this flag and run with it..
>>
>>
>>
>> +static uint32_t vmport_cmd_get_time(void *opaque, uint32_t addr)
>> +{
>> +    CPUState *env = opaque;
>> +    struct timeval tv;
>> +    gettimeofday(&tv, NULL);
>> +    env->regs[R_EAX] = tv.tv_sec;
>> +    env->regs[R_EBX] = tv.tv_usec;
>> +    return tv.tv_sec;
>> +}
>>
>>   
>
> FWIW, this is incorrect.  You are presenting the host's version of wall  
> clock time, not the guest's version.
>
> Regards,
>
> Anthony Liguori

  reply	other threads:[~2008-09-07  3:49 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-24 11:47 [Qemu-devel] [PATCH 1/1] Make vmport an optional feature at run time Ian Kirk
2008-08-24 19:38 ` Anthony Liguori
2008-08-24 19:54   ` Ian Kirk
2008-08-25  0:23     ` Anthony Liguori
2008-08-25  6:19       ` [Qemu-devel] [PATCH] 1/1: vmport update Todd T. Fries
2008-09-07  2:24         ` [Qemu-devel] [PATCH 1/1] Make vmport an optional feature at run time Anthony Liguori
2008-09-07  3:48           ` qemu-devel [this message]
2008-08-25  8:34       ` Thiemo Seufer
2008-08-25  9:26         ` Ian Kirk
2008-09-07  2:27         ` Anthony Liguori

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=20080907034822.GA31050@fries.net \
    --to=qemu-devel@email.fries.net \
    --cc=anthony@codemonkey.ws \
    --cc=qemu-devel@nongnu.org \
    --cc=todd@fries.net \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).