From patchwork Fri Mar 24 22:30:17 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Beau Belgrave X-Patchwork-Id: 13187425 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0F195C77B60 for ; Fri, 24 Mar 2023 22:30:41 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 1196B6B0071; Fri, 24 Mar 2023 18:30:41 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 0A31D6B0074; Fri, 24 Mar 2023 18:30:41 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id E855E6B0075; Fri, 24 Mar 2023 18:30:40 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id D24716B0071 for ; Fri, 24 Mar 2023 18:30:40 -0400 (EDT) Received: from smtpin03.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id A3AD7A0ACA for ; Fri, 24 Mar 2023 22:30:40 +0000 (UTC) X-FDA: 80605237440.03.7E9DF21 Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by imf11.hostedemail.com (Postfix) with ESMTP id D66A74001A for ; Fri, 24 Mar 2023 22:30:38 +0000 (UTC) Authentication-Results: imf11.hostedemail.com; dkim=pass header.d=linux.microsoft.com header.s=default header.b="A/F3EOFN"; spf=pass (imf11.hostedemail.com: domain of beaub@linux.microsoft.com designates 13.77.154.182 as permitted sender) smtp.mailfrom=beaub@linux.microsoft.com; dmarc=pass (policy=none) header.from=linux.microsoft.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1679697039; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-transfer-encoding:content-transfer-encoding: in-reply-to:references:dkim-signature; bh=Ep5nNPz4lz96sDudAV5L59mwRJdJA/ImnhxaQXHg9kI=; b=yd3JAyPa3Lu4Z8kEGEuAVXQK+QWtdg2tpvU596wXuA40F+eIJaKXe052SX930IlP+OQgsl 94p7eILXhkHudi90NfrYYGoU9T9WdaTqXOZfChymvDlihfaYXF8DytGqq8TOCwLgD0hlB/ +njDai5buFyBLYHz/Kg9fWp0QYeSsjM= ARC-Authentication-Results: i=1; imf11.hostedemail.com; dkim=pass header.d=linux.microsoft.com header.s=default header.b="A/F3EOFN"; spf=pass (imf11.hostedemail.com: domain of beaub@linux.microsoft.com designates 13.77.154.182 as permitted sender) smtp.mailfrom=beaub@linux.microsoft.com; dmarc=pass (policy=none) header.from=linux.microsoft.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1679697039; a=rsa-sha256; cv=none; b=kRG173qPmj0l5HYpYlNquIghTIONgimH1/Bjz8jl6REENB4ZvF6ItFQ7SC8mo/+5N0/Pia 8SeknMeG/iQL71WMTQgBSwJ8QCxdZqhJ8sCgRP+YsPpP3QiEioPqLxJ59qLGUitLdor/7y tialtDaG2sDM0GLLpRUgy7biLemp8a0= Received: from W11-BEAU-MD.localdomain (unknown [76.135.27.212]) by linux.microsoft.com (Postfix) with ESMTPSA id 2F6A720FC3CF; Fri, 24 Mar 2023 15:30:37 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 2F6A720FC3CF DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1679697037; bh=Ep5nNPz4lz96sDudAV5L59mwRJdJA/ImnhxaQXHg9kI=; h=From:To:Cc:Subject:Date:From; b=A/F3EOFNblU/sc5n51dqJJwUBTvDJZZafJgVO3fx2k4p5PQdDkY/mcF4QPAaRHoa3 872wTxDQkO4tWk2h4RULYaI6x1XlWpHfd8TijS9pa5tLk697JyOKWqV1n3KsGpGf4H Ra5Y488Y4HYs/GwbHyeGKEFYrD5Z5FFLlV6G3pAA= From: Beau Belgrave To: rostedt@goodmis.org, mhiramat@kernel.org, mathieu.desnoyers@efficios.com, dcook@linux.microsoft.com, alanau@linux.microsoft.com, brauner@kernel.org, akpm@linux-foundation.org, ebiederm@xmission.com, keescook@chromium.org, tglx@linutronix.de Cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-trace-kernel@vger.kernel.org Subject: [PATCH v9 00/11] tracing/user_events: Remote write ABI Date: Fri, 24 Mar 2023 15:30:17 -0700 Message-Id: <20230324223028.172-1-beaub@linux.microsoft.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-Rspam-User: X-Rspamd-Server: rspam04 X-Rspamd-Queue-Id: D66A74001A X-Stat-Signature: gygugfemh79jfixb5ydddhwff9qgyrcz X-HE-Tag: 1679697038-194121 X-HE-Meta: U2FsdGVkX18CkpHieNS7G8UXlyiyXtt7qEwHHQ7GQnItn7j55nj+8KB+58md9otJXnZ0trNSwHIv7si7ZbDhkk5Ofhaqxf61V7UQ1hztxW5IRyIyz3HgtHQ4Xy5p8G9GW20golF6MrbkJLJCBDhZfbBu8rf6xWqEvb4kkO/UkKpXQ5kHSnatD5HC1GaIdfAcckqjeQfcYIpczKNhRL7u9NWzUnTMz+xBis0jIOl2vZEEIKVDs1AD3zfU3sDefhOtyUf3k9/1Fu9JB6CStIv/FfGhZiTdeS4rxvIKPrgDUOwqsR9izavEJ040B+7KP9PwlHUb8xMR7zJLKnPS1zW/rfyMU1ptaV+9bWU2oUiLZhvF7h5piSTS7ZOS8Mhecssl7Bi6jZKgGbrvg7S8ZcZ5ilBBbofzTQkDcLNSDf9Pw9UAiua/hw6jO1xoxQdzV3ORX//C0CZeTcaMQjlq9s5Al2VzmkgiXp3Qvoj2GRhFZxNgnGcZYXCa/MMUgSK/cxR86aockpORhUgQqvw+kxFznuRAVCav7un7BLFTvfo1WFvf8I+otqZSah2c0BMqqduzDdYEf/KmjeZ546HLlALaqU/lr9yVQSFTSGknItVwB5MITsyclxXcZdOj0w9uY5Zy9dNJtvxS7DF+C/aarRrdWRCGvC1Kjbsr9opG5hI205ujlNwikhDW1YCrJGKE7RMNtOQGHSjGal9d9rctFX7OhAqzRdsHpekHlEHAZPVb7YvLfLSZZS8Vg9aeuVoV3/4MzVrcVwEgEaKznw3Hsy6FAXE2lcOu54a6CAzsJ6Y071vB8Zt4POPjm1/6Lj7w6YK0g6R7K6gmzk48w3FMKQh8WCdTuCEW1n6jC6YAuxEKG5h1DrQJBqpFzAazQsYsCW+VUsap3R+xRq3ewAJqCtFifsHlnF7CaUW1TgVGtlaH5eshCJjP71HdSAcni/A4/0ih+A4746nXec7U8eXnCCl VHLG81fE hB14ze7qAhPdpz3y2XwjX9aq/txUeOX8Bq7e9mEOBIzBnaato42E5kgNF4I5AYt9Z1R8CD8RplQYMpoM4hBPTSm858BhNMdGVJdqrtezpeJIFbOD/5ey2zheMuhkX32qyaeXm0B7QlZ86wbN1WJ/eSn+ncmeqebzrcfDrCqRxQAvAORBGe/jFTr27bMdQU9IwENK3gdg0SEMeLy1c5wMTgK1KTWgxWoaGXDY6wCy9lu86olpMVMsviIGKr/4eWVlEpDseoUIiMgiiqZRkZ72JrOuBog== X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: As part of the discussions for user_events aligned with user space tracers, it was determined that user programs should register a aligned value to set or clear a bit when an event becomes enabled. Currently a shared page is being used that requires mmap(). Remove the shared page implementation and move to a user registered address implementation. In this new model during the event registration from user programs 3 new values are specified. The first is the address to update when the event is either enabled or disabled. The second is the bit to set/clear to reflect the event being enabled. The third is the size of the value at the specified address. This allows for a local 32/64-bit value in user programs to support both kernel and user tracers. As an example, setting bit 31 for kernel tracers when the event becomes enabled allows for user tracers to use the other bits for ref counts or other flags. The kernel side updates the bit atomically, user programs need to also update these values atomically. User provided addresses must be aligned on a natural boundary, this allows for single page checking and prevents odd behaviors such as a enable value straddling 2 pages instead of a single page. When page faults are encountered they are done asyncly via a workqueue. If the page faults back in, the write update is attempted again. If the page cannot fault-in, then we log and wait until the next time the event is enabled/disabled. This is to prevent possible infinite loops resulting from bad user processes unmapping or changing protection values after registering the address. Change history V9: Updated documentation to include __packed attributed in example payload. Removed boot argument for max events, since sysctl can now be set via kernel boot parameters. V8: Rebase to 6.2-rc8. V7: Rebase to 6.2-rc4. Added flags to register ioctl, validates it's 0 for now. Future patches will enable other types of formats/options as needed. V6: Rebase to 6.2-rc2. Fixed small typos, code style. Changed from synchronize_rcu() to queue_rcu_work() to allow an rcu delay asyncly when mm is being removed and in an appropriate context for mmdrop(). V5: GFP_NOWAIT is still needed in user_event_enabler_dup(), due to rcu lock. V4: Rebase to 6.1-rc7. Moved user_events_fork() out of task signal lock and dropped use of GFP_NOWAIT. All allocations are now GFP_KERNEL or GFP_KERNEL_ACCOUNT. Added boot parameter user_events_max= to limit global events. Added sysctl value kernel.user_events_max to limit global events. Added cgroup tracking of memory allocated for events. V3: Rebase to 6.1-rc6. Removed RFC tag on series. Updated documentation to reflect ABI changes. Added self-test for ABI specific clone/fork cases. Moved user_event_mm removal into do_exit() to ensure RSS task accounting is done properly in async fault paths. Also lets us remove the delayed mmdrop(), saving memory in each user_event_mm struct. Fixed timing window where task exits, but write could be in-progress. During exit we now take mmap_write_lock to ensure we drain writes. V2: Rebase to 6.1-rc5. Added various comments based on feedback. Added enable_size to register struct, allows 32/64 bit addresses as long as the enable_bit fits and the address is naturally aligned. Changed user_event_enabler_write to accept a new flag indicating if a fault fixup should be done or not. This allows user_event_enabler_create to return back failures to the user ioctl reg call and retry to fault in data. Added tracking fork/exec/exit of tasks to have the user_event_mm lifetime tied more to the task than the file. This came with extra requirements around when you can lock, such as softirq cases, as well as a RCU pattern to ensure fork/exec/exit take minimal lock times. Changed enablers to use a single word-aligned value for saving the bit to set and any flags, such as faulting asyncly or being freed. This was required to ensure atomic bit set/test for fork cases where taking the event_mutex is not a good scalability decision. Added unregister IOCTL, since file lifetime no longer limits the enable time for any events (the mm does). Updated sample code to reflect the new remote write based ABI. Updated self-test code to reflect the new remote write based ABI. Beau Belgrave (11): tracing/user_events: Split header into uapi and kernel tracing/user_events: Track fork/exec/exit for mm lifetime tracing/user_events: Use remote writes for event enablement tracing/user_events: Fixup enable faults asyncly tracing/user_events: Add ioctl for disabling addresses tracing/user_events: Update self-tests to write ABI tracing/user_events: Add ABI self-test tracing/user_events: Use write ABI in example tracing/user_events: Update documentation for ABI tracing/user_events: Charge event allocs to cgroups tracing/user_events: Limit global user_event count Documentation/trace/user_events.rst | 179 ++-- fs/exec.c | 2 + include/linux/sched.h | 5 + include/linux/user_events.h | 101 ++- include/uapi/linux/user_events.h | 81 ++ kernel/exit.c | 2 + kernel/fork.c | 2 + kernel/trace/Kconfig | 5 +- kernel/trace/trace_events_user.c | 851 +++++++++++++++--- samples/user_events/example.c | 47 +- tools/testing/selftests/user_events/Makefile | 2 +- .../testing/selftests/user_events/abi_test.c | 226 +++++ .../testing/selftests/user_events/dyn_test.c | 2 +- .../selftests/user_events/ftrace_test.c | 162 ++-- .../testing/selftests/user_events/perf_test.c | 39 +- 15 files changed, 1306 insertions(+), 400 deletions(-) create mode 100644 include/uapi/linux/user_events.h create mode 100644 tools/testing/selftests/user_events/abi_test.c base-commit: ceaa837f96adb69c0df0397937cd74991d5d821a Acked-by: Masami Hiramatsu (Google)