From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH 10/14] uaccess: remove most CONFIG_SET_FS users Date: Mon, 14 Feb 2022 20:40:07 +0100 Message-ID: References: <20220214163452.1568807-1-arnd@kernel.org> <20220214163452.1568807-11-arnd@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: 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:Cc:To:Subject:Message-ID:Date:From: In-Reply-To:References:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=QGCdvMLs+y5szyo9mkpTXLks177+98b7gN0+EvDOXp0=; b=cjXsaJb51Ex7Uz 1vOSSTSyvHkeR7boVRzRTJ6ToXnV24SURGAUWNFYzH7gDj7Z9eSJvXpZ+Y6wLt8a43InA42y6F0bo USSNGLHwiC9rAqJY36FSUTZlaw29c7sv24n/W4Qrm2Mz1VseqYBWlIcY11rHLczyCJPBSq4rBepQ6 1lLz9sNp7Yc4BgcuZTU+t6ZsMh5vDj3owY6TP/QruLWz1jKDS+YP8H1W14nSz5Gb0d+ZwV7OHMtf/ cJmBo+pZcw3tb7oMmh6zxaPwPsNfAXs7P62HmTAt5K0GtlS+vB2wEl5cVmCWr++NI2lBdVHvZdckC sPAW0bfh74keI5CWDx6A==; DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1644867624; bh=5szbHF9etsKpvbwWwwc9H/Y1QakuvzjKYGUIBcYFA80=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=e8SkA3FlLGu70hI/WgHcE1Xcld6ahEqgd/8aP3tnN+Ui+6c+MZmJxArGum1LdtmAc TVm8LqkWXJbpel0WqJluZlN0q2Rc8pObOFXCjWsE1d3haVSN1sOjQ5hKRVHWsZ71lN 9Hnj9zqBHSBBx6qUNlA6sc7rmD8pMbOGHJ9SDDCXuRY9FNQ7binWt4bsfA/Yof8Egb Ksm8rfkNmn9T9/rsBeFI1yBap4xnb6nVmSKUusmotl5Q02Llm+mwF8WcKWget2RALt X3B6AavEfyIeX6XUTomqUIV78jVQAYplU/28sKTs9My6dabsF705ywSdaO+p8h0D4l JZS2fHyItMkuQ== In-Reply-To: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-riscv" Errors-To: linux-riscv-bounces+glpr-linux-riscv=m.gmane-mx.org@lists.infradead.org To: Christoph Hellwig Cc: Linus Torvalds , Christoph Hellwig , linux-arch , Linux-MM , Linux API , Arnd Bergmann , Linux Kernel Mailing List , Mark Rutland , Rich Felker , linux-ia64@vger.kernel.org, Linux-sh list , Peter Zijlstra , Max Filippov , Guo Ren , sparclinux , "open list:QUALCOMM HEXAGON..." , linux-riscv , Will Deacon , Ard Biesheuvel , linux-s390 , Brian Cain , Helge Deller , the arch/x86 maintainers On Mon, Feb 14, 2022 at 6:06 PM Christoph Hellwig wrote: > > On Mon, Feb 14, 2022 at 05:34:48PM +0100, Arnd Bergmann wrote: > > From: Arnd Bergmann > > > > On almost all architectures, there are no remaining callers > > of set_fs(), so CONFIG_SET_FS can be disabled, along with > > removing the thread_info field and any references to it. > > > > This turns access_ok() into a cheaper check against TASK_SIZE_MAX. > > Wouldn't it make more sense to just merge this into the last patch? Yes, sounds good. I wasn't sure at first if there is enough buy-in to get all architectures cleaned up, and I hadn't done the ia64 patch, so it seemed more important to do this part early, but now it seems that it will all go in at the same time, so doing this as part of a big removal at the end makes sense. Arnd