In previous bcache patches for Linux v5.2, the failure code path of
run_cache_set() is tested and fixed. So now the following comment
line can be removed from run_cache_set(),
        /* XXX: test this, it's broken */

Signed-off-by: Coly Li <[email protected]>
---
 drivers/md/bcache/super.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/md/bcache/super.c b/drivers/md/bcache/super.c
index 877113b62b0f..3364b20567eb 100644
--- a/drivers/md/bcache/super.c
+++ b/drivers/md/bcache/super.c
@@ -1946,7 +1946,7 @@ static int run_cache_set(struct cache_set *c)
        }
 
        closure_sync(&cl);
-       /* XXX: test this, it's broken */
+
        bch_cache_set_error(c, "%s", err);
 
        return -EIO;
-- 
2.16.4

Reply via email to