Commit 70d7ced2 authored by Shijie Luo's avatar Shijie Luo Committed by Theodore Ts'o
Browse files

ext4: change to use fallthrough macro 



Change to use fallthrough macro in switch case.

Signed-off-by: default avatarShijie Luo <luoshijie1@huawei.com>
Reviewed-by: default avatarRitesh Harjani <riteshh@linux.ibm.com>
Link: https://lore.kernel.org/r/20200810114435.24182-1-luoshijie1@huawei.com


Signed-off-by: default avatarTheodore Ts'o <tytso@mit.edu>
parent 2fe34d29

fs/ext4/hash.c

+2 −2
Original line number Diff line number Diff line
@@ -233,7 +233,7 @@ static int __ext4fs_dirhash(const char *name, int len, 
		break;

	case DX_HASH_HALF_MD4_UNSIGNED:

		str2hashbuf = str2hashbuf_unsigned;

		/* fall through */

		fallthrough;

	case DX_HASH_HALF_MD4:

		p = name;

		while (len > 0) {
@@ -247,7 +247,7 @@ static int __ext4fs_dirhash(const char *name, int len, 
		break;

	case DX_HASH_TEA_UNSIGNED:

		str2hashbuf = str2hashbuf_unsigned;

		/* fall through */

		fallthrough;

	case DX_HASH_TEA:

		p = name;

		while (len > 0) {

fs/ext4/indirect.c

+6 −6
Original line number Diff line number Diff line
@@ -1180,21 +1180,21 @@ void ext4_ind_truncate(handle_t *handle, struct inode *inode) 
			ext4_free_branches(handle, inode, NULL, &nr, &nr+1, 1);

			i_data[EXT4_IND_BLOCK] = 0;

		}

		/* fall through */

		fallthrough;

	case EXT4_IND_BLOCK:

		nr = i_data[EXT4_DIND_BLOCK];

		if (nr) {

			ext4_free_branches(handle, inode, NULL, &nr, &nr+1, 2);

			i_data[EXT4_DIND_BLOCK] = 0;

		}

		/* fall through */

		fallthrough;

	case EXT4_DIND_BLOCK:

		nr = i_data[EXT4_TIND_BLOCK];

		if (nr) {

			ext4_free_branches(handle, inode, NULL, &nr, &nr+1, 3);

			i_data[EXT4_TIND_BLOCK] = 0;

		}

		/* fall through */

		fallthrough;

	case EXT4_TIND_BLOCK:

		;

	}
@@ -1434,7 +1434,7 @@ int ext4_ind_remove_space(handle_t *handle, struct inode *inode, 
			ext4_free_branches(handle, inode, NULL, &nr, &nr+1, 1);

			i_data[EXT4_IND_BLOCK] = 0;

		}

		/* fall through */

		fallthrough;

	case EXT4_IND_BLOCK:

		if (++n >= n2)

			break;
@@ -1443,7 +1443,7 @@ int ext4_ind_remove_space(handle_t *handle, struct inode *inode, 
			ext4_free_branches(handle, inode, NULL, &nr, &nr+1, 2);

			i_data[EXT4_DIND_BLOCK] = 0;

		}

		/* fall through */

		fallthrough;

	case EXT4_DIND_BLOCK:

		if (++n >= n2)

			break;
@@ -1452,7 +1452,7 @@ int ext4_ind_remove_space(handle_t *handle, struct inode *inode, 
			ext4_free_branches(handle, inode, NULL, &nr, &nr+1, 3);

			i_data[EXT4_TIND_BLOCK] = 0;

		}

		/* fall through */

		fallthrough;

	case EXT4_TIND_BLOCK:

		;

	}

fs/ext4/readpage.c

+2 −2
Original line number Diff line number Diff line
@@ -140,7 +140,7 @@ static void bio_post_read_processing(struct bio_post_read_ctx *ctx) 
			return;

		}

		ctx->cur_step++;

		/* fall-through */

		fallthrough;

	case STEP_VERITY:

		if (ctx->enabled_steps & (1 << STEP_VERITY)) {

			INIT_WORK(&ctx->work, verity_work);
@@ -148,7 +148,7 @@ static void bio_post_read_processing(struct bio_post_read_ctx *ctx) 
			return;

		}

		ctx->cur_step++;

		/* fall-through */

		fallthrough;

	default:

		__read_end_io(ctx->bio);

	}