On 30/11/2018 09:23, Jan Beulich wrote:
>>>> On 29.11.18 at 20:20, <andrew.coop...@citrix.com> wrote:
>> get_cpuid_domain_info() has two conflicting return styles - either -error for
>> local failures, or -1/errno for hypercall failures.  Switch to consistently
>> use -error.
>>
>> While fixing the xc_get_cpu_featureset(), take the opportunity to remove the
>> redundancy and move it to be adjacent to the other featureset handling.
> Having noticed the redundancy before, I was assuming this was done
> to prevent altering host_nr_features early, just in case future code
> additions would want to use its original value. The way it is now, the
> code folding looks correct.

I don't recall a deliberate intention to do that, but it was a long time
ago now.

>
>> Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>
> Reviewed-by: Jan Beulich <jbeul...@suse.com>

Thanks.

~Andrew

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to