Re: Fwd: [PATCH v4.15.7 1/1] on Intel, VDSO should handle CLOCK_MONOTONIC_RAW and export 'tsc_calibration' pointer

2018-03-10 Thread Jason Vas Dias
Hi Thomas - Thanks very much for your help & guidance in previous mail: RE: On 08/03/2018, Thomas Gleixner wrote: > > The right way to do that is to put the raw conversion values and the raw > seconds base value into the vdso data and implement the counterpart of > getrawmonotonic64(). And if t

Re: Fwd: [PATCH v4.15.7 1/1] on Intel, VDSO should handle CLOCK_MONOTONIC_RAW and export 'tsc_calibration' pointer

2018-03-08 Thread Thomas Gleixner
On Thu, 8 Mar 2018, Jason Vas Dias wrote: > On 08/03/2018, Thomas Gleixner wrote: > I'd appreciate clarification on few points : > > > Well, you did not expose the raw conversion data in vsyscall_gtod_data. You > > are using: > > > > + tsc*= gtod->mult; > > + tsc >>= gtod->s

Re: Fwd: [PATCH v4.15.7 1/1] on Intel, VDSO should handle CLOCK_MONOTONIC_RAW and export 'tsc_calibration' pointer

2018-03-08 Thread Jason Vas Dias
On 08/03/2018, Thomas Gleixner wrote: > On Tue, 6 Mar 2018, Jason Vas Dias wrote: >> I will prepare a new patch that meets submission + coding style guidelines >> and >> does not expose pointers within the vsyscall_gtod_data region to >> user-space code - >> but I don't really understand why not,

Re: Fwd: [PATCH v4.15.7 1/1] on Intel, VDSO should handle CLOCK_MONOTONIC_RAW and export 'tsc_calibration' pointer

2018-03-08 Thread Thomas Gleixner
On Tue, 6 Mar 2018, Jason Vas Dias wrote: > I will prepare a new patch that meets submission + coding style guidelines and > does not expose pointers within the vsyscall_gtod_data region to > user-space code - > but I don't really understand why not, since only the gtod->mult value will > change as

Fwd: [PATCH v4.15.7 1/1] on Intel, VDSO should handle CLOCK_MONOTONIC_RAW and export 'tsc_calibration' pointer

2018-03-06 Thread Jason Vas Dias
On 06/03/2018, Thomas Gleixner wrote: > Jason, > > On Mon, 5 Mar 2018, Jason Vas Dias wrote: > > thanks for providing this. A few formal nits first. > > Please read Documentation/process/submitting-patches.rst > > Patches need a concise subject line and the subject line wants a prefix, in > this c