Whamcloud - gitweb
LU-18987 mdt: Remove stray assertion for dom read 73/59173/2
authorOleg Drokin <green@whamcloud.com>
Fri, 9 May 2025 23:21:10 +0000 (19:21 -0400)
committerOleg Drokin <green@whamcloud.com>
Wed, 14 May 2025 03:58:11 +0000 (03:58 +0000)
There's a strange assertion in mdt_dom_read_on_open()
that does not make sense and checks parameters in
a loop that don't even change.

Let's remove it.

Change-Id: I0662b621a39fbde79415676a8486ec4f9ab3a2ab
Test-Parameters: trivial
Fixes: 13372d6c243c ("LU-10181 mdt: read on open for DoM files")
Signed-off-by: Oleg Drokin <green@whamcloud.com>
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/59173
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Mikhail Pershin <mpershin@whamcloud.com>
lustre/mdt/mdt_io.c

index 690b110..f3cc37c 100644 (file)
@@ -1942,7 +1942,6 @@ int mdt_dom_read_on_open(struct mdt_thread_info *mti, struct mdt_device *mdt,
                memcpy(buf + (i << PAGE_SHIFT), p, lnb[i].lnb_len);
                kunmap(lnb[i].lnb_page);
                copied += lnb[i].lnb_len;
-               LASSERT(rc <= len);
        }
        CDEBUG(D_INFO, "Read %i (wanted %u) bytes from %llu\n", copied,
               len, offset);