From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e23smtp01.au.ibm.com (e23smtp01.au.ibm.com [202.81.31.143]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 52C431A0735 for ; Wed, 4 Feb 2015 19:22:05 +1100 (AEDT) Received: from /spool/local by e23smtp01.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 4 Feb 2015 18:22:03 +1000 Received: from d23relay07.au.ibm.com (d23relay07.au.ibm.com [9.190.26.37]) by d23dlp03.au.ibm.com (Postfix) with ESMTP id 784033578048 for ; Wed, 4 Feb 2015 19:21:59 +1100 (EST) Received: from d23av03.au.ibm.com (d23av03.au.ibm.com [9.190.234.97]) by d23relay07.au.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id t148Lpb248103520 for ; Wed, 4 Feb 2015 19:21:59 +1100 Received: from d23av03.au.ibm.com (localhost [127.0.0.1]) by d23av03.au.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id t148LPEN030037 for ; Wed, 4 Feb 2015 19:21:25 +1100 Message-ID: <54D1D66D.1030000@linux.vnet.ibm.com> Date: Wed, 04 Feb 2015 13:51:01 +0530 From: Anshuman Khandual MIME-Version: 1.0 To: linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH V2 00/12] POWER DSCR fixes, improvements, docs and tests References: <1421144560-15901-1-git-send-email-khandual@linux.vnet.ibm.com> In-Reply-To: <1421144560-15901-1-git-send-email-khandual@linux.vnet.ibm.com> Content-Type: text/plain; charset=UTF-8 Cc: mikey@neuling.org, shuahkh@osg.samsung.com, anton@samba.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 01/13/2015 03:52 PM, Anshuman Khandual wrote: > This patch series has patches for POWER DSCR fixes, improvements, > in code documentaion, kernel support user documentation and selftest based > test cases. It has got five test cases which are derived from Anton's DSCR > test bucket which can be listed as follows. > > (1) http://ozlabs.org/~anton/junkcode/dscr_default_test.c > (2) http://ozlabs.org/~anton/junkcode/dscr_explicit_test.c > (3) http://ozlabs.org/~anton/junkcode/dscr_inherit_exec_test.c > (4) http://ozlabs.org/~anton/junkcode/dscr_inherit_test.c > (5) http://ozlabs.org/~anton/junkcode/user_dscr_test.c > > So the derivied test cases have Anton's copyright along with mine but the > commit message as of now has only my signed-off-by statement. As Anton > mentioned before he would put his signed-off-by after reviewing these > modified test cases. > > NOTE1: Anton's original inherit exec test expected the child to have system > default DSCR value instead of the inherited DSCR value from it's parent. > But looks like thats not the case when we execute the test, it always > inherits it's parent's DSCR value over the exec call as well. So I have > changed the test program assuming its correct to have the inherited DSCR > value in the fork/execed child program. Please let me know if thats not > correct and I am missing something there. > > NOTE2: The selftests/powerpc/.gitignore will be added and will get updated > through a different patch series related to ptrace instead of this one. > > Changes in V2: > ------------- > - Updated the thread struct DSCR value inside mtspr facility exception path > - Modified the in code documentation to follow the kernel-doc format > - Added seven selftest based DSCR related test cases under powerpc Hey Michael, Did you get a chance to look into these patches ? After going through the discussions regarding all the selftest test cases, seems like they are good to go unless you disagree.