Re: [Xen-devel] [PATCH v3 4/6] x86/time: refactor read_platform_stime()

2016-08-29 Thread Jan Beulich
>>> On 26.08.16 at 17:13, wrote: > On 08/25/2016 11:17 AM, Jan Beulich wrote: > On 24.08.16 at 14:43, wrote: >>> In the case of clocksource=tsc we will >>> use it to set tsc_timestamp. >> >> I don't see how this relates to the patch here. >> >> The change itself looks reasonable assuming it

Re: [Xen-devel] [PATCH v3 4/6] x86/time: refactor read_platform_stime()

2016-08-26 Thread Joao Martins
On 08/25/2016 11:17 AM, Jan Beulich wrote: On 24.08.16 at 14:43, wrote: >> To fetch the last read from the clocksource which was used to >> calculate system_time. > > DYM "To allow the caller to fetch ..."? Yeap, sounds better that way. > >> In the case of clocksource=tsc we will >> use

Re: [Xen-devel] [PATCH v3 4/6] x86/time: refactor read_platform_stime()

2016-08-25 Thread Jan Beulich
>>> On 24.08.16 at 14:43, wrote: > To fetch the last read from the clocksource which was used to > calculate system_time. DYM "To allow the caller to fetch ..."? > In the case of clocksource=tsc we will > use it to set tsc_timestamp. I don't see how this relates to the patch here. The change i

[Xen-devel] [PATCH v3 4/6] x86/time: refactor read_platform_stime()

2016-08-24 Thread Joao Martins
To fetch the last read from the clocksource which was used to calculate system_time. In the case of clocksource=tsc we will use it to set tsc_timestamp. Signed-off-by: Joao Martins --- Cc: Jan Beulich Cc: Andrew Cooper Changes since v2: - s/plt_stamp_counter/plt_counter/g - Move conditional