On 02/16/2015 01:51 PM, Peter Krempa wrote: > Shuffling around the logic will allow to simplify the code quite a bit. > As an additional bonus the change in the logic now reports an error if > automatic placement is selected and individual placement is configured. > --- > src/conf/numa_conf.c | 53 +++++++++++++++++++++------------------------------- > 1 file changed, 21 insertions(+), 32 deletions(-) > hmm... perhaps one extra thing that could be done... Now that the last/5th param to virDomainNumatuneSet isn't NULL for any caller (and the 4th isn't -1) - it looks like to me you can also modify numa_conf.h to add a ATTRIBUTE_NONNULL(5) for the function... ACK John -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list