lib/device/bcache.c | 69 ++++++++++++++++++++++++++++++++++++----------------- 1 file changed, 47 insertions(+), 22 deletions(-) diff --git a/lib/device/bcache.c b/lib/device/bcache.c index 1cb1b2f..6886b74 100644 --- a/lib/device/bcache.c +++ b/lib/device/bcache.c @@ -320,7 +320,7 @@ struct io_engine *create_async_io_engine(void) e->aio_context = 0; r = io_setup(MAX_IO, &e->aio_context); if (r < 0) { - log_warn("io_setup failed"); + log_debug("io_setup failed %d", r); dm_free(e); return NULL; } @@ -363,8 +363,11 @@ static void _sync_destroy(struct io_engine *ioe) static bool _sync_issue(struct io_engine *ioe, enum dir d, int fd, sector_t sb, sector_t se, void *data, void *context) { - int r; - uint64_t len = (se - sb) * 512, where; + int rv; + off_t off; + uint64_t where; + uint64_t pos = 0; + uint64_t len = (se - sb) * 512; struct sync_engine *e = _to_sync(ioe); struct sync_io *io = malloc(sizeof(*io)); if (!io) { @@ -373,11 +376,17 @@ static bool _sync_issue(struct io_engine *ioe, enum dir d, int fd, } where = sb * 512; - r = lseek(fd, where, SEEK_SET); - if (r < 0) { - log_warn("unable to seek to position %llu", (unsigned long long) where); - free(io); - return false; + + off = lseek(fd, where, SEEK_SET); + if (off == (off_t) -1) { + log_warn("Device seek error %d for offset %llu", errno, (unsigned long long)where); + free(io); + return false; + } + if (off != (off_t) where) { + log_warn("Device seek failed for offset %llu", (unsigned long long)where); + free(io); + return false; } /* @@ -422,28 +431,44 @@ static bool _sync_issue(struct io_engine *ioe, enum dir d, int fd, len = nbytes; } - while (len) { - do { - if (d == DIR_READ) - r = read(fd, data, len); - else - r = write(fd, data, len); + while (pos < len) { + if (d == DIR_READ) + rv = read(fd, (char *)data + pos, len - pos); + else + rv = write(fd, (char *)data + pos, len - pos); - } while ((r < 0) && ((r == EINTR) || (r == EAGAIN))); + if (rv == -1 && errno == EINTR) + continue; + if (rv == -1 && errno == EAGAIN) + continue; + + if (!rv) + break; - if (r < 0) { - log_warn("io failed %d", r); + if (rv < 0) { + if (d == DIR_READ) + log_debug("Device read error %d offset %llu len %llu", errno, + (unsigned long long)(where + pos), + (unsigned long long)(len - pos)); + else + log_debug("Device write error %d offset %llu len %llu", errno, + (unsigned long long)(where + pos), + (unsigned long long)(len - pos)); free(io); return false; - } - - len -= r; + } + pos += rv; } - if (len) { - log_warn("short io %u bytes remaining", (unsigned) len); + if (pos < len) { + if (d == DIR_READ) + log_warn("Device read short %u bytes remaining", (unsigned)(len - pos)); + else + log_warn("Device write short %u bytes remaining", (unsigned)(len - pos)); + /* free(io); return false; + */ }