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-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
