On 11/25/2012 12:45 PM, Jan Kiszka wrote:
> On 2012-11-25 11:18, Avi Kivity wrote:
>> On 11/05/2012 02:37 PM, Jan Kiszka wrote:
As I noted, init and destroy cannot cause a topology update.
>>>
>>> Ah, right. Why are we wrapping them in transaction_begin/commit at all then?
>>>
>>
>> We a
On 2012-11-25 11:18, Avi Kivity wrote:
> On 11/05/2012 02:37 PM, Jan Kiszka wrote:
>>>
>>> As I noted, init and destroy cannot cause a topology update.
>>
>> Ah, right. Why are we wrapping them in transaction_begin/commit at all then?
>>
>
> We aren't.
>
>
> void memory_region_destroy(MemoryRegi
On 11/05/2012 02:37 PM, Jan Kiszka wrote:
>>
>> As I noted, init and destroy cannot cause a topology update.
>
> Ah, right. Why are we wrapping them in transaction_begin/commit at all then?
>
We aren't.
void memory_region_destroy(MemoryRegion *mr)
{
assert(QTAILQ_EMPTY(&mr->subregions));
On 2012-11-05 13:33, Avi Kivity wrote:
> On 11/05/2012 10:51 AM, Jan Kiszka wrote:
>> On 2012-11-05 09:12, Avi Kivity wrote:
>>> On 11/05/2012 08:26 AM, Jan Kiszka wrote:
On 2012-11-04 20:21, Avi Kivity wrote:
> On 11/04/2012 10:30 AM, Jan Kiszka wrote:
>> From: Jan Kiszka
>>
On 11/05/2012 10:51 AM, Jan Kiszka wrote:
> On 2012-11-05 09:12, Avi Kivity wrote:
> > On 11/05/2012 08:26 AM, Jan Kiszka wrote:
> >> On 2012-11-04 20:21, Avi Kivity wrote:
> >>> On 11/04/2012 10:30 AM, Jan Kiszka wrote:
> From: Jan Kiszka
>
> Cirrus is triggering this, e.g. during
On 2012-11-05 09:12, Avi Kivity wrote:
> On 11/05/2012 08:26 AM, Jan Kiszka wrote:
>> On 2012-11-04 20:21, Avi Kivity wrote:
>>> On 11/04/2012 10:30 AM, Jan Kiszka wrote:
From: Jan Kiszka
Cirrus is triggering this, e.g. during Win2k boot: Changes only on
disabled regions requir
On 11/05/2012 08:26 AM, Jan Kiszka wrote:
> On 2012-11-04 20:21, Avi Kivity wrote:
> > On 11/04/2012 10:30 AM, Jan Kiszka wrote:
> >> From: Jan Kiszka
> >>
> >> Cirrus is triggering this, e.g. during Win2k boot: Changes only on
> >> disabled regions require no topology update when transaction dept
On 2012-11-04 20:21, Avi Kivity wrote:
> On 11/04/2012 10:30 AM, Jan Kiszka wrote:
>> From: Jan Kiszka
>>
>> Cirrus is triggering this, e.g. during Win2k boot: Changes only on
>> disabled regions require no topology update when transaction depth drops
>> to 0 again.
>
> 817dcc5368988b0 (pci: give
On 11/04/2012 10:30 AM, Jan Kiszka wrote:
> From: Jan Kiszka
>
> Cirrus is triggering this, e.g. during Win2k boot: Changes only on
> disabled regions require no topology update when transaction depth drops
> to 0 again.
817dcc5368988b0 (pci: give each device its own address space) mad this
much
From: Jan Kiszka
Cirrus is triggering this, e.g. during Win2k boot: Changes only on
disabled regions require no topology update when transaction depth drops
to 0 again.
Signed-off-by: Jan Kiszka
---
memory.c | 17 +++--
1 files changed, 15 insertions(+), 2 deletions(-)
diff --gi
10 matches
Mail list logo