Later in this series the function will work on a newly introduced virCPUx86Signatures structure. Let's move it to the place were all related functions will be added and rename the function as virCPUx86SignaturesMatch for easier review of the virCPUx86Signatures patch. Signed-off-by: Jiri Denemark <jdenemar@xxxxxxxxxx> --- src/cpu/cpu_x86.c | 38 +++++++++++++++++++------------------- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/src/cpu/cpu_x86.c b/src/cpu/cpu_x86.c index 4b6eaf8060..ff44cae77e 100644 --- a/src/cpu/cpu_x86.c +++ b/src/cpu/cpu_x86.c @@ -1110,6 +1110,21 @@ virCPUx86SignaturesCopy(virCPUx86ModelPtr dst, } +static bool +virCPUx86SignaturesMatch(virCPUx86ModelPtr model, + uint32_t signature) +{ + size_t i; + + for (i = 0; i < model->nsignatures; i++) { + if (model->signatures[i] == signature) + return true; + } + + return false; +} + + static void x86ModelFree(virCPUx86ModelPtr model) { @@ -1877,21 +1892,6 @@ virCPUx86Compare(virCPUDefPtr host, } -static bool -x86ModelHasSignature(virCPUx86ModelPtr model, - uint32_t signature) -{ - size_t i; - - for (i = 0; i < model->nsignatures; i++) { - if (model->signatures[i] == signature) - return true; - } - - return false; -} - - static char * x86FormatSignatures(virCPUx86ModelPtr model) { @@ -1963,8 +1963,8 @@ x86DecodeUseCandidate(virCPUx86ModelPtr current, * consider candidates with matching family/model. */ if (signature && - x86ModelHasSignature(current, signature) && - !x86ModelHasSignature(candidate, signature)) { + virCPUx86SignaturesMatch(current, signature) && + !virCPUx86SignaturesMatch(candidate, signature)) { VIR_DEBUG("%s differs in signature from matching %s", cpuCandidate->model, cpuCurrent->model); return 0; @@ -1980,8 +1980,8 @@ x86DecodeUseCandidate(virCPUx86ModelPtr current, * result in longer list of features. */ if (signature && - x86ModelHasSignature(candidate, signature) && - !x86ModelHasSignature(current, signature)) { + virCPUx86SignaturesMatch(candidate, signature) && + !virCPUx86SignaturesMatch(current, signature)) { VIR_DEBUG("%s provides matching signature", cpuCandidate->model); return 1; } -- 2.26.0