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 3r54MP6rKDzDqBs for ; Thu, 12 May 2016 17:27:45 +1000 (AEST) Subject: Re: [PATCH] powerpc: Fix definition of SIAR register To: Paul Mackerras , Alexander Graf References: <1460130851-29021-1-git-send-email-thuth@redhat.com> <571DD072.6040305@linux.vnet.ibm.com> <182287E9-40B9-4774-A932-08A0AA499AE3@suse.de> <20160512045756.GB1421@oak.ozlabs.ibm.com> Cc: Madhavan Srinivasan , Michael Ellerman , linuxppc-dev@lists.ozlabs.org, kvm-ppc@vger.kernel.org From: Thomas Huth Message-ID: <5734306C.7070407@redhat.com> Date: Thu, 12 May 2016 09:27:40 +0200 MIME-Version: 1.0 In-Reply-To: <20160512045756.GB1421@oak.ozlabs.ibm.com> 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 12.05.2016 06:57, Paul Mackerras wrote: > On Mon, Apr 25, 2016 at 10:15:47AM +0200, Alexander Graf wrote: >> >>>> +#define SPRN_SIAR 796 >> >> I'm sure there's a reason (iSeries?) we used the r/o version before. Better introduce a new constant that gives us rw access and use that in the kvm entry/exit code. > > I don't think we ever did any performance monitoring on iSeries, and > in any case, we have removed the iSeries code. I think we should just > use the privileged, read/write number (i.e. 796). Ok, then I'll simply add the fix for SDAR to my patch as well, test it and submit it again. Thomas