Whamcloud - gitweb
LU-3035 clio: improper LASSERT in ll_file_io_generic()
authorNiu Yawei <yawei.niu@intel.com>
Thu, 28 Mar 2013 03:20:57 +0000 (23:20 -0400)
committerOleg Drokin <oleg.drokin@intel.com>
Sat, 30 Mar 2013 18:44:59 +0000 (14:44 -0400)
This LASSERT was introduced from the fix of LU-2910, and
which is incorrect, since the crw_count could be changed
in lov_io_rw_iter_init() even if no any read/write done.

Signed-off-by: Niu Yawei <yawei.niu@intel.com>
Change-Id: Ib2cee88f0a7f75f8fb63330912aa053fb5b9393e
Reviewed-on: http://review.whamcloud.com/5864
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Tested-by: Hudson
Reviewed-by: Bobi Jam <bobijam@gmail.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/llite/file.c

index 14e38cf..bd0e91f 100644 (file)
@@ -926,8 +926,7 @@ out:
                CDEBUG(D_VFSTRACE, "Restart %s on %s from %lld, count:%zd\n",
                       iot == CIT_READ ? "read" : "write",
                       file->f_dentry->d_name.name, *ppos, count);
-               LASSERTF(io->u.ci_rw.crw_count == count, "%zd != %zd\n",
-                        io->u.ci_rw.crw_count, count);
+               LASSERTF(io->ci_nob == 0, "%zd", io->ci_nob);
                goto restart;
        }