On Wed, Nov 13, 2019 at 01:24:53PM +0800, Wu Hao wrote: > Hi Moritz and all, Hi Will and Mark, Could you please help us on review this patchset? as this patchset mainly introduced a new perf driver following the similar way as drivers/perf/*. Hi Moritz, Could you please help us on review the other parts? This patchset has been submitted for a long time but didn't receive any comment after v4. we appreciate any review comments! thanks in advance. :) Hao > > This patchset adds performance reporting support for FPGA DFL drivers. It > introduces one pmu to expose userspace interfaces via standard perf API. > User could use standard perf tool to access perf events exposed via pmu. > > This patchset is generated based on fpga/for-next branch. > > Main changes from v5: > - use dev_ext_attribute instead of fme_perf_event_attr. > - use is_visible function to decide which events to expose per > hardware capability, and add event_init checking for all events. > > Main changes from v4: > - rebase and clean up. > - update Kconfig for PERF_EVENTS dependency. > > Main changes from v3: > - add more descriptions in doc, including how to use perf tool for these > hardware counters. (patch #1) > - use standard perf API instead of sysfs entries. (patch #2) > > Wu Hao (1): > fpga: dfl: fme: add performance reporting support > > Xu Yilun (1): > Documentation: fpga: dfl: add description for performance reporting > support > > Documentation/fpga/dfl.rst | 83 ++++ > drivers/fpga/Makefile | 1 + > drivers/fpga/dfl-fme-main.c | 4 + > drivers/fpga/dfl-fme-perf.c | 943 ++++++++++++++++++++++++++++++++++++++++++++ > drivers/fpga/dfl-fme.h | 2 + > drivers/fpga/dfl.h | 2 + > 6 files changed, 1035 insertions(+) > create mode 100644 drivers/fpga/dfl-fme-perf.c > > -- > 1.8.3.1
