From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Chinner Subject: Re: [RFC 0/6] vfs: Add timestamp range check support Date: Fri, 4 Nov 2016 10:48:02 +1100 Message-ID: <20161103234802.GK14023@dastard> References: <1478099096-25637-1-git-send-email-deepa.kernel@gmail.com> <20161102224827.GJ14023@dastard> <20161103204357.r7lgvbsv3euujijn@thunk.org> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 To: Theodore Ts'o , Deepa Dinamani , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, arnd@arndb.de, tglx@linutronix.de, gregkh@linuxfoundation.org, akpm@linux-foundation.org, viro@zeniv.linux.org.uk, y2038@lists.linaro.org, linux-afs@lists.infradead.org, Andreas Dilger , linux-ext4@vger.kernel.org Return-path: Content-Disposition: inline In-Reply-To: <20161103204357.r7lgvbsv3euujijn@thunk.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: y2038-bounces@lists.linaro.org Sender: "Y2038" List-Id: linux-ext4.vger.kernel.org T24gVGh1LCBOb3YgMDMsIDIwMTYgYXQgMDQ6NDM6NTdQTSAtMDQwMCwgVGhlb2RvcmUgVHMnbyB3 cm90ZToKPiBPbiBUaHUsIE5vdiAwMywgMjAxNiBhdCAwOTo0ODoyN0FNICsxMTAwLCBEYXZlIENo aW5uZXIgd3JvdGU6Cj4gPiAKPiA+IFdlJ3JlIGdvaW5nIHRvIG5lZWQgcmVncmVzc2lvbiB0ZXN0 cyBmb3IgdGhpcyB0byBlbnN1cmUgdGhhdCBpdAo+ID4gd29ya3MgcHJvcGVybHkgYW5kIHRoYXQg d2UgZG9uJ3QgaW5hZHZlcnRhbnRseSBicmVhayBpdCBpbiBmdXR1cmUuCj4gPiBDYW4geW91IHdy aXRlIHNvbWUgeGZzdGVzdHMgdGhhdCBleGVyY2lzZSB0aGlzIGZ1bmN0aW9uYWxpdHkgYW5kCj4g PiB2YWxpZGF0ZSB0aGF0IHRoZSBtb3VudCBiZWhhdmlvdXIsIGNsYW1waW5nIGFuZCByYW5nZSBs aW1pdGluZyBpcwo+ID4gd29ya2luZyBhcyBpbnRlbmRlZD8KPiAKPiBJbiBvcmRlciB0byBoYXZl IGF1dG9tYXRlZCByZWdyZXNzaW9uIHRlc3RzIHdoaWNoIGFyZSBmaWxlIHN5c3RlbQo+IGluZGVw ZW5kZW50LCB3ZSBuZWVkIGEgd2F5IHRvIHF1ZXJ5IHdoYXQgYXJlIHRoZSB0aW1lc3RhbXBzIHRo YXQgYQo+IHBhcnRpY3VsYXIgbW91bnRlZCBmaWxlIHN5c3RtZSBzdXBwb3J0cy4KCldlIGRvbid0 IG5lZWQgdGhhdCAtIHdlIHNpbXBseSBjb2RlIGl0IGRpcmVjdGx5IGludG8gdGhlIHRlc3QKaW5m cmFzdHJ1Y3R1cmUsIGxpa2Ugd2UndmUgZG9uZSBmb3IgdGhpbmdzIGxpa2UgdGhlIG1heGltdW0g bnVtYmVyCm9mIEFDTHMgYSBmaWxlc3lzdGVtIHN1cHBvcnRzIChjb21tb24vYXR0cjo6X2FjbF9n ZXRfbWF4KCkpLgoKPiBUaGUgbGFzdCBvcHRpb24sIHdoaWNoIGlzIGFkbWl0dGVkbHkgdWdseSwg d291bGQgYmUgdG8gY3JlYXRlIGFuIHNoZWxsCj4gZnVuY3Rpb24gd2hpY2gga25vd3MgaG93IHRv IGZpZ3VyZSBvdXQgdGhlIG1heF90aW1lc3RhbXAgYW5kCj4gbWluX3RpbWVzdGFtcCBieSB1c2lu ZyB0aGUgZmlsZSBzeXN0ZW0gbmFtZSBhbmQgcXVlcnlpbmcgdGhlCj4gc3VwZXJibG9jayB1c2lu ZyBkdW1wZTJmcywgeGZzX2RiLCBldGMuCgpZdXAsIHByZWNpc2VseSB0aGF0LiBXZSBzaG91bGRu J3QgdHJ1c3QgdGhlIGtlcm5lbCB0byB0ZWxsIHVzIHRoZQpjb3JyZWN0IHRoaW5nIHRvIGVuYWJs ZSB0aGUgdGVzdCB0aGF0IHRlbGxzIHVzIHRoYXQgdGhpbmcgaXMgd29ya2luZwpjb3JyZWN0bHkg b3Igbm90Li4uCgo+IEknZCBhcmd1ZSBmb3IgdGhlIGxhc3Qgb3B0aW9uIGJlY2F1c2Ugb25jZSB3 ZSBkbyBnZXQgYSBwcm9ncmFtbXRpYyB3YXkKPiB0byBnZXQgdGhlIGluZm9ybWF0aW9uIHZpYSBh IHN5c3RlbSBjYWxsIHN1Y2ggYXMgZnNpbmZvKDIpLCB3ZSBjYW4KPiBjb252ZXJ0IHhmc3Rlc3Rz IHRvIHVzZSBpdCwgd2hlcmUgYXMgaWYgd2UgYWRkIGFuIGlvY3RsIHRvIHJldHVybiB0aGlzCj4g aW5mb3JtYXRpb24sIHdlJ2xsIGhhdmUgdG8gc3VwcG9ydCB0aGUgaW9jdGwgZm9yZXZlci4KCldl IGhhdmUgdG8gc3VwcG9ydCBrZXJuZWxzIHRoYXQgd29uJ3QgZXZlciBoYXZlIHNvbWV0aGluZyBs aWtlCmZzaW5mbywgc28gaXQgaGFzIHRvIGJlIGRvbmUgdGhlICJ1Z2x5IHdheSIuCgpDaGVlcnMs CgpEYXZlLgotLSAKRGF2ZSBDaGlubmVyCmRhdmlkQGZyb21vcmJpdC5jb20KX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KWTIwMzggbWFpbGluZyBsaXN0Clky MDM4QGxpc3RzLmxpbmFyby5vcmcKaHR0cHM6Ly9saXN0cy5saW5hcm8ub3JnL21haWxtYW4vbGlz dGluZm8veTIwMzgK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 4 Nov 2016 10:48:02 +1100 From: Dave Chinner To: Theodore Ts'o , Deepa Dinamani , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, arnd@arndb.de, tglx@linutronix.de, gregkh@linuxfoundation.org, akpm@linux-foundation.org, viro@zeniv.linux.org.uk, y2038@lists.linaro.org, linux-afs@lists.infradead.org, Andreas Dilger , linux-ext4@vger.kernel.org Subject: Re: [RFC 0/6] vfs: Add timestamp range check support Message-ID: <20161103234802.GK14023@dastard> References: <1478099096-25637-1-git-send-email-deepa.kernel@gmail.com> <20161102224827.GJ14023@dastard> <20161103204357.r7lgvbsv3euujijn@thunk.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161103204357.r7lgvbsv3euujijn@thunk.org> Sender: linux-kernel-owner@vger.kernel.org List-ID: On Thu, Nov 03, 2016 at 04:43:57PM -0400, Theodore Ts'o wrote: > On Thu, Nov 03, 2016 at 09:48:27AM +1100, Dave Chinner wrote: > > > > We're going to need regression tests for this to ensure that it > > works properly and that we don't inadvertantly break it in future. > > Can you write some xfstests that exercise this functionality and > > validate that the mount behaviour, clamping and range limiting is > > working as intended? > > In order to have automated regression tests which are file system > independent, we need a way to query what are the timestamps that a > particular mounted file systme supports. We don't need that - we simply code it directly into the test infrastructure, like we've done for things like the maximum number of ACLs a filesystem supports (common/attr::_acl_get_max()). > The last option, which is admittedly ugly, would be to create an shell > function which knows how to figure out the max_timestamp and > min_timestamp by using the file system name and querying the > superblock using dumpe2fs, xfs_db, etc. Yup, precisely that. We shouldn't trust the kernel to tell us the correct thing to enable the test that tells us that thing is working correctly or not... > I'd argue for the last option because once we do get a programmtic way > to get the information via a system call such as fsinfo(2), we can > convert xfstests to use it, where as if we add an ioctl to return this > information, we'll have to support the ioctl forever. We have to support kernels that won't ever have something like fsinfo, so it has to be done the "ugly way". Cheers, Dave. -- Dave Chinner david@fromorbit.com