From nobody Mon Jul 4 18:09:17 2022 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 87CC31C3EF04; Mon, 4 Jul 2022 18:09:18 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4LcDMx5VhKz4kMH; Mon, 4 Jul 2022 18:09:17 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1656958157; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=5FmZusZi66hvkIwldaL3IhU4vmq/bZ0vGOGEEU5jRmw=; b=Z8N0R2+pY+Ddoy/roU6nSgyKPW0jIz08oA4NUI0f+0YsgByNx73NpEWFAtw9OYkn+aNyYh edKvjTLBvwxoGUHS2538X5VrjMVAW/JJ/s2kSmibF6RKZX8XGP7D6x/Sjmrc8tYpnZLEZQ 26Skyvy81gHkYPTh9x5nq6QR1MiT87pMdN7BZzcJpfF+aYh6qRdY+oW1WDy80OZx7aJv6j +JsdzbLNJhhEeWOJ4+Q0BRTWAwVK60NoZ5gN74QQcUUsHD0pc3DvRDt82d6y+n2wA6sj2F jCTPZrr4QzKpo57hsIDd7UVRmUu8jW+jnpPMgJ6ObQHR5hv5OA54z9Gvb9Vsew== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4807415E3C; Mon, 4 Jul 2022 18:09:17 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 264I9HUD064277; Mon, 4 Jul 2022 18:09:17 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 264I9HfR064276; Mon, 4 Jul 2022 18:09:17 GMT (envelope-from git) Date: Mon, 4 Jul 2022 18:09:17 GMT Message-Id: <202207041809.264I9HfR064276@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Alexander Motin Subject: git: 1b4b65dedfc1 - stable/13 - Revert "libpmc: jevents: Partial style(9) pass" List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: mav X-Git-Repository: src X-Git-Refname: refs/heads/stable/13 X-Git-Reftype: branch X-Git-Commit: 1b4b65dedfc13f0f942ef9bf849d0b2003b3ddd2 Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1656958157; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=5FmZusZi66hvkIwldaL3IhU4vmq/bZ0vGOGEEU5jRmw=; b=hh0mnDXrTd/fmqJLC90U0Sjtkx16GV60f3y224Ix2Bx0WROVIDHgOokg8BSd8jLBMEBYkI gXWuL7AGEgbB4nFQ5gcQNpAV22OGibmLQxIc+T+FiJPPKuiK8M9YFfMPIx0iyltrf0RpsW G04YRebxafhaxXe3VUtjRA3//0paG15ln/WDVTjNIuOY65Nn1+s+iZD4ihU/Jl6OHTy1iK tX+XTj6zsWZBhQ3NABNKmw9lxQQAgDoG3eQ9eM5HVN9/Jn4vjjRyOX6XjySfEdyFK/VHY1 4OVaPs3UvcioV8xNGHsVOg4l8cAvccVBpLNVz2ew2u1fE61JJA6fXYKUWvAuzQ== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1656958157; a=rsa-sha256; cv=none; b=LJ17LazJCTS71nTh9Dr5UgIX744iqdtolljP0IjYYyDqxsQZEEGU6NpFUQpPyKvsh1eJY5 VVNNjXIa9yvzNxV2xppufDM1kVIPpmWnCmCkJBNcGKMZbNKRI3nSx2u5p6/ew84u4J/BrN ixlaZTZQIRMkTY6RyklJaI9J2xmCXRkmlRF1FNXe2lM5fdkdBM1db625QdsNsCne/lTSi2 /eNTxZvO0gB/zlBuaURpRTQyVN6DqpIc9UKSOe4W7YQ5ygj4d7djzEKhOULTEiG1YNj3JS XySA2FzRxw0QCm6CMtp5wud+UwgMN2gxQnaMnRR9vnTo04jbu1qEJBTBeHmyGg== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N The branch stable/13 has been updated by mav: URL: https://cgit.FreeBSD.org/src/commit/?id=1b4b65dedfc13f0f942ef9bf849d0b2003b3ddd2 commit 1b4b65dedfc13f0f942ef9bf849d0b2003b3ddd2 Author: Alexander Motin AuthorDate: 2022-05-30 14:07:31 +0000 Commit: Alexander Motin CommitDate: 2022-07-04 17:48:09 +0000 Revert "libpmc: jevents: Partial style(9) pass" This file is shared with Linux. Having local style patches does not help with updating it. This reverts commit 3ea691c2612952dc5e6963649ad64312bd6dff4f. (cherry picked from commit 51f329660f1c56f295fb13bb3a34fb0895f485a1) --- lib/libpmc/pmu-events/jevents.c | 123 ++++++++++++++++------------------------ 1 file changed, 49 insertions(+), 74 deletions(-) diff --git a/lib/libpmc/pmu-events/jevents.c b/lib/libpmc/pmu-events/jevents.c index cca088e8b924..b411329dd35a 100644 --- a/lib/libpmc/pmu-events/jevents.c +++ b/lib/libpmc/pmu-events/jevents.c @@ -35,22 +35,18 @@ #include /* getrlimit */ #include #include /* getrlimit */ - #include #include #include -#include -#include #include #include #include -#include #include #include #include #include #include - +#include #include "list.h" #include "jsmn.h" #include "json.h" @@ -60,11 +56,12 @@ static int nftw_ordered(const char *path, int (*fn)(const char *, const struct stat *, int, struct FTW *), int nfds, int ftwflags); +_Noreturn void _Exit(int); + int verbose; static char *prog; -int -eprintf(int level, int var, const char *fmt, ...) +int eprintf(int level, int var, const char *fmt, ...) { int ret; @@ -82,14 +79,13 @@ eprintf(int level, int var, const char *fmt, ...) return ret; } -__attribute__((weak)) char * -get_cpu_str(void) +__attribute__((weak)) char *get_cpu_str(void) { return NULL; } -static void -addfield(char *map, char **dst, const char *sep, const char *a, jsmntok_t *bt) +static void addfield(char *map, char **dst, const char *sep, + const char *a, jsmntok_t *bt) { unsigned int len = strlen(a) + 1 + strlen(sep); int olen = *dst ? strlen(*dst) : 0; @@ -112,15 +108,13 @@ addfield(char *map, char **dst, const char *sep, const char *a, jsmntok_t *bt) strncat(*dst, map + bt->start, blen); } -static void -fixname(char *s) +static void fixname(char *s) { for (; *s; s++) *s = tolower(*s); } -static void -fixdesc(char *s) +static void fixdesc(char *s) { char *e = s + strlen(s); @@ -133,8 +127,7 @@ fixdesc(char *s) } /* Add escapes for '\' so they are proper C strings. */ -static char * -fixregex(char *s) +static char *fixregex(char *s) { int len = 0; int esc_count = 0; @@ -198,8 +191,7 @@ static struct field { { NULL, NULL } }; -static void -cut_comma(char *map, jsmntok_t *newval) +static void cut_comma(char *map, jsmntok_t *newval) { int i; @@ -210,8 +202,8 @@ cut_comma(char *map, jsmntok_t *newval) } } -static int -match_field(char *map, jsmntok_t *field, int nz, char **event, jsmntok_t *val) +static int match_field(char *map, jsmntok_t *field, int nz, + char **event, jsmntok_t *val) { struct field *f; jsmntok_t newval = *val; @@ -225,8 +217,7 @@ match_field(char *map, jsmntok_t *field, int nz, char **event, jsmntok_t *val) return 0; } -static struct msrmap * -lookup_msr(char *map, jsmntok_t *val) +static struct msrmap *lookup_msr(char *map, jsmntok_t *val) { jsmntok_t newval = *val; static bool warned; @@ -260,8 +251,7 @@ static struct map { {} }; -static const char * -field_to_perf(struct map *table, char *map, jsmntok_t *val) +static const char *field_to_perf(struct map *table, char *map, jsmntok_t *val) { int i; @@ -285,8 +275,7 @@ field_to_perf(struct map *table, char *map, jsmntok_t *val) static char *topic; -static char * -get_topic(void) +static char *get_topic(void) { char *tp; int i; @@ -312,8 +301,7 @@ get_topic(void) return tp; } -static int -add_topic(const char *bname) +static int add_topic(const char *bname) { free(topic); topic = strdup(bname); @@ -332,17 +320,17 @@ struct perf_entry_data { static int close_table; -static void -print_events_table_prefix(FILE *fp, const char *tblname) +static void print_events_table_prefix(FILE *fp, const char *tblname) { fprintf(fp, "static struct pmu_event %s[] = {\n", tblname); close_table = 1; } -static int -print_events_table_entry(void *data, char *name, const char *event, char *desc, - char *long_desc, char *pmu, char *unit, char *perpkg, char *metric_expr, - char *metric_name, char *metric_group) +static int print_events_table_entry(void *data, char *name, const char *event, + char *desc, char *long_desc, + char *pmu, char *unit, char *perpkg, + char *metric_expr, + char *metric_name, char *metric_group) { struct perf_entry_data *pd = data; FILE *outfp = pd->outfp; @@ -422,8 +410,7 @@ struct event_struct { static LIST_HEAD(arch_std_events); -static void -free_arch_std_events(void) +static void free_arch_std_events(void) { struct event_struct *es, *next; @@ -434,10 +421,10 @@ free_arch_std_events(void) } } -static int -save_arch_std_events(void *data __unused, char *name, const char *event, - char *desc, char *long_desc, char *pmu, char *unit, char *perpkg, - char *metric_expr, char *metric_name, char *metric_group) +static int save_arch_std_events(void *data __unused, char *name, const char *event, + char *desc, char *long_desc, char *pmu, + char *unit, char *perpkg, char *metric_expr, + char *metric_name, char *metric_group) { struct event_struct *es; @@ -454,8 +441,7 @@ out_free: return -ENOMEM; } -static void -print_events_table_suffix(FILE *outfp) +static void print_events_table_suffix(FILE *outfp) { fprintf(outfp, "{\n"); @@ -483,8 +469,7 @@ static struct fixed { /* * Handle different fixed counter encodings between JSON and perf. */ -static const char * -real_event(const char *name, char *event) +static const char *real_event(const char *name, char *event) { int i; @@ -499,9 +484,9 @@ real_event(const char *name, char *event) static int try_fixup(const char *fn, char *arch_std, char **event, char **desc, - char **name, char **long_desc, char **pmu, char **filter __unused, - char **perpkg, char **unit, char **metric_expr, char **metric_name, - char **metric_group, unsigned long long eventcode) + char **name, char **long_desc, char **pmu, char **filter __unused, + char **perpkg, char **unit, char **metric_expr, char **metric_name, + char **metric_group, unsigned long long eventcode) { /* try to find matching event from arch standard values */ struct event_struct *es; @@ -524,8 +509,7 @@ try_fixup(const char *fn, char *arch_std, char **event, char **desc, } /* Call func with each event in the json file */ -int -json_events(const char *fn, +int json_events(const char *fn, int (*func)(void *data, char *name, const char *event, char *desc, char *long_desc, char *pmu, char *unit, char *perpkg, @@ -713,8 +697,7 @@ out_free: return err; } -static char * -file_name_to_table_name(const char *fname) +static char *file_name_to_table_name(const char *fname) { unsigned int i; int n; @@ -756,14 +739,12 @@ file_name_to_table_name(const char *fname) return tblname; } -static void -print_mapping_table_prefix(FILE *outfp) +static void print_mapping_table_prefix(FILE *outfp) { fprintf(outfp, "struct pmu_events_map pmu_events_map[] = {\n"); } -static void -print_mapping_table_suffix(FILE *outfp) +static void print_mapping_table_suffix(FILE *outfp) { /* * Print the terminating, NULL entry. @@ -779,8 +760,7 @@ print_mapping_table_suffix(FILE *outfp) fprintf(outfp, "};\n"); } -static int -process_mapfile(FILE *outfp, char *fpath) +static int process_mapfile(FILE *outfp, char *fpath) { int n = 16384; FILE *mapfp; @@ -868,8 +848,7 @@ out: * table. This would at least allow perf to build even if we can't find/use * the aliases. */ -static void -create_empty_mapping(const char *output_file) +static void create_empty_mapping(const char *output_file) { FILE *outfp; @@ -888,8 +867,7 @@ create_empty_mapping(const char *output_file) fclose(outfp); } -static int -get_maxfds(void) +static int get_maxfds(void) { struct rlimit rlim; @@ -909,8 +887,7 @@ get_maxfds(void) static FILE *eventsfp; static char *mapfile; -static int -is_leaf_dir(const char *fpath) +static int is_leaf_dir(const char *fpath) { DIR *d; struct dirent *dir; @@ -948,8 +925,7 @@ is_leaf_dir(const char *fpath) return res; } -static int -is_json_file(const char *name) +static int is_json_file(const char *name) { const char *suffix; @@ -963,9 +939,8 @@ is_json_file(const char *name) return 0; } -static int -preprocess_arch_std_files(const char *fpath, const struct stat *sb, - int typeflag, struct FTW *ftwbuf) +static int preprocess_arch_std_files(const char *fpath, const struct stat *sb, + int typeflag, struct FTW *ftwbuf) { int level = ftwbuf->level; int is_file = typeflag == FTW_F; @@ -976,9 +951,8 @@ preprocess_arch_std_files(const char *fpath, const struct stat *sb, return 0; } -static int -process_one_file(const char *fpath, const struct stat *sb, int typeflag, - struct FTW *ftwbuf) +static int process_one_file(const char *fpath, const struct stat *sb, + int typeflag, struct FTW *ftwbuf) { char *tblname; const char *bname; @@ -1103,8 +1077,7 @@ process_one_file(const char *fpath, const struct stat *sb, int typeflag, * * Write out the PMU events tables and the mapping table to pmu-event.c. */ -int -main(int argc, char *argv[]) +int main(int argc, char *argv[]) { int rc; int maxfds; @@ -1206,6 +1179,8 @@ empty_map: return 0; } +#include + static int fts_compare(const FTSENT * const *a, const FTSENT * const *b) {