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, §or_size) < 0) > +#if defined(BLKPBSZGET) && defined(BLKSSZGET) > + if ((ioctl(fd, BLKPBSZGET, §or_size) < 0) && (ioctl(fd, > BLKSSZGET, §or_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