bpf.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jiri Olsa <olsajiri@gmail.com>
To: chenyuan_fl@163.com
Cc: olsajiri@gmail.com,
	aef2617b-ce03-4830-96a7-39df0c93aaad@kernel.org,
	andrii@kernel.org, ast@kernel.org, bpf@vger.kernel.org,
	chenyuan@kylinos.cn, daniel@iogearbox.net,
	linux-kernel@vger.kernel.org, qmo@kernel.org,
	yonghong.song@linux.dev
Subject: Re: [PATCH v7 0/2] bpftool: Refactor config parsing and add CET symbol matching
Date: Tue, 26 Aug 2025 00:44:36 +0200	[thread overview]
Message-ID: <aKznVD6BjdBCvm0e@krava> (raw)
In-Reply-To: <20250825022002.13760-1-chenyuan_fl@163.com>

On Mon, Aug 25, 2025 at 03:20:00AM +0100, chenyuan_fl@163.com wrote:
> From: Yuan CHen <chenyuan@kylinos.cn>
> 
> 1. **Refactor kernel config parsing**  
>    - Moves duplicate config file handling from feature.c to common.c  
>    - Keeps all existing functionality while enabling code reuse  
> 
> 2. **Add CET-aware symbol matching**  
>    - Adjusts kprobe hook detection for x86_64 CET (endbr32/64 prefixes)  
>    - Matches symbols at both original and CET-adjusted addresses  
> 
> Changed in PATCH v4:
> * Refactor repeated code into a function.
> * Add detection for the x86 architecture.
> 
> Changed int PATH v5:
> * Remove detection for the x86 architecture.
> 
>  Changed in PATCH v6:
> * Add new helper patch (1/2) to refactor kernel config reading
> * Use the new read_kernel_config() in CET symbol matching (2/2) to check CONFIG_X86_KERNEL_IBT
> 
> Changed in PATCH v7:
> * Display actual kprobe attachment addresses instead of symbol addresses

Acked-by: Jiri Olsa <jolsa@kernel.org>

thanks,
jirka


> 
> Yuan Chen (2):
>   bpftool: Refactor kernel config reading into common helper
>   bpftool: Add CET-aware symbol matching for x86_64 architectures
> 
>  tools/bpf/bpftool/common.c  | 93 +++++++++++++++++++++++++++++++++++++
>  tools/bpf/bpftool/feature.c | 86 ++--------------------------------
>  tools/bpf/bpftool/link.c    | 38 ++++++++++++++-
>  tools/bpf/bpftool/main.h    |  9 ++++
>  4 files changed, 142 insertions(+), 84 deletions(-)
> 
> -- 
> 2.39.5
> 

  parent reply	other threads:[~2025-08-25 22:44 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-07-23  2:20 [PATCH v5] bpftool: Add CET-aware symbol matching for x86_64 architectures chenyuan_fl
2025-07-23 10:24 ` Quentin Monnet
2025-07-23 13:08 ` Jiri Olsa
2025-07-23 15:52   ` Yonghong Song
2025-08-15  2:52     ` [PATCH 0/2] bpftool: Refactor config parsing and add CET symbol matching chenyuan_fl
2025-08-15  2:52       ` [PATCH v6 1/2] bpftool: Refactor kernel config reading into common helper chenyuan_fl
2025-08-15  2:52       ` [PATCH v6 2/2] bpftool: Add CET-aware symbol matching for x86_64 architectures chenyuan_fl
2025-08-18  9:55         ` Jiri Olsa
2025-08-25  2:20           ` [PATCH v7 0/2] bpftool: Refactor config parsing and add CET symbol matching chenyuan_fl
2025-08-25  2:20             ` [PATCH v7 1/2] bpftool: Refactor kernel config reading into common helper chenyuan_fl
2025-08-25 20:29               ` Yonghong Song
2025-08-25  2:20             ` [PATCH v7 2/2] bpftool: Add CET-aware symbol matching for x86_64 architectures chenyuan_fl
2025-08-25 20:39               ` Yonghong Song
2025-08-25 22:44             ` Jiri Olsa [this message]
2025-08-27 21:53             ` [PATCH v7 0/2] bpftool: Refactor config parsing and add CET symbol matching Andrii Nakryiko
2025-08-28 21:50               ` Quentin Monnet

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=aKznVD6BjdBCvm0e@krava \
    --to=olsajiri@gmail.com \
    --cc=aef2617b-ce03-4830-96a7-39df0c93aaad@kernel.org \
    --cc=andrii@kernel.org \
    --cc=ast@kernel.org \
    --cc=bpf@vger.kernel.org \
    --cc=chenyuan@kylinos.cn \
    --cc=chenyuan_fl@163.com \
    --cc=daniel@iogearbox.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=qmo@kernel.org \
    --cc=yonghong.song@linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).