* [PATCH v3] xfs_copy: flush target devices before exiting
@ 2020-06-18 18:18 Darrick J. Wong
2020-06-18 23:17 ` Dave Chinner
0 siblings, 1 reply; 2+ messages in thread
From: Darrick J. Wong @ 2020-06-18 18:18 UTC (permalink / raw)
To: Eric Sandeen; +Cc: xfs
From: Darrick J. Wong <darrick.wong@oracle.com>
Flush the devices we're copying to before exiting, so that we can report
any write errors.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
---
v3: flush before we yell about copies failing to complete
v2: fix thinko in v1 patch
---
copy/xfs_copy.c | 24 ++++++++++++++++++++++--
1 file changed, 22 insertions(+), 2 deletions(-)
diff --git a/copy/xfs_copy.c b/copy/xfs_copy.c
index 2d087f71..38a20d37 100644
--- a/copy/xfs_copy.c
+++ b/copy/xfs_copy.c
@@ -12,6 +12,7 @@
#include <stdarg.h>
#include "xfs_copy.h"
#include "libxlog.h"
+#include "libfrog/platform.h"
#define rounddown(x, y) (((x)/(y))*(y))
#define uuid_equal(s,d) (platform_uuid_compare((s),(d)) == 0)
@@ -138,6 +139,14 @@ check_errors(void)
int i, first_error = 0;
for (i = 0; i < num_targets; i++) {
+ if (target[i].state != INACTIVE) {
+ if (platform_flush_device(target[i].fd, 0)) {
+ target[i].error = errno;
+ target[i].state = INACTIVE;
+ target[i].err_type = 2;
+ }
+ }
+
if (target[i].state == INACTIVE) {
if (first_error == 0) {
first_error++;
@@ -145,10 +154,21 @@ check_errors(void)
_("THE FOLLOWING COPIES FAILED TO COMPLETE\n"));
}
do_log(" %s -- ", target[i].name);
- if (target[i].err_type == 0)
+ switch (target[i].err_type) {
+ case 0:
do_log(_("write error"));
- else
+ break;
+ case 1:
do_log(_("lseek error"));
+ break;
+ case 2:
+ do_log(_("flush error"));
+ break;
+ default:
+ do_log(_("unknown error type %d"),
+ target[i].err_type);
+ break;
+ }
do_log(_(" at offset %lld\n"), target[i].position);
}
}
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v3] xfs_copy: flush target devices before exiting
2020-06-18 18:18 [PATCH v3] xfs_copy: flush target devices before exiting Darrick J. Wong
@ 2020-06-18 23:17 ` Dave Chinner
0 siblings, 0 replies; 2+ messages in thread
From: Dave Chinner @ 2020-06-18 23:17 UTC (permalink / raw)
To: Darrick J. Wong; +Cc: Eric Sandeen, xfs
On Thu, Jun 18, 2020 at 11:18:25AM -0700, Darrick J. Wong wrote:
> From: Darrick J. Wong <darrick.wong@oracle.com>
>
> Flush the devices we're copying to before exiting, so that we can report
> any write errors.
>
> Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Looks ok. Flushing in check_errors() is a bit nasty, but it's a copy
completion path that is called from many places so there's no
obvious alternative...
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Cheers,
Dave.
--
Dave Chinner
david@fromorbit.com
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-06-18 23:17 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-06-18 18:18 [PATCH v3] xfs_copy: flush target devices before exiting Darrick J. Wong
2020-06-18 23:17 ` Dave Chinner
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.