Re: [PATCH] OProfile: correct CPU type for x86-64

[Andi Kleen]

John Levon <levon@xxxxxxxxxxxxxxxxx> writes:

> +				break;
> +#if defined(CONFIG_X86_64)
> +			case 0xf:
> +				model = &op_athlon_spec;
> +				nmi_ops.cpu_type = "x86-64/hammer";
> +				break;
> +#endif /* CONFIG_X86_64 */

The ifdef is not needed. The case is valid for i386 too, when running
an Opteron with an 32bit kernel. In that case you also want the user tools
to select the Opteron events.

-Andi

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/