[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 00/10] Cleanups around the 'current_machine' global variab
|
From: |
Markus Armbruster |
|
Subject: |
Re: [PATCH v2 00/10] Cleanups around the 'current_machine' global variable |
|
Date: |
Tue, 21 Jan 2020 13:47:42 +0100 |
|
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Philippe Mathieu-Daudé <address@hidden> writes:
> v1 was "Replace current_machine by qdev_get_machine()":
> https://www.mail-archive.com/address@hidden/msg669611.html
>
> But Markus objected, see:
> https://www.mail-archive.com/address@hidden/msg670122.html
> and older discussion:
> https://www.mail-archive.com/address@hidden/msg611338.html
>
> This series salvage patches from v1, and add other trivial cleanups.
>
> Can the ARM/PPC/S390 patches could go via their own tree, and the
> rest via Paolo's 'misc' tree?
>
> Supersedes: <address@hidden>
Separating off uncontroversial patches is always a good idea. These all
look okay to me on first glance, with one minor remark on PATCH 7. I
think you got or are likely to get competent review for all of them. If
you need me to have a closer look, just ask.
Thanks!
- Re: [PATCH v2 06/10] qom/object: Display more helpful message when a parent is missing, (continued)
- [PATCH v2 07/10] qdev: Abort if the root machine container is missing, Philippe Mathieu-Daudé, 2020/01/21
- [PATCH v2 08/10] accel: Introduce the current_accel() wrapper, Philippe Mathieu-Daudé, 2020/01/21
- [PATCH v2 09/10] accel: Replace current_machine->accelerator by current_accel() wrapper, Philippe Mathieu-Daudé, 2020/01/21
- [PATCH v2 10/10] accel/tcg: Sanitize include path, Philippe Mathieu-Daudé, 2020/01/21
- Re: [PATCH v2 00/10] Cleanups around the 'current_machine' global variable,
Markus Armbruster <=
- Re: [PATCH v2 00/10] Cleanups around the 'current_machine' global variable, Paolo Bonzini, 2020/01/21