From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1O63fG-0002lN-QV for qemu-devel@nongnu.org; Sun, 25 Apr 2010 11:22:34 -0400 Received: from [140.186.70.92] (port=50238 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1O63fE-0002kv-Ro for qemu-devel@nongnu.org; Sun, 25 Apr 2010 11:22:34 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1O63fB-0006Pq-N5 for qemu-devel@nongnu.org; Sun, 25 Apr 2010 11:22:31 -0400 Received: from mx1.redhat.com ([209.132.183.28]:31814) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1O63fB-0006PZ-Fh for qemu-devel@nongnu.org; Sun, 25 Apr 2010 11:22:29 -0400 Message-ID: <4BD45E32.1070406@redhat.com> Date: Sun, 25 Apr 2010 18:22:26 +0300 From: Avi Kivity MIME-Version: 1.0 References: <20100423170410.914857113@amt.cnet> <20100423170645.675040544@amt.cnet> <4BD29F22.8020806@web.de> <4BD4367F.5060307@redhat.com> <4BD44A4D.4060008@web.de> <4BD44F13.3070000@redhat.com> <4BD451D9.4090209@web.de> <4BD4547C.5060907@redhat.com> <4BD45709.9070705@web.de> <4BD458AD.9020500@redhat.com> <4BD45A96.3080800@web.de> In-Reply-To: <4BD45A96.3080800@web.de> Content-Type: text/plain; charset=ISO-8859-15; format=flowed Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: [uq/master patch 2/5] kvm: add logging count to slots List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jan Kiszka Cc: Marcelo Tosatti , qemu-devel@nongnu.org, kvm@vger.kernel.org On 04/25/2010 06:07 PM, Jan Kiszka wrote: > >> The fact that the API assumes a single user is what's broken IMO. >> >> If the API were to take a memory slot as parameter you could say it is >> the responsibility of the slot's owner to multiplex (and since vga has a >> single owner, no need to multiplex). But it takes a range. >> > No, the API accepts only a single slot. If you try passing bogus ranges > that span multiple or incomplete slots, you get what you deserve - a bug > message. > I see. In its qemu-kvm iteration, it would iterate over slots and accept incomplete slots (it's okay to log more than requested). If the API is for a slot, it should accept a slot, not a range (when we move to a slots representation in qemu). Unrelated: cpu_physical_sync_dirty_bitmap(isa_mem_base + 0xa0000, 0xa8000); cpu_physical_sync_dirty_bitmap(isa_mem_base + 0xa8000, 0xb0000); Will this sync to the right place (whatever those windows alias to)? -- error compiling committee.c: too many arguments to function