| From 89baaaa98a10cad5cc8516c7208b02d9fc711890 Mon Sep 17 00:00:00 2001 |
| From: Robert Richter <robert.richter@amd.com> |
| Date: Thu, 28 Jan 2010 16:50:45 +0100 |
| Subject: oprofile/x86: remove node check in AMD IBS initialization |
| |
| From: Robert Richter <robert.richter@amd.com> |
| |
| commit 89baaaa98a10cad5cc8516c7208b02d9fc711890 upstream. |
| |
| Standard AMD systems have the same number of nodes as there are |
| northbridge devices. However, there may kernel configurations |
| (especially for 32 bit) or system setups exist, where the node number |
| is different or it can not be detected properly. Thus the check is not |
| reliable and may fail though IBS setup was fine. For this reason it is |
| better to remove the check. |
| |
| Signed-off-by: Robert Richter <robert.richter@amd.com> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de> |
| |
| --- |
| arch/x86/oprofile/op_model_amd.c | 10 ---------- |
| 1 file changed, 10 deletions(-) |
| |
| --- a/arch/x86/oprofile/op_model_amd.c |
| +++ b/arch/x86/oprofile/op_model_amd.c |
| @@ -386,16 +386,6 @@ static int init_ibs_nmi(void) |
| return 1; |
| } |
| |
| -#ifdef CONFIG_NUMA |
| - /* Sanity check */ |
| - /* Works only for 64bit with proper numa implementation. */ |
| - if (nodes != num_possible_nodes()) { |
| - printk(KERN_DEBUG "Failed to setup CPU node(s) for IBS, " |
| - "found: %d, expected %d", |
| - nodes, num_possible_nodes()); |
| - return 1; |
| - } |
| -#endif |
| return 0; |
| } |
| |
