Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932264Ab1EXNaf (ORCPT ); Tue, 24 May 2011 09:30:35 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49489 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756105Ab1EXNae (ORCPT ); Tue, 24 May 2011 09:30:34 -0400 Date: Tue, 24 May 2011 15:30:17 +0200 (CEST) From: Lukas Czerner X-X-Sender: lukas@dhcp-27-109.brq.redhat.com To: OGAWA Hirofumi cc: Lukas Czerner , Kyungmin Park , Arnd Bergmann , linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org Subject: Re: [PATCH v6] fat: Batched discard support for fat In-Reply-To: <87sjs4i93w.fsf@devron.myhome.or.jp> Message-ID: References: <20110328103431.GA22323@july> <201103301620.50263.arnd@arndb.de> <201103301706.36214.arnd@arndb.de> <87wrhgk8lp.fsf@devron.myhome.or.jp> <87r57ok3fp.fsf@devron.myhome.or.jp> <87mxick0zj.fsf@devron.myhome.or.jp> <874o4kjtsy.fsf@devron.myhome.or.jp> <87y61wic4h.fsf@devron.myhome.or.jp> <87sjs4i93w.fsf@devron.myhome.or.jp> User-Agent: Alpine 2.00 (LFD 1167 2008-08-23) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2114 Lines: 48 On Tue, 24 May 2011, OGAWA Hirofumi wrote: > Lukas Czerner writes: > > >> Are you read my email? So, FAT adjust 2 blocks, ext* 1 block, and what > >> is other? The middle was guaranteed as continued? So, which is end of > >> blocks? > > > > I am sorry but I am not sure what you are asking for. Just grab the size > > of the file system or even better the size of underlying device (since > > the filesyste would not be bigger than that) and use that. > > > > Of course if you grab the count of filesystem blocks (and the file > > system does not account for the first data block) you'll end up with len > > smaller than first data block. So it might make sense to not to > > decrement the len and just start from first data block since it is not > > accounted for anyway. However it is not a big deal, is it ? Are you > > expecting any problems with this behaviour ? > > What is size of file system or underlying devices? You force to find the > device which target FS is using? Even if you can get size of underlying > devices, you force to user to insane loop in size of devices? Look, I do not have time to argue with you forever and I do not even understand what is your point. Just go and read other filesystems implementation of FITRIM (ext4,ext3,xfs,btrfs?) and you'll see what you need to do. If you do not want to get the file system size, then FINE! just pass the damn UULONG_MAX as length. I have no clue what insane loop are you talking about! It is *easy* just discard the whole thing (with UULONG_MAX) or, if you want to do it per-partes, then do it as long as it does not return EINVAL, once it does you know that your "start" is out of the filesystem and you are done! > > Why can you guarantee it's not big deal in design? Why can't you admit > userland can't make optimized loop? And what do you mean by that ? -Lukas -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/