-
Notifications
You must be signed in to change notification settings - Fork 17.3k
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
syscall: support O_SYNC flag for os.OpenFile on windows #64027
Conversation
os.OpenFile on windows did not use the O_SYNC flag. This meant that even if the user set O_SYNC, os.OpenFile would ignore it. This change adds a new flag FILE_FLAG_WRITE_THROUGH, which is the equivalent of O_SYNC flag on Linux and is documented in https://docs.microsoft.com/en-us/windows/win32/api/fileapi/nf-fileapi-createfilea Fixes #35358
This PR (HEAD: 15acd01) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/541015. Important tips:
|
This PR (HEAD: 4701606) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/541015. Important tips:
|
Message from Jes Cok: Patch Set 2: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Gopher Robot: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Gopher Robot: Patch Set 2: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Jes Cok: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Jes Cok: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Emmanuel Odeke: Patch Set 2: Code-Review+2 Run-TryBot+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Quim Muntal: Patch Set 2: (3 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Quim Muntal: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Jes Cok: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Quim Muntal: Patch Set 2: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
This PR (HEAD: 82c6275) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/541015. Important tips:
|
Message from Jes Cok: Patch Set 3: Run-TryBot+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Gopher Robot: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Gopher Robot: Patch Set 3: TryBot-Result+1 (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Jes Cok: Patch Set 3: (3 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Than McIntosh: Patch Set 3: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from David Chase: Patch Set 3: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Jes Cok: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Emmanuel Odeke: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Jes Cok: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Emmanuel Odeke: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
Message from Jes Cok: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
os.OpenFile on windows did not use the O_SYNC flag. This meant that even if the user set O_SYNC, os.OpenFile would ignore it. This change adds a new flag FILE_FLAG_WRITE_THROUGH, which is the equivalent of O_SYNC flag on Linux and is documented in https://docs.microsoft.com/en-us/windows/win32/api/fileapi/nf-fileapi-createfilea Fixes golang#35358 Change-Id: Ib338caed5bb2f215723bfe30a2551a83998d92c9 GitHub-Last-Rev: 82c6275 GitHub-Pull-Request: golang#64027
Message from Jes Cok: Patch Set 3: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/541015. |
os.OpenFile on windows did not use the O_SYNC flag. This meant that even if the user set O_SYNC, os.OpenFile would ignore it. This change adds a new flag FILE_FLAG_WRITE_THROUGH, which is the equivalent of O_SYNC flag on Linux and is documented in https://docs.microsoft.com/en-us/windows/win32/api/fileapi/nf-fileapi-createfilea Fixes #35358 Change-Id: Ib338caed5bb2f215723bfe30a2551a83998d92c9 GitHub-Last-Rev: 82c6275 GitHub-Pull-Request: #64027 Reviewed-on: https://go-review.googlesource.com/c/go/+/541015 Reviewed-by: Than McIntosh <thanm@google.com> Reviewed-by: Emmanuel Odeke <emmanuel@orijtech.com> Run-TryBot: Jes Cok <xigua67damn@gmail.com> TryBot-Result: Gopher Robot <gobot@golang.org> Reviewed-by: David Chase <drchase@google.com> Reviewed-by: Quim Muntal <quimmuntal@gmail.com>
This PR is being closed because golang.org/cl/541015 has been merged. |
os.OpenFile on windows did not use the O_SYNC flag. This meant
that even if the user set O_SYNC, os.OpenFile would ignore it.
This change adds a new flag FILE_FLAG_WRITE_THROUGH, which is
the equivalent of O_SYNC flag on Linux and is documented in
https://docs.microsoft.com/en-us/windows/win32/api/fileapi/nf-fileapi-createfilea
Fixes #35358