From mboxrd@z Thu Jan 1 00:00:00 1970 From: Josh Boyer Subject: Re: Oops in minixfs_statfs Date: Thu, 18 Aug 2011 18:24:10 -0400 Message-ID: <20110818222409.GT2227@zod.bos.redhat.com> References: <20110816164809.GE2227@zod.bos.redhat.com> <20110817021820.GU2203@ZenIV.linux.org.uk> <20110817171829.GG2227@zod.bos.redhat.com> <20110818211320.GQ2227@zod.bos.redhat.com> <20110818211620.GX2203@ZenIV.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org To: Al Viro Return-path: Received: from mx1.redhat.com ([209.132.183.28]:62138 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754989Ab1HRWYN (ORCPT ); Thu, 18 Aug 2011 18:24:13 -0400 Content-Disposition: inline In-Reply-To: <20110818211620.GX2203@ZenIV.linux.org.uk> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Thu, Aug 18, 2011 at 10:16:20PM +0100, Al Viro wrote: > On Thu, Aug 18, 2011 at 05:13:21PM -0400, Josh Boyer wrote: > > > + if (blocks != blocks_needed) > > + blocks = blocks_needed; > > ???? Er.. yeah. That's a left over from before I moved some verification into fill_super, and then it got simplified to the point of being utterly stupid. It's been a long day. I guess I could either just reassign sbi->s_[iz]map_blocks right then and truly ignore the extra block, or just always pass the calculated blocks_needed to count_free. Which would you prefer? josh