[Ocfs2-devel] [PATCH 1/1] Ocfs2: Add new OCFS2_IOC_INFO ioctl for ocfs2 v7.

[tristan]

Joel Becker wrote:
> On Tue, May 11, 2010 at 01:40:35PM -0700, Joel Becker wrote:
>   
>> On Tue, May 11, 2010 at 03:21:49PM +0800, Tristan Ye wrote:
>>     
>>> +/*
>>> + * This call is void because we are already reporting an error that may
>>> + * be -EFAULT.  The error will be returned from the ioctl(2) call.  It's
>>> + * just a best-effort to tell userspace that this request caused the error.
>>> + */
>>> +static inline void __o2info_error_to_user(struct ocfs2_info_request *kreq,
>>> +					  struct ocfs2_info_request __user *req)
>>> +{
>>> +	kreq->ir_flags |= OCFS2_INFO_FL_ERROR;
>>> +	(void)put_user(kreq->ir_flags, (__u32 __user *)&(req->ir_flags));
>>> +}
>>> +
>>> +#define o2info_error_to_user(a, b) \
>>> +		__o2info_error_to_user((struct ocfs2_info_request *)&(a), b)
>>>       
>> 	Hmm, since we're not actually passing the error up, perhaps it
>> should be called "[__]o2info_set_request_error(a, b)".
>>     
>
> Can you make this change so I can push this to .35?
>   

Sent out.

> Joel
>
>