On 2014年08月06日 21:57, Gene Heskett wrote:
> On Wednesday 06 August 2014, Lan, Tianyu wrote:
>> Have you tried my attached kernel config file? When someone reported
>> the issue to me, I also was very hard to reproduce the issue by my own
>> config file. Maybe once 100 tries. But I can reproduced the issue every
>> time with the attached configure file on several my machines and even
>> on server.
> 
> There was no attachment.

Hi Gene:
	Sorry. I meant the config file I attached in the my last reply.

>>
>> -----Original Message-----
>> From: Borislav Petkov [mailto:bp@alien8.de]
>> Sent: Wednesday, August 06, 2014 7:08 PM
>> To: Lan, Tianyu
>> Cc: tglx@linutronix.de; mingo@redhat.com; hpa@zytor.com; x86@kernel.org;
>> toshi.kani@hp.com; imammedo@redhat.com; jan.kiszka@siemens.com;
>> mingo@kernel.org; huawei.libin@huawei.com; prarit@redhat.com;
>> linux-kernel@vger.kernel.org; Brandt, Todd E Subject: Re: [PATCH]
>> X86/CPU: Avoid 100ms sleep for cpu offline during S3
>>
>> On Tue, Aug 05, 2014 at 05:29:18PM +0800, Lan Tianyu wrote:
>>> I think you can pull Tod's tool from git hub and do test from your
>>> laptop. I attached a configure file come from ubuntu which is easier
>>> to produce the issue. You can use it to reproduce the issue.
>>
>> Right, so I ran it on an old dualcore AMD laptop I have lying around
>> here and it didn't show any difference with or without your patch.
>>
>> For all 4 runs I did with the tool running it without any arguments (so
>> it did the default thing), I got suspend time for CPU1 of 4-6 msec and
>> resume of ~20msec.
>>
>> I don't know whether it is the dual core or AMD but your patch doesn't
>> change anything on that laptop which is ok, I'd guess.
>>
>>> Ok. BTW, I have tested it on the Ivbridge and Haswell machines.
>>
>> Yeah, add those numbers to the commit message too.
>>
>> Thanks.
>>
>> --
>> Regards/Gruss,
>>    Boris.
>>
>> Sent from a fat crate under my desk. Formatting is fine.
>> --
> 
> 
> Cheers, Gene Heskett
> 


-- 
Best regards
Tianyu Lan