diff options
author | pjd <pjd@FreeBSD.org> | 2010-09-25 10:31:05 +0000 |
---|---|---|
committer | pjd <pjd@FreeBSD.org> | 2010-09-25 10:31:05 +0000 |
commit | 60321cfa67d499937ac91500e248f7f96adccd3e (patch) | |
tree | b2b3954b1c5a49e54abb2f331ef9ace8f5ffb0b2 /sys/geom/eli/g_eli_ctl.c | |
parent | 6b8f97b12813cf038733bee39d12c57bc4905748 (diff) | |
download | FreeBSD-src-60321cfa67d499937ac91500e248f7f96adccd3e.zip FreeBSD-src-60321cfa67d499937ac91500e248f7f96adccd3e.tar.gz |
Ignore errors from BIO_FLUSH. It might confuse users that provider wasn't
really killed. What we really care about are write errors only.
MFC after: 2 weeks
Diffstat (limited to 'sys/geom/eli/g_eli_ctl.c')
-rw-r--r-- | sys/geom/eli/g_eli_ctl.c | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/sys/geom/eli/g_eli_ctl.c b/sys/geom/eli/g_eli_ctl.c index 3a7ba0f..02ede13 100644 --- a/sys/geom/eli/g_eli_ctl.c +++ b/sys/geom/eli/g_eli_ctl.c @@ -688,7 +688,7 @@ g_eli_ctl_delkey(struct gctl_req *req, struct g_class *mp) * Flush write cache so we don't overwrite data N times in cache * and only once on disk. */ - g_io_flush(cp); + (void)g_io_flush(cp); } bzero(&md, sizeof(md)); bzero(sector, sizeof(sector)); @@ -739,13 +739,11 @@ g_eli_kill_one(struct g_eli_softc *sc) if (error == 0) error = err; } - err = g_io_flush(cp); - if (err != 0) { - G_ELI_DEBUG(0, "Cannot flush %s (error=%d).", - pp->name, err); - if (error == 0) - error = err; - } + /* + * Flush write cache so we don't overwrite data N times + * in cache and only once on disk. + */ + (void)g_io_flush(cp); } free(sector, M_ELI); } |