ext4: Simplify last in leaf check in ext4_map_query_blocks
authorRitesh Harjani (IBM) <ritesh.list@gmail.com>
Mon, 19 May 2025 18:19:27 +0000 (23:49 +0530)
committerTheodore Ts'o <tytso@mit.edu>
Tue, 20 May 2025 18:21:00 +0000 (14:21 -0400)
This simplifies the check for last in leaf in ext4_map_query_blocks()
and fixes this cocci warning.

cocci warnings: (new ones prefixed by >>)
>> fs/ext4/inode.c:573:49-51: WARNING !A || A && B is equivalent to !A || B

Fixes: 5bb12b1837c0 ("ext4: Add support for EXT4_GET_BLOCKS_QUERY_LEAF_BLOCKS")
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202505191524.auftmOwK-lkp@intel.com/
Signed-off-by: Ritesh Harjani (IBM) <ritesh.list@gmail.com>
Reviewed-by: Ojaswin Mujoo <ojaswin@linux.ibm.com>
Link: https://patch.msgid.link/5fd5c806218c83f603c578c95997cf7f6da29d74.1747677758.git.ritesh.list@gmail.com
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
fs/ext4/inode.c

index ce0632094c50c15608bb69a9471f34c287b56ef0..459ffc6af1d32296cf85470e2797b27188a3f45f 100644 (file)
@@ -570,8 +570,7 @@ static int ext4_map_query_blocks(handle_t *handle, struct inode *inode,
         * - if the last in leaf is the full requested range
         */
        if (!(map->m_flags & EXT4_MAP_QUERY_LAST_IN_LEAF) ||
-                       ((map->m_flags & EXT4_MAP_QUERY_LAST_IN_LEAF) &&
-                        (map->m_len == orig_mlen))) {
+                       map->m_len == orig_mlen) {
                status = map->m_flags & EXT4_MAP_UNWRITTEN ?
                                EXTENT_STATUS_UNWRITTEN : EXTENT_STATUS_WRITTEN;
                ext4_es_insert_extent(inode, map->m_lblk, map->m_len,