From patchwork Tue May 28 11:48:10 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alejandro Colomar X-Patchwork-Id: 91014 Return-Path: X-Original-To: patchwork@sourceware.org Delivered-To: patchwork@sourceware.org Received: from server2.sourceware.org (localhost [IPv6:::1]) by sourceware.org (Postfix) with ESMTP id 98504385E458 for ; Tue, 28 May 2024 11:49:15 +0000 (GMT) X-Original-To: libc-alpha@sourceware.org Delivered-To: libc-alpha@sourceware.org Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by sourceware.org (Postfix) with ESMTPS id F088C385EC59 for ; Tue, 28 May 2024 11:48:13 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org F088C385EC59 Authentication-Results: sourceware.org; dmarc=pass (p=none dis=none) header.from=kernel.org Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=kernel.org ARC-Filter: OpenARC Filter v1.0.0 sourceware.org F088C385EC59 Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=139.178.84.217 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1716896896; cv=none; b=ClN293t75r5Y6D7/FGHKXhNa22/5Hc7xKC/W6p68RfoQiFwdTB1JgmoerPepqgvxUDhP7Pl+f0pgJzP0yDxFhvIQhu45vk7SbWU2T7BVfD95Jn4zMoYyiF8TT1hSpT6Edz48j6KEhl2ygVOY5U8tdK6J1Nc7Ze/0g0v63UO0xSE= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1716896896; c=relaxed/simple; bh=XeJxIi6a14rjilHcl8l3kQ5okk7lmMem0yEff2Bo8mg=; h=DKIM-Signature:Date:From:To:Subject:Message-ID:MIME-Version; b=kmfpqILZsLn6m5XHD5fKgSVF8jY9mVNIcYQzPuIlSGcO3XNaYxxkSlWcv6SVYqDjJil9SFd4xRoHVvyhg4bG8q2B22QC1eyBbxwpkh70C6MvuBpvKo1+VVB4eCr17puo3Qqv9vaRkE9iGzSIyZktdyIiPMvcdQpngRK3jNKr00M= ARC-Authentication-Results: i=1; server2.sourceware.org Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 8C40A6202B for ; Tue, 28 May 2024 11:48:13 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 39872C3277B; Tue, 28 May 2024 11:48:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1716896893; bh=XeJxIi6a14rjilHcl8l3kQ5okk7lmMem0yEff2Bo8mg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=nG44VyNIHnIx++HWkJExvRPBLAlMzM4Z7uZRrF2a3yBn7+MXwMm8U2u85VSB2J1SJ w2vVen2+NivkzUQ67RYEwdnw28Cizh8IFwlfMs62YHsM0HNF8UnPF3lrL3kW9lsm99 82IcvBePDFuHb6f67FPbVQ9LFaoeTHk9rzo1FXUiRq6XXGYVJgBoPoMCG4tJ3HDYfi GzmZWfrcwKA9V72kRGvtD09rPjO0AT8mkadlrXpBH4+FVnfEbeHoqUqjs5wlSM2IEf SgKbBCsQZUvJTvoo6PibWJOVVtZ5d42Oe4/3tUwCPWIKxNAK52beln3ZQa3XVVCnje d2dxdSLsotFoA== Date: Tue, 28 May 2024 13:48:10 +0200 From: Alejandro Colomar To: linux-api@vger.kernel.org Cc: linux-man@vger.kernel.org, libc-alpha@sourceware.org, Alejandro Colomar Subject: [PATCH v1 2/2] uapi/linux/prctl: Use the UL integer suffix for bit fields of width long Message-ID: <20240528114750.106187-3-alx@kernel.org> X-Mailer: git-send-email 2.45.1 References: <20240528114750.106187-1-alx@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20240528114750.106187-1-alx@kernel.org> X-Spam-Status: No, score=-10.0 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, GIT_PATCH_0, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org X-BeenThere: libc-alpha@sourceware.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Libc-alpha mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: libc-alpha-bounces+patchwork=sourceware.org@sourceware.org The prctl(2) wrapper provided by glibc uses a variadic argument list. This means that the arguments *must* have the right type (and more importantly, the right width). To avoid the user having to cast these constants, provide them with the appropriate width, that of a long. These are bit fields, and bitwise operations are dangerous on signed integers, so let's make sure they are unsigned. Link: Cc: Cc: Cc: Signed-off-by: Alejandro Colomar --- include/uapi/linux/prctl.h | 60 +++++++++++++++++++------------------- 1 file changed, 30 insertions(+), 30 deletions(-) diff --git a/include/uapi/linux/prctl.h b/include/uapi/linux/prctl.h index 60e14adb8d20..c016c316f3c5 100644 --- a/include/uapi/linux/prctl.h +++ b/include/uapi/linux/prctl.h @@ -33,12 +33,12 @@ /* Get/set floating-point exception mode (if meaningful) */ #define PR_GET_FPEXC 11 #define PR_SET_FPEXC 12 -# define PR_FP_EXC_SW_ENABLE 0x80 /* Use FPEXC for FP exception enables */ -# define PR_FP_EXC_DIV 0x010000 /* floating point divide by zero */ -# define PR_FP_EXC_OVF 0x020000 /* floating point overflow */ -# define PR_FP_EXC_UND 0x040000 /* floating point underflow */ -# define PR_FP_EXC_RES 0x080000 /* floating point inexact result */ -# define PR_FP_EXC_INV 0x100000 /* floating point invalid operation */ +# define PR_FP_EXC_SW_ENABLE 0x80UL /* Use FPEXC for FP exception enables */ +# define PR_FP_EXC_DIV 0x010000UL /* floating point divide by zero */ +# define PR_FP_EXC_OVF 0x020000UL /* floating point overflow */ +# define PR_FP_EXC_UND 0x040000UL /* floating point underflow */ +# define PR_FP_EXC_RES 0x080000UL /* floating point inexact result */ +# define PR_FP_EXC_INV 0x100000UL /* floating point invalid operation */ # define PR_FP_EXC_DISABLED 0L /* FP exceptions disabled */ # define PR_FP_EXC_NONRECOV 1L /* async non-recoverable exc. mode */ # define PR_FP_EXC_ASYNC 2L /* async recoverable exception mode */ @@ -188,8 +188,8 @@ struct prctl_mm_map { #define PR_SET_FP_MODE 45 #define PR_GET_FP_MODE 46 -# define PR_FP_MODE_FR (1 << 0) /* 64b FP registers */ -# define PR_FP_MODE_FRE (1 << 1) /* 32b compatibility */ +# define PR_FP_MODE_FR (1UL << 0) /* 64b FP registers */ +# define PR_FP_MODE_FRE (1UL << 1) /* 32b compatibility */ /* Control the ambient capability set */ #define PR_CAP_AMBIENT 47 @@ -201,11 +201,11 @@ struct prctl_mm_map { /* arm64 Scalable Vector Extension controls */ /* Flag values must be kept in sync with ptrace NT_ARM_SVE interface */ #define PR_SVE_SET_VL 50 /* set task vector length */ -# define PR_SVE_SET_VL_ONEXEC (1 << 18) /* defer effect until exec */ +# define PR_SVE_SET_VL_ONEXEC (1UL << 18) /* defer effect until exec */ #define PR_SVE_GET_VL 51 /* get task vector length */ /* Bits common to PR_SVE_SET_VL and PR_SVE_GET_VL */ -# define PR_SVE_VL_LEN_MASK 0xffff -# define PR_SVE_VL_INHERIT (1 << 17) /* inherit across exec */ +# define PR_SVE_VL_LEN_MASK 0xffffUL +# define PR_SVE_VL_INHERIT (1UL << 17) /* inherit across exec */ /* Per task speculation control */ #define PR_GET_SPECULATION_CTRL 52 @@ -215,7 +215,7 @@ struct prctl_mm_map { # define PR_SPEC_INDIRECT_BRANCH 1L # define PR_SPEC_L1D_FLUSH 2L /* Return and control values for PR_SET/GET_SPECULATION_CTRL */ -# define PR_SPEC_NOT_AFFECTED 0 +# define PR_SPEC_NOT_AFFECTED 0UL # define PR_SPEC_PRCTL (1UL << 0) # define PR_SPEC_ENABLE (1UL << 1) # define PR_SPEC_DISABLE (1UL << 2) @@ -240,10 +240,10 @@ struct prctl_mm_map { # define PR_MTE_TCF_ASYNC (1UL << 2) # define PR_MTE_TCF_MASK (PR_MTE_TCF_SYNC | PR_MTE_TCF_ASYNC) /* MTE tag inclusion mask */ -# define PR_MTE_TAG_SHIFT 3 +# define PR_MTE_TAG_SHIFT 3UL # define PR_MTE_TAG_MASK (0xffffUL << PR_MTE_TAG_SHIFT) /* Unused; kept only for source compatibility */ -# define PR_MTE_TCF_SHIFT 1 +# define PR_MTE_TCF_SHIFT 1UL /* Control reclaim behavior when allocating memory */ #define PR_SET_IO_FLUSHER 57 @@ -275,11 +275,11 @@ struct prctl_mm_map { /* arm64 Scalable Matrix Extension controls */ /* Flag values must be in sync with SVE versions */ #define PR_SME_SET_VL 63 /* set task vector length */ -# define PR_SME_SET_VL_ONEXEC (1 << 18) /* defer effect until exec */ +# define PR_SME_SET_VL_ONEXEC (1UL << 18) /* defer effect until exec */ #define PR_SME_GET_VL 64 /* get task vector length */ /* Bits common to PR_SME_SET_VL and PR_SME_GET_VL */ -# define PR_SME_VL_LEN_MASK 0xffff -# define PR_SME_VL_INHERIT (1 << 17) /* inherit across exec */ +# define PR_SME_VL_LEN_MASK 0xffffUL +# define PR_SME_VL_INHERIT (1UL << 17) /* inherit across exec */ /* Memory deny write / execute */ #define PR_SET_MDWE 65 @@ -298,13 +298,13 @@ struct prctl_mm_map { #define PR_RISCV_V_SET_CONTROL 69 #define PR_RISCV_V_GET_CONTROL 70 -# define PR_RISCV_V_VSTATE_CTRL_DEFAULT 0 -# define PR_RISCV_V_VSTATE_CTRL_OFF 1 -# define PR_RISCV_V_VSTATE_CTRL_ON 2 -# define PR_RISCV_V_VSTATE_CTRL_INHERIT (1 << 4) -# define PR_RISCV_V_VSTATE_CTRL_CUR_MASK 0x3 -# define PR_RISCV_V_VSTATE_CTRL_NEXT_MASK 0xc -# define PR_RISCV_V_VSTATE_CTRL_MASK 0x1f +# define PR_RISCV_V_VSTATE_CTRL_DEFAULT 0UL +# define PR_RISCV_V_VSTATE_CTRL_OFF 1UL +# define PR_RISCV_V_VSTATE_CTRL_ON 2UL +# define PR_RISCV_V_VSTATE_CTRL_INHERIT (1UL << 4) +# define PR_RISCV_V_VSTATE_CTRL_CUR_MASK 0x3UL +# define PR_RISCV_V_VSTATE_CTRL_NEXT_MASK 0xcUL +# define PR_RISCV_V_VSTATE_CTRL_MASK 0x1fUL #define PR_RISCV_SET_ICACHE_FLUSH_CTX 71 # define PR_RISCV_CTX_SW_FENCEI_ON 0L @@ -321,11 +321,11 @@ struct prctl_mm_map { # define PR_PPC_DEXCR_SRAPD 2L /* Subroutine return address prediction disable */ # define PR_PPC_DEXCR_NPHIE 3L /* Non-privileged hash instruction enable */ /* Action to apply / return */ -# define PR_PPC_DEXCR_CTRL_EDITABLE 0x1 /* Aspect can be modified with PR_PPC_SET_DEXCR */ -# define PR_PPC_DEXCR_CTRL_SET 0x2 /* Set the aspect for this process */ -# define PR_PPC_DEXCR_CTRL_CLEAR 0x4 /* Clear the aspect for this process */ -# define PR_PPC_DEXCR_CTRL_SET_ONEXEC 0x8 /* Set the aspect on exec */ -# define PR_PPC_DEXCR_CTRL_CLEAR_ONEXEC 0x10 /* Clear the aspect on exec */ -# define PR_PPC_DEXCR_CTRL_MASK 0x1f +# define PR_PPC_DEXCR_CTRL_EDITABLE 0x1UL /* Aspect can be modified with PR_PPC_SET_DEXCR */ +# define PR_PPC_DEXCR_CTRL_SET 0x2UL /* Set the aspect for this process */ +# define PR_PPC_DEXCR_CTRL_CLEAR 0x4UL /* Clear the aspect for this process */ +# define PR_PPC_DEXCR_CTRL_SET_ONEXEC 0x8UL /* Set the aspect on exec */ +# define PR_PPC_DEXCR_CTRL_CLEAR_ONEXEC 0x10UL /* Clear the aspect on exec */ +# define PR_PPC_DEXCR_CTRL_MASK 0x1fUL #endif /* _LINUX_PRCTL_H */