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 EB162C433EF for ; Wed, 30 Mar 2022 18:52:07 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1350244AbiC3Sxq (ORCPT ); Wed, 30 Mar 2022 14:53:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52018 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1350105AbiC3Sxk (ORCPT ); Wed, 30 Mar 2022 14:53:40 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [145.40.68.75]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3D0A14DF5A for ; Wed, 30 Mar 2022 11:51:55 -0700 (PDT) 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 ams.source.kernel.org (Postfix) with ESMTPS id D7BFBB81D85 for ; Wed, 30 Mar 2022 18:51:53 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 7ADCBC340F0; Wed, 30 Mar 2022 18:51:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1648666312; bh=ibrcnI4Sh2c6Hpc0kjpQBswypoj77BsVt19VYyJOL4E=; h=Date:From:To:Cc:Subject:Reply-To:References:In-Reply-To:From; b=pJLcyWJd3S6r6tVrlN0fz7PGG6FPoP7C33G3cuHsUM0hdDOc6/g6eb9ROMLaLW8BM Xp16xq6HxTrMII6JkiBq7/HSUXfYT2AtRj1AQkEkfqqrxYujQ/gnUW+B2nRi85+TlS QDpnu0EP9sNjzFNn56m7+n/sGYkpBSm4I8wRCwRvyuzq+Xx4bokWzu2xItauI+kF9q hQY0SnO4QLLF2jhbEs625Zqntg8AbmLy+C00zWuR3lTgbrjDigda70YDCGKRjt6RqV H4XIKtI+yBJTHYWN3a4K2uol+wiCh1dRNX8UhG1uOBhlDiwRrJPVeqHg9RLDh1jCW1 r1Gt3V3RqpWHA== Received: by paulmck-ThinkPad-P17-Gen-1.home (Postfix, from userid 1000) id 0FE7B5C12E4; Wed, 30 Mar 2022 11:51:52 -0700 (PDT) Date: Wed, 30 Mar 2022 11:51:52 -0700 From: "Paul E. McKenney" To: Willy Tarreau Cc: Ammar Faizi , Alviro Iskandar Setiawan , Nugraha , Linux Kernel Mailing List , GNU/Weeb Mailing List Subject: Re: [PATCH v2 0/9] Add dynamic memory allocator support for nolibc Message-ID: <20220330185152.GK4285@paulmck-ThinkPad-P17-Gen-1> Reply-To: paulmck@kernel.org References: <20220329101737.58985-1-ammarfaizi2@gnuweeb.org> <2504aa03-e3cf-a120-44dc-af70ec7c9d5c@gnuweeb.org> <20220330024114.GA18892@1wt.eu> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220330024114.GA18892@1wt.eu> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Mar 30, 2022 at 04:41:14AM +0200, Willy Tarreau wrote: > Hi Ammar, > > On Tue, Mar 29, 2022 at 05:20:31PM +0700, Ammar Faizi wrote: > > On 3/29/22 5:17 PM, Ammar Faizi wrote: > > > Hi, > > > > > > This is a patchset v2 to add dynamic memory allocator support > > > for nolibc after 2 RFCs, please review the changes carefully. > > Thank you! For me it's OK for all the series: > > Acked-by: Willy Tarreau > > I do have a minor comment about the use of __builtin_mul_overflow() here. > While this code is included in the kernel and mostly for use with kernel > related stuff, till now I've been careful to support older compilers (I'm > still seeing 4.8, 4.7 and 4.4 commonly in field). I don't find it urgent, > but I think that sooner or later it would be nice to implement an > alternative for compilers missing this builtin, especially if it's the > only one that prevents older compilers from being used. Probably that > something like this (untested) would do the job: > > if (nmemb && ~(size_t)0 / nmemb < size) { > SET_ERRNO(ENOMEM); > return NULL; > } > size *= nmemb; > > But again, for me it's not a showstopper and can be improved later. > > > Sorry, forgot to replace === with --- in for each patch. > > Should I resend? > > Let's see what Paul prefers. sed 's/===/---/' on the mbox should fix > it otherwise a resend will be needed. Given that I am adding your Acked-by anyway, why not? ;-) But please check the commits to make sure that this had the desired effect. Thanx, Paul