[{"id":1758955,"web_url":"http://patchwork.ozlabs.org/comment/1758955/","msgid":"<20170829001437.GF12629@fergus.ozlabs.ibm.com>","date":"2017-08-29T00:14:37","subject":"Re: [PATCH v3 3/4] powerpc/64s: idle POWER9 can execute stop in\n\tvirtual mode","submitter":{"id":67079,"url":"http://patchwork.ozlabs.org/api/people/67079/","name":"Paul Mackerras","email":"paulus@ozlabs.org"},"content":"On Fri, Aug 25, 2017 at 02:30:35PM +1000, Nicholas Piggin wrote:\n> The hardware can execute stop in any context, and KVM does not\n> require real mode because siblings do not share MMU state. This\n> saves a switch to real-mode when going idle.\n> \n> Acked-by: Gautham R. Shenoy \n> Signed-off-by: Nicholas Piggin \n> ---\n> arch/powerpc/kernel/idle_book3s.S | 9 +++++++++\n> 1 file changed, 9 insertions(+)\n> \n> diff --git a/arch/powerpc/kernel/idle_book3s.S b/arch/powerpc/kernel/idle_book3s.S\n> index 14e97f442167..32d65ee323a0 100644\n> --- a/arch/powerpc/kernel/idle_book3s.S\n> +++ b/arch/powerpc/kernel/idle_book3s.S\n> @@ -195,7 +195,16 @@ pnv_powersave_common:\n> \tstd\tr5,_CCR(r1)\n> \tstd\tr1,PACAR1(r13)\n> \n> +BEGIN_FTR_SECTION\n> +\t/*\n> +\t * POWER9 does not require real mode to stop, and presently does not\n> +\t * set hwthread_state for KVM (threads don't share MMU context), so\n> +\t * we can remain in virtual mode for this.\n> +\t */\n> +\tbctr\n> +END_FTR_SECTION_IFSET(CPU_FTR_ARCH_300)\n> \t/*\n> +\t * POWER8\n\n... and POWER7 too.\n\nSince that's just a comment,\n\nReviewed-by: Paul Mackerras ","headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xh8XB18nfz9sNr\n\tfor ;\n\tTue, 29 Aug 2017 10:24:26 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3xh8X972mszDqJs\n\tfor ;\n\tTue, 29 Aug 2017 10:24:25 +1000 (AEST)","from ozlabs.org (ozlabs.org [IPv6:2401:3900:2:1::2])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3xh8S412qGzDqGW\n\tfor ;\n\tTue, 29 Aug 2017 10:20:52 +1000 (AEST)","by ozlabs.org (Postfix, from userid 1003)\n\tid 3xh8S36rFNz9s65; Tue, 29 Aug 2017 10:20:51 +1000 (AEST)"],"Authentication-Results":["ozlabs.org; dkim=pass (2048-bit key;\n\tsecure) header.d=ozlabs.org header.i=@ozlabs.org header.b=\"x8DyLFYm\";\n\tdkim-atps=neutral","lists.ozlabs.org; dkim=pass (2048-bit key;\n\tsecure) header.d=ozlabs.org header.i=@ozlabs.org header.b=\"x8DyLFYm\";\n\tdkim-atps=neutral","lists.ozlabs.org; dkim=pass (2048-bit key;\n\tsecure) header.d=ozlabs.org header.i=@ozlabs.org header.b=\"x8DyLFYm\"; \n\tdkim-atps=neutral"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ozlabs.org; s=201707; \n\tt=1503966051; bh=/bUdDyMVkPUMcBeFlKgZ2gEJXMQAr0VRhlTjoTNgFuw=;\n\th=Date:From:To:Cc:Subject:References:In-Reply-To:From;\n\tb=x8DyLFYm9MCbzPpZ4ptD85mSTnG3zyQUDKL8wnObohAXGRXjYYSSVWzz6SberwWda\n\twrrtDnhELjNe7DSW4o9fE3RqIHESrYcqtjEDMKgoU7QeD6b3DN/UwXpH4jeqOq7YWG\n\tX774hTuX8rW9KlJpeN8nse4SoASUlEYleIW16h/+pCfa271yMsHCai21lBjLN+O9si\n\td+NXjW50r5EPL69VtLENCwLi8j0BIuYjfvGFNvpZuqGihMixBEYwyAlRXrqVUARxfo\n\thvh/x7mwfSrwnl3nlsTJZ9OwwG1F65gH6QYNKUvW2I6sH7Ry5eV9eq68CkfvlBUKeq\n\tDN6A+om2IDm6A==","Date":"Tue, 29 Aug 2017 10:14:37 +1000","From":"Paul Mackerras ","To":"Nicholas Piggin ","Subject":"Re: [PATCH v3 3/4] powerpc/64s: idle POWER9 can execute stop in\n\tvirtual mode","Message-ID":"<20170829001437.GF12629@fergus.ozlabs.ibm.com>","References":"<20170825043036.18236-1-npiggin@gmail.com>\n\t<20170825043036.18236-4-npiggin@gmail.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<20170825043036.18236-4-npiggin@gmail.com>","User-Agent":"Mutt/1.5.24 (2015-08-30)","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"linuxppc-dev@lists.ozlabs.org, kvm-ppc@vger.kernel.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"}},{"id":1760902,"web_url":"http://patchwork.ozlabs.org/comment/1760902/","msgid":"<3xjgLX3Dgkz9sQl@ozlabs.org>","date":"2017-08-31T11:36:19","subject":"Re: [v3,\n\t3/4] powerpc/64s: idle POWER9 can execute stop in virtual mode","submitter":{"id":69509,"url":"http://patchwork.ozlabs.org/api/people/69509/","name":"Michael Ellerman","email":"patch-notifications@ellerman.id.au"},"content":"On Fri, 2017-08-25 at 04:30:35 UTC, Nicholas Piggin wrote:\n> The hardware can execute stop in any context, and KVM does not\n> require real mode because siblings do not share MMU state. This\n> saves a switch to real-mode when going idle.\n> \n> Acked-by: Gautham R. Shenoy \n> Signed-off-by: Nicholas Piggin \n> Reviewed-by: Paul Mackerras \n\nApplied to powerpc next, thanks.\n\nhttps://git.kernel.org/powerpc/c/72b0d51d973beab5a06c97279b61a0\n\ncheers","headers":{"Return-Path":"","X-Original-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Delivered-To":["patchwork-incoming@ozlabs.org","linuxppc-dev@lists.ozlabs.org"],"Received":["from lists.ozlabs.org (lists.ozlabs.org [103.22.144.68])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xjhCg54qXz9t16\n\tfor ;\n\tThu, 31 Aug 2017 22:15:27 +1000 (AEST)","from lists.ozlabs.org (lists.ozlabs.org [IPv6:2401:3900:2:1::3])\n\tby lists.ozlabs.org (Postfix) with ESMTP id 3xjhCg4HK8zDrG1\n\tfor ;\n\tThu, 31 Aug 2017 22:15:27 +1000 (AEST)","from ozlabs.org (ozlabs.org [IPv6:2401:3900:2:1::2])\n\t(using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby lists.ozlabs.org (Postfix) with ESMTPS id 3xjgLX5NwWzDqTm\n\tfor ;\n\tThu, 31 Aug 2017 21:36:20 +1000 (AEST)","by ozlabs.org (Postfix, from userid 1034)\n\tid 3xjgLX3Dgkz9sQl; Thu, 31 Aug 2017 21:36:19 +1000 (AEST)"],"X-powerpc-patch-notification":"thanks","X-powerpc-patch-commit":"72b0d51d973beab5a06c97279b61a0a58ccf33c8","In-Reply-To":"<20170825043036.18236-4-npiggin@gmail.com>","To":"Nicholas Piggin , linuxppc-dev@lists.ozlabs.org","From":"Michael Ellerman ","Subject":"Re: [v3,\n\t3/4] powerpc/64s: idle POWER9 can execute stop in virtual mode","Message-Id":"<3xjgLX3Dgkz9sQl@ozlabs.org>","Date":"Thu, 31 Aug 2017 21:36:19 +1000 (AEST)","X-BeenThere":"linuxppc-dev@lists.ozlabs.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"Linux on PowerPC Developers Mail List\n\t","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"Nicholas Piggin , kvm-ppc@vger.kernel.org","Errors-To":"linuxppc-dev-bounces+patchwork-incoming=ozlabs.org@lists.ozlabs.org","Sender":"\"Linuxppc-dev\"\n\t"}}]