>>> On 08.01.16 at 14:55, <haozhong.zh...@intel.com> wrote: > On 01/08/16 02:44, Jan Beulich wrote: >> >>> On 31.12.15 at 04:03, <haozhong.zh...@intel.com> wrote: >> > + ratio = mul_u64_u32_div(1ULL << hvm_funcs.tsc_scaling_ratio_frac_bits, >> > + gtsc_khz, cpu_khz); >> > + >> > + return (!ratio || ratio > hvm_funcs.max_tsc_scaling_ratio) ? FALSE : >> > TRUE; >> >> There no point in using a conditional expression here. > > Ah, right, I'll change it to > return !!(ratio && ratio <= hvm_funcs.max_tsc_scaling_ratio);
Except that you don't need the !!() here either. Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel