<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Mar 8, 2023 at 12:16 PM Michal Privoznik <<a href="mailto:mprivozn@redhat.com">mprivozn@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Technically, there's nothing libnuma specific about<br>
virNumaNodesetToCPUset(). It just implements a generic algorithm<br>
over virNumaGetNodeCPUs() (which is then libnuma dependant).<br>
Nevertheless, there's no need to have this function living inside<br>
WITH_NUMACTL block. Any error returned from virNumaGetNodeCPUs()<br>
(including the one that !WITH_NUMACTL stub returns) is propagated<br>
properly.<br>
<br>
Move the function out of the block into a generic one and drop<br>
the !WITH_NUMACTL stub.<br>
<br>
Signed-off-by: Michal Privoznik <<a href="mailto:mprivozn@redhat.com" target="_blank">mprivozn@redhat.com</a>><br>
---<br>
 src/util/virnuma.c | 115 +++++++++++++++++++++------------------------<br>
 1 file changed, 53 insertions(+), 62 deletions(-)<br></blockquote><div><br></div><div> <div>Reviewed-by: Kristina Hanicova <<a href="mailto:khanicov@redhat.com" target="_blank">khanicov@redhat.com</a>> <br></div><font style="--darkreader-inline-color: #b2a696;" color="#888888"><div><br></div><div>Kristina</div></font></div></div></div>