Re: [f2fs-dev] [PATCH] f2fs-tools: prefer physical over logical block size

[Chao Yu via Linux-f2fs-devel]

On 4/25/25 09:36, nl6720 via Linux-f2fs-devel wrote:
> Some drives operate in "512e" configuration with their logical block
> size set to 512 bytes for legacy compatibility reasons while providing
> a more optimal 4096 byte value as the physical block size.
> 
> Since the physical block size is the smallest unit a physical storage
> device can write atomically, prefer it over the logical block size.
> 
> Closes: https://github.com/jaegeuk/f2fs-tools/issues/29

f2fs uses PAGE_SIZE as block size, commonly it's 4096 rather than 512?

Thanks,

> 
> Signed-off-by: nl6720 <devn...@nl6720.me>
> ---
>  lib/libf2fs.c | 9 +++++++--
>  1 file changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/lib/libf2fs.c b/lib/libf2fs.c
> index 
> d2579d76a5c7eaf141dc8aad11c176eeadabad83..040ea4bc87288fa075cb4bf51df08b9db28a65b3
>  100644
> --- a/lib/libf2fs.c
> +++ b/lib/libf2fs.c
> @@ -933,10 +933,15 @@ void get_kernel_uname_version(__u8 *version)
>  #define BLKSSZGET    _IO(0x12,104)
>  #endif
>  
> +#if defined(__linux__) && defined(_IO) && !defined(BLKPBSZGET)
> +#define BLKPBSZGET   _IO(0x12,123)
> +#endif
> +
>  #if defined(__APPLE__)
>  #include <sys/disk.h>
>  #define BLKGETSIZE   DKIOCGETBLOCKCOUNT
>  #define BLKSSZGET    DKIOCGETBLOCKCOUNT
> +#define BLKPBSZGET   DKIOCGETBLOCKCOUNT
>  #endif /* APPLE_DARWIN */
>  
>  #ifndef _WIN32
> @@ -1050,8 +1055,8 @@ int get_device_info(int i)
>       } else if (S_ISREG(stat_buf->st_mode)) {
>               dev->total_sectors = stat_buf->st_size / dev->sector_size;
>       } else if (S_ISBLK(stat_buf->st_mode)) {
> -#ifdef BLKSSZGET
> -             if (ioctl(fd, BLKSSZGET, &sector_size) < 0)
> +#if defined(BLKPBSZGET) && defined(BLKSSZGET)
> +             if ((ioctl(fd, BLKPBSZGET, &sector_size) < 0) && (ioctl(fd, 
> BLKSSZGET, &sector_size) < 0))
>                       MSG(0, "\tError: Using the default sector size\n");
>               else if (dev->sector_size < sector_size)
>                       dev->sector_size = sector_size;



_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel