From mboxrd@z Thu Jan 1 00:00:00 1970 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C37E0C43217 for ; Tue, 8 Feb 2022 01:06:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233604AbiBHBGW (ORCPT ); Mon, 7 Feb 2022 20:06:22 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48834 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1343853AbiBHAAb (ORCPT ); Mon, 7 Feb 2022 19:00:31 -0500 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3BFD0C061355 for ; Mon, 7 Feb 2022 16:00:30 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id CA21360ED2 for ; Tue, 8 Feb 2022 00:00:29 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 35B22C004E1; Tue, 8 Feb 2022 00:00:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1644278429; bh=7PSS6jLI4iYgo5KR7l9I8htyhSIG+wba1OL4q9uxO0A=; h=Date:From:To:Cc:Subject:Reply-To:References:In-Reply-To:From; b=rT8N85SaIX7ed1iwiguGhkZs2v/ci9v1zkM5KrJlFLGx8Xl5UhG/8YI5cHDebrpbM WzR9aNeGASh81O6ZfmMdsxBJVmQ91f7+CsCKocg9nKsEkqvYLpJUoqPpSg0CXVNnKW HwaZD0yj2EE/DJZBHRyWZaWUMf5V7lJRMbhaYJzwWkNhuMT7Tjq7zdmbM8NAWm/d+2 G1XeFB54L6fAucOHVwftmBxqul2YEZwf9pqRA/229nY61N9hda2VT+PyLFcsAbUzYI fvRzCHfamu7DK3NDXdnIpXyDsr+ppQ62eXcej6as88jvhy9A/Up4Yfof6zMg5vRIdg Gwsm5lQHTIjDQ== Received: by paulmck-ThinkPad-P17-Gen-1.home (Postfix, from userid 1000) id EAD965C0348; Mon, 7 Feb 2022 16:00:28 -0800 (PST) Date: Mon, 7 Feb 2022 16:00:28 -0800 From: "Paul E. McKenney" To: Willy Tarreau Cc: Mark Brown , linux-kernel@vger.kernel.org Subject: Re: [PATCH 00/42] nolibc: update to resync with out-of-tree project Message-ID: <20220208000028.GG4285@paulmck-ThinkPad-P17-Gen-1> Reply-To: paulmck@kernel.org References: <20220207162354.14293-1-w@1wt.eu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220207162354.14293-1-w@1wt.eu> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Feb 07, 2022 at 05:23:12PM +0100, Willy Tarreau wrote: > Hello Paul, > > Here comes an update to the nolibc subsystem to keep it in sync with > the out-of-tree project and to bring a number of usability improvements. > > The approach here consists in better splitting the layers so that > arch-specific syscall functions are now in their own files (one per > arch), then syscalls are implemented on top of this in a generic file, > then the generic libc stuff is exposed. This way it becomes easier to > add support for new architectures if needed. > > I exported 3 functions as weak symbols: memset(), memmove() and abort(). > For the first two ones, gcc/clang may issue calls to them and will not > always fall back to the builtin equivalent. For abort(), I've seen it > referenced by libgcc from some divide functions. > > A few stdio-level functions were added (unbuffered), including a quite > limited but usable printf() which significantly eases debugging of test > programs by allowing to print return codes or pointers for example. > > A few functions and syscalls like abort(), raise(), time() and usleep() > were added. More str* functions were added as well. open() now takes a > vararg on the 3rd argument so as to ease porting of existing programs. > > Finally, the functions and definitions were moved to the appropriate > files as documented in their man pages when relevant, so that simple > programs which only use a few include files among stdio.h, stdlib.h, > string.h, unistd.h, errno.h, ctype.h, signal.h, time.h or types.h can > build without modification. Obviously this remains fairly limited in > scope, but for test programs that's usually fine. > > I could verify that a few of my programs and that your rcutorture test > continue to work (not much surprising since most of the work only > consists in moving functions between files). > > There's obviously no rush, but I preferred to resync the version here > given that Mark expressed some interest recently, and I'd hate it if > he had to add new syscalls that already existed out of tree! Comments, > suggestions and feature requests welcome as usual. If you prefer to > pull, it's also available here, on top of 5.17-rc3: > > git://git.kernel.org/pub/scm/linux/kernel/git/wtarreau/nolibc.git 20220206-nolibc-split-2 I queued these from email to get some time on them. If an update is needed (for example, based on discussion with David Laight), please feel free to send me an update and I can drop the current set in favor of a new version. Good stuff, by the way, thank you!!! Hmmm... I should make rcutorture able to smoke-test this. My thought would be to add some code to rcutorture's init.c that tried out the system calls. But I bet that you already have some test code. ;-) One approach would be to place the test code in tools/nolibc, and have rcutorture's init.c either #include that or link to it. Thoughts? Especially thoughts about better testing approaches? Thanx, Paul > Thank you! > Willy > > --- > Willy Tarreau (42): > tools/nolibc: use pselect6 on RISCV > tools/nolibc: guard the main file against multiple inclusion > tools/nolibc/std: move the standard type definitions to std.h > tools/nolibc/types: split syscall-specific definitions into their own files > tools/nolibc/arch: split arch-specific code into individual files > tools/nolibc/sys: split the syscall definitions into their own file > tools/nolibc/stdlib: extract the stdlib-specific functions to their own file > tools/nolibc/string: split the string functions into string.h > tools/nolibc/ctype: split the is* functions to ctype.h > tools/nolibc/ctype: add the missing is* functions > tools/nolibc/types: move the FD_* functions to macros in types.h > tools/nolibc/types: make FD_SETSIZE configurable > tools/nolibc/types: move makedev to types.h and make it a macro > tools/nolibc/stdlib: move ltoa() to stdlib.h > tools/nolibc/stdlib: replace the ltoa() function with more efficient ones > tools/nolibc/stdlib: add i64toa() and u64toa() > tools/nolibc/stdlib: add utoh() and u64toh() > tools/nolibc/stdio: add a minimal set of stdio functions > tools/nolibc/stdio: add stdin/stdout/stderr and fget*/fput* functions > tools/nolibc/stdio: add fwrite() to stdio > tools/nolibc/stdio: add a minimal [vf]printf() implementation > tools/nolibc/types: define EXIT_SUCCESS and EXIT_FAILURE > tools/nolibc/stdio: add perror() to report the errno value > tools/nolibc/sys: make open() take a vararg on the 3rd argument > tools/nolibc/stdlib: avoid a 64-bit shift in u64toh_r() > tools/nolibc/stdlib: make raise() use the lower level syscalls only > tools/nolibc/sys: make getpgrp(), getpid(), gettid() not set errno > tools/nolibc/string: use unidirectional variants for memcpy() > tools/nolibc/string: slightly simplify memmove() > tools/nolibc/string: add strncpy() and strlcpy() > tools/nolibc/string: add tiny versions of strncat() and strlcat() > tools/nolibc: move exported functions to their own section > tools/nolibc/arch: mark the _start symbol as weak > tools/nolibc/types: define PATH_MAX and MAXPATHLEN > tools/nolibc/string: export memset() and memmove() > tools/nolibc/errno: extract errno.h from sys.h > tools/nolibc/unistd: extract msleep(), sleep(), tcsetpgrp() to unistd.h > tools/nolibc/unistd: add usleep() > tools/nolibc/signal: move raise() to signal.h > tools/nolibc/time: create time.h with time() > tools/nolibc: also mention how to build by just setting the include path > tools/nolibc/stdlib: implement abort() > > tools/include/nolibc/arch-aarch64.h | 200 +++ > tools/include/nolibc/arch-arm.h | 205 +++ > tools/include/nolibc/arch-i386.h | 197 +++ > tools/include/nolibc/arch-mips.h | 216 +++ > tools/include/nolibc/arch-riscv.h | 205 +++ > tools/include/nolibc/arch-x86_64.h | 216 +++ > tools/include/nolibc/arch.h | 32 + > tools/include/nolibc/ctype.h | 99 ++ > tools/include/nolibc/errno.h | 27 + > tools/include/nolibc/nolibc.h | 2540 +-------------------------- > tools/include/nolibc/signal.h | 22 + > tools/include/nolibc/std.h | 49 + > tools/include/nolibc/stdio.h | 296 ++++ > tools/include/nolibc/stdlib.h | 311 ++++ > tools/include/nolibc/string.h | 212 +++ > tools/include/nolibc/sys.h | 1168 ++++++++++++ > tools/include/nolibc/time.h | 28 + > tools/include/nolibc/types.h | 184 ++ > tools/include/nolibc/unistd.h | 54 + > 19 files changed, 3756 insertions(+), 2505 deletions(-) > create mode 100644 tools/include/nolibc/arch-aarch64.h > create mode 100644 tools/include/nolibc/arch-arm.h > create mode 100644 tools/include/nolibc/arch-i386.h > create mode 100644 tools/include/nolibc/arch-mips.h > create mode 100644 tools/include/nolibc/arch-riscv.h > create mode 100644 tools/include/nolibc/arch-x86_64.h > create mode 100644 tools/include/nolibc/arch.h > create mode 100644 tools/include/nolibc/ctype.h > create mode 100644 tools/include/nolibc/errno.h > create mode 100644 tools/include/nolibc/signal.h > create mode 100644 tools/include/nolibc/std.h > create mode 100644 tools/include/nolibc/stdio.h > create mode 100644 tools/include/nolibc/stdlib.h > create mode 100644 tools/include/nolibc/string.h > create mode 100644 tools/include/nolibc/sys.h > create mode 100644 tools/include/nolibc/time.h > create mode 100644 tools/include/nolibc/types.h > create mode 100644 tools/include/nolibc/unistd.h > > -- > 2.35.1 >