From owner-freebsd-bugs@FreeBSD.ORG Sat Mar 23 23:10:01 2013 Return-Path: Delivered-To: freebsd-bugs@smarthost.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id B246A3E4 for ; Sat, 23 Mar 2013 23:10:01 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) by mx1.freebsd.org (Postfix) with ESMTP id A329CFD5 for ; Sat, 23 Mar 2013 23:10:01 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.6/8.14.6) with ESMTP id r2NNA060070902 for ; Sat, 23 Mar 2013 23:10:00 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.6/8.14.6/Submit) id r2NNA0fu070901; Sat, 23 Mar 2013 23:10:00 GMT (envelope-from gnats) Date: Sat, 23 Mar 2013 23:10:00 GMT Message-Id: <201303232310.r2NNA0fu070901@freefall.freebsd.org> To: freebsd-bugs@FreeBSD.org Cc: From: Oliver Pinter Subject: Re: kern/177319: Add more x86 CPU feature flag definitions X-BeenThere: freebsd-bugs@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: Oliver Pinter List-Id: Bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 23 Mar 2013 23:10:01 -0000 The following reply was made to PR kern/177319; it has been noted by GNATS. From: Oliver Pinter To: Jeremy Chadwick Cc: FreeBSD-gnats-submit@freebsd.org Subject: Re: kern/177319: Add more x86 CPU feature flag definitions Date: Sun, 24 Mar 2013 00:06:09 +0100 The cpu_stdext_feature section looks fine based on Intel Architecture Instruction Set Extensions Programming Reference (aug. 2012) 319433-014.pdf. On 3/23/13, Jeremy Chadwick wrote: > >>Number: 177319 >>Category: kern >>Synopsis: Add more x86 CPU feature flag definitions >>Confidential: no >>Severity: non-critical >>Priority: low >>Responsible: freebsd-bugs >>State: open >>Quarter: >>Keywords: >>Date-Required: >>Class: update >>Submitter-Id: current-users >>Arrival-Date: Sat Mar 23 19:20:02 UTC 2013 >>Closed-Date: >>Last-Modified: >>Originator: Jeremy Chadwick >>Release: FreeBSD 9.1-STABLE amd64 >>Organization: >>Environment: > System: FreeBSD icarus.home.lan 9.1-STABLE FreeBSD 9.1-STABLE #0 r248403: > Sat Mar 16 20:33:01 PDT 2013 > root@icarus.home.lan:/usr/obj/usr/src/sys/X7SBA_RELENG_9_amd64 amd64 >>Description: > While reading this thread: > > http://lists.freebsd.org/pipermail/freebsd-current/2013-March/040666.html > > It occurred to me that we might be missing some x86 CPU feature flag > definitions in comparison to other OSes (e.g. Linux). > > My comparison source was the Linux 3.8.4 source code, specifically > arch/x86/include/asm/cpufeature.h. That can be viewed here: > > http://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/tree/arch/x86/include/asm/cpufeature.h?h=linux-3.8.y >>How-To-Repeat: > n/a >>Fix: > Apply below patch (intended for stable/9, but can be backported to head > I'm sure). Patch is also available here once I get a PR number: > > http://jdc.koitsu.org/freebsd/{prnum}/ > > I strongly recommend someone familiar with these bits review this patch > first, as AFAIK these are only for cosmetical representation rather than > operational, but I could be wrong. > > > Index: sys/amd64/amd64/identcpu.c > =================================================================== > --- sys/amd64/amd64/identcpu.c (revision 248655) > +++ sys/amd64/amd64/identcpu.c (working copy) > @@ -366,13 +366,13 @@ printcpuinfo(void) > "\017" > "\020LWP" /* Lightweight Profiling */ > "\021FMA4" /* 4-operand FMA instructions */ > - "\022" > + "\022TCE" /* Translation Cache Extension */ > "\023" > "\024NodeId" /* NodeId MSR support */ > "\025" > "\026TBM" /* Trailing Bit Manipulation */ > "\027Topology" /* Topology Extensions */ > - "\030" > + "\030CorePerfC" /* Core Perf. Counter Extensions */ > "\031" > "\032" > "\033" > @@ -390,9 +390,17 @@ printcpuinfo(void) > "\020" > "\001GSFSBASE" > "\002TSCADJ" > + "\004BMI1" > + "\005HLE" > + "\006AVX2" > "\010SMEP" > + "\011BMI2" > "\012ENHMOVSB" > "\013INVPCID" > + "\014RTM" > + "\023RDSEED" > + "\024ADX" > + "\025SMAP" > ); > } > > Index: sys/i386/i386/identcpu.c > =================================================================== > --- sys/i386/i386/identcpu.c (revision 248655) > +++ sys/i386/i386/identcpu.c (working copy) > @@ -842,13 +842,13 @@ printcpuinfo(void) > "\017" > "\020LWP" /* Lightweight Profiling */ > "\021FMA4" /* 4-operand FMA instructions */ > - "\022" > + "\022TCE" /* Translation Cache Extension */ > "\023" > "\024NodeId" /* NodeId MSR support */ > "\025" > "\026TBM" /* Trailing Bit Manipulation */ > "\027Topology" /* Topology Extensions */ > - "\030" > + "\030CorePerfC" /* Core Perf. Counter Extensions */ > "\031" > "\032" > "\033" >>Release-Note: >>Audit-Trail: >>Unformatted: > _______________________________________________ > freebsd-bugs@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-bugs > To unsubscribe, send any mail to "freebsd-bugs-unsubscribe@freebsd.org" >