From: Javier González <jav...@javigon.com>

Return a meaningful error when the sanity vector I/O check fails.

Signed-off-by: Javier González <jav...@cnexlabs.com>
Signed-off-by: Matias Bjørling <m...@lightnvm.io>
---
 drivers/lightnvm/pblk-core.c | 22 ++++++++--------------
 1 file changed, 8 insertions(+), 14 deletions(-)

diff --git a/drivers/lightnvm/pblk-core.c b/drivers/lightnvm/pblk-core.c
index 2cad918434a7..0d4078805ecc 100644
--- a/drivers/lightnvm/pblk-core.c
+++ b/drivers/lightnvm/pblk-core.c
@@ -467,16 +467,13 @@ int pblk_submit_io(struct pblk *pblk, struct nvm_rq *rqd)
 {
        struct nvm_tgt_dev *dev = pblk->dev;
 
+       atomic_inc(&pblk->inflight_io);
+
 #ifdef CONFIG_NVM_DEBUG
-       int ret;
-
-       ret = pblk_check_io(pblk, rqd);
-       if (ret)
-               return ret;
+       if (pblk_check_io(pblk, rqd))
+               return NVM_IO_ERR;
 #endif
 
-       atomic_inc(&pblk->inflight_io);
-
        return nvm_submit_io(dev, rqd);
 }
 
@@ -484,16 +481,13 @@ int pblk_submit_io_sync(struct pblk *pblk, struct nvm_rq 
*rqd)
 {
        struct nvm_tgt_dev *dev = pblk->dev;
 
+       atomic_inc(&pblk->inflight_io);
+
 #ifdef CONFIG_NVM_DEBUG
-       int ret;
-
-       ret = pblk_check_io(pblk, rqd);
-       if (ret)
-               return ret;
+       if (pblk_check_io(pblk, rqd))
+               return NVM_IO_ERR;
 #endif
 
-       atomic_inc(&pblk->inflight_io);
-
        return nvm_submit_io_sync(dev, rqd);
 }
 
-- 
2.11.0

Reply via email to