From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ryan Harper Subject: Re: Remove qemu_alloc_physram() Date: Thu, 28 May 2009 16:53:52 -0500 Message-ID: <20090528215352.GG11675@us.ibm.com> References: <2803e73b0905270308l70cf562jaca72bf67d1aafff@mail.gmail.com> <4A1D3711.2070603@redhat.com> <2803e73b0905270630t5579f141lcb1013ae9aa146a9@mail.gmail.com> <2803e73b0905270640n56d9bfa3x15c0ecabd0b4676f@mail.gmail.com> <4A1D4739.4070208@redhat.com> <4A1D68AC.3050300@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: nicolas prochazka , kvm@vger.kernel.org To: Avi Kivity Return-path: Received: from e34.co.us.ibm.com ([32.97.110.152]:38932 "EHLO e34.co.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751021AbZE1Vx6 (ORCPT ); Thu, 28 May 2009 17:53:58 -0400 Received: from d03relay02.boulder.ibm.com (d03relay02.boulder.ibm.com [9.17.195.227]) by e34.co.us.ibm.com (8.13.1/8.13.1) with ESMTP id n4SLp9HX004633 for ; Thu, 28 May 2009 15:51:09 -0600 Received: from d03av03.boulder.ibm.com (d03av03.boulder.ibm.com [9.17.195.169]) by d03relay02.boulder.ibm.com (8.13.8/8.13.8/NCO v9.2) with ESMTP id n4SLruX5206960 for ; Thu, 28 May 2009 15:53:58 -0600 Received: from d03av03.boulder.ibm.com (loopback [127.0.0.1]) by d03av03.boulder.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id n4SLru3x024505 for ; Thu, 28 May 2009 15:53:56 -0600 Content-Disposition: inline In-Reply-To: <4A1D68AC.3050300@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: * Avi Kivity [2009-05-27 11:23]: > Avi Kivity wrote: > >nicolas prochazka wrote: > >>without -mem-prealloc > >>HugePages_Total: 2560 > >>HugePages_Free: 2296 > >>HugePages_Rsvd: 0 > >> > >>so after minimum test, i can say that's your patch seems to be correct > >>this problem. > >> > > > >It isn't correct, it doesn't generate the right alignment. > > > Better patch attached. This patch restores -mempath working for me on latest qemu-kvm.git. btw, why'd we go from -mem-path (kvm-84/stable) to -mempath (kvm-85 and newer)? Changing these breaks existing scripts. Not a big deal, but wondering what was the motivation for the change. -- Ryan Harper Software Engineer; Linux Technology Center IBM Corp., Austin, Tx ryanh@us.ibm.com