Bug 191793 - [PATCH] i386: Move phys_proc_id/early intel workaround to correct function
Summary: [PATCH] i386: Move phys_proc_id/early intel workaround to correct function
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Red Hat Enterprise Linux 4
Classification: Red Hat
Component: kernel
Version: 4.0
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
: ---
Assignee: John Villalovos
QA Contact: Brian Brock
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-05-15 20:33 UTC by Jason Baron
Modified: 2015-05-08 13:57 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2009-04-24 19:45:27 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
intel_early_workaround() moved to generic_identify() as early_detect_cpus() only run at BP. (574 bytes, patch)
2006-05-15 20:50 UTC, Bhavna Sarathy
no flags Details | Diff

Description Jason Baron 2006-05-15 20:33:24 UTC
Description of problem:

please see:


http://kernel.org/git/?p=linux/kernel/git/stable/linux-2.6.15.y.git;a=commitdiff;h=bcf2887b1416a506e3461c504642a1b7fad52ddc
Version-Release number of selected component (if applicable):


How reproducible:


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


Expected results:


Additional info:

Comment 1 Bhavna Sarathy 2006-05-15 20:50:39 UTC
Created attachment 129118 [details]
intel_early_workaround() moved to generic_identify() as early_detect_cpus() only run at BP.

Geoff, please test this patch, please.	 I can submit it to rhkernel-list soon
after.

Comment 2 Geoff Gustafson 2006-05-17 20:16:52 UTC
I sanity-checked on one box. Oddly generic_identify gets run twice on each proc,
but same upstream. Looks fine.


Comment 3 Geoff Gustafson 2008-02-05 19:46:40 UTC
We've made it this far without anyone tearing their hair out, I'm tempted to
think this couldn't possibly matter. But I'll check with the mothership before
giving up on it.


Comment 4 Geoff Gustafson 2008-02-06 17:19:03 UTC
Suresh pointed out, and I confirmed in the latest EL4 source, that the
x86_cache_alignment field this function sets is only ever read with the macro
cache_line_size() which reads it from the boot cpu. So the patch isn't really
needed.

The one exception to this is reporting to /proc/cpuinfo, so on some machine I
guess it will show 128 for boot cpu and incorrectly show 64 for others. But I'm
not sure that makes it worth posting.


Comment 5 John Villalovos 2009-04-24 19:45:27 UTC
From Comment 4, it doesn't seem like this is a bug.  Or at least not worth posting a patch.

Please re-open if disagree.


Note You need to log in before you can comment on or make changes to this bug.