Bug 550183

Summary: Make oprofile recognize Nehalem
Product: Red Hat Enterprise Linux 4 Reporter: Herbert van den Bergh <herbert.van.den.bergh>
Component: kernelAssignee: John Villalovos <jvillalo>
Status: CLOSED DUPLICATE QA Contact: Red Hat Kernel QE team <kernel-qe>
Severity: medium Docs Contact:
Priority: low    
Version: 4.8CC: prarit, vgoyal
Target Milestone: rc   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: 458441 Environment:
Last Closed: 2010-01-05 20:27:03 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Proposed patch for bare metal oprofile none

Description Herbert van den Bergh 2009-12-23 23:00:29 UTC 
Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:
Comment 1 Herbert van den Bergh 2009-12-23 23:01:42 UTC 
Description of problem:
Oprofile needs to recognize the CPU model to make use of its PMU; it can only
use timer interrupt otherwise.  It doesn't recognize Nehalem, yet.

Additional info:
Fixed in Linus's tree:

commit 4b9f12a3779c548b68bc9af7d94030868ad3aa1b
Author: Linus Torvalds <torvalds>
Date:   Thu Jul 24 17:29:00 2008 -0700

    x86/oprofile/nmi_int: add Nehalem to list of ppro cores

    ..otherwise oprofile will fall back on that poor timer interrupt.

    Also replace the unreadable chain of if-statements with a "switch()"
    statement instead. It generates better code, and is a lot clearer.

    Signed-off-by: Linus Torvalds <torvalds>
Comment 2 Herbert van den Bergh 2009-12-23 23:02:53 UTC 
Created attachment 380098 [details]
Proposed patch for bare metal oprofile
Comment 3 John Villalovos 2010-01-05 20:27:03 UTC 
The attached patch is not nearly enough to make it work.

I am going to close this bug as a duplicate of bug 509423

*** This bug has been marked as a duplicate of bug 509423 ***