Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753487Ab0ASG4N (ORCPT ); Tue, 19 Jan 2010 01:56:13 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752922Ab0ASG4M (ORCPT ); Tue, 19 Jan 2010 01:56:12 -0500 Received: from mx1.redhat.com ([209.132.183.28]:56508 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752332Ab0ASG4L (ORCPT ); Tue, 19 Jan 2010 01:56:11 -0500 Date: Tue, 19 Jan 2010 08:55:37 +0200 From: Gleb Natapov To: "H. Peter Anvin" Cc: Peter Zijlstra , kvm@vger.kernel.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org, avi@redhat.com, mingo@elte.hu, tglx@linutronix.de, riel@redhat.com, cl@linux-foundation.org Subject: Re: [PATCH v3 04/12] Add "handle page fault" PV helper. Message-ID: <20100119065537.GF14345@redhat.com> References: <1262700774-1808-1-git-send-email-gleb@redhat.com> <1262700774-1808-5-git-send-email-gleb@redhat.com> <1263490267.4244.340.camel@laptop> <20100117144411.GI31692@redhat.com> <4B541D08.9040802@zytor.com> <20100118085022.GA30698@redhat.com> <4B5510B1.9010202@zytor.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <4B5510B1.9010202@zytor.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2067 Lines: 43 On Mon, Jan 18, 2010 at 05:53:53PM -0800, H. Peter Anvin wrote: > On 01/18/2010 12:50 AM, Gleb Natapov wrote: > > On Mon, Jan 18, 2010 at 12:34:16AM -0800, H. Peter Anvin wrote: > >> On 01/17/2010 06:44 AM, Gleb Natapov wrote: > >>> On Thu, Jan 14, 2010 at 06:31:07PM +0100, Peter Zijlstra wrote: > >>>> On Tue, 2010-01-05 at 16:12 +0200, Gleb Natapov wrote: > >>>>> Allow paravirtualized guest to do special handling for some page faults. > >>>>> > >>>>> The patch adds one 'if' to do_page_fault() function. The call is patched > >>>>> out when running on physical HW. I ran kernbech on the kernel with and > >>>>> without that additional 'if' and result were rawly the same: > >>>> > >>>> So why not program a different handler address for the #PF/#GP faults > >>>> and avoid the if all together? > >>> I would gladly use fault vector reserved by x86 architecture, but I am > >>> not sure Intel will be happy about it. > >>> > >> > >> That's what it's there for... see Peter Z.'s response. > >> > > Do you mean I can use one of exception vectors reserved by Intel > > (20-31)? What Peter Z says is that I can register my own handler for > > #PF and avoid the 'if' in non PV case as far as I understand him. > > > > What I mean is that vector 14 is page faults -- that's what it is all > about. Why on Earth do you need another vector? > Because this is not usual pagefault that tell the OS that page is not mapped. This is a notification to a guest OS that the page it is trying to access is swapped out by the host OS. There is nothing guest can do about it except schedule another task. So the guest should handle both type of exceptions: usual #PF when page is not mapped by the guest and new type of notifications. Ideally we would use one of unused exception vectors for new type of notifications. -- Gleb. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/