On 06/09/2010 10:46 PM, Fred Isaman wrote:
>>>>> static inline enum pnfs_try_status
>>>>> pnfs_try_to_read_data(struct nfs_read_data *data,
>>>>> const struct rpc_call_ops *call_ops)
>>>>
>>>> Don't think this needs to be inline, whats the point?
>>>>
>>>
>>> The point is that it is in the header file, not a c file.
>>>
>>
>> That's what I meant. Why is it in the header file. Why not
>> in .c file and declared.
>>
>
> To make it easy to ifdef out if CONFIG_NFS_V4_1 is not set.
>
> Fred
>
It's not easier, just:
#ifdef CONFIG_NFS_V4_1
enum pnfs_try_status
pnfs_try_to_read_data(struct nfs_read_data *data,
const struct rpc_call_ops *call_ops); /* declaration */
#else
static inline enum pnfs_try_status
pnfs_try_to_read_data(struct nfs_read_data *data,
const struct rpc_call_ops *call_ops) /* Empty inline */
{
return ...;
}
#endif
We do that everywhere. Should also here
Boaz
>>> Fred
>>>
>>
>> (-Bz
>>
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
