[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] xen/x86: Add Xenoprofile support for AMD Family 17h

On 5/23/2017 4:46 PM, Boris Ostrovsky wrote:
On 05/23/2017 05:28 PM, Gary R Hook wrote:
Signed-off-by: Gary R Hook <gary.hook@xxxxxxx>
  xen/arch/x86/oprofile/nmi_int.c |    4 ++++
  1 file changed, 4 insertions(+)

diff --git a/xen/arch/x86/oprofile/nmi_int.c b/xen/arch/x86/oprofile/nmi_int.c
index 13534d491405..5ad48c12e515 100644
--- a/xen/arch/x86/oprofile/nmi_int.c
+++ b/xen/arch/x86/oprofile/nmi_int.c
@@ -419,6 +419,10 @@ static int __init nmi_init(void)
                                model = &op_athlon_spec;
                                cpu_type = "x86-64/family16h";
+                       case 0x17:
+                                model = &op_amd_fam15h_spec;
+                               cpu_type = "x86-64/family17h";
+                               break;

Have you actually tried this? I don't know whether oprofile still works
since corresponding kernel patches that I am aware of are at least 5
years old.

Yes, I was getting a complaint during boot. That's why I did it. Works a treat on my family 17 system :-)

I don't know whether the code is relevant (and if not, it should be
removed, right?) but I don't like complaints, no matter how spurious. Thus, I minor patch.

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.