Re: [Qemu-devel] [PATCH] numa: fixup parsed NumaNodeOptions earlier

2017-11-09 Thread Markus Armbruster
Igor Mammedov writes: > On Thu, 12 Oct 2017 12:43:46 -0300 > Eduardo Habkost wrote: > >> On Thu, Oct 12, 2017 at 11:39:58AM +0200, Igor Mammedov wrote: >> > numa 'mem' option with suffix or without one is possible >> > only on CLI/HMP. Instead of fixing up special suffix less >> > CLI case deep

Re: [Qemu-devel] [PATCH] numa: fixup parsed NumaNodeOptions earlier

2017-10-12 Thread Igor Mammedov
On Thu, 12 Oct 2017 12:43:46 -0300 Eduardo Habkost wrote: > On Thu, Oct 12, 2017 at 11:39:58AM +0200, Igor Mammedov wrote: > > numa 'mem' option with suffix or without one is possible > > only on CLI/HMP. Instead of fixing up special suffix less > > CLI case deep in parse_numa_node() do it earlie

Re: [Qemu-devel] [PATCH] numa: fixup parsed NumaNodeOptions earlier

2017-10-12 Thread Eduardo Habkost
On Thu, Oct 12, 2017 at 11:39:58AM +0200, Igor Mammedov wrote: > numa 'mem' option with suffix or without one is possible > only on CLI/HMP. Instead of fixing up special suffix less > CLI case deep in parse_numa_node() do it earlier right > after option is parsed into NumaNodeOptions with OptVisist

[Qemu-devel] [PATCH] numa: fixup parsed NumaNodeOptions earlier

2017-10-12 Thread Igor Mammedov
numa 'mem' option with suffix or without one is possible only on CLI/HMP. Instead of fixing up special suffix less CLI case deep in parse_numa_node() do it earlier right after option is parsed into NumaNodeOptions with OptVisistor so that the rest of the code would use valid values in NumaNodeOptio