linux-kernel - Re: [PATCH 1/5] perf, x86: Remove incorrect model number from Haswell perf

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 27 Aug 2014 11:53:57 +0200 (CEST)
From:	Thomas Gleixner <tglx@...utronix.de>
To:	Andi Kleen <andi@...stfloor.org>
cc:	peterz@...radead.org, linux-kernel@...r.kernel.org,
	mingo@...nel.org, eranian@...gle.com,
	Andi Kleen <ak@...ux.intel.com>
Subject: Re: [PATCH 1/5] perf, x86: Remove incorrect model number from Haswell
 perf

On Wed, 27 Aug 2014, Thomas Gleixner wrote:

> On Tue, 26 Aug 2014, Andi Kleen wrote:
> 
> > > So what's the point of making the obvious onliner patch
> > > 
> > > -	case 71:
> > > 
> > > into something which reorders the sorted case numbers?
> >  
> > This was a merging mistake. I'll fix it to be a one liner.
> > 
> > > And of course, this patch is missing any explanation WHY 71 is
> > > incorrect and how it got there in the first place.
> > 
> > 71 is a Broadwell, as you would know if you had read the next patch.
> 
> And why on earth do I need to read the next patch to figure out what
> the heck this patch is doing?
> 
> Just because it's written by someone who gives a shit?
> 
> Your unwillingness to cooperate and your advisory resistance are
> slowly approaching the Krause level...

And after reading the next patch I still can't see that 71 is a
broadwell, because the next patch merily adds haswell names to the
numbers. Neither of the following patches has the magic 71
incorporated. The only model number related to broadwell is 61 in
patch 3/5.

Try again when you figured yourself out what number means what and why
71 is bogus.

Thanks,

	tglx


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/