[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory policy
From: |
Wanlong Gao |
Subject: |
Re: [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory policy |
Date: |
Tue, 18 Jun 2013 17:54:24 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 |
On 06/18/2013 05:20 PM, Paolo Bonzini wrote:
> Il 18/06/2013 10:09, Wanlong Gao ha scritto:
>> +static unsigned int numa_node_parse_mpol(const char *str, unsigned long *bm)
>> +{
>> + unsigned long long value, endvalue;
>> + char *endptr;
>> + unsigned int flags = 0;
>> +
>> + if (str[0] == '!') {
>> + flags |= 2;
>
> clear = true;
>
>> + bitmap_fill(bm, MAX_CPUMASK_BITS);
>> + str++;
>> + }
>> + if (str[0] == '+') {
>> + flags |= 1;
>
> flags = NODE_HOST_RELATIVE
>
>> + str++;
>> + }
>> +
>> + if (!strcmp(str, "all")) {
>> + bitmap_fill(bm, MAX_CPUMASK_BITS);
>> + return flags;
>> + }
>> +
>> + if (parse_uint(str, &value, &endptr, 10) < 0)
>> + goto error;
>> + if (*endptr == '-') {
>> + if (parse_uint_full(endptr + 1, &endvalue, 10) < 0) {
>> + goto error;
>> + }
>> + } else if (*endptr == '\0') {
>> + endvalue = value;
>> + } else {
>> + goto error;
>> + }
>> +
>> + if (endvalue >= MAX_CPUMASK_BITS) {
>> + endvalue = MAX_CPUMASK_BITS - 1;
>> + fprintf(stderr,
>> + "qemu: NUMA: A max of %d host nodes are supported\n",
>> + MAX_CPUMASK_BITS);
>> + }
>> +
>> + if (endvalue < value) {
>> + goto error;
>> + }
>> +
>> + if (flags & 2)
>
> if (clear)
>
>> + bitmap_clear(bm, value, endvalue - value + 1);
>> + else
>> + bitmap_set(bm, value, endvalue - value + 1);
>> +
>> + return flags;
>> +
>> +error:
>> + fprintf(stderr, "qemu: Invalid host NUMA nodes range: %s\n", str);
>
> Please change the functions (numa_add and numa_node_parse_mpol) to
> accept an Error *. This will make it much easier to reuse them for e.g.
> memory hotplug in the future.
Got it, I'll try. Thank you.
>
>> + return 4;
>
> return -EINVAL;
>
>> +}
>
>> + if (get_param_value(option, 128, "interleave", optarg) != 0)
>> + numa_info[nodenr].flags |= NODE_HOST_INTERLEAVE;
>> + else if (get_param_value(option, 128, "preferred", optarg) != 0)
>> + numa_info[nodenr].flags |= NODE_HOST_PREFERRED;
>> + else if (get_param_value(option, 128, "membind", optarg) != 0)
>> + numa_info[nodenr].flags |= NODE_HOST_BIND;
>
> You're not handling the case where someone specifies more than one option.
>
> What about:
>
> policy={interleave,preferred,bind},mem-hostnode=0
>
> ?
OK, will follow this, thank you.
>
> Also, please use QemuOpts instead of yet another homegrown parser.
> Eduardo, I think you had the most recent attempt to convert -numa to
> QemuOpts?
So, any patches I can based on or change it myself? Eduardo?
Thanks,
Wanlong Gao
>
>> + if (option[0] != 0) {
>> + ret = numa_node_parse_mpol(option, numa_info[nodenr].host_mem);
>> + if (ret == 4) {
>
> if (ret < 0)
>
>> + exit(1);
>> + } else if (ret & 1) {
>> + numa_info[nodenr].flags |= NODE_HOST_RELATIVE;
>
> else {
> numa_info[nodenr].flags |= ret;
> }
>
>> + }
>> + }
>> +
>
> Paolo
>
- [Qemu-devel] [PATCH 0/7] Add support for binding guest numa nodes to host numa nodes, Wanlong Gao, 2013/06/18
- [Qemu-devel] [PATCH 4/7] NUMA: set guest numa nodes memory policy, Wanlong Gao, 2013/06/18
- [Qemu-devel] [PATCH 1/7] Add numa_info structure to contain numa nodes info, Wanlong Gao, 2013/06/18
- [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory policy, Wanlong Gao, 2013/06/18
- Re: [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory policy, Paolo Bonzini, 2013/06/18
- Re: [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory policy,
Wanlong Gao <=
- Re: [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory policy, Eduardo Habkost, 2013/06/18
- Re: [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory policy, Bandan Das, 2013/06/18
- Re: [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory policy, Wanlong Gao, 2013/06/19
- Re: [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory policy, Paolo Bonzini, 2013/06/19
- Re: [Qemu-devel] [PATCH 3/7] NUMA: parse guest numa nodes memory policy, Wanlong Gao, 2013/06/19
[Qemu-devel] [PATCH 6/7] NUMA: add hmp command set-mpol, Wanlong Gao, 2013/06/18
[Qemu-devel] [PATCH 7/7] NUMA: show host memory policy info in info numa command, Wanlong Gao, 2013/06/18
[Qemu-devel] [PATCH 5/7] NUMA: add qmp command set-mpol to set memory policy for NUMA node, Wanlong Gao, 2013/06/18