Re: [PATCH 1/9] extents for ext4

Mingming Cao
2006-08-10T16:46:45Z

Andrew Morton wrote:

> On Wed, 09 Aug 2006 18:20:26 -0700
> Mingming Cao <cmm@us.ibm.com> wrote:
>
>
>>Add extent map support to ext4. Patch from Alex Tomas.
>>
>>On disk extents format:
>>/*
>> * this is extent on-disk structure
>> * it's used at the bottom of the tree
>> */
>>struct ext3_extent {
>> __le32 ee_block; /* first logical block extent covers */
>> __le16 ee_len; /* number of blocks covered by extent */
>> __le16 ee_start_hi; /* high 16 bits of physical block */
>> __le32 ee_start; /* low 32 bigs of physical block */
>>};
>>
>
>
>>From a quick scan:
>

> - There are several places which appear to be putting block numbers into
> an `int'.
>

This is fixed in [PATCH 4/9] 48bit support in extents, where we
converted those "int" type block numbers to ext4_fsblk_t (which is
typedefined as sector_t to support 48bit)

Thanks,
Mingming