Bug 224344 - [PATCH] powerpc64: Add POWER9 revision
Summary: [PATCH] powerpc64: Add POWER9 revision
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: kern (show other bugs)
Version: CURRENT
Hardware: powerpc Any
: --- Affects Only Me
Assignee: Justin Hibbits
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2017-12-14 17:50 UTC by Breno Leitao
Modified: 2018-03-24 18:07 UTC (History)
1 user (show)

See Also:


Attachments
Patch with the fix (1.52 KB, patch)
2017-12-14 17:50 UTC, Breno Leitao
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Breno Leitao 2017-12-14 17:50:06 UTC
Created attachment 188836 [details]
Patch with the fix

I currently booted FreeBSD on a POWER9 machine (on top of KVM), and I
got the following message:

cpu0: Unknown PowerPC CPU revision 0x1201, 2550.00 MHz

This is because of POWER9 is not listed as a valid CPU model on FreeBSD
code. This patch simply adds it with the same feature set as POWER8 for
now, so, new features could be added as they are implemented.
Comment 1 Justin Hibbits freebsd_committer freebsd_triage 2017-12-14 20:01:32 UTC
Committed as r326859.  Thanks!
Comment 2 commit-hook freebsd_committer freebsd_triage 2017-12-14 20:01:35 UTC
A commit references this bug:

Author: jhibbits
Date: Thu Dec 14 20:01:04 UTC 2017
New revision: 326859
URL: https://svnweb.freebsd.org/changeset/base/326859

Log:
  Add identifier for POWER9 CPU to CPU list

  Without the identifier in the list booting FreeBSD results in printing the
  following (from a PowerKVM boot):

  cpu0: Unknown PowerPC CPU revision 0x1201, 2550.00 MHz

  For now, add the same feature list as POWER8.  As new capabilities are added to
  support POWER9 specific features, they will be added to this.

  PR:		224344
  Submitted by:	Breno Leitao <breno_DOT_leitao_AT_gmail_DOT_com>

Changes:
  head/sys/powerpc/powerpc/cpu.c
Comment 3 commit-hook freebsd_committer freebsd_triage 2018-03-19 06:50:04 UTC
A commit references this bug:

Author: eadler
Date: Mon Mar 19 06:49:49 UTC 2018
New revision: 331184
URL: https://svnweb.freebsd.org/changeset/base/331184

Log:
  MFC r326859:

  Add identifier for POWER9 CPU to CPU list

  Without the identifier in the list booting FreeBSD results in printing the
  following (from a PowerKVM boot):

  cpu0: Unknown PowerPC CPU revision 0x1201, 2550.00 MHz

  For now, add the same feature list as POWER8.  As new capabilities are added to
  support POWER9 specific features, they will be added to this.

  PR:		224344

Changes:
_U  stable/11/
  stable/11/sys/powerpc/powerpc/cpu.c
Comment 4 commit-hook freebsd_committer freebsd_triage 2018-03-24 18:07:28 UTC
A commit references this bug:

Author: ian
Date: Sat Mar 24 18:06:35 UTC 2018
New revision: 331492
URL: https://svnweb.freebsd.org/changeset/base/331492

Log:
  To fix the powerpc build, back out r331184, which was this mfc:

  MFC r326859:

  Add identifier for POWER9 CPU to CPU list

  Without the identifier in the list booting FreeBSD results in printing the
  following (from a PowerKVM boot):

  cpu0: Unknown PowerPC CPU revision 0x1201, 2550.00 MHz

  For now, add the same feature list as POWER8.  As new capabilities are added to
  support POWER9 specific features, they will be added to this.

  PR:		224344

Changes:
_U  stable/11/
  stable/11/sys/powerpc/powerpc/cpu.c