lkml.org 
[lkml]   [2022]   [Aug]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v2 1/2] libfs: Introduce tokenize_user_input()
From
On 2022-08-30 8:33 PM, Matthew Wilcox wrote:
> On Thu, Aug 25, 2022 at 06:48:32PM +0200, Cezary Rojewski wrote:
>> Add new helper function to allow for splitting specified user string
>> into a sequence of integers. Internally it makes use of get_options() so
>> the returned sequence contains the integers extracted plus an additional
>> element that begins the sequence and specifies the integers count.
>>
>> Suggested-by: Andy Shevchenko <andy.shevchenko@gmail.com>
>> Signed-off-by: Cezary Rojewski <cezary.rojewski@intel.com>
>> ---
>> fs/libfs.c | 45 +++++++++++++++++++++++++++++++++++++++++++++
>> include/linux/fs.h | 1 +
>
> This really has nothing to do with filesystems. Surely
> string_helpers.[ch] is the appropriate place for this code?
> Also get_options() should probably move its prototype from kernel.h to
> string_helpers.h.


Hello Matthew,

Thanks for your input. The initial version of the change was located in
the string_helpers.[ch] just like you propose and I have no problem
moving it back again. string_helpers are devoid of __user content though
and that's why in v2 it's part of libfs instead.

I'll give a day or two to see if there are other suggestions and then
send v3 relocating the implementation.


Regards,
Czarek

\
 
 \ /
  Last update: 2022-08-31 18:16    [W:0.341 / U:0.768 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site