Project

General

Profile

Actions
Copy link

Bug #2650

closed

AMD family 0x15 PG support

Added by Hans Rosenfeld about 11 years ago. Updated about 11 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Hans Rosenfeld
Category:
kernel
Start date:
2012-04-23
Due date:
% Done:

100%

Estimated time:
Difficulty:
Medium
Tags:
amd
Gerrit CR:
External Bug:

Description

In AMD family 0x15 processors cores come in pairs called compute units, sharing an FPU and L2 and L1I caches. This topology information needs to be detected and implemented as a Processor Group (PG).

Actions
Copy link
 #1

Updated by Hans Rosenfeld about 11 years ago

  • % Done changed from 0 to 80

Preliminary webrev: http://grumpf.hope-2000.org/illumos-2650-webrev/

Introduces cpi_compunitid, cpi_cores_per_compunit to struct cpuid_info, which are set according to the CPUID information. On systems not having compute units, cpi_compunitid is the same as cpi_coreid, while cpi_cores_per_compunit is 1.

A platform override for PGHW_FPU policy is added to account for the fact that L2 and L1I caches are shared, too, so it should include CMT_AFFINITY. I'm not sure that is really necessary, as I wasn't able to find any usage of CMT_AFFINITY in the code that would actually affect scheduling decisions.

Actions
Copy link
 #2

Updated by Hans Rosenfeld about 11 years ago

  • % Done changed from 80 to 100

Updated webrev: http://grumpf.hope-2000.org/illumos-2650-webrev-2/

Changes:
  • minor stylistic nits (L1I -> I$)
  • Bugfix: Forgot to set chipid on family 0x15, causing all kinds of interesting panics on big systems. Reworked chipid setting to set chipid based on procnodeid and procnodes_per_pkg on all CPU families.
  • Bugfix: The specification didn't tell whether the compute unit id in the CPUID data was system-wide just like the node id. It turns out it isn't, so work around it.
Tested on:
  • 4-core family 0x10
  • 8-core family 0x15 (1 node in 1 socket)
  • 32-core family 0x15 (4 nodes in 2 sockets)
Actions
Copy link
 #3

Updated by Albert Lee about 11 years ago

  • Status changed from In Progress to Resolved
  • Tags changed from needs-triage to amd

Fixed in:
changeset: 13681:73253247f9e5
tag: tip
user: Hans Rosenfeld <>
date: Thu May 03 15:56:05 2012 +0200
files: usr/src/uts/i86pc/os/cpuid.c usr/src/uts/i86pc/os/mp_machdep.c usr/
src/uts/intel/sys/x86_archext.h
description:
2650 AMD family 0x15 PG support
Reviewed by: Robert Mustacchi <>
Reviewed by: Richard Lowe <>
Approved by: Albert Lee <>

Actions
Copy link

Also available in: Atom PDF