RE: [PATCH 1/4] exfat: redefine PBR as boot_sector

From: Sungjong Seo
Date: Thu May 28 2020 - 01:31:27 EST


> Aggregate PBR related definitions and redefine as "boot_sector" to comply
> with the exFAT specification.
> And, rename variable names including 'pbr'.
>
> Signed-off-by: Tetsuhiro Kohada <kohada.t2@xxxxxxxxx>
> ---
> fs/exfat/exfat_fs.h | 2 +-
> fs/exfat/exfat_raw.h | 79 +++++++++++++++--------------------------
> fs/exfat/super.c | 84 ++++++++++++++++++++++----------------------
> 3 files changed, 72 insertions(+), 93 deletions(-)
>
[snip]
> +/* EXFAT: Main and Backup Boot Sector (512 bytes) */ struct boot_sector
> +{
> + __u8 jmp_boot[BOOTSEC_JUMP_BOOT_LEN];
> + __u8 oem_name[BOOTSEC_OEM_NAME_LEN];

According to the exFAT specification, fs_name and BOOTSEC_FS_NAME_LEN look
better.

> + __u8 must_be_zero[BOOTSEC_OLDBPB_LEN];
> + __le64 partition_offset;
> + __le64 vol_length;
> + __le32 fat_offset;
> + __le32 fat_length;
> + __le32 clu_offset;
> + __le32 clu_count;
> + __le32 root_cluster;
> + __le32 vol_serial;
> + __u8 fs_revision[2];
> + __le16 vol_flags;
> + __u8 sect_size_bits;
> + __u8 sect_per_clus_bits;
> + __u8 num_fats;
> + __u8 drv_sel;
> + __u8 percent_in_use;
> + __u8 reserved[7];
> + __u8 boot_code[390];
> + __le16 signature;
> } __packed;