[PATCH] XFS: Fix gcc 4.6 set but not read and unused statement warnings v2

Andi Kleen andi at firstfloor.org
Tue Jun 15 02:24:52 CDT 2010


> 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 at linux.intel.com -- Speaking for myself only.




More information about the xfs mailing list