[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] qdev: Keep global allocation counter per bus
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2] qdev: Keep global allocation counter per bus |
Date: |
Wed, 05 Feb 2014 09:45:22 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux) |
Peter Crosthwaite <address@hidden> writes:
> On Tue, Feb 4, 2014 at 7:28 PM, Markus Armbruster <address@hidden> wrote:
>> Markus Armbruster <address@hidden> writes:
>>
>>> Peter Crosthwaite <address@hidden> writes:
>>>
>>>> On Wed, Jan 8, 2014 at 11:47 PM, Paolo Bonzini <address@hidden> wrote:
>>>>> Il 08/01/2014 14:40, Andreas Färber ha scritto:
>>>>>> > Either you fix "info qtree" to cope with your change to the device
>>>>>> > graph, or the change needs to be reverted until somebody fixes it or it
>>>>>> > goes away.
>>>>>> Sharing a backtrace would be a start, rather than just throwing around
>>>>>> the word "crash" to justify reverting patches. :)
>>>>>
>>>>> I mentioned the root cause in the previous message: a Device cannot be
>>>>> added to main_system_bus, but that's what the patch does. The fix isn't
>>>>> trivial, because most of the affected board are not even qdevified.
>>>>>
>>>>
>>>> So I made progress here with the needed QOMification. Finally I have a
>>>> sane info qtree WRT NAND:
>>>>
>>>> $ arm-softmmu/qemu-system-arm -M spitz -nographic -S
>>>> (qemu) info qtree
>>>> bus: main-system-bus
>>>> type System
>>>> ...
>>>> dev: sl-nand, id ""
>>>> manf_id = 236
>>>> chip_id = 115
>>>> irq 0
>>>> mmio 000000000c000000/0000000000000040
>>>> bus: nand
>>>> type nand-bus
>>>> dev: nand, id ""
>>>> manufacturer_id = 236
>>>> chip_id = 115
>>>> drive = <null>
>>>>
>>>> With just the proposed revert info qtree does work again, but is bogus:
>>>>
>>>> (qemu) info qtree
>>>> bus: main-system-bus
>>>> type System
>>>> ...
>>>> dev: nand, id ""
>>>> manufacturer_id = 236
>>>> chip_id = 115
>>>> drive = <null>
>>>> irq 0
>>>> dev: sl-nand, id ""
>>>> manf_id = 236
>>>> chip_id = 115
>>>> irq 0
>>>> mmio 000000000c000000/0000000000000040
>>>
>>> Progress!
>>>
>>>> Patches sometime next week hopefully.
>>>
>>> I think we can wait that long :)
>>
>> I just ran into the info qtree crash again, and I can't find your fix
>> right now. Got a pointer for me?
>>
>
> Hi Markus,
>
> That series got very big on me with complications. I think near term
> we just proceed with the revert. Sorry for the delay.
>
> For a straight revert of my patch nand s/SYSBUS/DEVICE patch:
>
> Acked-by: Peter Crosthwaite <address@hidden>
Done. Thanks!