Re: [PATCH 1/2] lib/strtox: introduce kstrtoull_suffix() helper
From: Andy Shevchenko
Date: Thu Dec 21 2023 - 15:55:40 EST
On Fri, Dec 22, 2023 at 07:07:31AM +1030, Qu Wenruo wrote:
> On 2023/12/21 22:30, Andy Shevchenko wrote:
...
> Then what about going the following path for the whole memparse() rabbit
> hole?
>
> - Mark the old memparse() deprecated
> - Add a new function memparse_safe() (or rename the older one to
> __memparse, and let the new one to be named memparse()?)
> - Add unit test for the new memparse_safe() or whatever the name is
> - Try my best to migrate as many call sites as possible
> Only the two btrfs ones I'm 100% confident for now
>
> Would that be a sounding plan?
As long as it does not break any ABI (like kernel command line parsing),
sounds good to me.
--
With Best Regards,
Andy Shevchenko