Re: [PATCH RESEND 13/62] ext2: remove ext2_image_size and associated code

Jan Kara <[email protected]>
Newsgroups gmane.linux.uml.devel,gmane.linux.file-systems,gmane.linux.kernel,gmane.linux.kernel.cross-arch,gmane.linux.ports.alpha,gmane.linux.kernel.arc,gmane.linux.ports.arm.kernel,gmane.linux.ports.hexagon,gmane.linux.ports.mips,gmane.linux.ports.parisc,gmane.linux.ports.ppc64.devel,gmane.linux.ports.riscv,gmane.linux.ports.sh.devel,gmane.linux.ports.sparc,gmane.linux.block,gmane.linux.kernel.initramfs,gmane.linux.kernel.api,gmane.linux.documentation,gmane.linux.kernel.efi,gmane.comp.file-systems.ext4,gmane.linux.acpi.devel,gmane.linux.drivers.devicetree
Message-ID <5xr5efvf4dhy43fchbvfsxspzgde5bxezhszdgqcya4eqrocgy@lqqkaq5wok6a>
On Sat 13-09-25 00:37:52, Askar Safin wrote:
> It is not used anymore
> 
> Signed-off-by: Askar Safin <[email protected]>

Looks good.

Acked-by: Jan Kara <[email protected]>

								Honza

> ---
>  fs/ext2/ext2.h          |  9 ---------
>  include/linux/ext2_fs.h | 13 -------------
>  2 files changed, 22 deletions(-)
> 
> diff --git a/fs/ext2/ext2.h b/fs/ext2/ext2.h
> index cf97b76e9fd3..d623a14040d9 100644
> --- a/fs/ext2/ext2.h
> +++ b/fs/ext2/ext2.h
> @@ -608,15 +608,6 @@ struct ext2_dir_entry_2 {
>  					 ~EXT2_DIR_ROUND)
>  #define EXT2_MAX_REC_LEN		((1<<16)-1)
>  
> -static inline void verify_offsets(void)
> -{
> -#define A(x,y) BUILD_BUG_ON(x != offsetof(struct ext2_super_block, y));
> -	A(EXT2_SB_MAGIC_OFFSET, s_magic);
> -	A(EXT2_SB_BLOCKS_OFFSET, s_blocks_count);
> -	A(EXT2_SB_BSIZE_OFFSET, s_log_block_size);
> -#undef A
> -}
> -
>  /*
>   * ext2 mount options
>   */
> diff --git a/include/linux/ext2_fs.h b/include/linux/ext2_fs.h
> index 1fef88569037..e5ebe6cdf06c 100644
> --- a/include/linux/ext2_fs.h
> +++ b/include/linux/ext2_fs.h
> @@ -27,17 +27,4 @@
>   */
>  #define EXT2_LINK_MAX		32000
>  
> -#define EXT2_SB_MAGIC_OFFSET	0x38
> -#define EXT2_SB_BLOCKS_OFFSET	0x04
> -#define EXT2_SB_BSIZE_OFFSET	0x18
> -
> -static inline u64 ext2_image_size(void *ext2_sb)
> -{
> -	__u8 *p = ext2_sb;
> -	if (*(__le16 *)(p + EXT2_SB_MAGIC_OFFSET) != cpu_to_le16(EXT2_SUPER_MAGIC))
> -		return 0;
> -	return (u64)le32_to_cpup((__le32 *)(p + EXT2_SB_BLOCKS_OFFSET)) <<
> -		le32_to_cpup((__le32 *)(p + EXT2_SB_BSIZE_OFFSET));
> -}
> -
>  #endif	/* _LINUX_EXT2_FS_H */
> -- 
> 2.47.2
> 
-- 
Jan Kara <[email protected]>
SUSE Labs, CR
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.