From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: [PATCH v2] xen: handle paged gfn in wrmsr_hypervisor_regs Date: Fri, 03 May 2013 16:55:15 +0100 Message-ID: References: <20130503151913.GA29247@aepfle.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20130503151913.GA29247@aepfle.de> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Olaf Hering , Jan Beulich Cc: xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On 03/05/2013 16:19, "Olaf Hering" wrote: > On Fri, May 03, Jan Beulich wrote: > >>>>> On 03.05.13 at 16:26, Keir Fraser wrote: >>> I suggest the following return codes: >>> 0: not handled >>> 1: handled >>> -EINVAL: error during handling >>> -EAGAIN: retry >>> >>> The HVM callers should then handle as follows: >>> -EAGAIN: rc = X86EMUL_RETRY >>> -EINVAL: goto gp_fault >>> 0: try other msr handlers (if any) >>> 1: we're done, return X86EMUL_OKAY >>> >>> Does that make sense? >> >> Sure - you may have seen my later reply where I also notide >> this mistake in my first response to Olaf. > > I sent v3 of the patch. I had to guess which 'return 0' is supposed to > be an error. You guessed correctly ;) > Olaf