From nobody Fri Jun 17 19:37:40 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 E739585AB8B; Fri, 17 Jun 2022 19:37:41 +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 4LPq7m6Q0Pz3Pwf; Fri, 17 Jun 2022 19:37:40 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1655494661; 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=Tten+7RaCDhXD7J7PuJ5nhUGIind9ZH7kx+JB5YYTck=; b=KNLF+sjgxsObxdrzgKsb+gfMs4cpk5pQjfDMkbSv9YeU5R08UvvQz2VmxebbsJMywlz5Is EWJaq30Pu6kQmQIHdFfMAOepeQlIM/cp3RsZV6SQLL5eo4pzI6A+3O/d+IUcLy+JXQq8e/ 5ipwjGkJz9/lQGGANLDbCwUJk0WZgpMQs0Q4dCBLw26/4cWU7sSK/M2XuDDSGsDGMYXTDT VZoRknF1TQMq+ZjBh5azvgVslMNKug8CAB6rRnULe3ySD6df/fBH5e0JmzUq7PsmZDsUgF fMw952N/9r2o4dR1lrTj/hSEfnXh/XAM5AwOFLPdb9f0xgH+L9fxSVFZGzrfng== 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 7E7D025C3B; Fri, 17 Jun 2022 19:37:40 +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 25HJbefH013665; Fri, 17 Jun 2022 19:37:40 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 25HJbehe013664; Fri, 17 Jun 2022 19:37:40 GMT (envelope-from git) Date: Fri, 17 Jun 2022 19:37:40 GMT Message-Id: <202206171937.25HJbehe013664@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Dmitry Chagin Subject: git: 0783310f4d94 - stable/13 - linux(4): Eliminating remnants of futex sdt. 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: dchagin X-Git-Repository: src X-Git-Refname: refs/heads/stable/13 X-Git-Reftype: branch X-Git-Commit: 0783310f4d94e6152c346f6372f57b30240e57e6 Auto-Submitted: auto-generated ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1655494661; 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=Tten+7RaCDhXD7J7PuJ5nhUGIind9ZH7kx+JB5YYTck=; b=V/NzKlboEDU9ygNaXJxoggtYzBoRJtD6qvS0Nsy6JvdjYYsYAIm+ztnOB+gRXIzvjVMfBD Bv0Y+SBWK8rtJmetMv3UdwXElIodL+e7Oox7kNOHwnKCrqIiMmP4+0sQlD1KQZk00pswPo TQWQSGcsUhORMYV4Ect/vHSuaNPlqZHjILYVcMlYLKJ2Cv6+5RzfcMuctXiXiKdEF+vD1u KEaNHnfhiXqcNVJjCiXisspZTbbbSOeYcKHQ2dBVoDJJId96aunEfbynBSfYltzzs9wWU6 R6P6NEdAXQck9PzOynLrIU1WyMZzcCU5TGJTH0kxUnqU3X/APqyzqlQoHKNdRA== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1655494661; a=rsa-sha256; cv=none; b=qul9NVWP2BepXxq+Hn/n0CeEec2Pfz9azNzc22k4lgxlLAKUBwFcWXKtIanJy7tPZjQH+S SSjMZE+0XdIVAr8GcbbK8f/wrlVmfBqwvczO5pVtZmoumvjA8v+tEF9O0lX/SxWmaHfRMl st8Qysp7bchuBvAtBgc8D94v9OpD3pZZWukVRNoVYGNnpjARnQchz/EitUonN0ZX711yXN Wx4WVh7acMh3i7Hd3Fa94VgGmpQR7aJOjig4/C8PFnsaKRW7zwhJD4EcdseZ2HTnNgHMjV 5iwzIb9xCSecO16ZbJ5XdXnszYOV2UDNmVrBZ/1hQv/2rlBMDAQCnXVpSiNzdQ== ARC-Authentication-Results: i=1; mx1.freebsd.org; none X-ThisMailContainsUnwantedMimeParts: N The branch stable/13 has been updated by dchagin: URL: https://cgit.FreeBSD.org/src/commit/?id=0783310f4d94e6152c346f6372f57b30240e57e6 commit 0783310f4d94e6152c346f6372f57b30240e57e6 Author: Dmitry Chagin AuthorDate: 2021-07-29 09:52:36 +0000 Commit: Dmitry Chagin CommitDate: 2022-06-17 19:33:22 +0000 linux(4): Eliminating remnants of futex sdt. MFC after: 2 weeks (cherry picked from commit edd44176aa0e1acdb3a97746c6fe595d12be0c45) --- sys/compat/linux/check_error.d | 25 -------------- sys/compat/linux/linux_futex.c | 76 +++--------------------------------------- 2 files changed, 5 insertions(+), 96 deletions(-) diff --git a/sys/compat/linux/check_error.d b/sys/compat/linux/check_error.d index 389e768aa3f6..8ed87a397863 100644 --- a/sys/compat/linux/check_error.d +++ b/sys/compat/linux/check_error.d @@ -39,28 +39,6 @@ linuxulator*:dummy::not_implemented, linuxulator*:emul:linux_thread_detach:child_clear_tid_error, linuxulator*:emul:linux_thread_detach:futex_failed, linuxulator*:emul:linux_schedtail:copyout_error, -linuxulator*:futex:futex_get:error, -linuxulator*:futex:futex_sleep:requeue_error, -linuxulator*:futex:futex_sleep:sleep_error, -linuxulator*:futex:futex_wait:copyin_error, -linuxulator*:futex:futex_wait:itimerfix_error, -linuxulator*:futex:futex_wait:sleep_error, -linuxulator*:futex:futex_atomic_op:missing_access_check, -linuxulator*:futex:futex_atomic_op:unimplemented_op, -linuxulator*:futex:futex_atomic_op:unimplemented_cmp, -linuxulator*:futex:linux_sys_futex:unimplemented_clockswitch, -linuxulator*:futex:linux_sys_futex:copyin_error, -linuxulator*:futex:linux_sys_futex:unhandled_efault, -linuxulator*:futex:linux_sys_futex:unimplemented_lock_pi, -linuxulator*:futex:linux_sys_futex:unimplemented_unlock_pi, -linuxulator*:futex:linux_sys_futex:unimplemented_trylock_pi, -linuxulator*:futex:linux_sys_futex:unimplemented_wait_requeue_pi, -linuxulator*:futex:linux_sys_futex:unimplemented_cmp_requeue_pi, -linuxulator*:futex:linux_sys_futex:unknown_operation, -linuxulator*:futex:linux_get_robust_list:copyout_error, -linuxulator*:futex:handle_futex_death:copyin_error, -linuxulator*:futex:fetch_robust_entry:copyin_error, -linuxulator*:futex:release_futexes:copyin_error, linuxulator*:time:linux_clock_gettime:conversion_error, linuxulator*:time:linux_clock_gettime:gettime_error, linuxulator*:time:linux_clock_gettime:copyout_error, @@ -98,9 +76,6 @@ linuxulator*:mib:linux_prison_get:vfs_setopts_error linuxulator*:util:linux_driver_get_name_dev:nullcall, linuxulator*:util:linux_driver_get_major_minor:nullcall, -linuxulator*:futex:linux_sys_futex:invalid_cmp_requeue_use, -linuxulator*:futex:linux_sys_futex:deprecated_requeue, -linuxulator*:futex:linux_set_robust_list:size_error, linuxulator*:time:linux_clock_getres:nullcall { printf("WARNING: %s:%s:%s:%s in application %s, maybe an application error?\n", probename, probeprov, probemod, probefunc, execname); diff --git a/sys/compat/linux/linux_futex.c b/sys/compat/linux/linux_futex.c index 8bc8879c92c6..510b43f11ca4 100644 --- a/sys/compat/linux/linux_futex.c +++ b/sys/compat/linux/linux_futex.c @@ -56,7 +56,6 @@ __KERNEL_RCSID(1, "$NetBSD: linux_futex.c,v 1.7 2006/07/24 19:01:49 manu Exp $") #include #include #include -#include #include #include @@ -68,40 +67,12 @@ __KERNEL_RCSID(1, "$NetBSD: linux_futex.c,v 1.7 2006/07/24 19:01:49 manu Exp $") #include #include #endif -#include #include #include #include #include #include -/* DTrace init */ -LIN_SDT_PROVIDER_DECLARE(LINUX_DTRACE); - -/** - * DTrace probes in this module. - */ -LIN_SDT_PROBE_DEFINE4(futex, futex_atomic_op, decoded_op, "int", "int", "int", - "int"); -LIN_SDT_PROBE_DEFINE1(futex, futex_atomic_op, unimplemented_op, "int"); -LIN_SDT_PROBE_DEFINE1(futex, futex_atomic_op, unimplemented_cmp, "int"); -LIN_SDT_PROBE_DEFINE3(futex, linux_futex, debug_wait, "uint32_t *", - "uint32_t", "uint32_t"); -LIN_SDT_PROBE_DEFINE3(futex, linux_futex, debug_wake, "uint32_t *", - "uint32_t", "uint32_t"); -LIN_SDT_PROBE_DEFINE5(futex, linux_futex, debug_cmp_requeue, "uint32_t *", - "uint32_t", "uint32_t", "uint32_t *", "struct l_timespec *"); -LIN_SDT_PROBE_DEFINE5(futex, linux_futex, debug_wake_op, "uint32_t *", - "int", "uint32_t", "uint32_t *", "uint32_t"); -LIN_SDT_PROBE_DEFINE0(futex, linux_futex, deprecated_requeue); -LIN_SDT_PROBE_DEFINE0(futex, linux_futex, unimplemented_wait_requeue_pi); -LIN_SDT_PROBE_DEFINE0(futex, linux_futex, unimplemented_cmp_requeue_pi); -LIN_SDT_PROBE_DEFINE1(futex, linux_futex, unknown_operation, "int"); -LIN_SDT_PROBE_DEFINE0(futex, linux_set_robust_list, size_error); -LIN_SDT_PROBE_DEFINE1(futex, linux_get_robust_list, copyout_error, "int"); -LIN_SDT_PROBE_DEFINE1(futex, fetch_robust_entry, copyin_error, "int"); -LIN_SDT_PROBE_DEFINE1(futex, release_futexes, copyin_error, "int"); - #define FUTEX_SHARED 0x8 /* shared futex */ #define GET_SHARED(a) (a->flags & FUTEX_SHARED) ? AUTO_SHARE : THREAD_SHARE @@ -188,9 +159,6 @@ futex_atomic_op(struct thread *td, int encoded_op, uint32_t *uaddr) if (encoded_op & (FUTEX_OP_OPARG_SHIFT << 28)) oparg = 1 << oparg; - LIN_SDT_PROBE4(futex, futex_atomic_op, decoded_op, op, cmp, oparg, - cmparg); - switch (op) { case FUTEX_OP_SET: ret = futex_xchgl(oparg, uaddr, &oldval); @@ -208,7 +176,6 @@ futex_atomic_op(struct thread *td, int encoded_op, uint32_t *uaddr) ret = futex_xorl(oparg, uaddr, &oldval); break; default: - LIN_SDT_PROBE1(futex, futex_atomic_op, unimplemented_op, op); ret = -ENOSYS; break; } @@ -236,7 +203,6 @@ futex_atomic_op(struct thread *td, int encoded_op, uint32_t *uaddr) ret = (oldval > cmparg); break; default: - LIN_SDT_PROBE1(futex, futex_atomic_op, unimplemented_cmp, cmp); ret = -ENOSYS; } @@ -264,8 +230,6 @@ linux_futex(struct thread *td, struct linux_futex_args *args) /* FALLTHROUGH */ case LINUX_FUTEX_WAIT_BITSET: - LIN_SDT_PROBE3(futex, linux_futex, debug_wait, args->uaddr, - args->val, args->val3); LINUX_CTR3(sys_futex, "WAIT uaddr %p val 0x%x bitset 0x%x", args->uaddr, args->val, args->val3); @@ -276,8 +240,6 @@ linux_futex(struct thread *td, struct linux_futex_args *args) /* FALLTHROUGH */ case LINUX_FUTEX_WAKE_BITSET: - LIN_SDT_PROBE3(futex, linux_futex, debug_wake, args->uaddr, - args->val, args->val3); LINUX_CTR3(sys_futex, "WAKE uaddr %p nrwake 0x%x bitset 0x%x", args->uaddr, args->val, args->val3); @@ -294,8 +256,6 @@ linux_futex(struct thread *td, struct linux_futex_args *args) if ((pem->flags & LINUX_XDEPR_REQUEUEOP) == 0) { linux_msg(td, "unsupported FUTEX_REQUEUE"); pem->flags |= LINUX_XDEPR_REQUEUEOP; - LIN_SDT_PROBE0(futex, linux_futex, - deprecated_requeue); } /* @@ -311,9 +271,6 @@ linux_futex(struct thread *td, struct linux_futex_args *args) /* FALLTHROUGH */ case LINUX_FUTEX_CMP_REQUEUE: - LIN_SDT_PROBE5(futex, linux_futex, debug_cmp_requeue, - args->uaddr, args->val, args->val3, args->uaddr2, - args->ts); LINUX_CTR5(sys_futex, "CMP_REQUEUE uaddr %p " "nrwake 0x%x uval 0x%x uaddr2 %p nrequeue 0x%x", args->uaddr, args->val, args->val3, args->uaddr2, @@ -322,8 +279,6 @@ linux_futex(struct thread *td, struct linux_futex_args *args) return (linux_futex_requeue(td, args)); case LINUX_FUTEX_WAKE_OP: - LIN_SDT_PROBE5(futex, linux_futex, debug_wake_op, - args->uaddr, args->op, args->val, args->uaddr2, args->val3); LINUX_CTR5(sys_futex, "WAKE_OP " "uaddr %p nrwake 0x%x uaddr2 %p op 0x%x nrwake2 0x%x", args->uaddr, args->val, args->uaddr2, args->val3, @@ -356,8 +311,6 @@ linux_futex(struct thread *td, struct linux_futex_args *args) if ((pem->flags & LINUX_XUNSUP_FUTEXPIOP) == 0) { linux_msg(td, "unsupported FUTEX_WAIT_REQUEUE_PI"); pem->flags |= LINUX_XUNSUP_FUTEXPIOP; - LIN_SDT_PROBE0(futex, linux_futex, - unimplemented_wait_requeue_pi); } return (ENOSYS); @@ -367,15 +320,11 @@ linux_futex(struct thread *td, struct linux_futex_args *args) if ((pem->flags & LINUX_XUNSUP_FUTEXPIOP) == 0) { linux_msg(td, "unsupported FUTEX_CMP_REQUEUE_PI"); pem->flags |= LINUX_XUNSUP_FUTEXPIOP; - LIN_SDT_PROBE0(futex, linux_futex, - unimplemented_cmp_requeue_pi); } return (ENOSYS); default: linux_msg(td, "unsupported futex op %d", args->op); - LIN_SDT_PROBE1(futex, linux_futex, unknown_operation, - args->op); return (ENOSYS); } } @@ -930,10 +879,8 @@ linux_set_robust_list(struct thread *td, struct linux_set_robust_list_args *args { struct linux_emuldata *em; - if (args->len != sizeof(struct linux_robust_list_head)) { - LIN_SDT_PROBE0(futex, linux_set_robust_list, size_error); + if (args->len != sizeof(struct linux_robust_list_head)) return (EINVAL); - } em = em_find(td); em->robust_futexes = args->head; @@ -978,19 +925,10 @@ linux_get_robust_list(struct thread *td, struct linux_get_robust_list_args *args } error = copyout(&len, args->len, sizeof(l_size_t)); - if (error) { - LIN_SDT_PROBE1(futex, linux_get_robust_list, copyout_error, - error); + if (error) return (EFAULT); - } - - error = copyout(&head, args->head, sizeof(head)); - if (error) { - LIN_SDT_PROBE1(futex, linux_get_robust_list, copyout_error, - error); - } - return (error); + return (copyout(&head, args->head, sizeof(head))); } static int @@ -1063,10 +1001,8 @@ fetch_robust_entry(struct linux_robust_list **entry, int error; error = copyin((const void *)head, &uentry, sizeof(l_ulong)); - if (error) { - LIN_SDT_PROBE1(futex, fetch_robust_entry, copyin_error, error); + if (error) return (EFAULT); - } *entry = (void *)(uentry & ~1UL); *pi = uentry & 1; @@ -1098,10 +1034,8 @@ release_futexes(struct thread *td, struct linux_emuldata *em) error = copyin(&head->futex_offset, &futex_offset, sizeof(futex_offset)); - if (error) { - LIN_SDT_PROBE1(futex, release_futexes, copyin_error, error); + if (error) return; - } if (fetch_robust_entry(&pending, PTRIN(&head->pending_list), &pip)) return;