NetBSD Problem Report #39950

From bernd@arresum.veego.de  Tue Nov 18 12:41:39 2008
Return-Path: <bernd@arresum.veego.de>
Received: from mail.netbsd.org (mail.netbsd.org [204.152.190.11])
	by narn.NetBSD.org (Postfix) with ESMTP id 3CDB763B874
	for <gnats-bugs@gnats.NetBSD.org>; Tue, 18 Nov 2008 12:41:39 +0000 (UTC)
Message-Id: <200811181241.mAICf2Sm018191@arresum.veego.de>
Date: Tue, 18 Nov 2008 13:41:02 +0100 (CET)
From: Bernd Ernesti <pr200824-2@veego.de>
Reply-To: Bernd Ernesti <pr200824-2@veego.de>
To: gnats-bugs@gnats.NetBSD.org
Subject: Missing features 2-4 support in /proc/cpuinfo for AMD CPUs on i386
X-Send-Pr-Version: 3.95

>Number:         39950
>Category:       port-i386
>Synopsis:       Missing features 2-4 support in /proc/cpuinfo for AMD CPUs on i386
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    msaitoh
>State:          closed
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Tue Nov 18 12:45:00 +0000 2008
>Closed-Date:    Mon Aug 08 09:42:36 +0000 2016
>Last-Modified:  Mon Oct 22 20:08:55 +0000 2018
>Originator:     Bernd Ernesti
>Release:        NetBSD 5.99.02
>Organization:
>Environment:
System: NetBSD 5.99.02
Architecture: i386
Machine: i386
>Description:
	/proc/cpuinfo is incomplete because it doesn't show all cpu flags.

# cpuctl identify 0
cpu0: AMD Athlon X2 or Athlon 64 X2 (686-class), 2506.49 MHz, id 0x60fb2
cpu0: features 178bfbff<FPU,VME,DE,PSE,TSC,MSR,PAE,MCE,CX8,APIC,SEP,MTRR>
cpu0: features 178bfbff<PGE,MCA,CMOV,PAT,PSE36,CFLUSH,MMX>
cpu0: features 178bfbff<FXSR,SSE,SSE2,HTT>
cpu0: features2 2001<SSE3,CX16>
cpu0: features3 ebd3fbff<SCALL/RET,NOX,MXX,FFXSR,RDTSCP,LONG,3DNOW2,3DNOW>
cpu0: features4 11f<LAHF,CMPLEGACY,SVM,EAPIC,ALTMOVCR0,3DNOWPREFETCH>
cpu0: "AMD Athlon(tm) Dual Core Processor 4850e"

# cat /proc/cpuinfo
[..]     
flags : fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht  

	It seems that features 2-4 are not in the flags output and so programs like
	the configure check in mplayer doesn't find all flags. (where sse3 doesn't
	work due too the old binutils, but thats another problem)

	This may affect port-amd64 too, but i have no amd64 installed to check the
	output.

>How-To-Repeat:
	Mount /proc with the option linux and notice that /proc/cpuinfo doesn't
	contain all flags.
>Fix:

>Release-Note:

>Audit-Trail:
From: Nicolas Joly <njoly@pasteur.fr>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: port-i386/39950: Missing features 2-4 support in /proc/cpuinfo for AMD CPUs on i386
Date: Wed, 14 Jul 2010 18:57:58 +0200

 --lrZ03NoBR/3+SXJZ
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: inline

 On Tue, Nov 18, 2008 at 12:45:00PM +0000, Bernd Ernesti wrote:
 > >Number:         39950
 > >Category:       port-i386
 > >Synopsis:       Missing features 2-4 support in /proc/cpuinfo for AMD CPUs on i386
 > >Confidential:   no
 > >Severity:       non-critical
 > >Priority:       medium
 > >Responsible:    port-i386-maintainer
 > >State:          open
 > >Class:          sw-bug
 > >Submitter-Id:   net
 > >Arrival-Date:   Tue Nov 18 12:45:00 +0000 2008
 > >Originator:     Bernd Ernesti
 > >Release:        NetBSD 5.99.02
 > >Organization:
 > >Environment:
 > System: NetBSD 5.99.02
 > Architecture: i386
 > Machine: i386
 > >Description:
 > 	/proc/cpuinfo is incomplete because it doesn't show all cpu flags.

 Can you try the attached patch ?

 -- 
 Nicolas Joly

 Biological Software and Databanks.
 Institut Pasteur, Paris.

 --lrZ03NoBR/3+SXJZ
 Content-Type: text/plain; charset=us-ascii
 Content-Disposition: attachment; filename="netbsd-x86cpuinfo.diff"

 Index: sys/arch/x86/x86/procfs_machdep.c
 ===================================================================
 RCS file: /cvsroot/src/sys/arch/x86/x86/procfs_machdep.c,v
 retrieving revision 1.1
 diff -u -p -r1.1 procfs_machdep.c
 --- sys/arch/x86/x86/procfs_machdep.c	8 Jul 2010 11:25:00 -0000	1.1
 +++ sys/arch/x86/x86/procfs_machdep.c	14 Jul 2010 16:53:26 -0000
 @@ -53,6 +53,8 @@ __KERNEL_RCSID(0, "$NetBSD: procfs_machd
  #include <miscfs/procfs/procfs.h>

  #include <machine/cpu.h>
 +#include <machine/cputypes.h>
 +#include <machine/cpuvar.h>
  #include <machine/reg.h>
  #include <machine/specialreg.h>

 @@ -64,13 +66,13 @@ static const char * const x86_features[]
  	"fpu", "vme", "de", "pse", "tsc", "msr", "pae", "mce",
  	"cx8", "apic", NULL, "sep", "mtrr", "pge", "mca", "cmov",
  	"pat", "pse36", "pn", "clflush", NULL, "dts", "acpi", "mmx",
 -	"fxsr", "sse", "sse2", "ss", "ht", "tm", "ia64", NULL,
 -#ifdef __x86_64__
 +	"fxsr", "sse", "sse2", "ss", "ht", "tm", "ia64", "pbe",
 +
  	/* AMD-defined */
  	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 
  	NULL, NULL, NULL, "syscall", NULL, NULL, NULL, NULL, 
 -	NULL, NULL, NULL, NULL, "nx", NULL, "mmxext", NULL, 
 -	NULL, "fxsr_opt", "rdtscp", NULL, NULL, "lm", "3dnowext", "3dnow",
 +	NULL, NULL, NULL, "mp", "nx", NULL, "mmxext", NULL, 
 +	NULL, "fxsr_opt", "pdpe1gb", "rdtscp", NULL, "lm", "3dnowext", "3dnow",

  	/* Transmeta-defined */
  	"recovery", "longrun", NULL, "lrti", NULL, NULL, NULL, NULL, 
 @@ -79,31 +81,30 @@ static const char * const x86_features[]
  	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 

  	/* Linux-defined */
 -	"cxmmx", NULL, "cyrix_arr", "centaur_mcr", NULL, 
 -	"constant_tsc", NULL, NULL, 
 -	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 
 -	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 
 +	"cxmmx", "k6_mtrr", "cyrix_arr", "centaur_mcr", NULL, NULL, NULL, NULL,
 +	"constant_tsc", "up", NULL, "arch_perfmon", "pebs", "bts", NULL, NULL,
 +	"rep_good", NULL, NULL, NULL, "nopl", "amdc1e", "xtopology", NULL, 
  	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 

  	/* Intel-defined */
 -	"pni", NULL, NULL, "monitor", "ds_cpi", "vmx", NULL, "est", 
 -	"tm2", NULL, "cid", NULL, NULL, "cx16", "xtpr", NULL, 
 -	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 
 -	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 
 +	"pni", "pclmulqdq", "dtes64", "monitor", "ds_cpl", "vmx", "smx", "est", 
 +	"tm2", "ssse3", "cid", NULL, "fma", "cx16", "xtpr", "pdcm", 
 +	NULL, NULL, "dca", "sse4_1", "sse4_2", "x2apic", NULL, "popcnt", 
 +	NULL, "aes", "xsave", NULL, "avx", NULL, NULL, NULL, 

  	/* VIA/Cyrix/Centaur-defined */
  	NULL, NULL, "rng", "rng_en", NULL, NULL, "ace", "ace_en", 
 -	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 
 +	"ace2", "ace2_en", "phe", "phe_en", "pmm", "pmm_en", NULL, NULL, 
  	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 
  	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 

  	/* AMD defined */
 -	"lahf_lm", "cmp_legacy", "svm", NULL, "cr8_legacy", NULL, NULL, NULL,
 -	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 
 +	"lahf_lm", "cmp_legacy", "svm", "extapic", "cr8_legacy", "abm", "sse4a", "misalignsse",
 +	"3dnowprefetch", "osvw", "ibs", "sse5", "skinit", "wdt", NULL, NULL, 
  	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 
 -	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, 
 -#endif
 +	NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL
  };
 +CTASSERT(__arraycount(x86_features) % 32 == 0);

  static int	procfs_getonecpu(int, struct cpu_info *, char *, int *);

 @@ -147,12 +148,26 @@ static int
  procfs_getonecpu(int xcpu, struct cpu_info *ci, char *bf, int *len)
  {
  	int left, l, i;
 -	char featurebuf[256], *p;
 +	char featurebuf[512], *p;
 +	uint32_t features[__arraycount(x86_features) / 32];
 +
 +	memset(features, 0, sizeof(features));
 +	features[0] = ci->ci_feat_val[0];
 +	switch (cpu_vendor) {
 +	case CPUVENDOR_AMD:
 +	case CPUVENDOR_INTEL:
 +		features[1] = ci->ci_feat_val[2];
 +		features[4] = ci->ci_feat_val[1];
 +		features[6] = ci->ci_feat_val[3];
 +		break;
 +	default:
 +		break;
 +	}

  	p = featurebuf;
  	left = sizeof(featurebuf);
 -	for (i = 0; i < 32; i++) {
 -		if ((ci->ci_feat_val[0] & (1 << i)) && x86_features[i]) {
 +	for (i = 0; i < __arraycount(x86_features); i++) {
 +		if ((features[i / 32] & (1 << (i % 32))) && x86_features[i]) {
  			l = snprintf(p, left, "%s ", x86_features[i]);
  			left -= l;
  			p += l;

 --lrZ03NoBR/3+SXJZ--

From: Bernd Ernesti <pr200824@veego.de>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: Re: port-i386/39950: Missing features 2-4 support in /proc/cpuinfo for AMD CPUs on i386
Date: Sat, 17 Jul 2010 21:26:59 +0200

 On Wed, Jul 14, 2010 at 05:00:08PM +0000, Nicolas Joly wrote:
 >  On Tue, Nov 18, 2008 at 12:45:00PM +0000, Bernd Ernesti wrote:
 [..]

 >  > >Environment:
 >  > System: NetBSD 5.99.02
 >  > Architecture: i386
 >  > Machine: i386
 >  > >Description:
 >  > 	/proc/cpuinfo is incomplete because it doesn't show all cpu flags.
 >  
 >  Can you try the attached patch ?

 Hmm, the patch looks like it is for -current. I switched to netbsd-5 for the host
 where I reported this problem. I try to adapt it to the branch when I have some time.

 Thanks,
 Bernd

State-Changed-From-To: open->feedback
State-Changed-By: dholland@NetBSD.org
State-Changed-When: Mon, 08 Aug 2016 05:21:25 +0000
State-Changed-Why:
What's the state of this nowadays?


Responsible-Changed-From-To: port-i386-maintainer->msaitoh
Responsible-Changed-By: msaitoh@NetBSD.org
Responsible-Changed-When: Mon, 08 Aug 2016 09:32:04 +0000
Responsible-Changed-Why:
take.


From: "SAITOH Masanobu" <msaitoh@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/39950 CVS commit: src/sys/arch/x86/x86
Date: Mon, 8 Aug 2016 09:39:06 +0000

 Module Name:	src
 Committed By:	msaitoh
 Date:		Mon Aug  8 09:39:06 UTC 2016

 Modified Files:
 	src/sys/arch/x86/x86: procfs_machdep.c

 Log Message:
 - Update VIA/Cyrix/Centaur-defined bits. Part of PR#39950
 - Fix comment. x86_features[4] is not 0x80000001 but 0x00000001
 - Update comment


 To generate a diff of this commit:
 cvs rdiff -u -r1.12 -r1.13 src/sys/arch/x86/x86/procfs_machdep.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

State-Changed-From-To: feedback->closed
State-Changed-By: msaitoh@NetBSD.org
State-Changed-When: Mon, 08 Aug 2016 09:42:36 +0000
State-Changed-Why:
For AMD's, it already fixed in netbsd-7.
For VIA/Cyrix/Centaur, it's fixed in -current.
Thanks.


From: "Soren Jacobsen" <snj@netbsd.org>
To: gnats-bugs@gnats.NetBSD.org
Cc: 
Subject: PR/39950 CVS commit: [netbsd-7] src/sys/arch/x86/x86
Date: Thu, 8 Dec 2016 08:06:38 +0000

 Module Name:	src
 Committed By:	snj
 Date:		Thu Dec  8 08:06:38 UTC 2016

 Modified Files:
 	src/sys/arch/x86/x86 [netbsd-7]: procfs_machdep.c

 Log Message:
 Pull up following revision(s) (requested by msaitoh in ticket #1293):
 	sys/arch/x86/x86/procfs_machdep.c: revisions 1.12-1.14
 Update for x86 /proc/cpuinfo:
 - Add ptsc, avx512dq, avx512bw, avx512vl and rdt_a.
 - Update VIA/Cyrix/Centaur-defined bits. Part of PR#39950.
 - Remove pcommit.
 - Update some Linux mapping unused in /proc/cpuinfo.


 To generate a diff of this commit:
 cvs rdiff -u -r1.6.4.1 -r1.6.4.2 src/sys/arch/x86/x86/procfs_machdep.c

 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.

>Unformatted:

NetBSD Home
NetBSD PR Database Search

(Contact us) $NetBSD: query-full-pr,v 1.43 2018/01/16 07:36:43 maya Exp $
$NetBSD: gnats_config.sh,v 1.9 2014/08/02 14:16:04 spz Exp $
Copyright © 1994-2017 The NetBSD Foundation, Inc. ALL RIGHTS RESERVED.