Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gfs_pio_stat() may change st_mtime/st_atime, if it's called against a never accessed 0-byte file #958

Open
gfarm-admin opened this issue May 16, 2016 · 5 comments
Labels
auto-migrated bug Something isn't working sourceforge
Milestone

Comments

@gfarm-admin
Copy link
Member

Probably this is since the beginning of the gfarm-2.0.0.
And currently the fix for this problem hasn't been scheduled, because it's not known that this causes a severe problem.

NOTE: this problem can be fixed by making gfmd REOPEN protocol return st_mtime/st_atime, and letting gfsd call futimes(2), if the underlying local filesystem supports same or finer time resolution with the corresponding metadata.

Reported by: n-soda

Original Ticket: gfarm/tickets/958

@gfarm-admin
Copy link
Member Author

Original comment by: n-soda

@gfarm-admin
Copy link
Member Author

a regression test for this problem is added in [r10155] as regress/regress/lib/libgfarm/gfarm/gfs_pio_stat/gfs_pio_stat_mtime.sh

Original comment by: n-soda

@gfarm-admin
Copy link
Member Author

  • Description has changed:

Diff:


--- old
+++ new
@@ -1,2 +1,4 @@
 Probably this is since the beginning of the gfarm-2.0.0.
 And currently the fix for this problem hasn't been scheduled, because it's not known that this causes a severe problem. 
+
+NOTE: this problem can be fixed by making gfmd REOPEN protocol return st_mtime/st_atime, and letting gfsd call futimes(2)

Original comment by: n-soda

@gfarm-admin
Copy link
Member Author

  • Description has changed:

Diff:


--- old
+++ new
@@ -1,4 +1,4 @@
 Probably this is since the beginning of the gfarm-2.0.0.
 And currently the fix for this problem hasn't been scheduled, because it's not known that this causes a severe problem. 
 
-NOTE: this problem can be fixed by making gfmd REOPEN protocol return st_mtime/st_atime, and letting gfsd call futimes(2)
+NOTE: this problem can be fixed by making gfmd REOPEN protocol return st_mtime/st_atime, and letting gfsd call futimes(2), if the underlying local filesystem supports same or finer time resolution with the corresponding metadata.

Original comment by: n-soda

@gfarm-admin
Copy link
Member Author

  • Milestone: gfarm-2.6.11 --> gfarm-2.6.26

Original comment by: n-soda

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-migrated bug Something isn't working sourceforge
Projects
None yet
Development

No branches or pull requests

1 participant