Re: [PATCH 4/5] libxg: drop dead assignment to "rc" from xc_cpuid_apply_policy()

2023-06-13 Thread Anthony PERARD
On Mon, Jun 12, 2023 at 01:47:13PM +0200, Jan Beulich wrote: > "rc" is written immediately below the outer if(). Fold the remaining two > if()s. > > Coverity ID: 1532320 > Fixes: 685e922d6f30 ("tools/libxc: Rework xc_cpuid_apply_policy() to use > {get,set}_cpu_policy()") > Signed-off-by: Jan Beul

Re: [PATCH 4/5] libxg: drop dead assignment to "rc" from xc_cpuid_apply_policy()

2023-06-12 Thread Juergen Gross
On 12.06.23 13:47, Jan Beulich wrote: "rc" is written immediately below the outer if(). Fold the remaining two if()s. Coverity ID: 1532320 Fixes: 685e922d6f30 ("tools/libxc: Rework xc_cpuid_apply_policy() to use {get,set}_cpu_policy()") Signed-off-by: Jan Beulich Reviewed-by: Juergen Gross

[PATCH 4/5] libxg: drop dead assignment to "rc" from xc_cpuid_apply_policy()

2023-06-12 Thread Jan Beulich
"rc" is written immediately below the outer if(). Fold the remaining two if()s. Coverity ID: 1532320 Fixes: 685e922d6f30 ("tools/libxc: Rework xc_cpuid_apply_policy() to use {get,set}_cpu_policy()") Signed-off-by: Jan Beulich --- The code in question was subsequently moved by 54463aa79dac ("x86/