Messages in this thread Patch in this message | | | From | Andrew Murray <> | Subject | [PATCH v1 2/2] perf: Use consistent style of comments for optional callbacks | Date | Fri, 3 May 2019 11:23:15 +0100 |
| |
To improve the readability of perf_event.h, let's be consistent in how we annotate optional struct pmu callbacks. Where a multi-line comment block is present before a single function prototype we add the annotation at the end of the block. Otherwise we use a single line comment at the end of the function prototype.
Signed-off-by: Andrew Murray <andrew.murray@arm.com> --- include/linux/perf_event.h | 25 ++++++++++++++++++------- 1 file changed, 18 insertions(+), 7 deletions(-)
diff --git a/include/linux/perf_event.h b/include/linux/perf_event.h index 27c1cb3cddf1..f0479c99d2ec 100644 --- a/include/linux/perf_event.h +++ b/include/linux/perf_event.h @@ -393,8 +393,10 @@ struct pmu { /* * Will return the value for perf_event_mmap_page::index for this event, * if no implementation is provided it will default to: 0. + * + * Optional. */ - int (*event_idx) (struct perf_event *event); /*optional */ + int (*event_idx) (struct perf_event *event); /* * context-switches callback @@ -409,15 +411,18 @@ struct pmu { /* * Set up pmu-private data structures for an AUX area + * + * Optional. */ void *(*setup_aux) (struct perf_event *event, void **pages, int nr_pages, bool overwrite); - /* optional */ /* * Free pmu-private AUX data structures + * + * Optional. */ - void (*free_aux) (void *aux); /* optional */ + void (*free_aux) (void *aux); /* * Validate address range filters: make sure the HW supports the @@ -426,9 +431,10 @@ struct pmu { * * Runs in the context of the ioctl()ing process and is not serialized * with the rest of the PMU callbacks. + * + * Optional. */ int (*addr_filters_validate) (struct list_head *filters); - /* optional */ /* * Synchronize address range filter configuration: @@ -440,19 +446,24 @@ struct pmu { * * May (and should) traverse event::addr_filters::list, for which its * caller provides necessary serialization. + * + * Optional. */ void (*addr_filters_sync) (struct perf_event *event); - /* optional */ /* * Filter events for PMU-specific reasons. + * + * Optional. */ - int (*filter_match) (struct perf_event *event); /* optional */ + int (*filter_match) (struct perf_event *event); /* * Check period value for PERF_EVENT_IOC_PERIOD ioctl. + * + * Optional. */ - int (*check_period) (struct perf_event *event, u64 value); /* optional */ + int (*check_period) (struct perf_event *event, u64 value); }; enum perf_addr_filter_action_t { -- 2.21.0
| |