From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-9.6 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 71377C432C0 for ; Tue, 3 Dec 2019 10:35:35 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 3AD55205ED for ; Tue, 3 Dec 2019 10:35:35 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="ZbyxxI07" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726107AbfLCKff (ORCPT ); Tue, 3 Dec 2019 05:35:35 -0500 Received: from mail-lj1-f196.google.com ([209.85.208.196]:42796 "EHLO mail-lj1-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726074AbfLCKfe (ORCPT ); Tue, 3 Dec 2019 05:35:34 -0500 Received: by mail-lj1-f196.google.com with SMTP id e28so3152001ljo.9 for ; Tue, 03 Dec 2019 02:35:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=jZjcSHRXcQc9LXYl9gLx4saHozoSnnADGecLzjfnxP4=; b=ZbyxxI07RR1XWPG4qlFq9t3wf8zXQ9W0SkeySzOooj9EL4CBwMkAdKURU3ShChVTXE UhxdSWReOEKhmNzWZjWJmW8ZvtC1WISp9lz941g+wV3tCkmdMv/hwk+g22kG1j9rQ/MX iJ8BtZ6fuFFUmB0kdVHW8aHgynj+HwuoS1O/L2t8q4cL+A9snrunkzZzPzSU5sbUEk3d qxeHV7Xma4sxwGAeny+RuPddxElnhVst1hw3R8hWBfNZQhu9/K3cfNpOSnPkH8DRHLrT B9VO7P/8w3pF5CXl8IxjnVeNtutHPnLhLDL117GIkTgAM5VPhNa43hUCs1zPDBO+a5ud VDxw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=jZjcSHRXcQc9LXYl9gLx4saHozoSnnADGecLzjfnxP4=; b=lOfaPZLL7y6eVb849nJlFMgQvG3U9Z/VD7WFu5YXb8qVp0VhTtVLykN91K8aS0/e65 q7c1BlXKQbnYGG9aLUpKFdegPrmKOOURhJwQj7WNIVayYkTf/NdVTNn3EAEI7Up3f2ls rJ8f4pewQ8yTSvbeVsX6h6Evi0IsYgvjztCY/G1Z7xkbdm6pv25+998CD8U90HfQO7fO /rkUohq2/3OOCJ0lhItcgPRhgjTtOLuXPutBWV1TdilTaX/lPMyB/NaHdFjL7cfRdzBQ bCcdNPzUmOR6UeZ8GHI0z2YIN2pNmqr/dPDlqPmf+DeLQXAZReYWrMW9/VfycqNsYDi5 cZPA== X-Gm-Message-State: APjAAAXf5yrZRL/XohpZztoKZ6mg2vIvQTXC0iihdWXb4t0WiEeQtC6F qnm70jd8mirapMbKgZjq2jE= X-Google-Smtp-Source: APXvYqx9hniq9+vAHMHBy3DaTAEq80tUif1w/A4JQlIczl59waGZfs7y4onF6NR/GM+hsx2wm70wAQ== X-Received: by 2002:a05:651c:3dc:: with SMTP id f28mr2229659ljp.35.1575369332610; Tue, 03 Dec 2019 02:35:32 -0800 (PST) Received: from oberon.eng.vmware.com ([146.247.46.5]) by smtp.gmail.com with ESMTPSA id t9sm1053023lfl.51.2019.12.03.02.35.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 03 Dec 2019 02:35:32 -0800 (PST) From: "Tzvetomir Stoyanov (VMware)" To: rostedt@goodmis.org Cc: linux-trace-devel@vger.kernel.org Subject: [PATCH v17 05/18] trace-cmd: Add new library API for local CPU count Date: Tue, 3 Dec 2019 12:35:09 +0200 Message-Id: <20191203103522.482684-6-tz.stoyanov@gmail.com> X-Mailer: git-send-email 2.23.0 In-Reply-To: <20191203103522.482684-1-tz.stoyanov@gmail.com> References: <20191203103522.482684-1-tz.stoyanov@gmail.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-trace-devel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-trace-devel@vger.kernel.org In order to reuse the code, the function detecting the number of local CPU is moved from trace-cmd application to libtracecmd. The following new library API is introduced: int tracecmd_count_cpus(void); Signed-off-by: Tzvetomir Stoyanov (VMware) --- include/trace-cmd/trace-cmd.h | 3 +++ lib/trace-cmd/trace-util.c | 40 +++++++++++++++++++++++++++++ tracecmd/include/trace-local.h | 2 -- tracecmd/trace-agent.c | 2 +- tracecmd/trace-profile.c | 2 +- tracecmd/trace-record.c | 46 +++------------------------------- tracecmd/trace-stat.c | 4 +-- 7 files changed, 50 insertions(+), 49 deletions(-) diff --git a/include/trace-cmd/trace-cmd.h b/include/trace-cmd/trace-cmd.h index 5287d23..5d4292e 100644 --- a/include/trace-cmd/trace-cmd.h +++ b/include/trace-cmd/trace-cmd.h @@ -427,6 +427,9 @@ void tracecmd_plog(const char *fmt, ...); void tracecmd_plog_error(const char *fmt, ...); int tracecmd_set_logfile(char *logfile); +/* --- System --- */ +int tracecmd_count_cpus(void); + /* --- Hack! --- */ int tracecmd_blk_hack(struct tracecmd_input *handle); diff --git a/lib/trace-cmd/trace-util.c b/lib/trace-cmd/trace-util.c index 4c70d9a..e019dce 100644 --- a/lib/trace-cmd/trace-util.c +++ b/lib/trace-cmd/trace-util.c @@ -1112,6 +1112,46 @@ int tracecmd_stack_tracer_status(int *status) return 1; /* full success */ } +int tracecmd_count_cpus(void) +{ + FILE *fp; + char buf[1024]; + int cpus = 0; + char *pbuf; + size_t *pn; + size_t n; + int r; + + cpus = sysconf(_SC_NPROCESSORS_CONF); + if (cpus > 0) + return cpus; + + warning("sysconf could not determine number of CPUS"); + + /* Do the hack to figure out # of CPUS */ + n = 1024; + pn = &n; + pbuf = buf; + + fp = fopen("/proc/cpuinfo", "r"); + if (!fp) + die("Can not read cpuinfo"); + + while ((r = getline(&pbuf, pn, fp)) >= 0) { + char *p; + + if (strncmp(buf, "processor", 9) != 0) + continue; + for (p = buf+9; isspace(*p); p++) + ; + if (*p == ':') + cpus++; + } + fclose(fp); + + return cpus; +} + /* * tracecmd_write_file - Write in trace file * @file: Full name of the trace file. diff --git a/tracecmd/include/trace-local.h b/tracecmd/include/trace-local.h index 3a05e9c..a51f0d0 100644 --- a/tracecmd/include/trace-local.h +++ b/tracecmd/include/trace-local.h @@ -243,8 +243,6 @@ void update_first_instance(struct buffer_instance *instance, int topt); void show_instance_file(struct buffer_instance *instance, const char *name); -int count_cpus(void); - /* moved from trace-cmd.h */ void tracecmd_create_top_instance(char *name); void tracecmd_remove_instances(void); diff --git a/tracecmd/trace-agent.c b/tracecmd/trace-agent.c index 93e3edf..1c6e0a3 100644 --- a/tracecmd/trace-agent.c +++ b/tracecmd/trace-agent.c @@ -203,7 +203,7 @@ static void agent_serve(unsigned int port) signal(SIGCHLD, handle_sigchld); - nr_cpus = count_cpus(); + nr_cpus = tracecmd_count_cpus(); page_size = getpagesize(); sd = make_vsock(port); diff --git a/tracecmd/trace-profile.c b/tracecmd/trace-profile.c index 231e381..cfae2a2 100644 --- a/tracecmd/trace-profile.c +++ b/tracecmd/trace-profile.c @@ -1314,7 +1314,7 @@ void trace_init_profile(struct tracecmd_input *handle, struct hook_list *hook, * system. */ if (!h->cpus) - h->cpus = count_cpus(); + h->cpus = tracecmd_count_cpus(); list_head_init(&h->migrate_starts); h->cpu_starts = malloc(sizeof(*h->cpu_starts) * h->cpus); diff --git a/tracecmd/trace-record.c b/tracecmd/trace-record.c index 13a7cda..4459c90 100644 --- a/tracecmd/trace-record.c +++ b/tracecmd/trace-record.c @@ -2726,46 +2726,6 @@ static void expand_event_list(void) expand_event_instance(instance); } -int count_cpus(void) -{ - FILE *fp; - char buf[1024]; - int cpus = 0; - char *pbuf; - size_t *pn; - size_t n; - int r; - - cpus = sysconf(_SC_NPROCESSORS_CONF); - if (cpus > 0) - return cpus; - - warning("sysconf could not determine number of CPUS"); - - /* Do the hack to figure out # of CPUS */ - n = 1024; - pn = &n; - pbuf = buf; - - fp = fopen("/proc/cpuinfo", "r"); - if (!fp) - die("Can not read cpuinfo"); - - while ((r = getline(&pbuf, pn, fp)) >= 0) { - char *p; - - if (strncmp(buf, "processor", 9) != 0) - continue; - for (p = buf+9; isspace(*p); p++) - ; - if (*p == ':') - cpus++; - } - fclose(fp); - - return cpus; -} - static void finish(int sig) { /* all done */ @@ -4487,7 +4447,7 @@ static void reset_clock(void) static void reset_cpu_mask(void) { struct buffer_instance *instance; - int cpus = count_cpus(); + int cpus = tracecmd_count_cpus(); int fullwords = (cpus - 1) / 32; int bits = (cpus - 1) % 32 + 1; int len = (fullwords + 1) * 9; @@ -5043,7 +5003,7 @@ void init_top_instance(void) { if (!top_instance.ftrace) top_instance.ftrace = tracecmd_create_instance(NULL); - top_instance.cpu_count = count_cpus(); + top_instance.cpu_count = tracecmd_count_cpus(); top_instance.flags = BUFFER_FL_KEEP; tracecmd_init_instance(&top_instance); } @@ -5240,7 +5200,7 @@ static void init_common_record_context(struct common_record_context *ctx, memset(ctx, 0, sizeof(*ctx)); ctx->instance = &top_instance; ctx->curr_cmd = curr_cmd; - local_cpu_count = count_cpus(); + local_cpu_count = tracecmd_count_cpus(); init_top_instance(); } diff --git a/tracecmd/trace-stat.c b/tracecmd/trace-stat.c index 32aebc5..44ee559 100644 --- a/tracecmd/trace-stat.c +++ b/tracecmd/trace-stat.c @@ -737,7 +737,7 @@ static void report_cpumask(struct buffer_instance *instance) cont = strstrip(str); /* check to make sure all CPUs on this machine are set */ - cpus = count_cpus(); + cpus = tracecmd_count_cpus(); for (i = strlen(cont) - 1; i >= 0 && cpus > 0; i--) { if (cont[i] == ',') @@ -896,7 +896,7 @@ void trace_stat (int argc, char **argv) instance = create_instance(optarg); if (!instance) die("Failed to create instance"); - add_instance(instance, count_cpus()); + add_instance(instance, tracecmd_count_cpus()); /* top instance requires direct access */ if (!topt && is_top_instance(first_instance)) first_instance = instance; -- 2.23.0