From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45849) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xo4Jj-0007yi-8l for qemu-devel@nongnu.org; Tue, 11 Nov 2014 00:48:48 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Xo4Ja-0006Pn-8n for qemu-devel@nongnu.org; Tue, 11 Nov 2014 00:48:39 -0500 Received: from e38.co.us.ibm.com ([32.97.110.159]:60616) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Xo4Ja-0006PQ-18 for qemu-devel@nongnu.org; Tue, 11 Nov 2014 00:48:30 -0500 Received: from /spool/local by e38.co.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Mon, 10 Nov 2014 22:48:28 -0700 Message-ID: <5461A315.3040602@linux.vnet.ibm.com> Date: Tue, 11 Nov 2014 11:18:05 +0530 From: Aravinda Prasad MIME-Version: 1.0 References: <20141105071019.26196.93729.stgit@aravindap> <20141105071315.26196.68104.stgit@aravindap> <5459E0B4.3040402@suse.de> <545A464F.7080906@gmail.com> <545B46A1.6060009@linux.vnet.ibm.com> <20141111031923.GG15270@voom.redhat.com> In-Reply-To: <20141111031923.GG15270@voom.redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [Qemu-ppc] [PATCH v3 4/4] target-ppc: Handle ibm, nmi-register RTAS call List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: David Gibson Cc: Tom Musta , benh@au1.ibm.com, aik@au1.ibm.com, qemu-devel@nongnu.org, qemu-ppc@nongnu.org, paulus@samba.org On Tuesday 11 November 2014 08:49 AM, David Gibson wrote: > On Thu, Nov 06, 2014 at 03:30:01PM +0530, Aravinda Prasad wrote: >> On Wednesday 05 November 2014 09:16 PM, Tom Musta wrote: >>> On 11/5/2014 2:32 AM, Alexander Graf wrote: >>>> On 05.11.14 08:13, Aravinda Prasad wrote: > > [snip] >>>>> + /* >>>>> + * ori r3,r3,KVMPPC_H_REPORT_MC_ERR. The KVMPPC_H_REPORT_MC_ERR >>>>> + * value is patched below >>>>> + */ >>>>> +1: ori 3,3,0 >>> >>> Why do "li 3,0" followed by "ori 3,3,X"? Isn't this just "li 3,X" ? (aka "addi 3,0,X") >> >> I remember I first tried doing li r3,X but faced some problem (but not >> able to exactly recall what was the problem) may be due to not familiar >> with ppc assembly. > > This would be because with the offset to the private hcalls, the > actual hcall number is 0xf003, which means an li instruction will sign > extend it incorrectly. So you will need two instructions to load the > number. > hmm.. ok -- Regards, Aravinda