[PATCH 03/14] repair: kill B_IS_META flag

[Alex Elder]

On , Eric Sandeen wrote:
> Christoph Hellwig wrote:
>> On Mon, Oct 12, 2009 at 02:45:08PM -0500, Eric Sandeen wrote:
>>> Christoph Hellwig wrote:
>>> 
>>>> B_IS_META is the inverse flag of B_IS_INODE which is not really obvious
>>>> from it's use.  So just use !B_IS_INODE to make it more clear.
>>>> 
>>> Logic-wise it's fine, but is this change really helpful?   The comment says:
>>> 
>>> /*
>>> * Test if bit 0 or 2 is set in the "priority tag" of the buffer to see if
>>> * the buffer is for an inode or other metadata.
>>> */
>>> 
>>> so basically it distinguishes inodes from other metadata right.
>> 
>> Yes, with the key on other.  In my books inodes are meta-data.
>> 
>>> B_IS_INODE is clear; B_IS_META is pretty clear, "!B_IS_INODE" seems muddy; so
>>> very many things are "not inodes" :)
>> 
>> In a buffercache (and in fact a whole application) that only deals with
>> metadata at all !B_IS_INODE meaning other metadata seems a lot more clear to me than B_IS_META.
>> 
> 
> Ok, I'm fine with that I suppose.

Reviewed-by: Alex Elder <aelder at sgi.com>