[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] numa: check for busy memory backend
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] numa: check for busy memory backend |
Date: |
Wed, 25 Jun 2014 13:48:05 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Igor Mammedov <address@hidden> writes:
> On Wed, 25 Jun 2014 17:04:14 +0800
> Hu Tao <address@hidden> wrote:
>
>> ..to prevent one memory backend from being used by more than one numa
>> node.
>>
>> Signed-off-by: Hu Tao <address@hidden>
>> ---
>> numa.c | 7 +++++++
>> 1 file changed, 7 insertions(+)
>>
>> diff --git a/numa.c b/numa.c
>> index e471afe..6c1c554 100644
>> --- a/numa.c
>> +++ b/numa.c
>> @@ -279,6 +279,13 @@ void memory_region_allocate_system_memory(MemoryRegion
>> *mr, Object *owner,
>> exit(1);
>> }
>>
>> + if (memory_region_is_mapped(seg)) {
>> + char *path =
>> object_get_canonical_path_component(OBJECT(backend));
>> + error_report("memory backend %s is busy", path);
>> + g_free(path);
>> + exit(1);
> s/1/EXIT_FAILURE/ please
I count >600 instances of exit() with a numeric argument (not counting
some 50 sys.exit() in Python code), but less than 40 using EXIT_SUCCESS
or EXIT_FAILURE.
The abstraction provided by EXIT_SUCCESS / EXIT_FAILURE is basically
worthless anyway.
>
>> + }
>> +
>> memory_region_add_subregion(mr, addr, seg);
>> vmstate_register_ram_global(seg);
>> addr += size;
- [Qemu-devel] [PATCH] numa: check for busy memory backend, Hu Tao, 2014/06/25
- Re: [Qemu-devel] [PATCH] numa: check for busy memory backend, Igor Mammedov, 2014/06/25
- Re: [Qemu-devel] [PATCH] numa: check for busy memory backend,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH] numa: check for busy memory backend, Michael S. Tsirkin, 2014/06/29
- Re: [Qemu-devel] [PATCH] numa: check for busy memory backend, Hu Tao, 2014/06/30
- Re: [Qemu-devel] [PATCH] numa: check for busy memory backend, Michael S. Tsirkin, 2014/06/30
- Re: [Qemu-devel] [PATCH] numa: check for busy memory backend, Hu Tao, 2014/06/30
- Re: [Qemu-devel] [PATCH] numa: check for busy memory backend, Michael S. Tsirkin, 2014/06/30
- Re: [Qemu-devel] [PATCH] numa: check for busy memory backend, Igor Mammedov, 2014/06/30
- Re: [Qemu-devel] [PATCH] numa: check for busy memory backend, Michael S. Tsirkin, 2014/06/30
- Re: [Qemu-devel] [PATCH] numa: check for busy memory backend, Hu Tao, 2014/06/30