[PATCH v3 0/3] perf probe: Improve logs for long name string

Leo Yan posted 3 patches 1 month, 2 weeks ago
tools/perf/util/probe-event.c | 21 ++++++++++++++++++---
1 file changed, 18 insertions(+), 3 deletions(-)
[PATCH v3 0/3] perf probe: Improve logs for long name string
Posted by Leo Yan 1 month, 2 weeks ago
The series [1] tries to use hashed symbol names for event names. This
can avoid failures due to long symbol names. Somehow, it is preferred to
specify a distinct event name when adding a probe.

This series follows up on the comment to improve the logs for long event
strings.

[1] https://lore.kernel.org/linux-perf-users/20241010234818.ab279974ef597f02b61ab850@kernel.org/T/

Changes from v2:
- Printed a more abstract syntax "EVENT=PROBEDEF" in log.
- Amended warning for adding suffix failure.

Changes from v1:
- Used a patch to add MAX_EVENT_NAME_LEN macro.
- Refined log for long event name failure to replace hashed event name
  (Masami).


Leo Yan (3):
  perf probe: Use the MAX_EVENT_NAME_LEN macro
  perf probe: Check group string length
  perf probe: Improve log for long event name failure

 tools/perf/util/probe-event.c | 21 ++++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

-- 
2.25.1
Re: [PATCH v3 0/3] perf probe: Improve logs for long name string
Posted by Namhyung Kim 1 month, 1 week ago
On Sat, 12 Oct 2024 21:47:22 +0100, Leo Yan wrote:

> The series [1] tries to use hashed symbol names for event names. This
> can avoid failures due to long symbol names. Somehow, it is preferred to
> specify a distinct event name when adding a probe.
> 
> This series follows up on the comment to improve the logs for long event
> strings.
> 
> [...]

Applied to perf-tools-next, thanks!

Best regards,
Namhyung