Hello there, linux-4.7/fs/xfs/xfs_icache.c:1298]: (style) Redundant condition: eofb. '!eofb || (eofb && eofb.eof_scan_owner!=0)' is equivalent to '!eofb || eofb.eof_scan_owner!=0' Source code is ASSERT(!eofb || (eofb && eofb->eof_scan_owner != 0)); Maybe better code ASSERT(!eofb || (eofb->eof_scan_owner != 0)); Regards David Binderman _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs