From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:52936) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QN4PJ-0004kj-Hg for qemu-devel@nongnu.org; Thu, 19 May 2011 10:41:01 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QN4PI-0001rW-Mw for qemu-devel@nongnu.org; Thu, 19 May 2011 10:40:57 -0400 Received: from mx1.redhat.com ([209.132.183.28]:5719) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QN4PI-0001r9-Cp for qemu-devel@nongnu.org; Thu, 19 May 2011 10:40:56 -0400 Message-ID: <4DD52BF2.2080506@redhat.com> Date: Thu, 19 May 2011 17:40:50 +0300 From: Avi Kivity MIME-Version: 1.0 References: <4DD3D236.90708@siemens.com> <4DD3D95E.2060105@redhat.com> <4DD3E1B3.3020405@siemens.com> <4DD3E47F.9060104@redhat.com> <4DD3E782.8090208@siemens.com> <4DD3E8D6.6090807@redhat.com> <20110519090851.GD28399@redhat.com> <4DD4DE8E.8030402@redhat.com> <20110519091404.GE28399@redhat.com> <4DD5029D.6000700@redhat.com> <20110519115405.GG28399@redhat.com> <4DD505C4.6010604@redhat.com> <4DD50B17.7000205@siemens.com> <4DD511FB.3080901@redhat.com> <4DD51413.1050202@siemens.com> <4DD51468.7050509@redhat.com> <4DD51531.7000701@siemens.com> <4DD515F9.1020902@redhat.com> <4DD51A82.7060205@siemens.com> <4DD51B64.8000306@redhat.com> <4DD51FDA.3010107@codemonkey.ws> <4DD520ED.8010606@redhat.com> <4DD5260A.1080309@codemonkey.ws> <4DD5272F.5000003@siemens.com> <4DD52848.6030102@codemonkey.ws> <4DD52910.4080106@siemens.com> <4DD52B0E.2080604@codemonkey.ws> In-Reply-To: <4DD52B0E.2080604@codemonkey.ws> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [RFC] Memory API List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: Jan Kiszka , qemu-devel , Gleb Natapov On 05/19/2011 05:37 PM, Anthony Liguori wrote: > > So.... do you do: > > isa_register_region(ISABus *bus, MemoryRegion *mr, int priority) > { > chipset_register_region(bus->chipset, mr, priority + 1); > } > > I don't really understand how you can fold everything into one table > and not allow devices to override their parents using priorities. Think of how a window manager folds windows with priorities onto a flat framebuffer. You do a depth-first walk of the tree. For each child list, you iterate it from the lowest to highest priority, allowing later subregions override earlier subregions. -- error compiling committee.c: too many arguments to function