From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kees Cook Subject: Re: [PATCH 16/17] arch: remove HAVE_COPY_THREAD_TLS Date: Mon, 22 Jun 2020 17:44:52 -0700 Message-ID: <202006221743.1BE5670@keescook> References: <20200622234326.906346-1-christian.brauner@ubuntu.com> <20200622234326.906346-17-christian.brauner@ubuntu.com> Mime-Version: 1.0 Return-path: DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=WOoBj34wrpkLm3cxJNNb0KejtEcGHXihiKs5tn9Bl2A=; b=Fyi9BekJ6P5kCaEZI4v0Q5UGFFijbkzvYxlDhZfmBSOVaRIA/BSafm9pzLuUQlIJo0 akWxcy93XZBgnvmXLQQBdCNAgHijbBuOyOHThfXz86makirRrM3AruNfhYt0fz3mFMJt ULQfJMf3BQL8GiNmwlac0RpP8ipddRm3w9CIM= Content-Disposition: inline In-Reply-To: <20200622234326.906346-17-christian.brauner@ubuntu.com> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: linuxppc-dev-bounces+glppe-linuxppc-embedded-2=m.gmane-mx.org@lists.ozlabs.org Sender: "Linuxppc-dev" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Christian Brauner Cc: Rich Felker , linux-sh@vger.kernel.org, Peter Zijlstra , Catalin Marinas , Heiko Carstens , linux-mips@vger.kernel.org, "James E.J. Bottomley" , Guo Ren , linux-csky@vger.kernel.org, sparclinux@vger.kernel.org, linux-hexagon@vger.kernel.org, linux-riscv@lists.infradead.org, Vincent Chen , Will Deacon , Thomas Gleixner , Anton Ivanov , Jonas Bonn , linux-s390@vger.kernel.org, linux-ia64@vger.kernel.org, linux-c6x-dev@linux-c6x.org, Brian Cain , linux-xtensa@linux-xtensa.org, Helge Deller , x86@kernel.org, Russell King , Ley Foon Tan On Tue, Jun 23, 2020 at 01:43:25AM +0200, Christian Brauner wrote: > All architectures support copy_thread_tls() now, so remove the legacy > copy_thread() function and the HAVE_COPY_THREAD_TLS config option. Everyone > uses the same process creation calling convention based on > copy_thread_tls() and struct kernel_clone_args. This will make it easier to > maintain the core process creation code under kernel/, simplifies the > callpaths and makes the identical for all architectures. > [...] > Signed-off-by: Christian Brauner Massive CC list. ;) linux-arch@ may be sufficient. Reviewed-by: Kees Cook -- Kees Cook