[lkml]   [2011]   [Dec]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v2 0/2] vfs: truncate bug fixes
On Thu, Dec 22, 2011 at 11:10 AM, Al Viro <> wrote:
> On Thu, Dec 22, 2011 at 10:55:24AM +0200, Dmitry Kasatkin wrote:
>> When a file is opened with O_TRUNC or truncated with truncate()/ftruncate(),
>> and then closed, iversion is not updated. This patch uses ATTR_OPEN flag,
>> which is currently used to identify files opened with O_TRUNC,
>> as an indication to increment iversion.
>> Also it adds passing of this flag from do_sys_truncate and do_sys_ftruncate.
>> To reflect real meaning, ATTR_OPEN is suggested to be renamed to ATTR_TRUNC.
> Um...  Is there any difference left between that and ATTR_SIZE, then?

Oops. Indeed. It becomes simpler..

Comments and using ATTR_OPEN in fs/fuse/dir.c confused me.

/* If it's an open(O_TRUNC) or an ftruncate(), don't update */
if ((ivalid & ATTR_SIZE) && (ivalid & (ATTR_OPEN | ATTR_FILE)))

do_sys_truncate() calls do_truncate(path.dentry, length, 0, NULL);
while do_sys_ftruncate calls do_truncate(dentry, length,

Notice the last file parameter... NULL or file..

As truncate and ftruncate syscalls should do the same job as fstat or stat.
It will lead to different results for FUSE.
Following will be 0: (ivalid & (ATTR_OPEN | ATTR_FILE)

FUSE comments says.. OR
/* If it's an open(O_TRUNC) or an ftruncate(), don't update */

Will make new version..

- Dmitry
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2011-12-22 11:33    [W:0.048 / U:1.428 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site