> I'm still not happy with it. With the BUG_ON? > > We'll commit a modified version of one of your patches into the XFS > tree as you seem unable to accept that changes might be necessary to > get your code to pass review. Sure modify however you like as long as you fix the warnings. Also it was good to have review on the cases it warned about, luckily there were no real bugs found. [BTW I should add there were some more mostly from array overflows. I have not investigated in detail, but I suspect they were harmless] My goal was also mostly to get my build reasonably warning free, but if I can't get this patch in I can also just disable the warning for fs/xfs. -Andi -- ak@xxxxxxxxxxxxxxx -- Speaking for myself only. _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs