From: Riccardo Mancini <rickyman7@gmail.com> To: Arnaldo Carvalho de Melo <acme@kernel.org> Cc: Ian Rogers <irogers@google.com>, Namhyung Kim <namhyung@kernel.org>, Peter Zijlstra <peterz@infradead.org>, Ingo Molnar <mingo@redhat.com>, Mark Rutland <mark.rutland@arm.com>, Jiri Olsa <jolsa@redhat.com>, linux-kernel@vger.kernel.org, linux-perf-users@vger.kernel.org, Alexey Bayduraev <alexey.v.bayduraev@linux.intel.com>, Riccardo Mancini <rickyman7@gmail.com> Subject: [RFC PATCH v2 01/10] perf workqueue: threadpool creation and destruction Date: Fri, 30 Jul 2021 17:34:08 +0200 [thread overview] Message-ID: <4f0cd6c8e77c0b4f4d4b8d553a7032757b976e61.1627657061.git.rickyman7@gmail.com> (raw) In-Reply-To: <cover.1627657061.git.rickyman7@gmail.com> The workqueue library is made up by two components: - threadpool: handles the lifetime of the threads - workqueue: handles work distribution among the threads This first patch introduces the threadpool, starting from its creation and destruction functions. Thread management is based on the prototype from Alexey: https://lore.kernel.org/lkml/cover.1625227739.git.alexey.v.bayduraev@linux.intel.com/ Each thread in the threadpool executes the same function (aka task) with a different argument tidx. Threads use a pair of pipes to communicate with the main process. The threadpool is static (all threads will be spawned at the same time). Future work could include making it resizable and adding affinity support (as in Alexey prototype). Suggested-by: Alexey Bayduraev <alexey.v.bayduraev@linux.intel.com> Signed-off-by: Riccardo Mancini <rickyman7@gmail.com> --- tools/perf/util/Build | 1 + tools/perf/util/workqueue/Build | 1 + tools/perf/util/workqueue/threadpool.c | 208 +++++++++++++++++++++++++ tools/perf/util/workqueue/threadpool.h | 30 ++++ 4 files changed, 240 insertions(+) create mode 100644 tools/perf/util/workqueue/Build create mode 100644 tools/perf/util/workqueue/threadpool.c create mode 100644 tools/perf/util/workqueue/threadpool.h diff --git a/tools/perf/util/Build b/tools/perf/util/Build index 2d4fa13041789cd6..c7b09701661c869d 100644 --- a/tools/perf/util/Build +++ b/tools/perf/util/Build @@ -180,6 +180,7 @@ perf-$(CONFIG_LIBBABELTRACE) += data-convert-bt.o perf-y += data-convert-json.o perf-y += scripting-engines/ +perf-y += workqueue/ perf-$(CONFIG_ZLIB) += zlib.o perf-$(CONFIG_LZMA) += lzma.o diff --git a/tools/perf/util/workqueue/Build b/tools/perf/util/workqueue/Build new file mode 100644 index 0000000000000000..8b72a6cd4e2cba0d --- /dev/null +++ b/tools/perf/util/workqueue/Build @@ -0,0 +1 @@ +perf-y += threadpool.o diff --git a/tools/perf/util/workqueue/threadpool.c b/tools/perf/util/workqueue/threadpool.c new file mode 100644 index 0000000000000000..0004ce606d5fa73d --- /dev/null +++ b/tools/perf/util/workqueue/threadpool.c @@ -0,0 +1,208 @@ +// SPDX-License-Identifier: GPL-2.0 +#include <stdlib.h> +#include <stdio.h> +#include <unistd.h> +#include <errno.h> +#include <string.h> +#include "debug.h" +#include <asm/bug.h> +#include <linux/zalloc.h> +#include <linux/string.h> +#include <linux/err.h> +#include <linux/kernel.h> +#include "threadpool.h" + +enum threadpool_status { + THREADPOOL_STATUS__STOPPED, /* no threads */ + THREADPOOL_STATUS__ERROR, /* errors */ + THREADPOOL_STATUS__MAX +}; + +struct threadpool { + int nr_threads; /* number of threads in the pool */ + struct threadpool_entry *threads; /* array of threads in the pool */ + struct task_struct *current_task; /* current executing function */ + enum threadpool_status status; /* current status of the pool */ +}; + +struct threadpool_entry { + int idx; /* idx of thread in pool->threads */ + pid_t tid; /* tid of thread */ + struct threadpool *pool; /* parent threadpool */ + struct { + int ack[2]; /* messages from thread (acks) */ + int cmd[2]; /* messages to thread (commands) */ + } pipes; +}; + +/** + * threadpool_entry__init_pipes - initialize all pipes of @thread + */ +static void threadpool_entry__init_pipes(struct threadpool_entry *thread) +{ + thread->pipes.ack[0] = -1; + thread->pipes.ack[1] = -1; + thread->pipes.cmd[0] = -1; + thread->pipes.cmd[1] = -1; +} + +/** + * threadpool_entry__open_pipes - open all pipes of @thread + */ +static int threadpool_entry__open_pipes(struct threadpool_entry *thread) +{ + if (pipe(thread->pipes.ack)) { + pr_debug2("threadpool: failed to create comm pipe 'from': %s\n", + strerror(errno)); + return -ENOMEM; + } + + if (pipe(thread->pipes.cmd)) { + pr_debug2("threadpool: failed to create comm pipe 'to': %s\n", + strerror(errno)); + close(thread->pipes.ack[0]); + thread->pipes.ack[0] = -1; + close(thread->pipes.ack[1]); + thread->pipes.ack[1] = -1; + return -ENOMEM; + } + + return 0; +} + +/** + * threadpool_entry__close_pipes - close all communication pipes of @thread + */ +static void threadpool_entry__close_pipes(struct threadpool_entry *thread) +{ + if (thread->pipes.ack[0] != -1) { + close(thread->pipes.ack[0]); + thread->pipes.ack[0] = -1; + } + if (thread->pipes.ack[1] != -1) { + close(thread->pipes.ack[1]); + thread->pipes.ack[1] = -1; + } + if (thread->pipes.cmd[0] != -1) { + close(thread->pipes.cmd[0]); + thread->pipes.cmd[0] = -1; + } + if (thread->pipes.cmd[1] != -1) { + close(thread->pipes.cmd[1]); + thread->pipes.cmd[1] = -1; + } +} + +/** + * threadpool__new - create a fixed threadpool with @n_threads threads + */ +struct threadpool *threadpool__new(int n_threads) +{ + int ret, err, t; + struct threadpool *pool = malloc(sizeof(*pool)); + + if (!pool) { + pr_debug2("threadpool: cannot allocate pool: %s\n", + strerror(errno)); + err = -ENOMEM; + goto out_return; + } + + if (n_threads <= 0) { + pr_debug2("threadpool: invalid number of threads: %d\n", + n_threads); + err = -EINVAL; + goto out_free_pool; + } + + pool->nr_threads = n_threads; + pool->current_task = NULL; + + pool->threads = calloc(n_threads, sizeof(*pool->threads)); + if (!pool->threads) { + pr_debug2("threadpool: cannot allocate threads: %s\n", + strerror(errno)); + err = -ENOMEM; + goto out_free_pool; + } + + for (t = 0; t < n_threads; t++) { + pool->threads[t].idx = t; + pool->threads[t].tid = -1; + pool->threads[t].pool = pool; + threadpool_entry__init_pipes(&pool->threads[t]); + } + + for (t = 0; t < n_threads; t++) { + ret = threadpool_entry__open_pipes(&pool->threads[t]); + if (ret) { + err = -ret; + goto out_close_pipes; + } + } + + pool->status = THREADPOOL_STATUS__STOPPED; + + return pool; + +out_close_pipes: + for (t = 0; t < n_threads; t++) + threadpool_entry__close_pipes(&pool->threads[t]); + + zfree(&pool->threads); +out_free_pool: + free(pool); +out_return: + return ERR_PTR(err); +} + +/** + * threadpool__strerror - print message regarding given @err in @pool + * + * Buffer size should be at least THREADPOOL_STRERR_BUFSIZE bytes. + */ +int threadpool__strerror(struct threadpool *pool __maybe_unused, int err, char *buf, size_t size) +{ + char sbuf[STRERR_BUFSIZE], *emsg; + + emsg = str_error_r(err, sbuf, sizeof(sbuf)); + return scnprintf(buf, size, "Error: %s.\n", emsg); +} + +/** + * threadpool__new_strerror - print message regarding @err_ptr + * + * Buffer size should be at least THREADPOOL_STRERR_BUFSIZE bytes. + */ +int threadpool__new_strerror(struct threadpool *err_ptr, char *buf, size_t size) +{ + return threadpool__strerror(err_ptr, PTR_ERR(err_ptr), buf, size); +} + +/** + * threadpool__delete - free the @pool and all its resources + */ +void threadpool__delete(struct threadpool *pool) +{ + int t; + + if (IS_ERR_OR_NULL(pool)) + return; + + WARN_ON(pool->status != THREADPOOL_STATUS__STOPPED + && pool->status != THREADPOOL_STATUS__ERROR); + + for (t = 0; t < pool->nr_threads; t++) + threadpool_entry__close_pipes(&pool->threads[t]); + + zfree(&pool->threads); + free(pool); +} + +/** + * threadpool__size - get number of threads in the threadpool + */ +int threadpool__size(struct threadpool *pool) +{ + return pool->nr_threads; +} diff --git a/tools/perf/util/workqueue/threadpool.h b/tools/perf/util/workqueue/threadpool.h new file mode 100644 index 0000000000000000..fb18aa32fb64f671 --- /dev/null +++ b/tools/perf/util/workqueue/threadpool.h @@ -0,0 +1,30 @@ +/* SPDX-License-Identifier: GPL-2.0 */ +#ifndef __WORKQUEUE_THREADPOOL_H +#define __WORKQUEUE_THREADPOOL_H + +struct threadpool; +struct task_struct; + +typedef void (*task_func_t)(int tidx, struct task_struct *task); + +struct task_struct { + task_func_t fn; +}; + +extern struct threadpool *threadpool__new(int n_threads); +extern void threadpool__delete(struct threadpool *pool); + +extern int threadpool__start(struct threadpool *pool); +extern int threadpool__stop(struct threadpool *pool); + +extern int threadpool__execute(struct threadpool *pool, struct task_struct *task); +extern int threadpool__wait(struct threadpool *pool); + +extern int threadpool__size(struct threadpool *pool); + +/* Error management */ +#define THREADPOOL_STRERR_BUFSIZE (128+STRERR_BUFSIZE) +extern int threadpool__strerror(struct threadpool *pool, int err, char *buf, size_t size); +extern int threadpool__new_strerror(struct threadpool *err_ptr, char *buf, size_t size); + +#endif /* __WORKQUEUE_THREADPOOL_H */ -- 2.31.1
next parent reply other threads:[~2021-07-30 15:37 UTC|newest] Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top [not found] <cover.1627657061.git.rickyman7@gmail.com> 2021-07-30 15:34 ` Riccardo Mancini [this message] 2021-08-07 2:24 ` Namhyung Kim 2021-08-09 10:30 ` Riccardo Mancini 2021-08-10 18:54 ` Namhyung Kim 2021-08-10 20:24 ` Arnaldo Carvalho de Melo 2021-08-11 17:55 ` Riccardo Mancini 2021-07-30 15:34 ` [RFC PATCH v2 02/10] perf tests: add test for workqueue Riccardo Mancini 2021-07-30 15:34 ` [RFC PATCH v2 03/10] perf workqueue: add threadpool start and stop functions Riccardo Mancini 2021-08-07 2:43 ` Namhyung Kim 2021-08-09 10:35 ` Riccardo Mancini 2021-07-30 15:34 ` [RFC PATCH v2 04/10] perf workqueue: add threadpool execute and wait functions Riccardo Mancini 2021-08-07 2:56 ` Namhyung Kim 2021-07-30 15:34 ` [RFC PATCH v2 05/10] tools: add sparse context/locking annotations in compiler-types.h Riccardo Mancini 2021-07-30 15:34 ` [RFC PATCH v2 06/10] perf workqueue: introduce workqueue struct Riccardo Mancini 2021-08-09 12:04 ` Jiri Olsa 2021-07-30 15:34 ` [RFC PATCH v2 07/10] perf workqueue: implement worker thread and management Riccardo Mancini 2021-07-30 15:34 ` [RFC PATCH v2 08/10] perf workqueue: add queue_work and flush_workqueue functions Riccardo Mancini 2021-07-30 15:34 ` [RFC PATCH v2 09/10] perf workqueue: add utility to execute a for loop in parallel Riccardo Mancini 2021-07-30 15:34 ` [RFC PATCH v2 10/10] perf synthetic-events: use workqueue parallel_for Riccardo Mancini 2021-08-09 12:04 ` Jiri Olsa 2021-08-09 13:24 ` Riccardo Mancini
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=4f0cd6c8e77c0b4f4d4b8d553a7032757b976e61.1627657061.git.rickyman7@gmail.com \ --to=rickyman7@gmail.com \ --cc=acme@kernel.org \ --cc=alexey.v.bayduraev@linux.intel.com \ --cc=irogers@google.com \ --cc=jolsa@redhat.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-perf-users@vger.kernel.org \ --cc=mark.rutland@arm.com \ --cc=mingo@redhat.com \ --cc=namhyung@kernel.org \ --cc=peterz@infradead.org \ --subject='Re: [RFC PATCH v2 01/10] perf workqueue: threadpool creation and destruction' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).