Re: [PATCH] xfs: stop using simple_strtoul()

[Abhijit Pawar <abhi.c.pawar@xxxxxxxxx>]

Hi Jeff,
On 01/08/2013 12:51 PM, Jeff Liu wrote:
> Hi Ben and Abhijit,
> 
> On 01/08/2013 04:49 AM, Ben Myers wrote:
>> Hey Jeff & Abhijit,
>>
>> On Sun, Jan 06, 2013 at 11:50:08AM +0800, Jeff Liu wrote:
>>> This small patch convert xfs_parseargs() and suffix_strtoul() to
>>> use kstrtoint() instead of simple_strtoul() which is deprecated.
>>
>> It looks like Jeff's patch will conflict with a patch from Abhijit Pawar, 
>> 'fs:
>> remove obsolete simple_strto<foo>' which we saw Dec 7.  You guys went after 
>> the
>> same thing and came up with slightly differing results.  If Abhijit would
>> prefer to factor out the xfs changes I'd be happy to take either that, or
>> Jeff's patch, but it would be better if they didn't conflict.  Can y'all come
>> to an agreement as to what the xfs changes should be?
> Sorry, I missed that patch from Abhijit.
> 
> Looks the previous patch did not replaced the simple_strtoul() with
> kstrtoint() in suffix_strtoul().
> 
> Abhijit, would you like to fix it as well?
Yes,it did not replace the call. Looks like I have missed that piece of
code. Once Ben applies my original patch, I will send another patch for
this on top of that.

> 
> Thanks,
> -Jeff
> 


-- 
-
Abhijit