Re: [PATCH] Add CONFIG_VFAT_NO_CREATE_WITH_LONGNAMES option

From: vimal singh
Date: Wed May 27 2009 - 08:06:15 EST


Any conclusion has been reached reagarding this patch...?

---
vimal

On Sat, May 2, 2009 at 5:29 PM, OGAWA Hirofumi
<hirofumi@xxxxxxxxxxxxxxxxxx> wrote:
> tridge@xxxxxxxxx writes:
>
>>  > Yes. I guess -ENAMETOOLONG would not good for !valid, and !(is_shortname
>>  > && base_info.valid && ext_info.valid) or add "else" part is more prefer
>>  > though.
>>
>> ok, new patch below.
>
> Looks good to me as technical stand.
>
>>  > Yes. The case-insensitive is ok, the dcache can handle case-insensitive.
>>
>> good, that makes life simpler :-)
>>
>> Are you happy with the shortname_flags hackery in
>> vfat_create_shortname() ?
>
> Ah. What is the intent to force WINNT option (lcase field)? To force it
> may not be good. So, change default if config is on?
> --
> OGAWA Hirofumi <hirofumi@xxxxxxxxxxxxxxxxxx>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/