On Thu, Feb 17, 2011 at 11:03:30AM +0100, Tejun Heo wrote: > > I'm not particularly inclined toward including this patch in upstream > as long as it can't be enabled by default. It's just gonna be an > extra piece of code to carry around which only a handful of people use > and nobody really knows on which hardware it works or not and to what > extent. I have a feedback for the chipsets listed in the patch. The protection via the extra parameter is owned to the fact that intel recommends against the OS doing this. Quote: "SW shall not manipulate SMS during runtime operation; that is, the OS will not do this." I would not see early fixup as runtime, however, better save than sorry. Advantages are better performance and lower power consumption. I don't wanna break anybody's data with that patch, hence the precaution. Bye, Joerg
Attachment:
signature.asc
Description: Digital signature