File bug-1150021_09-bcache-reverse-earlier-patch.patch of Package lvm2-clvm.16468

Overview Repositories Revisions Requests Users Attributes Meta

File bug-1150021_09-bcache-reverse-earlier-patch.patch of Package lvm2-clvm.16468

From 1e2e12f19c584c3568c255c5d89ed8aadbe61b9c Mon Sep 17 00:00:00 2001
From: Joe Thornber <ejt@redhat.com>
Date: Tue, 29 Oct 2019 15:14:07 +0000
Subject: [PATCH 7/9] [bcache] reverse earlier patch.

It broke some unit tests, for v. little benefit
---
 lib/device/bcache.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/lib/device/bcache.c b/lib/device/bcache.c
index 56e3db3..72cc599 100644
--- a/lib/device/bcache.c
+++ b/lib/device/bcache.c
@@ -246,10 +246,6 @@ static bool _async_issue(struct io_engine *ioe, enum dir d, int fd,
 
 	if (r < 0) {
 		_cb_free(e->cbs, cb);
-	       ((struct block *) context)->error = r;
-	       log_warn("io_submit <%c> off %llu bytes %llu return %d:%s",
-	                (d == DIR_READ) ? 'R' : 'W', (long long unsigned)offset,
-                        (long long unsigned) nbytes, r, strerror(-r));
 		return false;
 	}
 
@@ -788,7 +784,8 @@ static void _issue_low_level(struct block *b, enum dir d)
 	dm_list_move(&cache->io_pending, &b->list);
 
 	if (!cache->engine->issue(cache->engine, d, b->fd, sb, se, b->data, b)) {
-		_complete_io(b, b->error);
+		/* FIXME: if io_submit() set an errno, return that instead of EIO? */
+		_complete_io(b, -EIO);
 		return;
 	}
 }
-- 
1.8.3.1

Places

File bug-1150021_09-bcache-reverse-earlier-patch.patch of Package lvm2-clvm.16468

Places