RE: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2017-01-26 Thread Stanton, Kevin B
On Thurs, 15 Dec 2016, Thomas Gleixner wrote: >See the SDM. It suggests that the deadline is compared to the TSC value. >I don't care how it is implemented, but I very much care about it being >documented in the way it is implemented, which is obviously not the case. The TSC_DEADLINE behavior

RE: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2017-01-26 Thread Stanton, Kevin B
On Thurs, 15 Dec 2016, Thomas Gleixner wrote: >See the SDM. It suggests that the deadline is compared to the TSC value. >I don't care how it is implemented, but I very much care about it being >documented in the way it is implemented, which is obviously not the case. The TSC_DEADLINE behavior

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-15 Thread Thomas Gleixner
On Wed, 14 Dec 2016, Roland Scheidegger wrote: > Am 14.12.2016 um 22:40 schrieb Thomas Gleixner: > > And the whole mess stems from the fact that the deadline is not as one > > would expect simply compared against the sum of the counter and the adjust > > MSR. > Why would it be compared against the

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-15 Thread Thomas Gleixner
On Wed, 14 Dec 2016, Roland Scheidegger wrote: > Am 14.12.2016 um 22:40 schrieb Thomas Gleixner: > > And the whole mess stems from the fact that the deadline is not as one > > would expect simply compared against the sum of the counter and the adjust > > MSR. > Why would it be compared against the

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-14 Thread Roland Scheidegger
Am 14.12.2016 um 22:40 schrieb Thomas Gleixner: > On Wed, 14 Dec 2016, Thomas Gleixner wrote: >> Positive space, results in timer not firing anymore - at least not in a >> time frame you are willing to wait for. >> >> 0x 8000 >> >> Negative space, results in an interrupt storm.

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-14 Thread Roland Scheidegger
Am 14.12.2016 um 22:40 schrieb Thomas Gleixner: > On Wed, 14 Dec 2016, Thomas Gleixner wrote: >> Positive space, results in timer not firing anymore - at least not in a >> time frame you are willing to wait for. >> >> 0x 8000 >> >> Negative space, results in an interrupt storm.

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-14 Thread Thomas Gleixner
On Wed, 14 Dec 2016, Thomas Gleixner wrote: > Positive space, results in timer not firing anymore - at least not in a > time frame you are willing to wait for. > > 0x 8000 > > Negative space, results in an interrupt storm. > > 0x > 0x fffe

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-14 Thread Thomas Gleixner
On Wed, 14 Dec 2016, Thomas Gleixner wrote: > Positive space, results in timer not firing anymore - at least not in a > time frame you are willing to wait for. > > 0x 8000 > > Negative space, results in an interrupt storm. > > 0x > 0x fffe

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-14 Thread Thomas Gleixner
On Wed, 14 Dec 2016, Thomas Gleixner wrote: > On Wed, 14 Dec 2016, Roland Scheidegger wrote: > > Am 13.12.2016 um 17:46 schrieb Thomas Gleixner: > > > What are the adjust values after a warm boot? > > > > So, after cold boot with a kernel which doesn't adjust TSCs, then warm > > boot I got: > > [

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-14 Thread Thomas Gleixner
On Wed, 14 Dec 2016, Thomas Gleixner wrote: > On Wed, 14 Dec 2016, Roland Scheidegger wrote: > > Am 13.12.2016 um 17:46 schrieb Thomas Gleixner: > > > What are the adjust values after a warm boot? > > > > So, after cold boot with a kernel which doesn't adjust TSCs, then warm > > boot I got: > > [

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-14 Thread Thomas Gleixner
On Wed, 14 Dec 2016, Roland Scheidegger wrote: > Am 13.12.2016 um 17:46 schrieb Thomas Gleixner: > > What are the adjust values after a warm boot? > > So, after cold boot with a kernel which doesn't adjust TSCs, then warm > boot I got: > [0.00] TSC ADJUST: CPU0: -602358264300 176072418728

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-14 Thread Thomas Gleixner
On Wed, 14 Dec 2016, Roland Scheidegger wrote: > Am 13.12.2016 um 17:46 schrieb Thomas Gleixner: > > What are the adjust values after a warm boot? > > So, after cold boot with a kernel which doesn't adjust TSCs, then warm > boot I got: > [0.00] TSC ADJUST: CPU0: -602358264300 176072418728

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-13 Thread Roland Scheidegger
Am 13.12.2016 um 17:46 schrieb Thomas Gleixner: > On Tue, 13 Dec 2016, Roland Scheidegger wrote: > >> Am 13.12.2016 um 14:14 schrieb Thomas Gleixner: >>> Roland reported interesting TSC ADJUST register wreckage on his DELL >>> machine, which seems to populate that MSR with a random number

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-13 Thread Roland Scheidegger
Am 13.12.2016 um 17:46 schrieb Thomas Gleixner: > On Tue, 13 Dec 2016, Roland Scheidegger wrote: > >> Am 13.12.2016 um 14:14 schrieb Thomas Gleixner: >>> Roland reported interesting TSC ADJUST register wreckage on his DELL >>> machine, which seems to populate that MSR with a random number

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-13 Thread Thomas Gleixner
On Tue, 13 Dec 2016, Roland Scheidegger wrote: > Am 13.12.2016 um 14:14 schrieb Thomas Gleixner: > > Roland reported interesting TSC ADJUST register wreckage on his DELL > > machine, which seems to populate that MSR with a random number generator. > > FWIW, I thought about the actual values some

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-13 Thread Thomas Gleixner
On Tue, 13 Dec 2016, Roland Scheidegger wrote: > Am 13.12.2016 um 14:14 schrieb Thomas Gleixner: > > Roland reported interesting TSC ADJUST register wreckage on his DELL > > machine, which seems to populate that MSR with a random number generator. > > FWIW, I thought about the actual values some

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-13 Thread Roland Scheidegger
Am 13.12.2016 um 14:14 schrieb Thomas Gleixner: > Roland reported interesting TSC ADJUST register wreckage on his DELL > machine, which seems to populate that MSR with a random number generator. FWIW, I thought about the actual values some more and I don't actually think they are all that random

Re: [patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-13 Thread Roland Scheidegger
Am 13.12.2016 um 14:14 schrieb Thomas Gleixner: > Roland reported interesting TSC ADJUST register wreckage on his DELL > machine, which seems to populate that MSR with a random number generator. FWIW, I thought about the actual values some more and I don't actually think they are all that random

[patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-13 Thread Thomas Gleixner
Roland reported interesting TSC ADJUST register wreckage on his DELL machine, which seems to populate that MSR with a random number generator. Deeper investagation into fixing this wreckage unearthed another special feature which is designed by Intel: Negative TSC adjuste values cause interrupt

[patch 0/2] tsc/adjust: Cure suspend/resume issues and prevent TSC deadline timer irq storm

2016-12-13 Thread Thomas Gleixner
Roland reported interesting TSC ADJUST register wreckage on his DELL machine, which seems to populate that MSR with a random number generator. Deeper investagation into fixing this wreckage unearthed another special feature which is designed by Intel: Negative TSC adjuste values cause interrupt