summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPádraig Brady <P@draigBrady.com>2011-02-25 12:27:25 +0000
committerPádraig Brady <P@draigBrady.com>2011-03-05 01:48:55 +0000
commite1788d9ed9c22811ab73163270cfc11ed36c8abd (patch)
tree2dc6e40a1ce16c99c5fe261adb45a8a73494d2ed
parentcaaf2899f67d312d76af91add2a4d9f7be2d5c61 (diff)
downloadcoreutils-e1788d9ed9c22811ab73163270cfc11ed36c8abd.tar.xz
dd: warn when we disable oflag=direct not at EOF
An alternative to this is to auto enable iflag=fullblock when oflag=direct and bs= is specified. It was thought better though, to warn about the specific issue, and give full control of dd's options to the user. * src/dd.c (iwrite): Warn, when we write after having disabled O_DIRECT. See https://bugzilla.redhat.com/show_bug.cgi?id=614605
-rw-r--r--src/dd.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/src/dd.c b/src/dd.c
index daddc1ed6..fd468a647 100644
--- a/src/dd.c
+++ b/src/dd.c
@@ -837,6 +837,12 @@ iwrite (int fd, char const *buf, size_t size)
{
size_t total_written = 0;
+ if ((output_flags & O_DIRECT) && w_partial == 1)
+ {
+ error (0, 0, _("dd: warning: partial read; oflag=direct disabled; "
+ "suggest iflag=fullblock"));
+ }
+
if ((output_flags & O_DIRECT) && size < output_blocksize)
{
int old_flags = fcntl (STDOUT_FILENO, F_GETFL);