On 05/17/2010 05:39 PM, Eric Blake wrote:
On 05/17/2010 03:27 PM, Chris Lalancette wrote:
>> + <dt><code>track</code></dt>
>> + <dd>
>> + The<code>track</code> attribute specifies what the timer
>> + tracks, and can be "boot", "guest", or
"wall".
>> + Only valid for<code>name="rtc"</code>
>> + or<code>name="platform"</code>.
>>
> Tough one. This doesn't tell people much about what they should be
> choosing for "track", and hence isn't that useful. You may want to
> add more about what this means. From what I can tell, it's actually
> keeping track of how the RTC is getting updated. If it's set to
"guest",
> then it is always set relative to the number of guest cycles that have
> passed. If it's set to "wall", it's relative to the actual time
that
> has passed from the host's POV. I'm not sure about "boot".
>
Now you see my quandary of why I asked for feedback! I'm documenting
someone else's work, and getting lost in all the options. Laine, since
you own the commit that added<timer>, would you care to lend a hand, so
we can make this documentation useful? :-)
Actually, documenting that had bee on my to-do list, but constantly
slipped due to the fact that I wrote the code based on specs provided by
others, so I don't consider myself to be all that knowledgeable about
the "why" of things either! :-)
I'll gladly help with shaping it up further, but first need a few days
to get my dev environment back into shape and check on a pretty ugly bug
that still bothers me.
Thanks for tackling this, btw. I agree that your time so far was well spent.