On Thu, Jun 07, 2012 at 10:35:04AM +0900, Greg KH wrote: > On Wed, Jun 06, 2012 at 05:13:29PM +0200, Robert Richter wrote: > > Minimal stable backport of f227d4306cf30e1d5b6f231e8ef9006c34f3d186. > > Minimal doesn't begin to describe this. A totally different patch is > more like it. > > Why wasn't it done this way originally? I really don't like diverging > from Linus's tree in large ways like this. > > Borislav, what do you think about this version? Any reason why your > original patch shouldn't just be applied to 3.4 instead?
Right, so Robert thought the original patch is too big for stable, that's why. But the main change is lvt_interrupt_supported() while the rest are contextual changes to accomodate that first thing. And I don't think it is too big, so if you were to ask me, I'd backport the original patch which is already upstream and widely tested... Oh, and the other thing is, this patch indirectly fixes IBS registration so if you want to use IBS with perf on -stable, you need that patch and I didn't say that in the commit message. And this is actually the main reason why we want it in stable - to enable precise instruction tracing on AMD. Thanks. -- Regards/Gruss, Boris. Advanced Micro Devices GmbH Einsteinring 24, 85609 Dornach GM: Alberto Bozzo Reg: Dornach, Landkreis Muenchen HRB Nr. 43632 WEEE Registernr: 129 19551 -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
