[PATCH SLOF v3 3/5] disk-label: rename confusing "block" word

Nikunj A Dadhania nikunj at linux.vnet.ibm.com
Thu Jul 2 15:47:49 AEST 2015


Hi Segher,

Segher Boessenkool <segher at kernel.crashing.org> writes:
> On Tue, Jun 30, 2015 at 04:31:19PM +0530, Nikunj A Dadhania wrote:
>> "block" word is not a block number, actually its an allocated host
>> address.  Rename it to disk-buf along with a associated
>> size(disk-buf-size=4096) for using during allocation/free.
>> 
>> Also renaming the helper routine read-sector to read-disk-buf. This
>> routine assumes the address to be disk-buf and only takes sector number
>> as argument.
>
> This isn't what I suggested, and I think it is a terrible idea.

The comment was against the "has-fat-filesystem". As the complete
disk-label.fs had that same assumption, I went ahead and renamed "block"
across the file.

Are you suggesting to drop complete patch or just the rename of
"read-sector" ?

> Just FWIW :-)

Regards
Nikunj



More information about the Linuxppc-dev mailing list