Messages in this thread Patch in this message | | | From | Xiaoming Ni <> | Subject | [PATCH] mtd:Fix issue where write_cached_data() fails but write() still returns success | Date | Fri, 20 Mar 2020 11:15:11 +0800 |
| |
mtdblock_flush() -->write_cached_data() --->erase_write() mtdblock: erase of region [0x40000, 0x20000] on "xxx" failed
Because mtdblock_flush() always returns 0, even if write_cached_data() fails and data is not written to the device, syscall_write() still returns success
Signed-off-by: Xiaoming Ni <nixiaoming@huawei.com> --- drivers/mtd/mtdblock.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/mtdblock.c b/drivers/mtd/mtdblock.c index c06b532..078e0f6 100644 --- a/drivers/mtd/mtdblock.c +++ b/drivers/mtd/mtdblock.c @@ -294,12 +294,13 @@ static void mtdblock_release(struct mtd_blktrans_dev *mbd) static int mtdblock_flush(struct mtd_blktrans_dev *dev) { struct mtdblk_dev *mtdblk = container_of(dev, struct mtdblk_dev, mbd); + int ret; mutex_lock(&mtdblk->cache_mutex); - write_cached_data(mtdblk); + ret = write_cached_data(mtdblk); mutex_unlock(&mtdblk->cache_mutex); mtd_sync(dev->mtd); - return 0; + return ret; } static void mtdblock_add_mtd(struct mtd_blktrans_ops *tr, struct mtd_info *mtd) -- 1.8.5.6
| |