[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 6/7] vl: Set current_machine early
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 6/7] vl: Set current_machine early |
Date: |
Mon, 19 Aug 2013 11:35:13 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux) |
Andreas Färber <address@hidden> writes:
> Am 16.08.2013 15:18, schrieb address@hidden:
>> From: Markus Armbruster <address@hidden>
>>
>> I'd like to access QEMUMachine from a QEMUMachine init() method, which
>> is currently not possible. Instead of passing it as an argument, I
>> simply set current_machine earlier.
>
> We had such a patch for CPU hot-add and decided against doing this.
> Currently current_machine != signals that it has been initialized. And
Does any code actually depend on this undocumented condition? I found
none.
> generally we have been trying to get away from accessing globals from
> random parts of code.
Global state need to be managed with care. Global variables have their
place in that.
In my experience, the most common kind of carelessness involving global
variables is indisciplined *updating* of global state via global
variables.
Unchanging global state is relatively harmless, and referring to it via
global variables is often the easiest and most obvious way to manage
such state.
> Can't you pass either QEMUMachine or the specific fields needed from PC
> code to those SMBIOS functions? You did add a bool argument.
Can't see how to do that without passing the machine to QEMUMachine
method init(), which involves touching all boards. I doubt that's a
good idea, but if you insist, I can do it.
- [Qemu-devel] [PATCH v2 0/7] smbios cleanup & nicer defaults for type 1, armbru, 2013/08/16
- [Qemu-devel] [PATCH v2 3/7] smbios: Improve diagnostics for conflicting entries, armbru, 2013/08/16
- [Qemu-devel] [PATCH v2 1/7] smbios: Normalize smbios_entry_add()'s error handling to exit(1), armbru, 2013/08/16
- [Qemu-devel] [PATCH v2 2/7] smbios: Convert to QemuOpts, armbru, 2013/08/16
- [Qemu-devel] [PATCH v2 4/7] smbios: Make multiple -smbios type= accumulate sanely, armbru, 2013/08/16
- [Qemu-devel] [PATCH v2 6/7] vl: Set current_machine early, armbru, 2013/08/16
[Qemu-devel] [PATCH v2 5/7] smbios: Factor out smbios_maybe_add_str(), armbru, 2013/08/16
[Qemu-devel] [PATCH v2 7/7] smbios: Set system manufacturer, product & version by default, armbru, 2013/08/16
Re: [Qemu-devel] [PATCH v2 0/7] smbios cleanup & nicer defaults for type 1, Laszlo Ersek, 2013/08/17