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 X-Spam-Level: X-Spam-Status: No, score=-3.9 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 388B6C28CBC for ; Mon, 4 May 2020 01:26:04 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id D23D8206EB for ; Mon, 4 May 2020 01:26:03 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=canb.auug.org.au header.i=@canb.auug.org.au header.b="P+yJzoL4" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D23D8206EB Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=canb.auug.org.au Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 314398E0005; Sun, 3 May 2020 21:26:03 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 29EB38E0001; Sun, 3 May 2020 21:26:03 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 18D978E0005; Sun, 3 May 2020 21:26:03 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0189.hostedemail.com [216.40.44.189]) by kanga.kvack.org (Postfix) with ESMTP id F0CEA8E0001 for ; Sun, 3 May 2020 21:26:02 -0400 (EDT) Received: from smtpin26.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay01.hostedemail.com (Postfix) with ESMTP id B6172180AD811 for ; Mon, 4 May 2020 01:26:02 +0000 (UTC) X-FDA: 76777295364.26.cause21_51b574d9ff743 X-HE-Tag: cause21_51b574d9ff743 X-Filterd-Recvd-Size: 3918 Received: from ozlabs.org (bilbo.ozlabs.org [203.11.71.1]) by imf29.hostedemail.com (Postfix) with ESMTP for ; Mon, 4 May 2020 01:26:01 +0000 (UTC) Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mail.ozlabs.org (Postfix) with ESMTPSA id 49FlYH0HjSz9sSr; Mon, 4 May 2020 11:25:55 +1000 (AEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=canb.auug.org.au; s=201702; t=1588555556; bh=vc2ocq2LVzS7oNXSMpHcVG3xe33e0fBbGhPZyRYagfM=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=P+yJzoL4bKmQgxZi1HZa5TPP9rwqF/h2NThx+pjTcMdtdBDNff/3h228jqwzsXE2b NQJQhDYncXGB0zYQeDkYjscr4OF+6Rqxpr31OS5Obez0CGvi8oBiJ3Y2azikMftqQt U+jzCD47pLK/nuTaLkh3aiYDzUWWzf1IbXyuYsgSqVM3RNmpilPBYaN/E+R7RNNqo1 WT9ab3XNgwxWwK5N/cas1cph5490Z5afc6RVZtTmbZpHUnnVlwPo1486wsKlO+zmPg yu1OUHXML1YrIkeJQ8m9spCLWXR6FJr3ec9jOEGd6cjZcxZe1mrlNfBUbSa4Vnfgr4 ZNwuqxT3rnTxQ== Date: Mon, 4 May 2020 11:25:52 +1000 From: Stephen Rothwell To: Christoph Hellwig Cc: Kees Cook , Iurii Zaikin , Alexei Starovoitov , Daniel Borkmann , linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, netdev@vger.kernel.org, bpf@vger.kernel.org Subject: Re: [PATCH 3/5] sysctl: remove all extern declaration from sysctl.c Message-ID: <20200504112552.4dbdd2a9@canb.auug.org.au> In-Reply-To: <20200424064338.538313-4-hch@lst.de> References: <20200424064338.538313-1-hch@lst.de> <20200424064338.538313-4-hch@lst.de> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="Sig_/Ze/S33mgpR/AlQdumg8GhV7"; protocol="application/pgp-signature"; micalg=pgp-sha256 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: --Sig_/Ze/S33mgpR/AlQdumg8GhV7 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable Hi Christoph, On Fri, 24 Apr 2020 08:43:36 +0200 Christoph Hellwig wrote: > > Extern declarations in .c files are a bad style and can lead to > mismatches. Use existing definitions in headers where they exist, > and otherwise move the external declarations to suitable header > files. >=20 > Signed-off-by: Christoph Hellwig > --- > include/linux/coredump.h | 4 ++++ > include/linux/file.h | 2 ++ > include/linux/mm.h | 2 ++ > include/linux/mmzone.h | 2 ++ > include/linux/pid.h | 3 +++ > include/linux/sysctl.h | 8 +++++++ > kernel/sysctl.c | 45 +++------------------------------------- > 7 files changed, 24 insertions(+), 42 deletions(-) A couple of suggestions for another patch (since this one is in a shared branch in Al's tree now): There is an "extern struct ctl_table random_table[];" in drivers/char/random.c which is redundant now (in fact always was). There is already an "extern struct ctl_table epoll_table[];" in include/linux/poll.h, so could have included that in kernel/sysctl.c instead of adding the new one in include/linux/sysctl.h --=20 Cheers, Stephen Rothwell --Sig_/Ze/S33mgpR/AlQdumg8GhV7 Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEENIC96giZ81tWdLgKAVBC80lX0GwFAl6vbyAACgkQAVBC80lX 0GxFGwf/WH/cv5RMadV9dmdAfJGI/cu7ozy5lgpWNwTKsdi4ITqe5NGRhPUzU4yW G2MRxjhFk0AMvRQWk9cdWriceLeojws2iI4W6ht10nYedCXG70ee+MJ/TEx7VJ5f +Cff6GVaFu+4/qF04l/XTgi75XrzhNxQx3EptjGJg89jeJoKB219icmdSfwb8lBE RzU/qAufZ62wf5ClRmbyMJ0rj0RqdpKI/Jg+gD/rYE9HnveOnyLkFRDY7Zn5s7H8 k+TPkw9I4xlWwDYr6cp43CX4rfGlZNoByoYat5ExYbvsjtPk9ygEYm9liKVKgIqh riEp/3lZakla9sctllvU4m6A6mIQJQ== =LZ1e -----END PGP SIGNATURE----- --Sig_/Ze/S33mgpR/AlQdumg8GhV7--