From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3rJqHN5492zDqlW for ; Tue, 31 May 2016 20:20:00 +1000 (AEST) Subject: Re: [PATCH] powerpc/pseries: Add POWER8NVL support to ibm,client-architecture-support call To: Michael Ellerman , Paul Mackerras , linuxppc-dev@lists.ozlabs.org References: <1464673877-30659-1-git-send-email-thuth@redhat.com> <1464689047.23025.1.camel@ellerman.id.au> Cc: Benjamin Herrenschmidt , kvm-ppc@vger.kernel.org From: Thomas Huth Message-ID: <574D6549.3050903@redhat.com> Date: Tue, 31 May 2016 12:19:53 +0200 MIME-Version: 1.0 In-Reply-To: <1464689047.23025.1.camel@ellerman.id.au> Content-Type: text/plain; charset=utf-8 List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 31.05.2016 12:04, Michael Ellerman wrote: > On Tue, 2016-05-31 at 07:51 +0200, Thomas Huth wrote: > >> If we do not provide the PVR for POWER8NVL, a guest on this >> system currently ends up in PowerISA 2.06 compatibility mode on >> KVM, since QEMU does not provide a generic PowerISA 2.07 mode yet. >> So some new instructions from POWER8 (like "mtvsrd") get disabled >> for the guest, resulting in crashes when using code compiled >> explicitly for POWER8 (e.g. with the "-mcpu=power8" option of GCC). >> >> Signed-off-by: Thomas Huth > > So this should say: > > Fixes: ddee09c099c3 ("powerpc: Add PVR for POWER8NVL processor") > > And therefore: > > Cc: stable@vger.kernel.org # v4.0+ > > Am I right? Right. (At least for virtualized systems ... for bare-metal systems, that original patch was enough). So shall I resubmit my patch with these two lines, or could you add them when you pick this patch up? Thomas