[Cluster-devel] [PATCH 1/1] GFS2: Replace PTR_RET with PTR_ERR_OR_ZERO
Rusty Russell
rusty at rustcorp.com.au
Tue Jul 16 06:35:56 UTC 2013
Steven Whitehouse <swhiteho at redhat.com> writes:
> Hi,
>
> On Mon, 2013-07-15 at 16:58 +0530, Sachin Kamat wrote:
>> PTR_RET is now deprecated. Use PTR_ERR_OR_ZERO instead.
>>
>> Signed-off-by: Sachin Kamat <sachin.kamat at linaro.org>
>> ---
>> Compile tested and based on the following tree:
>> git://git.kernel.org/pub/scm/linux/kernel/git/rusty/linux.git (PTR_RET)
>>
>> Dependent on [1]
>> [1] http://lkml.indiana.edu/hypermail/linux/kernel/1306.2/00010.html
>> ---
>> fs/gfs2/inode.c | 3 ++-
>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/fs/gfs2/inode.c b/fs/gfs2/inode.c
>> index bbb2715..a186ebd 100644
>> --- a/fs/gfs2/inode.c
>> +++ b/fs/gfs2/inode.c
>> @@ -19,6 +19,7 @@
>> #include <linux/crc32.h>
>> #include <linux/fiemap.h>
>> #include <linux/security.h>
>> +#include <linux/err.h>
>> #include <asm/uaccess.h>
>>
>> #include "gfs2.h"
>> @@ -594,7 +595,7 @@ static int gfs2_create_inode(struct inode *dir, struct dentry *dentry,
>> }
>> gfs2_glock_dq_uninit(ghs);
>> if (IS_ERR(d))
>> - return PTR_RET(d);
>> + return PTR_ERR_OR_ZERO(d);
>
> I'm not sure I follow what this is supposed to be doing... what is the
> reason for this change? This macro/function doesn't seem to be defined
> in the current kernel, so I assume that it is "coming soon" but the
> thread pointed to above wasn't very enlightening,
It's a clarification rename.
But this fix is wrong, it should just be changed to PTR_ERR(d). It
never needed PTR_RET() in the first place.
Thanks,
Rusty.
More information about the Cluster-devel
mailing list