On Mon, Jun 06, 2022 at 07:55:52PM +0800, Yicong Yang wrote: > From: Qi Liu <liuqi115@xxxxxxxxxx> > > HiSilicon PCIe tune and trace device (PTT) could dynamically tune > the PCIe link's events, and trace the TLP headers). > > This patch add support for PTT device in perf tool, so users could > use 'perf record' to get TLP headers trace data. > > Signed-off-by: Qi Liu <liuqi115@xxxxxxxxxx> > Signed-off-by: Yicong Yang <yangyicong@xxxxxxxxxxxxx> Just one minor comment. [...] > diff --git a/tools/perf/util/hisi-ptt.h b/tools/perf/util/hisi-ptt.h > new file mode 100644 > index 000000000000..2db9b4056214 > --- /dev/null > +++ b/tools/perf/util/hisi-ptt.h > @@ -0,0 +1,19 @@ > +/* SPDX-License-Identifier: GPL-2.0 */ > +/* > + * HiSilicon PCIe Trace and Tuning (PTT) support > + * Copyright (c) 2022 HiSilicon Technologies Co., Ltd. > + */ > + > +#ifndef INCLUDE__PERF_HISI_PTT_H__ > +#define INCLUDE__PERF_HISI_PTT_H__ > + > +#define HISI_PTT_PMU_NAME "hisi_ptt" > +#define HISI_PTT_AUXTRACE_PRIV_SIZE sizeof(u64) > + > +struct auxtrace_record *hisi_ptt_recording_init(int *err, > + struct perf_pmu *hisi_ptt_pmu); > + > +int hisi_ptt_process_auxtrace_info(union perf_event *event, > + struct perf_session *session); The function hisi_ptt_process_auxtrace_info() is introduced by next patch for support PTT decoding, for good practice (e.g. keep bisection), it's good to introduce function declaration and definition in one patch. With this fixing, this patch looks good to me: Reviewed-by: Leo Yan <leo.yan@xxxxxxxxxx> > + > +#endif > -- > 2.24.0
