Re: svn commit: r309351 - head/etc/defaults

2016-12-02 Thread Warner Losh
On Fri, Dec 2, 2016 at 9:18 AM, John Baldwin  wrote:
> On Thursday, December 01, 2016 02:41:41 PM Warner Losh wrote:
>> On Thu, Dec 1, 2016 at 9:48 AM, hiren panchasara
>>  wrote:
>> > On 12/01/16 at 04:35P, Warner Losh wrote:
>> >> Author: imp
>> >> Date: Thu Dec  1 04:35:43 2016
>> >> New Revision: 309351
>> >> URL: https://svnweb.freebsd.org/changeset/base/309351
>> >>
>> >> Log:
>> >>   Revert the 'performance' setting to 'NONE' from C2.
>> >
>> > Just a note that this commit reverts r282110.
>>
>> Yea, it's good for laptops, bad for servers. At least there's some
>> Intel CPUs that this causes corruption with USB. That's being sorted
>> out, but until that's done we need to fail more safe for our primary
>> market.
>
> It's also good for some servers as you need some cores in C-states
> to get the maximum Turbo Boost on other cores (though that may have
> been more true for earlier generations of i7 CPUs than more recent
> generations).

Once the corruption issues are solved, we can and should put this
back. Adrian is working on that.

Warner
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"


Re: svn commit: r309351 - head/etc/defaults

2016-12-02 Thread John Baldwin
On Thursday, December 01, 2016 02:41:41 PM Warner Losh wrote:
> On Thu, Dec 1, 2016 at 9:48 AM, hiren panchasara
>  wrote:
> > On 12/01/16 at 04:35P, Warner Losh wrote:
> >> Author: imp
> >> Date: Thu Dec  1 04:35:43 2016
> >> New Revision: 309351
> >> URL: https://svnweb.freebsd.org/changeset/base/309351
> >>
> >> Log:
> >>   Revert the 'performance' setting to 'NONE' from C2.
> >
> > Just a note that this commit reverts r282110.
> 
> Yea, it's good for laptops, bad for servers. At least there's some
> Intel CPUs that this causes corruption with USB. That's being sorted
> out, but until that's done we need to fail more safe for our primary
> market.

It's also good for some servers as you need some cores in C-states
to get the maximum Turbo Boost on other cores (though that may have
been more true for earlier generations of i7 CPUs than more recent
generations).

-- 
John Baldwin
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"


Re: svn commit: r309351 - head/etc/defaults

2016-12-01 Thread Adrian Chadd
+1 for reverting it from now (from the causer of the pain...)



-a


On 1 December 2016 at 13:41, Warner Losh  wrote:
> On Thu, Dec 1, 2016 at 9:48 AM, hiren panchasara
>  wrote:
>> On 12/01/16 at 04:35P, Warner Losh wrote:
>>> Author: imp
>>> Date: Thu Dec  1 04:35:43 2016
>>> New Revision: 309351
>>> URL: https://svnweb.freebsd.org/changeset/base/309351
>>>
>>> Log:
>>>   Revert the 'performance' setting to 'NONE' from C2.
>>
>> Just a note that this commit reverts r282110.
>
> Yea, it's good for laptops, bad for servers. At least there's some
> Intel CPUs that this causes corruption with USB. That's being sorted
> out, but until that's done we need to fail more safe for our primary
> market.
>
> Warner
>
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"


Re: svn commit: r309351 - head/etc/defaults

2016-12-01 Thread Warner Losh
On Thu, Dec 1, 2016 at 9:48 AM, hiren panchasara
 wrote:
> On 12/01/16 at 04:35P, Warner Losh wrote:
>> Author: imp
>> Date: Thu Dec  1 04:35:43 2016
>> New Revision: 309351
>> URL: https://svnweb.freebsd.org/changeset/base/309351
>>
>> Log:
>>   Revert the 'performance' setting to 'NONE' from C2.
>
> Just a note that this commit reverts r282110.

Yea, it's good for laptops, bad for servers. At least there's some
Intel CPUs that this causes corruption with USB. That's being sorted
out, but until that's done we need to fail more safe for our primary
market.

Warner
___
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"


Re: svn commit: r309351 - head/etc/defaults

2016-12-01 Thread hiren panchasara
On 12/01/16 at 08:48P, hiren panchasara wrote:
> On 12/01/16 at 04:35P, Warner Losh wrote:
> > Author: imp
> > Date: Thu Dec  1 04:35:43 2016
> > New Revision: 309351
> > URL: https://svnweb.freebsd.org/changeset/base/309351
> > 
> > Log:
> >   Revert the 'performance' setting to 'NONE' from C2.
> 
> Just a note that this commit reverts r282110.

Ah, my bad. performance_cx_lowest has more history than that. :-) 
(HIGH -> Cmax -> C2 -> NONE)

Cheers,
Hiren


pgppfedJR0kE5.pgp
Description: PGP signature


Re: svn commit: r309351 - head/etc/defaults

2016-12-01 Thread hiren panchasara
On 12/01/16 at 04:35P, Warner Losh wrote:
> Author: imp
> Date: Thu Dec  1 04:35:43 2016
> New Revision: 309351
> URL: https://svnweb.freebsd.org/changeset/base/309351
> 
> Log:
>   Revert the 'performance' setting to 'NONE' from C2.

Just a note that this commit reverts r282110.

Cheers,
Hiren


pgpg5ixLSD8tO.pgp
Description: PGP signature