[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 02/86] machine: introduce ram-memdev property
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH v2 02/86] machine: introduce ram-memdev property |
Date: |
Wed, 15 Jan 2020 16:56:30 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.1 |
On 15/01/20 16:06, Igor Mammedov wrote:
>
> + object_property_add_link(obj, "ram-memdev", TYPE_MEMORY_BACKEND,
> + (Object **)&ms->ram_memdev,
> + object_property_allow_set_link,
> + OBJ_PROP_LINK_STRONG, &error_abort);
> + object_property_set_description(obj, "ram-memdev",
> + "Set RAM backend"
> + "Valid value is ID of hostmem based
> backend",
> + &error_abort);
> +
Obligatory bikeshedding, why not just ram (the MachineState field can
remain "ram_memdev"). Or memory-backend matching the QOM type names.
Paolo
- [PATCH v2 00/86] refactor main RAM allocation to use hostmem backend, Igor Mammedov, 2020/01/15
- [PATCH v2 03/86] machine: alias -mem-path and -mem-prealloc into memory-foo backend, Igor Mammedov, 2020/01/15
- [PATCH v2 04/86] machine: introduce convenience MachineState::ram, Igor Mammedov, 2020/01/15
- [PATCH v2 05/86] initialize MachineState::ram in NUMA case, Igor Mammedov, 2020/01/15
- [PATCH v2 06/86] alpha:dp264: use memdev for RAM, Igor Mammedov, 2020/01/15
- [PATCH v2 08/86] arm:aspeed: actually check RAM size, Igor Mammedov, 2020/01/15