PBXT

Merge lp:~vkolesnikov/pbxt/pbxt-fullfsync into lp:pbxt

  1. pbxt-fullfsync
  2. Merge into trunk
Proposed by Vladimir Kolesnikov
Status: Merged
Merged at revision: not available
Proposed branch: lp:~vkolesnikov/pbxt/pbxt-fullfsync
Merge into: lp:pbxt
Diff against target: None lines
To merge this branch: bzr merge lp:~vkolesnikov/pbxt/pbxt-fullfsync
Reviewer Review Type Date Requested Status
PBXT Core Pending
Review via email: mp+8300@code.launchpad.net
To post a comment you must log in.
lp:~vkolesnikov/pbxt/pbxt-fullfsync updated
671. By Vladimir Kolesnikov

updated comment

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ChangeLog'
2--- ChangeLog 2009-07-02 12:13:58 +0000
3+++ ChangeLog 2009-07-07 09:52:36 +0000
4@@ -3,6 +3,8 @@
5
6 ------- 1.0.08 RC2 - 2009-06-30
7
8+RN253: Use fcntl(F_FULLFSYNC) instead of fsync on platforms that support it
9+
10 RN252: Fixed bug #368692: PBXT not reporting data size correctly in information_schema
11
12 RN251: A Windows-specific test update, also removed false assertion that failed on Windows.
13
14=== modified file 'src/filesys_xt.cc'
15--- src/filesys_xt.cc 2009-05-28 15:17:16 +0000
16+++ src/filesys_xt.cc 2009-07-07 09:52:36 +0000
17@@ -868,11 +868,18 @@
18 goto failed;
19 }
20 #else
21+#ifdef F_FULLFSYNC
22+ if (fcntl(of->of_filedes, F_FULLFSYNC, 0) == -1) {
23+ xt_register_ferrno(XT_REG_CONTEXT, errno, xt_file_path(of));
24+ goto failed;
25+ }
26+#else
27 if (fsync(of->of_filedes) == -1) {
28 xt_register_ferrno(XT_REG_CONTEXT, errno, xt_file_path(of));
29 goto failed;
30 }
31 #endif
32+#endif
33 #ifdef DEBUG_TRACE_IO
34 xt_trace("/* %s */ pbxt_file_sync(\"%s\");\n", xt_trace_clock_diff(timef, start), of->fr_file->fil_path);
35 #endif

Subscribers

People subscribed via source and target branches