[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 09/40] memory: create FlatView for new address s

From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH 09/40] memory: create FlatView for new address spaces
Date: Wed, 8 May 2013 04:41:59 -0400 (EDT)

> On 7 May 2013 15:16, Paolo Bonzini <address@hidden> wrote:
> > A new address space has an empty FlatView.  In order to create it, we
> > must (a) call memory_region_transaction_commit after the address space is
> > inserted into the list; (b) force memory_region_transaction_commit to
> > do something.
> >
> > This bug was latent so far because all address spaces started empty,
> > including
> > the PCI address space where the bus master region is initially disabled.
> > However, the target address space of an IOMMU is usually rooted at
> > get_system_memory(), which might not be empty at the time the IOMMU is
> > created.
> The code looks ok but I don't entirely understand the
> commit message. Surely we're definitely already creating
> the FlatView -- that's what the g_new and flatview_init do.
> What we're not doing is calling notifiers for the initial
> contents of the address space, right? (and thus in turn
> we're not setting up the data structures those notifiers
> control, but as far as I can see from the code that's not
> the FlatView.)

Right, it should be s/create/populate/g.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]