From patchwork Fri Aug 11 21:50:45 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Zhangjin Wu X-Patchwork-Id: 134727 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b824:0:b0:3f2:4152:657d with SMTP id z4csp1376285vqi; Fri, 11 Aug 2023 14:54:41 -0700 (PDT) X-Google-Smtp-Source: AGHT+IEXqLoHJ+hSu9BjKk8dkGty7zzeepgIN83rlpsF576St7Z6nlNaxOddBNIin8lhHBiFQWol X-Received: by 2002:a17:902:ec8d:b0:1b8:c1f:fcfc with SMTP id x13-20020a170902ec8d00b001b80c1ffcfcmr3089933plg.11.1691790881400; Fri, 11 Aug 2023 14:54:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691790881; cv=none; d=google.com; s=arc-20160816; b=PYLrNKB+500IwlNwMyfwlwgPCsrMumueZiOiFi8phF/W63mGBDGaLakuGLIdn4CplE uTfXPq4q1tZG56KXSIN26T9zcQXgEnv0KZSMnOSNZjmMXiUqSoCJadH0GnNsNqROK4mV x/3wfjMll5aCgr2NHv9nYHhU34riHLR+M76fVEimoN+mpEzdzpFwj2dW+d6Df/OWMzWU ExOXN6u076wKxuZhqaSvCWzKuUKnTdHO2Z1za8rFUwXHWLwrSlVtp0Zk4fNoq/Gj+lBo b+TbsiRmOu/cX9KbrEIiGIkhflCEMhFto2Ltt28QxgaIFKTgX5hOL3NmdxlrK//4SMha nuHA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:feedback-id:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from; bh=/Y1xos753N/BdPaeDDuyyrJmK6HalOaWtKnRU2U/rLU=; fh=2/wD3Wxskfa9AcaWsJc6DzZ67+LVSLroI/gZJKhbDmM=; b=n5o/lBn6a2NYVyOGzzXscSehltQVfvNoc8BzxCefUJC/3wbjGMWvaTN+gXX/jAAVg5 Ac4HsDD0XqywKjE4ZniedidMlR3o2smQcrFXqLIygleWqkX07qpSVUzI1thpNhteNn0T /piGeJeQNnwXv/L2zLCP5Kyh2waYvezLUM7ybsHox2osfQRJgRYOdJuD1JSpsrAgvEew g2/oh/rqCOAp3Wihmsr3gTfbcr/cwA8go7EbSHPHVDj8OcavISGeMRRJGpLs92gXXDGi EhPwGbAOk7EFPwwlCTqCDVQz0xoRMD7wuPb45K//8DlZyvOf6ynkzVVgBvAneR9uOkVN 1/aw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id y24-20020a170902b49800b001b89551a392si3800098plr.113.2023.08.11.14.54.27; Fri, 11 Aug 2023 14:54:41 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237084AbjHKVvF (ORCPT + 99 others); Fri, 11 Aug 2023 17:51:05 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45128 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237070AbjHKVvE (ORCPT ); Fri, 11 Aug 2023 17:51:04 -0400 Received: from bg4.exmail.qq.com (bg4.exmail.qq.com [43.154.54.12]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 964DD2712; Fri, 11 Aug 2023 14:51:03 -0700 (PDT) X-QQ-mid: bizesmtp63t1691790647tk467uon Received: from linux-lab-host.localdomain ( [116.30.128.116]) by bizesmtp.qq.com (ESMTP) with id ; Sat, 12 Aug 2023 05:50:45 +0800 (CST) X-QQ-SSF: 01200000000000E0X000000A0000000 X-QQ-FEAT: cknW069rtVdcYll5qSTwzsTfqXrvv75L0qlCqnV/4/VuNNA7AGW2lTfMec+rQ gEw8mbCx55sIMZILg5KY6QqKA+LhLlKg5bADNzSAvAlfol9VlvGPKaKhHJ/bXqcbpBQY9gV C0DVP4IGf98tFitIlQSyk5om2TtXNcx+SX7xsxiCPx7YbfwTS64JfQSNKLwptd5ADVtfSyf I5AdAfTL6bbgRExXfcQMn7T9lmasM9eDj+0hWlw5iNtV8lDfnf2oti4bBDVfPw6G52Dnyto jTDMKgGG4xzfJdBamFQYhuWsLwhYy4TPPJB72surXE19oa0ZzXscT77AHrYFWUEZLX8ySFy 7taB/RHQeSTab8kW2cW9spWFSz7nxNBDKqV9hqMf48knKOe2WazBXHc7rPhhA== X-QQ-GoodBg: 0 X-BIZMAIL-ID: 10867541119786147020 From: Zhangjin Wu To: w@1wt.eu Cc: falcon@tinylab.org, david.laight@aculab.com, arnd@arndb.de, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, tanyuan@tinylab.org, thomas@t-8ch.de Subject: [PATCH v6 1/2] tools/nolibc: let sys_brk, sys_mmap and sys_mmap2 return long Date: Sat, 12 Aug 2023 05:50:45 +0800 Message-Id: <82b584cbda5cee8d5318986644a2a64ba749a098.1691788036.git.falcon@tinylab.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: References: MIME-Version: 1.0 X-QQ-SENDSIZE: 520 Feedback-ID: bizesmtp:tinylab.org:qybglogicsvrgz:qybglogicsvrgz5a-1 X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RCVD_IN_MSPIKE_H2, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773971315151416644 X-GMAIL-MSGID: 1773971315151416644 Firstly, since the sys_* functions are internally used by our library routines, it is ok to let them preserve the 'long' return type of my_syscall macros, that means not necessary to return pointer like their library routines do. Secondly, in order to avoid the size inflating issues introduced by the sign extension, it is better to let __sysret() only accept integer input types, to do so, we must let all of the sys_* functions not return pointers. There are only three sys_* functions which return pointer, let's make them return 'long' instead of pointer. Link: https://lore.kernel.org/lkml/20230809221743.83107-1-falcon@tinylab.org/ Signed-off-by: Zhangjin Wu --- tools/include/nolibc/arch-s390.h | 4 ++-- tools/include/nolibc/sys.h | 16 ++++++++-------- 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/tools/include/nolibc/arch-s390.h b/tools/include/nolibc/arch-s390.h index 5d60fd43f883..6396c2a6bc3a 100644 --- a/tools/include/nolibc/arch-s390.h +++ b/tools/include/nolibc/arch-s390.h @@ -160,7 +160,7 @@ struct s390_mmap_arg_struct { }; static __attribute__((unused)) -void *sys_mmap(void *addr, size_t length, int prot, int flags, int fd, +long sys_mmap(void *addr, size_t length, int prot, int flags, int fd, off_t offset) { struct s390_mmap_arg_struct args = { @@ -172,7 +172,7 @@ void *sys_mmap(void *addr, size_t length, int prot, int flags, int fd, .offset = (unsigned long)offset }; - return (void *)my_syscall1(__NR_mmap, &args); + return my_syscall1(__NR_mmap, &args); } #define sys_mmap sys_mmap diff --git a/tools/include/nolibc/sys.h b/tools/include/nolibc/sys.h index 833d6c5e86dc..a28e7fbff448 100644 --- a/tools/include/nolibc/sys.h +++ b/tools/include/nolibc/sys.h @@ -74,9 +74,9 @@ long __sysret(unsigned long ret) */ static __attribute__((unused)) -void *sys_brk(void *addr) +long sys_brk(void *addr) { - return (void *)my_syscall1(__NR_brk, addr); + return my_syscall1(__NR_brk, addr); } static __attribute__((unused)) @@ -89,12 +89,12 @@ static __attribute__((unused)) void *sbrk(intptr_t inc) { /* first call to find current end */ - void *ret = sys_brk(0); + void *ret = (void *)sys_brk(0); - if (ret && sys_brk(ret + inc) == ret + inc) + if (ret && (void *)sys_brk(ret + inc) == ret + inc) return ret + inc; - return (void *)__sysret(-ENOMEM); + return (void *)__sysret((long)-ENOMEM); } @@ -658,7 +658,7 @@ int mknod(const char *path, mode_t mode, dev_t dev) #ifndef sys_mmap static __attribute__((unused)) -void *sys_mmap(void *addr, size_t length, int prot, int flags, int fd, +long sys_mmap(void *addr, size_t length, int prot, int flags, int fd, off_t offset) { int n; @@ -670,7 +670,7 @@ void *sys_mmap(void *addr, size_t length, int prot, int flags, int fd, n = __NR_mmap; #endif - return (void *)my_syscall6(n, addr, length, prot, flags, fd, offset); + return my_syscall6(n, addr, length, prot, flags, fd, offset); } #endif @@ -682,7 +682,7 @@ void *sys_mmap(void *addr, size_t length, int prot, int flags, int fd, static __attribute__((unused)) void *mmap(void *addr, size_t length, int prot, int flags, int fd, off_t offset) { - return (void *)__sysret((unsigned long)sys_mmap(addr, length, prot, flags, fd, offset)); + return (void *)__sysret(sys_mmap(addr, length, prot, flags, fd, offset)); } static __attribute__((unused)) From patchwork Fri Aug 11 21:51:53 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Zhangjin Wu X-Patchwork-Id: 134749 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b824:0:b0:3f2:4152:657d with SMTP id z4csp1390348vqi; Fri, 11 Aug 2023 15:25:26 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGHILmHMZMeTFm3ZQxoTXBOZJR/cVyU0e6QrBebkp2Nl1nj9db4vOV3wxCKZeEKZv1wrLRE X-Received: by 2002:a05:6a20:5482:b0:134:73f6:5832 with SMTP id i2-20020a056a20548200b0013473f65832mr4983116pzk.16.1691792726554; Fri, 11 Aug 2023 15:25:26 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1691792726; cv=none; d=google.com; s=arc-20160816; b=VgzSnNYnMSOxOU4uccCyZODdFkpuusG9/9XXOjB4gXXmSRE7oMsqu6x8W5eOhpLADJ 6+0/iwqTfC4ZNGwoJRPPFreSmubnXynVHjmNKhoTKH3xuZh2WKVPPA9ZbdtA0QyOtizM feQn1ZinH1IaKkYEgv+c0ZuNedS4zsQABO3/SnisUjlA/dkr7CQb7PuCtx0RiFjMMUb9 JCD9XhLJpXeW5xMPrgkJbf4OUdHYZiTp+hNPkx80Hgfq7tcLm7AgeD7ckEPUyFlzafEX iOAYURkChzlNRg8+19vnU3/cN65tLzPq4pjbHuXOk31XnagXIxDm8a4GlhNOhFa2YeTL JViQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:feedback-id:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from; bh=9RSN2EY2cJTW9zDsZRZuyf2aqRmAybo3jg1zS7CH0eo=; fh=2/wD3Wxskfa9AcaWsJc6DzZ67+LVSLroI/gZJKhbDmM=; b=G4tQxIWvHBzncEENhGh0/67HmFZ3379uhgKS6kIqFDfq/lf52c7o64Wg5tjdIa8LxQ 5B2YLHVKyWeVA+nGIR0+py1RUa94lCMz5pRTgnvyvfDdkoS1mtmd1gw+Kr7Z/zfV0tlZ x5LmOIo5raTU6iesQ+WIlbgot4fuC1640hOmFyqIviUOseXSI6wjQWs18YCdITrNNp2g L32uwPE2i/ooMowAAAGU+hM8xUBAo98E2mFUJD5ntZKatQN1jEF1/DRi1uHCAH69BbhO Z0BZL5sYz+gq6+JDLe0+pQOLCOiV33bS+Ni8IziKXR748RrdgUPGinBoTmm97xjXFsO9 9Ttg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id p28-20020a056a0026dc00b006871e641c8dsi3986077pfw.12.2023.08.11.15.25.12; Fri, 11 Aug 2023 15:25:26 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237000AbjHKVwP (ORCPT + 99 others); Fri, 11 Aug 2023 17:52:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42298 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235401AbjHKVwO (ORCPT ); Fri, 11 Aug 2023 17:52:14 -0400 Received: from bg4.exmail.qq.com (bg4.exmail.qq.com [43.155.65.254]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1E957271B; Fri, 11 Aug 2023 14:52:10 -0700 (PDT) X-QQ-mid: bizesmtp69t1691790715t2l3imm8 Received: from linux-lab-host.localdomain ( [116.30.128.116]) by bizesmtp.qq.com (ESMTP) with id ; Sat, 12 Aug 2023 05:51:53 +0800 (CST) X-QQ-SSF: 01200000000000E0X000000A0000000 X-QQ-FEAT: jGGC4gWX7WFEuWrQD10QYevA6rKFR5eyLN27zADU7mMJyUa1X6F5wj5b8c1PO ER8oXLqoaZcj68vp2XW1IvvzWMXYjLid4rTDbWxcHY9+xK0LZ6TboYz9xrN1UuKrfZPeivH Xhx+frlWCSJTPIl4qTvNiv+IIPg62gxZBtdjgSzG1OWsfAsHQZwMgx7V1TgW3uPw9evoaY9 f/3qSx8aKzolxDdGprJ9lCbNnqC8BdKQhSeoDf1ZVL0vz1omtIq9sC7KeFcpXHcLENpcFN+ ZEMfFnvLJZED4528TgM/+SpuC2OxOWzKeNxwVnytS785YBa97JXf5RSeKCPgFQkYpErIUmN JyXdyk2gkVM2i9Ad6ukNE5KRQK4tVquu5jqDZvRRUNb6m1LNpMtGj4GfqQLSUQA0SHWLMTn X-QQ-GoodBg: 0 X-BIZMAIL-ID: 14313426769615256479 From: Zhangjin Wu To: w@1wt.eu Cc: falcon@tinylab.org, david.laight@aculab.com, arnd@arndb.de, linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org, tanyuan@tinylab.org, thomas@t-8ch.de Subject: [PATCH v6 2/2] tools/nolibc: fix up size inflate regression Date: Sat, 12 Aug 2023 05:51:53 +0800 Message-Id: <96624cc918092737d35dd539d184de06dba7a9b8.1691788036.git.falcon@tinylab.org> X-Mailer: git-send-email 2.25.1 In-Reply-To: References: MIME-Version: 1.0 X-QQ-SENDSIZE: 520 Feedback-ID: bizesmtp:tinylab.org:qybglogicsvrgz:qybglogicsvrgz5a-1 X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1773973250235168189 X-GMAIL-MSGID: 1773973250235168189 As reported and suggested by Willy, the inline __sysret() helper introduces three types of conversions and increases the size: (1) the "unsigned long" argument to __sysret() forces a sign extension from all sys_* functions that used to return 'int' (2) the comparison with the error range now has to be performed on a 'unsigned long' instead of an 'int' (3) the return value from __sysret() is a 'long' (note, a signed long) which then has to be turned back to an 'int' before being returned by the caller to satisfy the caller's prototype. To fix up this, firstly, let's use macro instead of inline function to preserves the input type and avoids these useless conversions (1), (3). Secondly, since all of the sys_* functions have been converted to return integer, now, it is able to remove comparison to a 'unsigned long' -MAX_ERRNO (2) and restore the simple sign comparison as before. Suggested-by: Willy Tarreau Link: https://lore.kernel.org/lkml/20230806095846.GB10627@1wt.eu/ Signed-off-by: Zhangjin Wu --- tools/include/nolibc/sys.h | 27 ++++++++++----------------- 1 file changed, 10 insertions(+), 17 deletions(-) diff --git a/tools/include/nolibc/sys.h b/tools/include/nolibc/sys.h index a28e7fbff448..e0b68d3532b6 100644 --- a/tools/include/nolibc/sys.h +++ b/tools/include/nolibc/sys.h @@ -27,23 +27,16 @@ #include "errno.h" #include "types.h" - -/* Syscall return helper for library routines, set errno as -ret when ret is in - * range of [-MAX_ERRNO, -1] - * - * Note, No official reference states the errno range here aligns with musl - * (src/internal/syscall_ret.c) and glibc (sysdeps/unix/sysv/linux/sysdep.h) - */ - -static __inline__ __attribute__((unused, always_inline)) -long __sysret(unsigned long ret) -{ - if (ret >= (unsigned long)-MAX_ERRNO) { - SET_ERRNO(-(long)ret); - return -1; - } - return ret; -} +/* Syscall return helper, set errno as -ret when ret < 0 */ +#define __sysret(arg) \ +({ \ + __typeof__(arg) __ret = (arg); \ + if (__ret < 0) { \ + SET_ERRNO(-__ret); \ + __ret = -1L; \ + } \ + __ret; \ +}) /* Functions in this file only describe syscalls. They're declared static so * that the compiler usually decides to inline them while still being allowed