On 11/13/13, 12:42 PM, Brian Foster wrote:
> On 11/13/2013 12:52 PM, Eric Sandeen wrote:
>> The "verbose" argument has been unused since
>> ffe37436 xfs: stop using the page cache to back the buffer cache
>>
>> Remove it, and then give the function a better (?) name since
>> no flags are involved.
>>
>> Fix inconsistent comment spacing while we're at it.
>>
>> Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx>
>> ---
>>
>> diff --git a/fs/xfs/xfs_buf.c b/fs/xfs/xfs_buf.c
>> index 2634700..85d916a 100644
>> --- a/fs/xfs/xfs_buf.c
>> +++ b/fs/xfs/xfs_buf.c
>> @@ -1602,11 +1602,10 @@ xfs_free_buftarg(
>> }
>>
>> STATIC int
>> -xfs_setsize_buftarg_flags(
>> +__xfs_setsize_buftarg(
>> xfs_buftarg_t *btp,
>> unsigned int blocksize,
>> - unsigned int sectorsize,
>> - int verbose)
>> + unsigned int sectorsize)
>> {
>> btp->bt_bsize = blocksize;
>> btp->bt_sshift = ffs(sectorsize) - 1;
>> @@ -1627,17 +1626,17 @@ xfs_setsize_buftarg_flags(
>> }
>>
>> /*
>> - * When allocating the initial buffer target we have not yet
>> - * read in the superblock, so don't know what sized sectors
>> - * are being used at this early stage. Play safe.
>> + * When allocating the initial buffer target we have not yet
>> + * read in the superblock, so don't know what sized sectors
>> + * are being used at this early stage. Play safe.
>> */
>> STATIC int
>> xfs_setsize_buftarg_early(
>> xfs_buftarg_t *btp,
>> struct block_device *bdev)
>> {
>> - return xfs_setsize_buftarg_flags(btp,
>> - PAGE_SIZE, bdev_logical_block_size(bdev), 0);
>> + return __xfs_setsize_buftarg(btp,
>> + PAGE_SIZE, bdev_logical_block_size(bdev));
>> }
>>
>> int
>> @@ -1646,7 +1645,7 @@ xfs_setsize_buftarg(
>> unsigned int blocksize,
>> unsigned int sectorsize)
>> {
>> - return xfs_setsize_buftarg_flags(btp, blocksize, sectorsize, 1);
>> + return __xfs_setsize_buftarg(btp, blocksize, sectorsize);
>> }
>>
>
> At this point is there a need for the separate __xfs_setsize_buftarg()
> helper? E.g., fold the implementation into xfs_setsize_buftarg() and let
> xfs_setsize_buftarg_early() call xfs_setsize_buftarg()..?
>
> Brian
>
Oh that probably makes sense. :) I got tunnel vision. I'll send V2.
Thanks,
-Eric
|