On 26/02/18 10:50, Jan Beulich wrote: >>>> On 26.02.18 at 11:44, <roger....@citrix.com> wrote: >> On Mon, Feb 26, 2018 at 12:38:11AM -0700, Jan Beulich wrote: >>> Signed-off-by: Jan Beulich <jbeul...@suse.com> >> Reviewed-by: Roger Pau Monné <roger....@citrix.com> > Thanks. > >> I would however invert the check, my brain finds it easier to read >> as: >> >> (port + bytes) <= v->arch.pv_vcpu.iobmp_limit > My brain too, but I didn't want to make unrelated changes.
Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com>. I think this is a sufficiently small and well-describe change that rearranging the expression for clarity is fine. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel