From: Eric Sandeen Subject: Re: [PATCH, RFC V3] ext3: add ioctl to force 32-bit hashes from indexed dirs Date: Mon, 01 Apr 2013 15:05:35 -0500 Message-ID: <5159E88F.8030704@redhat.com> References: <51546EED.8030507@redhat.com> <5154AAB4.2000701@redhat.com> <5159A8D5.1000204@redhat.com> <20130401181718.GB22443@thunk.org> <5159D03F.5000606@redhat.com> <20130401190804.GD22443@thunk.org> <5159E4AC.7050307@redhat.com> <20130401200052.GA24097@thunk.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: ext4 development , Anand Avati , Jan Kara To: "Theodore Ts'o" Return-path: Received: from mx1.redhat.com ([209.132.183.28]:64732 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759161Ab3DAUGU (ORCPT ); Mon, 1 Apr 2013 16:06:20 -0400 In-Reply-To: <20130401200052.GA24097@thunk.org> Sender: linux-ext4-owner@vger.kernel.org List-ID: On 4/1/13 3:00 PM, Theodore Ts'o wrote: > On Mon, Apr 01, 2013 at 02:49:00PM -0500, Eric Sandeen wrote: >> >> Urgh, I guess if we are adding an interface which will live "forever," >> we may as well make it full featured & flexible, as long as the complexity >> isn't out of hand, and I don't think it will be in this case. So I'm at >> least half inclined to go ahead & allow toggling it on and off under the >> right circumstances, even though it goes against what I think is my better >> judgement. ;) > > If you want to have a fully flexible interface, then we probably > should have a way to both get and set the flag. And from there the > next step down the slippery slope would be to make this be a bit more > like a fcntl-style F_GETFL/F_SETFL style interface, so we can in the > future set and get other ext4-specific struct_file-specific flags. :-) > > I'll let you decide how far you want to go with this; I won't mind if > you keep it with the original KISS interface, but I also won't mind if > you want to create a somewhat more expansive interface. Meh, let's just keep it simple then. And I'd really like to know if gluster still even needs this, or if their new scheme will work instead, in which case we should drop it - but Samba made noise about needing it too, though I've not seen specifics, so I hate to merge it "just in case." I put it out mostly for review so it was ready if we needed it (since certain quarters seem anxious) ;) -Eric > Cheers, > > - Ted >