diff for duplicates of <1485188442.41392.1.camel@primarydata.com> diff --git a/a/1.txt b/N1/1.txt index 4540912..3d62c4d 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,21 +1,31 @@ -T24gTW9uLCAyMDE3LTAxLTIzIGF0IDE3OjA1ICswMTAwLCBDaHJpc3RvcGggSGVsbHdpZyB3cm90 -ZToNCj4gT24gTW9uLCBKYW4gMjMsIDIwMTcgYXQgMTA6NTI6MDlBTSAtMDUwMCwgSmVmZiBMYXl0 -b24gd3JvdGU6DQo+ID4gVG8gYmUgY2xlYXIsIHRoZSBjbGllbnQgaXMgcmVxdWVzdGluZyB0byBz -ZXQgdGhlIG10aW1lIHRvIGN1cnJlbnQNCj4gPiBzZXJ2ZXINCj4gPiB0aW1lIGFuZCBub3QgdG8g -YSBzcGVjaWZpYyBtdGltZSwgcmlnaHQ/DQo+IA0KPiBZZXMuwqDCoEFuZCBJIHRoaW5rIGl0J3Mg -bW9zdGx5IHRoZSBMaW51eCBjbGllbnQgYmVpbmcgbGF6eSAtDQo+IEFUVFJfTVRJTUUNCj4gaXMg -d2hhdCBpdCBnZXRzIGZyb20gdGhlIFZGUyBmb3IgYSB0cnVuY2F0ZSBvcGVyYXRpb24gKGJ1dCBu -b3QNCj4gZnRydW5hdGUsDQo+IHNvIHdlIHByb2JhYmx5IHdvbid0IHNlZSBpdCBvbiB0aGUgd2ly -ZSBpbiB0aGF0IGNhc2UsIGJ1dCBJIG5lZWQgdG8NCj4gdmVyaWZ5DQo+IHRoYXQgZmlyc3QpLsKg -wqBZZXQgYW5vdGhlciByZWFzb24gZm9yIC0+dHJ1bmNhdGUgOikNCj4gDQoNCk5vdGUgdGhhdCB0 -aGUgUE9TSVggc3BlYyBzZWVtcyB0byBoYXZlIGNoYW5nZWQgcmVjZW50bHkuIFRoZSBjdXJyZW50 -DQpzcGVjIGFwcGVhcnMgdG8gc3RhdGUgdGhhdCB3ZSBzaG91bGQgc2V0IHRoZSBtdGltZSBhbmQg -Y3RpbWUgKGFuZA0KY2hhbmdlIGF0dHJpYnV0ZSkgb24gc3VjY2VzcyBpbiBvcGVuKE9fVFJVTkMp -LCB0cnVuY2F0ZSgpIGFuZA0KZnRydW5jYXRlKCkuIEluIHByZXZpb3VzIGluY2FybmF0aW9ucyBv -ZiB0aGUgc3BlYywgdHJ1bmNhdGUoKSB3b3VsZA0Kb25seSBzZXQgdGhlIHRpbWUgaWYgdGhlIHNp -emUgd2FzIGNoYW5nZWQ6DQoNClNlZToNCmh0dHA6Ly9wdWJzLm9wZW5ncm91cC5vcmcvb25saW5l -cHVicy85Njk5OTE5Nzk5L2Z1bmN0aW9ucy9mdHJ1bmNhdGUuaHRtDQpsDQpodHRwOi8vcHVicy5v -cGVuZ3JvdXAub3JnL29ubGluZXB1YnMvOTY5OTkxOTc5OS9mdW5jdGlvbnMvdHJ1bmNhdGUuaHRt -bA0KaHR0cDovL3B1YnMub3Blbmdyb3VwLm9yZy9vbmxpbmVwdWJzLzk2OTk5MTk3OTkvZnVuY3Rp -b25zL29wZW4uaHRtbA0KDQotLSANClRyb25kIE15a2xlYnVzdA0KTGludXggTkZTIGNsaWVudCBt -YWludGFpbmVyLCBQcmltYXJ5RGF0YQ0KdHJvbmQubXlrbGVidXN0QHByaW1hcnlkYXRhLmNvbQ0K +On Mon, 2017-01-23 at 17:05 +0100, Christoph Hellwig wrote: +> On Mon, Jan 23, 2017 at 10:52:09AM -0500, Jeff Layton wrote: +> > To be clear, the client is requesting to set the mtime to current +> > server +> > time and not to a specific mtime, right? +> +> Yes. And I think it's mostly the Linux client being lazy - +> ATTR_MTIME +> is what it gets from the VFS for a truncate operation (but not +> ftrunate, +> so we probably won't see it on the wire in that case, but I need to +> verify +> that first). Yet another reason for ->truncate :) +> + +Note that the POSIX spec seems to have changed recently. The current +spec appears to state that we should set the mtime and ctime (and +change attribute) on success in open(O_TRUNC), truncate() and +ftruncate(). In previous incarnations of the spec, truncate() would +only set the time if the size was changed: + +See: +http://pubs.opengroup.org/onlinepubs/9699919799/functions/ftruncate.htm +l +http://pubs.opengroup.org/onlinepubs/9699919799/functions/truncate.html +http://pubs.opengroup.org/onlinepubs/9699919799/functions/open.html + +-- +Trond Myklebust +Linux NFS client maintainer, PrimaryData +trond.myklebust@primarydata.com diff --git a/a/content_digest b/N1/content_digest index 52c8da5..dd49284 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -15,26 +15,36 @@ " linux-fsdevel@vger.kernel.org <linux-fsdevel@vger.kernel.org>\0" "\00:1\0" "b\0" - "T24gTW9uLCAyMDE3LTAxLTIzIGF0IDE3OjA1ICswMTAwLCBDaHJpc3RvcGggSGVsbHdpZyB3cm90\n" - "ZToNCj4gT24gTW9uLCBKYW4gMjMsIDIwMTcgYXQgMTA6NTI6MDlBTSAtMDUwMCwgSmVmZiBMYXl0\n" - "b24gd3JvdGU6DQo+ID4gVG8gYmUgY2xlYXIsIHRoZSBjbGllbnQgaXMgcmVxdWVzdGluZyB0byBz\n" - "ZXQgdGhlIG10aW1lIHRvIGN1cnJlbnQNCj4gPiBzZXJ2ZXINCj4gPiB0aW1lIGFuZCBub3QgdG8g\n" - "YSBzcGVjaWZpYyBtdGltZSwgcmlnaHQ/DQo+IA0KPiBZZXMuwqDCoEFuZCBJIHRoaW5rIGl0J3Mg\n" - "bW9zdGx5IHRoZSBMaW51eCBjbGllbnQgYmVpbmcgbGF6eSAtDQo+IEFUVFJfTVRJTUUNCj4gaXMg\n" - "d2hhdCBpdCBnZXRzIGZyb20gdGhlIFZGUyBmb3IgYSB0cnVuY2F0ZSBvcGVyYXRpb24gKGJ1dCBu\n" - "b3QNCj4gZnRydW5hdGUsDQo+IHNvIHdlIHByb2JhYmx5IHdvbid0IHNlZSBpdCBvbiB0aGUgd2ly\n" - "ZSBpbiB0aGF0IGNhc2UsIGJ1dCBJIG5lZWQgdG8NCj4gdmVyaWZ5DQo+IHRoYXQgZmlyc3QpLsKg\n" - "wqBZZXQgYW5vdGhlciByZWFzb24gZm9yIC0+dHJ1bmNhdGUgOikNCj4gDQoNCk5vdGUgdGhhdCB0\n" - "aGUgUE9TSVggc3BlYyBzZWVtcyB0byBoYXZlIGNoYW5nZWQgcmVjZW50bHkuIFRoZSBjdXJyZW50\n" - "DQpzcGVjIGFwcGVhcnMgdG8gc3RhdGUgdGhhdCB3ZSBzaG91bGQgc2V0IHRoZSBtdGltZSBhbmQg\n" - "Y3RpbWUgKGFuZA0KY2hhbmdlIGF0dHJpYnV0ZSkgb24gc3VjY2VzcyBpbiBvcGVuKE9fVFJVTkMp\n" - "LCB0cnVuY2F0ZSgpIGFuZA0KZnRydW5jYXRlKCkuIEluIHByZXZpb3VzIGluY2FybmF0aW9ucyBv\n" - "ZiB0aGUgc3BlYywgdHJ1bmNhdGUoKSB3b3VsZA0Kb25seSBzZXQgdGhlIHRpbWUgaWYgdGhlIHNp\n" - "emUgd2FzIGNoYW5nZWQ6DQoNClNlZToNCmh0dHA6Ly9wdWJzLm9wZW5ncm91cC5vcmcvb25saW5l\n" - "cHVicy85Njk5OTE5Nzk5L2Z1bmN0aW9ucy9mdHJ1bmNhdGUuaHRtDQpsDQpodHRwOi8vcHVicy5v\n" - "cGVuZ3JvdXAub3JnL29ubGluZXB1YnMvOTY5OTkxOTc5OS9mdW5jdGlvbnMvdHJ1bmNhdGUuaHRt\n" - "bA0KaHR0cDovL3B1YnMub3Blbmdyb3VwLm9yZy9vbmxpbmVwdWJzLzk2OTk5MTk3OTkvZnVuY3Rp\n" - "b25zL29wZW4uaHRtbA0KDQotLSANClRyb25kIE15a2xlYnVzdA0KTGludXggTkZTIGNsaWVudCBt\n" - YWludGFpbmVyLCBQcmltYXJ5RGF0YQ0KdHJvbmQubXlrbGVidXN0QHByaW1hcnlkYXRhLmNvbQ0K + "On Mon, 2017-01-23 at 17:05 +0100, Christoph Hellwig wrote:\n" + "> On Mon, Jan 23, 2017 at 10:52:09AM -0500, Jeff Layton wrote:\n" + "> > To be clear, the client is requesting to set the mtime to current\n" + "> > server\n" + "> > time and not to a specific mtime, right?\n" + "> \n" + "> Yes.\302\240\302\240And I think it's mostly the Linux client being lazy -\n" + "> ATTR_MTIME\n" + "> is what it gets from the VFS for a truncate operation (but not\n" + "> ftrunate,\n" + "> so we probably won't see it on the wire in that case, but I need to\n" + "> verify\n" + "> that first).\302\240\302\240Yet another reason for ->truncate :)\n" + "> \n" + "\n" + "Note that the POSIX spec seems to have changed recently. The current\n" + "spec appears to state that we should set the mtime and ctime (and\n" + "change attribute) on success in open(O_TRUNC), truncate() and\n" + "ftruncate(). In previous incarnations of the spec, truncate() would\n" + "only set the time if the size was changed:\n" + "\n" + "See:\n" + "http://pubs.opengroup.org/onlinepubs/9699919799/functions/ftruncate.htm\n" + "l\n" + "http://pubs.opengroup.org/onlinepubs/9699919799/functions/truncate.html\n" + "http://pubs.opengroup.org/onlinepubs/9699919799/functions/open.html\n" + "\n" + "-- \n" + "Trond Myklebust\n" + "Linux NFS client maintainer, PrimaryData\n" + trond.myklebust@primarydata.com -0842ba7724411e7bd3e50fc5d901a3212c4c2b227b7af8b4dac3dadada6bbc7b +6c1ebf8fbdaa60170360508bf1679e43517a7ec1f50acc4c0ce6256ef24f87fc
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.