From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-10.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A, SPF_HELO_NONE,SPF_PASS,UNPARSEABLE_RELAY,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E9FEAC433DB for ; Wed, 24 Mar 2021 20:18:22 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id B30AD61A0A for ; Wed, 24 Mar 2021 20:18:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237915AbhCXURu (ORCPT ); Wed, 24 Mar 2021 16:17:50 -0400 Received: from bhuna.collabora.co.uk ([46.235.227.227]:38958 "EHLO bhuna.collabora.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238132AbhCXURf (ORCPT ); Wed, 24 Mar 2021 16:17:35 -0400 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: tonyk) with ESMTPSA id BCFFA1F45E77 Subject: Re: [RFC PATCH 2/4] mm: shmem: Support case-insensitive file name lookups To: Gabriel Krisman Bertazi Cc: Hugh Dickins , Andrew Morton , Alexander Viro , smcv@collabora.com, kernel@collabora.com, linux-mm@kvack.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Daniel Rosenberg References: <20210323195941.69720-1-andrealmeid@collabora.com> <20210323195941.69720-3-andrealmeid@collabora.com> <877dlxd3oc.fsf@collabora.com> From: =?UTF-8?Q?Andr=c3=a9_Almeida?= Message-ID: Date: Wed, 24 Mar 2021 17:17:21 -0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1 MIME-Version: 1.0 In-Reply-To: <877dlxd3oc.fsf@collabora.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org Às 17:18 de 23/03/21, Gabriel Krisman Bertazi escreveu: > André Almeida writes: >> opt = fs_parse(fc, shmem_fs_parameters, param, &result); >> if (opt < 0) >> @@ -3468,6 +3519,23 @@ static int shmem_parse_one(struct fs_context *fc, struct fs_parameter *param) >> ctx->full_inums = true; >> ctx->seen |= SHMEM_SEEN_INUMS; >> break; >> + case Opt_casefold: >> + if (strncmp(param->string, "utf8-", 5)) >> + return invalfc(fc, "Only utf8 encondings are supported"); >> + ret = strscpy(version, param->string + 5, sizeof(version)); > > Ugh. Now we are doing two strscpy for the parse api (in unicode_load). > Can change the unicode_load api to reuse it? > So instead of getting just the version number (e.g. "12.1.0") as parameter, utf8_load/unicode_load would get the full encoding string (e.g. "utf8-12.1.0") right?