| Message ID | 1396572808-21535-1-git-send-email-akong@redhat.com |
|---|---|
| State | New |
| Headers | show |
On 04.04.2014 02:53, Amos Kong wrote: > We mixedly used "guest time", "system time", "hardware time", "RTC" > in document, it's unclear. > > This patch just added two remarks of RTC and replace two "guest time" > by "guest's system time". > > Signed-off-by: Amos Kong <akong@redhat.com> Reviewed-by: Michal Privoznik <mprivozn@redhat.com> Michal
On 04/03/2014 06:53 PM, Amos Kong wrote: > We mixedly used "guest time", "system time", "hardware time", "RTC" s/mixedly used/mixed the use of/ > in document, it's unclear. s/document/documentation/ > > This patch just added two remarks of RTC and replace two "guest time" > by "guest's system time". > > Signed-off-by: Amos Kong <akong@redhat.com> > --- > qga/commands-posix.c | 2 +- > qga/qapi-schema.json | 14 +++++++------- > 2 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/qga/commands-posix.c b/qga/commands-posix.c > index 6b5f11f..25e4302 100644 > --- a/qga/commands-posix.c > +++ b/qga/commands-posix.c > @@ -171,7 +171,7 @@ void qmp_guest_set_time(bool has_time, int64_t time_ns, Error **errp) > /* Now, if user has passed a time to set and the system time is set, we > * just need to synchronize the hardware clock. However, if no time was > * passed, user is requesting the opposite: set the system time from the > - * hardware clock. */ > + * hardware clock(RTC). */ s/clock(/clock (/ > -# This command tries to set guest time to the given value, > -# then sets the Hardware Clock to the current System Time. > -# This will make it easier for a guest to resynchronize > -# without waiting for NTP. If no @time is specified, then > -# the time to set is read from RTC. > +# This command tries to set guest's System Time to the > +# given value, then sets the Hardware Clock(RTC) to the and again With those changes, Reviewed-by: Eric Blake <eblake@redhat.com>
diff --git a/qga/commands-posix.c b/qga/commands-posix.c index 6b5f11f..25e4302 100644 --- a/qga/commands-posix.c +++ b/qga/commands-posix.c @@ -171,7 +171,7 @@ void qmp_guest_set_time(bool has_time, int64_t time_ns, Error **errp) /* Now, if user has passed a time to set and the system time is set, we * just need to synchronize the hardware clock. However, if no time was * passed, user is requesting the opposite: set the system time from the - * hardware clock. */ + * hardware clock(RTC). */ pid = fork(); if (pid == 0) { setsid(); diff --git a/qga/qapi-schema.json b/qga/qapi-schema.json index 80edca1..de75005 100644 --- a/qga/qapi-schema.json +++ b/qga/qapi-schema.json @@ -96,8 +96,8 @@ ## # @guest-get-time: # -# Get the information about guest time relative to the Epoch -# of 1970-01-01 in UTC. +# Get the information about guest's System Time relative to +# the Epoch of 1970-01-01 in UTC. # # Returns: Time in nanoseconds. # @@ -117,11 +117,11 @@ # gap was, NTP might not be able to resynchronize the # guest. # -# This command tries to set guest time to the given value, -# then sets the Hardware Clock to the current System Time. -# This will make it easier for a guest to resynchronize -# without waiting for NTP. If no @time is specified, then -# the time to set is read from RTC. +# This command tries to set guest's System Time to the +# given value, then sets the Hardware Clock(RTC) to the +# current System Time. This will make it easier for a guest +# to resynchronize without waiting for NTP. If no @time is +# specified, then the time to set is read from RTC. # # @time: #optional time of nanoseconds, relative to the Epoch # of 1970-01-01 in UTC.
We mixedly used "guest time", "system time", "hardware time", "RTC" in document, it's unclear. This patch just added two remarks of RTC and replace two "guest time" by "guest's system time". Signed-off-by: Amos Kong <akong@redhat.com> --- qga/commands-posix.c | 2 +- qga/qapi-schema.json | 14 +++++++------- 2 files changed, 8 insertions(+), 8 deletions(-)