perf tools amd ibs: Fixes + future enhancements

[PATCH 1/6] perf tool ibs: Sync AMD IBS header file

Posted by Ravi Bangoria 2 weeks, 6 days ago

IBS_OP_DATA2 register will have two more fields: strm_st and rmt_socket
in future AMD CPUs. Kernel header file is already updated. Add those
fields in tools copy as well.

Signed-off-by: Ravi Bangoria <ravi.bangoria@amd.com>
---
 tools/arch/x86/include/asm/amd/ibs.h | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tools/arch/x86/include/asm/amd/ibs.h b/tools/arch/x86/include/asm/amd/ibs.h
index cbce54fec7b9..e9662370e690 100644
--- a/tools/arch/x86/include/asm/amd/ibs.h
+++ b/tools/arch/x86/include/asm/amd/ibs.h
@@ -99,7 +99,9 @@ union ibs_op_data2 {
 			rmt_node:1,	/* 4: destination node */
 			cache_hit_st:1,	/* 5: cache hit state */
 			data_src_hi:2,	/* 6-7: data source high */
-			reserved1:56;	/* 8-63: reserved */
+			strm_st:1,	/* 8: streaming store */
+			rmt_socket:1,   /* 9: remote socket */
+			reserved1:54;   /* 10-63: reserved */
 	};
 };
 
-- 
2.43.0

Re: [PATCH 1/6] perf tool ibs: Sync AMD IBS header file

Posted by Arnaldo Carvalho de Melo 2 weeks, 5 days ago

On Mon, Jan 19, 2026 at 02:43:22AM +0000, Ravi Bangoria wrote:
> IBS_OP_DATA2 register will have two more fields: strm_st and rmt_socket
> in future AMD CPUs. Kernel header file is already updated. Add those
> fields in tools copy as well.

Was this already merged in the kernel? 

- Arnaldo
 
> Signed-off-by: Ravi Bangoria <ravi.bangoria@amd.com>
> ---
>  tools/arch/x86/include/asm/amd/ibs.h | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/tools/arch/x86/include/asm/amd/ibs.h b/tools/arch/x86/include/asm/amd/ibs.h
> index cbce54fec7b9..e9662370e690 100644
> --- a/tools/arch/x86/include/asm/amd/ibs.h
> +++ b/tools/arch/x86/include/asm/amd/ibs.h
> @@ -99,7 +99,9 @@ union ibs_op_data2 {
>  			rmt_node:1,	/* 4: destination node */
>  			cache_hit_st:1,	/* 5: cache hit state */
>  			data_src_hi:2,	/* 6-7: data source high */
> -			reserved1:56;	/* 8-63: reserved */
> +			strm_st:1,	/* 8: streaming store */
> +			rmt_socket:1,   /* 9: remote socket */
> +			reserved1:54;   /* 10-63: reserved */
>  	};
>  };
>  
> -- 
> 2.43.0
>

Re: [PATCH 1/6] perf tool ibs: Sync AMD IBS header file

Posted by Ravi Bangoria 2 weeks, 4 days ago

Hi Arnaldo,

On 21-Jan-26 12:59 AM, Arnaldo Carvalho de Melo wrote:
> On Mon, Jan 19, 2026 at 02:43:22AM +0000, Ravi Bangoria wrote:
>> IBS_OP_DATA2 register will have two more fields: strm_st and rmt_socket
>> in future AMD CPUs. Kernel header file is already updated. Add those
>> fields in tools copy as well.
> 
> Was this already merged in the kernel?

Apologies for any confusion. By "Kernel header file is already updated",
I meant kernel copy has been updated by following patches. But kernel
patches are still under review.

https://lore.kernel.org/r/20260116033450.965-11-ravi.bangoria@amd.com
https://lore.kernel.org/r/20260116033450.965-12-ravi.bangoria@amd.com

Thanks,
Ravi