From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Fri, 25 Mar 2016 09:56:24 +0100 (CET) Received: from ozlabs.org ([103.22.144.67]:55188 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S27008482AbcCYI4VbXFf4 (ORCPT ); Fri, 25 Mar 2016 09:56:21 +0100 Received: by ozlabs.org (Postfix, from userid 1034) id 3qWc3w0GCtz9sdm; Fri, 25 Mar 2016 19:32:11 +1100 (AEDT) In-Reply-To: <1458824294-29733-1-git-send-email-jslaby@suse.cz> To: Jiri Slaby , akpm@linux-foundation.org From: Michael Ellerman Cc: linux-mips@linux-mips.org, linux-s390@vger.kernel.org, Rich Felker , Aurelien Jacquiot , Peter Zijlstra , Catalin Marinas , Will Deacon , David Howells , Max Filippov , Paul Mackerras , "H. Peter Anvin" , sparclinux@vger.kernel.org, linux-hexagon@vger.kernel.org, Guan Xuetao , Jiri Slaby , Lennox Wu , Hans-Christian Egtvedt , Jonas Bonn , Chen Liqin , Jesper Nilsson , linux-am33-list@redhat.com, Russell King , linux-c6x-dev@linux-c6x.org, Yoshinori Sato , linux-sh@vger.kernel.org, Helge Deller , x86@kernel.org, "James E.J. Bottomley" , Ingo Molnar , Geert Uytterhoeven , Mark Salter , Matt Turner , linux-snps-arc@lists.infradead.org, Haavard Skinnemoen , uclinux-h8-devel@lists.sourceforge.jp, Fenghua Yu , James Hogan , Chris Metcalf , user-mode-linux-devel@lists.sourceforge.net, Steven Miao , Heiko Carstens , Jeff Dike , linux-alpha@vger.kernel.org, adi-buildroot-devel@lists.sourceforge.net, Mikael Starvik , linux-m68k@lists.linux-m68k.org, Ivan Kokshaysky , user-mode-linux-user@lists.sourceforge.net, nios2-dev@lists.rocketboards.org, Thomas Gleixner , linux-xtensa@linux-xtensa.org, linux-metag@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Richard Henderson , Chris Zankel , Michal Simek , Tony Luck , linux-ia64@vger.kernel.org, linux-parisc@vger.kernel.org, linux-cris-kernel@axis.com, Vineet Gupta , linux-kernel@vger.kernel.org, Ralf Baechle , Richard Kuo , Richard Weinberger , Martin Schwidefsky , Ley Foon Tan , Koichi Yasutake , linuxppc-dev@lists.ozlabs.org, "David S. Miller" Subject: Re: [3/4] exit_thread: accept a task parameter to be exited Message-Id: <3qWc3w0GCtz9sdm@ozlabs.org> Date: Fri, 25 Mar 2016 19:32:11 +1100 (AEDT) Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 52699 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: mpe@ellerman.id.au Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips On Thu, 2016-24-03 at 12:58:13 UTC, Jiri Slaby wrote: > We need to call exit_thread from copy_process in a fail path. So make > it accept task_struct as a parameter. If I'm counting right 22 of those are empty functions, so would it be a good clean up to make it optional first? cheers From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from ozlabs.org ([103.22.144.67]:55188 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S27008482AbcCYI4VbXFf4 (ORCPT ); Fri, 25 Mar 2016 09:56:21 +0100 In-Reply-To: <1458824294-29733-1-git-send-email-jslaby@suse.cz> From: Michael Ellerman Subject: Re: [3/4] exit_thread: accept a task parameter to be exited Message-ID: <3qWc3w0GCtz9sdm@ozlabs.org> Date: Fri, 25 Mar 2016 19:32:11 +1100 (AEDT) Return-Path: Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-subscribe: List-owner: List-post: List-archive: To: Jiri Slaby , akpm@linux-foundation.org Cc: linux-mips@linux-mips.org, linux-s390@vger.kernel.org, Rich Felker , Aurelien Jacquiot , Peter Zijlstra , Catalin Marinas , Will Deacon , David Howells , Max Filippov , Paul Mackerras , "H. Peter Anvin" , sparclinux@vger.kernel.org, linux-hexagon@vger.kernel.org, Guan Xuetao , Lennox Wu , Hans-Christian Egtvedt , Jonas Bonn , Chen Liqin , Jesper Nilsson , linux-am33-list@redhat.com, Russell King , linux-c6x-dev@linux-c6x.org, Yoshinori Sato , linux-sh@vger.kernel.org, Helge Deller , x86@kernel.org, "James E.J. Bottomley" , Ingo Molnar , Geert Uytterhoeven , Mark Salter , Matt Turner , linux-snps-arc@lists.infradead.org, Haavard Skinnemoen , uclinux-h8-devel@lists.sourceforge.jp, Fenghua Yu , James Hogan , Chris Metcalf , user-mode-linux-devel@lists.sourceforge.net, Steven Miao , Heiko Carstens , Jeff Dike , linux-alpha@vger.kernel.org, adi-buildroot-devel@lists.sourceforge.net, Mikael Starvik , linux-m68k@lists.linux-m68k.org, Ivan Kokshaysky , user-mode-linux-user@lists.sourceforge.net, nios2-dev@lists.rocketboards.org, Thomas Gleixner , linux-xtensa@linux-xtensa.org, linux-metag@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Richard Henderson , Chris Zankel , Michal Simek , str.eu, Tony Luck , linux-ia64@vger.kernel.org, linux-parisc@vger.kernel.org, linux-cris-kernel@axis.com, Vineet Gupta , linux-kernel@vger.kernel.org, Ralf Baechle , Richard Kuo , Richard Weinberger , Martin Schwidefsky , Ley Foon Tan , Koichi Yasutake , linuxppc-dev@lists.ozlabs.org, "David S. Miller" Message-ID: <20160325083211.PilGcMZYdNQ742bK9SJpFKQik669dLJsnxnKawA8bkM@z> On Thu, 2016-24-03 at 12:58:13 UTC, Jiri Slaby wrote: > We need to call exit_thread from copy_process in a fail path. So make > it accept task_struct as a parameter. If I'm counting right 22 of those are empty functions, so would it be a good clean up to make it optional first? cheers