>>> On 27.06.18 at 14:27, <andrew.coop...@citrix.com> wrote: > Coverity complains at code which which performs a bitwise and with a constant > that happens to be zero. Both _PAGE_GNTTAB and PG_SH_enable may be 0 > depending on Kconfig settings. > > Rearrange the C to test the constant first and short circuit the bitwise and.
Hmm, well, this makes the code look quite, ehem, interesting. Normally such would seem a prime candidate for cleaning up, especially without any comment attached. But anyway, if it helps ... > No functional change. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> Acked-by: Jan Beulich <jbeul...@suse.com> Jan _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel