From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:55139) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S4I5j-0005bU-Sl for qemu-devel@nongnu.org; Sun, 04 Mar 2012 15:31:41 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1S4I5h-0006HL-Un for qemu-devel@nongnu.org; Sun, 04 Mar 2012 15:31:39 -0500 Received: from cantor2.suse.de ([195.135.220.15]:40875 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1S4I5h-0006H8-Kx for qemu-devel@nongnu.org; Sun, 04 Mar 2012 15:31:37 -0500 Message-ID: <4F53D127.3030206@suse.de> Date: Sun, 04 Mar 2012 21:31:35 +0100 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <1330043012-30556-1-git-send-email-david@gibson.dropbear.id.au> <1330043012-30556-7-git-send-email-david@gibson.dropbear.id.au> <4F4CC96B.6040003@redhat.com> <1330465707.11728.29.camel@pasglop> <4F5348CE.8070909@redhat.com> <1330861999.11728.79.camel@pasglop> <4F539C4E.2090304@suse.de> <7C5C662C-6D69-459A-90EA-246596648796@suse.de> <4F53CEB0.7030406@suse.de> <2BF15430-E518-4949-ABB7-D1095DE40340@suse.de> In-Reply-To: <2BF15430-E518-4949-ABB7-D1095DE40340@suse.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 6/6] kvm: Fix dirty tracking with large kernel page size List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Alexander Graf Cc: Marcelo Tosatti , "qemu-devel@nongnu.org" , Avi Kivity , "anthony@codemonkey.ws" , David Gibson Am 04.03.2012 21:25, schrieb Alexander Graf: >=20 >=20 > On 04.03.2012, at 21:21, Andreas F=C3=A4rber wrote: >=20 >> Am 04.03.2012 19:46, schrieb Alexander Graf: >>> >>> >>> On 04.03.2012, at 17:46, Andreas F=C3=A4rber wrote= : >>> >>>> Am 04.03.2012 12:53, schrieb Benjamin Herrenschmidt: >>>>> On Sun, 2012-03-04 at 12:49 +0200, Avi Kivity wrote: >>>>>> On 02/28/2012 11:48 PM, Benjamin Herrenschmidt wrote: >>>>>>> On Tue, 2012-02-28 at 14:32 +0200, Avi Kivity wrote: >>>>>>> >>>>>>>> What if TARGET_PAGE_SIZE > getpagesize()? Or is that impossible= ? >>>>>>> >>>>>>> We have yet to encounter such a case. It's not currently possible= on >>>>>>> power (some old embedded chips could do 1K and 2K page sizes in t= he TLB >>>>>>> iirc but we never supported that in Linux and it's being phased o= ut in >>>>>>> HW). >>>>>>> >>>>>>> I suggest that gets dealt with when/if it needs to, which means p= robably >>>>>>> never :-) >>>>>> >>>>>> Doesn't ppc support both 4k and 64k pages? Suppose you run a 4k g= uest >>>>>> on a 64k host? >>>>>> >>>>>> Maybe I'm misremembering or misunderstanding something. >>>> >>>>> TARGET_PAGE_SIZE in qemu is always 4k for powerpc, it's a compile t= ime >>>>> #define. >>>> >>>> Except for ppcemb-softmmu (1k), which is irrelevant for KVM AFAIU. >>>> >>>> Maybe just add an assert and be done with it? >>> >>> Assert for what? Linux page size of 64k is something perfectly normal= on ppc. The hardware can always do at least 4k maps however. >> >> g_assert(TARGET_PAGE_SIZE <=3D getpagesize()) >> >> Just declare the above case as unsupported and abort if we encounter i= t. >=20 > What I'm trying to tell you is that it's the default case on book3s ppc= ! ;) Exactly, which is why I'm saying just ignore the weird embedded case. :) Andreas --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3=BC= rnberg