From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33059) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X3h48-0006xn-0T for qemu-devel@nongnu.org; Sun, 06 Jul 2014 03:40:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1X3h41-00027N-Rd for qemu-devel@nongnu.org; Sun, 06 Jul 2014 03:40:51 -0400 Received: from mx1.redhat.com ([209.132.183.28]:14545) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1X3h41-00027H-JK for qemu-devel@nongnu.org; Sun, 06 Jul 2014 03:40:45 -0400 Date: Sun, 6 Jul 2014 09:42:42 +0300 From: "Michael S. Tsirkin" Message-ID: <20140706064242.GA28502@redhat.com> References: <53B54D98.2060800@redhat.com> <20140704025613.GC31329@G08FNSTD100614.fnst.cn.fujitsu.com> <53B62824.2010209@redhat.com> <20140704074320.GB2176@G08FNSTD100614.fnst.cn.fujitsu.com> <53B65C1F.5030304@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <53B65C1F.5030304@redhat.com> Subject: Re: [Qemu-devel] [PATCH for 2.1 2/2] memory-backend-file: improve error handling List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: Hu Tao , Igor Mammedov , qemu-devel@nongnu.org, Yasunori Goto On Fri, Jul 04, 2014 at 09:47:43AM +0200, Paolo Bonzini wrote: > Il 04/07/2014 09:43, Hu Tao ha scritto: > >The memory size is rounded up to hpagesize when allocating memory, we > >can waste at most hpagesize-1 bytes memory. I don't think it's a > >problem. > > > >In the other side, do you think we should give an error in the case like > >-object memory-backend-ram,size=1111M(given 2M hpagesize)? > > I think for gb pages it can make sense to waste a little memory (or even not > so little, like if you have a 2.5 GB guest). > > Paolo For 2.1, all we can take at this point is obvious bugfixes. Propagating error up the stack instead of aborting might be fine if it's ready before rc1, that is Monday at the latest. Afterwards, I'll only merge high priority bugfixes, improving handling of user errors doesn't count as there's an easy workaround. Whether rounding up to page size is a good idea or not, I'm inclined to say this is 2.2 material. -- MST