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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id DDC3AEE14A9 for ; Wed, 6 Sep 2023 21:59:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=azmJ7lyUEig0NVA1JzeTAek98WaXVirSF2AF0fyALOY=; b=CaEBwH3PUlftWx 7qdDiBklNirIM9DBQ4OAxdt5Tb6TY1xh7hlUKe10lP45qp8/vS3QBkZZoUOuqNuzruVqdc/+zIpzy YBZuCTQdcCGRg63coEXVOhcHugVTTkQoecU39IBIp9nhdQOfYH115eJJ4VucNE+8O5e1Dh7j1KtPy shQxSSPrmtjdGtmCbHYtQu2FzC6dTKZRSWb1g/66kZaWMvARTjkIkORTXlWBRNUJk8j0+65w/fYs/ eTwXV5Ze7nzPy1gyK10gQvTne9cm6FonjgntCBl0mXwyeE3P0D6KMqKJDHpuiopXoSnm3GFHEHoUY +7GvoclXQxZ784auzUXg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qe0Y8-00Awb7-1u; Wed, 06 Sep 2023 21:59:00 +0000 Received: from mail-wr1-x436.google.com ([2a00:1450:4864:20::436]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qe0Y6-00AwZu-0j; Wed, 06 Sep 2023 21:58:59 +0000 Received: by mail-wr1-x436.google.com with SMTP id ffacd0b85a97d-317c3ac7339so325125f8f.0; Wed, 06 Sep 2023 14:58:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1694037535; x=1694642335; darn=lists.infradead.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:from:to:cc:subject:date:message-id :reply-to; bh=fUEsUDVRFl9we3IvMz1iDQ5wrm80qIsR/HrhiCU8qwU=; b=lSL/1uPdhaIJ8y740T8StDBG0FcDYPTm4rR/6LB/Ug5b0T2T/KZgMBvOSYUsbyD2rp Owgv8gNeNEkCyWHVasKFro8IN8jV776OJxVhi6IqqauEv8dvHVrrLSftSzleYnt5Dcoc OzUvr0Nh5vHtJLf7GEWORU1SlH4HrBy4IM1kgTMKQVzQeKVn+lZkxzMeQcrDotZBXjQf 2B8/g/FgoGIUgx+gGgVTgG3SDVh3GCc/lh1QBrsU6lbiql4xeaS8T0OhLhQr9FZ33kYB 42UpbUWAcn+8c71Gwr5OhIrx83a8/tA4G1bLLXoYoPQI3WXB3ydq6xA0zdlNbu6Qj1cq LmvA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1694037535; x=1694642335; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=fUEsUDVRFl9we3IvMz1iDQ5wrm80qIsR/HrhiCU8qwU=; b=aKK02bS5vSZnYgI/z+dyKAaiGOjUiDB8A6Ds1Zj2aQ7KjcuuFjMiK2pKsR2a64xRtc CYR3yzxSnO5RfnXXFn747eKZwNYhTQC3c5t5qy2iBU+rsiDmYG0PGS/BxVcyTt67fPNo 4ex27Lz2CvwhqnUu2SK91KgcjXbym0ieaMHAft57e6YXyeQewB4cfjoPcFXcfip53IS+ p5mtZHAkL00Bdlgj+n64+gN0xlSLzC7at82ubaER7Fxr4BzLkpLvlkb0ZlR3Oltw8feG dJrIGJ++aktiVmFu3ernC54eMyuluq4nCjSOdc1bo8AbOASTZUidT0u49+0kAv8icw9n sbkg== X-Gm-Message-State: AOJu0YxRT+611y07CRW7nQ6qKVdSSxSkFAWaSCyZ8VwGNxrjNIRMWoov x/05yeZacx/3jCo+fLbH+MY= X-Google-Smtp-Source: AGHT+IG5nMeuL/VoGmcwCLwFLhOfCIhRsiIb9391lHp1yY958vfciaCR4r3uuUZhhDYYQHGkLNQJgQ== X-Received: by 2002:a5d:604b:0:b0:30e:3da5:46e5 with SMTP id j11-20020a5d604b000000b0030e3da546e5mr3830642wrt.59.1694037535283; Wed, 06 Sep 2023 14:58:55 -0700 (PDT) Received: from gmail.com (1F2EF6A2.nat.pool.telekom.hu. [31.46.246.162]) by smtp.gmail.com with ESMTPSA id n10-20020adffe0a000000b003140f47224csm21447397wrr.15.2023.09.06.14.58.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 06 Sep 2023 14:58:50 -0700 (PDT) Date: Wed, 6 Sep 2023 23:58:47 +0200 From: Ingo Molnar To: Dave Hansen Cc: j.granados@samsung.com, Luis Chamberlain , willy@infradead.org, josh@joshtriplett.org, Kees Cook , Iurii Zaikin , Heiko Carstens , Vasily Gorbik , Alexander Gordeev , Christian Borntraeger , Sven Schnelle , Gerald Schaefer , Russell King , Catalin Marinas , Will Deacon , Mark Rutland , Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , x86@kernel.org, "H. Peter Anvin" , Andy Lutomirski , Paul Walmsley , Palmer Dabbelt , Albert Ou , Michael Ellerman , Nicholas Piggin , Christophe Leroy , Guo Ren , linux-fsdevel@vger.kernel.org, linux-s390@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-riscv@lists.infradead.org, linuxppc-dev@lists.ozlabs.org, linux-ia64@vger.kernel.org, linux-csky@vger.kernel.org Subject: Re: [PATCH 3/8] arch/x86: Remove sentinel elem from ctl_table arrays Message-ID: References: <20230906-jag-sysctl_remove_empty_elem_arch-v1-0-3935d4854248@samsung.com> <20230906-jag-sysctl_remove_empty_elem_arch-v1-3-3935d4854248@samsung.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230906_145858_269073_20680165 X-CRM114-Status: GOOD ( 18.12 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org * Dave Hansen wrote: > On 9/6/23 03:03, Joel Granados via B4 Relay wrote: > > This commit comes at the tail end of a greater effort to remove the > > empty elements at the end of the ctl_table arrays (sentinels) which > > will reduce the overall build time size of the kernel and run time > > memory bloat by ~64 bytes per sentinel (further information Link : > > https://lore.kernel.org/all/ZO5Yx5JFogGi%2FcBo@bombadil.infradead.org/) > > > > Remove sentinel element from sld_sysctl and itmt_kern_table. > > There's a *LOT* of content to read for a reviewer to figure out what's > going on here between all the links. I would have appreciated one more > sentence here, maybe: > > This is now safe because the sysctl registration code > (register_sysctl()) implicitly uses ARRAY_SIZE() in addition > to checking for a sentinel. > > That needs to be more prominent _somewhere_. Maybe here, or maybe in > the cover letter, but _somewhere_. > > That said, feel free to add this to the two x86 patches: > > Acked-by: Dave Hansen # for x86 Absolutely needs to be in the title as well, something like: arch/x86: Remove now superfluous sentinel elem from ctl_table arrays With that propagated into the whole series: Reviewed-by: Ingo Molnar Thanks, Ingo _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel