>>> On 08.09.17 at 15:44, <wei.l...@citrix.com> wrote:
> @@ -41,7 +41,7 @@ bool_t hvm_monitor_cr(unsigned int index, unsigned long 
> value, unsigned long old
>            value != old) &&
>           (!((value ^ old) & ad->monitor.write_ctrlreg_mask[index])) )
>      {
> -        bool_t sync = !!(ad->monitor.write_ctrlreg_sync & ctrlreg_bitmask);
> +        bool sync = !!(ad->monitor.write_ctrlreg_sync & ctrlreg_bitmask);

If you do such conversion, please also get rid of any unnecessary
!! like is being used here (there are very few cases where these
can't be dropped).

Jan


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

Reply via email to