From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932820AbbBDIWI (ORCPT ); Wed, 4 Feb 2015 03:22:08 -0500 Received: from e23smtp04.au.ibm.com ([202.81.31.146]:54694 "EHLO e23smtp04.au.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932371AbbBDIWE (ORCPT ); Wed, 4 Feb 2015 03:22:04 -0500 Message-ID: <54D1D66D.1030000@linux.vnet.ibm.com> Date: Wed, 04 Feb 2015 13:51:01 +0530 From: Anshuman Khandual User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 MIME-Version: 1.0 To: linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org CC: mikey@neuling.org, shuahkh@osg.samsung.com, anton@samba.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 Content-Transfer-Encoding: 7bit X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 15020408-0013-0000-0000-000000C57EE0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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.