linux-kernel - [PATCH v4 0/2] Use of devname for interrupt descriptions and tracepoint support for smp2p

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <20240716173835.997259-1-quic_sudeepgo@quicinc.com>
Date: Tue, 16 Jul 2024 23:08:32 +0530
From: Sudeepgoud Patil <quic_sudeepgo@...cinc.com>
To: <quic_bjorande@...cinc.com>, <andersson@...nel.org>,
        <quic_clew@...cinc.com>, <mathieu.poirier@...aro.org>,
        <quic_deesin@...cinc.com>
CC: <linux-kernel@...r.kernel.org>, <linux-arm-msm@...r.kernel.org>,
        <linux-remoteproc@...r.kernel.org>, <quic_sudeepgo@...cinc.com>
Subject: [PATCH v4 0/2] Use of devname for interrupt descriptions and tracepoint support for smp2p

This commit enhances the smp2p driver by adding support for using the device
name in interrupt descriptions and introducing tracepoint functionality.
These improvements facilitate more effective debugging of smp2p-related issues.

The devname patch, along with the callback to print the irq chip name as the
device name and the removal of the ‘smp2p’ string from the irq request,
results in a unique interrupt description.

Tracepoint functionality captures essential details such as subsystem name,
negotiation specifics, supported features, bit changes, and subsystem restart
activity. These enhancements significantly improve debugging capabilities
for inter-subsystem issues.

Changes in v4:
- Add sign-off for "Use devname for interrupt descriptions" patch
- Update commit message in imperative mood for tracepoint patch
- Remove second argument of __assign_str() as per the commit 2c92ca8 ("tracing/treewide: Remove second parameter of __assign_str()") 
- Link to v3: https://lore.kernel.org/all/20240627104831.4176799-1-quic_sudeepgo@quicinc.com 

Changes in v3:
- Update patch to use devname for interrupt descriptions with a different approach
- Modify tracepoint patch by removing remote_pid field from all tracepoints
- Use SMP2P_FEATURE_SSR_ACK definition from smp2p.c instead of redefiniton
- Link to v2: https://lore.kernel.org/all/20240611123351.3813190-1-quic_sudeepgo@quicinc.com

Changes in v2:
- Add support to include the remote name in the smp2p IRQ devname, allowing for remote PID-name mapping
- Map the remote PID (Process ID) along with the remote name in tracepoints, as suggested by Chris
- Modify to capture all `out->features` instead of just the `ssr_ack`, following Chris's recommendation
- Expand the commit description to provide additional context
- Link to v1: https://lore.kernel.org/all/20240429075528.1723133-1-quic_sudeepgo@quicinc.com

Chris Lew (1):
  soc: qcom: smp2p: Use devname for interrupt descriptions

Sudeepgoud Patil (1):
  soc: qcom: smp2p: Introduce tracepoint support

 drivers/soc/qcom/Makefile      |  1 +
 drivers/soc/qcom/smp2p.c       | 20 ++++++-
 drivers/soc/qcom/trace-smp2p.h | 98 ++++++++++++++++++++++++++++++++++
 3 files changed, 118 insertions(+), 1 deletion(-)
 create mode 100644 drivers/soc/qcom/trace-smp2p.h


base-commit: d67978318827d06f1c0fa4c31343a279e9df6fde
--