Re: CONFIG_VFAT_FS_DUALNAMES regressions

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Jan Engelhardt <jengelh@...>
Cc: Theodore Tso <tytso@...>, Alan Cox <alan@...>, Rusty Russell <rusty@...>, Pavel Machek <pavel@...>, OGAWA Hirofumi <hirofumi@...>, <john.lanza@...>, Linux Kernel Mailing List <linux-kernel@...>, <linux-fsdevel@...>, Dave Kleikamp <shaggy@...>, <corbet@...>, <jcm@...>
Date: Thursday, July 2, 2009 - 8:14 pm

Hi Jan,

I should also mention that in the first patch I sent back in May, I
added some code that made the 8.3 name problem easier on the user. It
basically forced the shortname= option, overriding the mount option,
when the patch was triggered. The end result was that a name like
dcis3000.jpg would always be stored as a 8.3 name, and thus be visible
to digital cameras, even if the mount options were set to say that
lowercase names should be considered as "long" names.

Hirofumi objected, on the basis that I was overriding the VFAT
mount option. I think that overriding in this case would in fact be
worthwhile, as I don't think many users ever think carefully about
what shortname= option they are using, and unless you get it right
then you can get some real surprises.

Hirofumi-san, do you think that same objection you expressed
previously also applies to this new patch? For example, would you
think it reasonable to modify the patch to store the filename as 8.3
if it can fit in 8.3 in a case mapped manner? Or store as 8.3 if it is
all uppercase or all lowercase?

I think we will hit cases like the one Jan has pointed out where the
exising shortname= options will become even more confusing than they
were already when this new patch is applied, especially as I think
most distros these days magically supply a shortname= option when you
insert some VFAT formatted media.

Cheers, Tridge
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Pavel Machek, (Tue Jun 30, 2:31 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Pavel Machek, (Thu Jul 2, 6:00 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Pavel Machek, (Thu Jul 2, 6:41 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Boaz Harrosh, (Wed Jul 1, 7:48 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, James Bottomley, (Wed Jul 1, 11:44 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Jamie Lokier, (Mon Jul 6, 4:41 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Boaz Harrosh, (Tue Jul 7, 6:02 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Jamie Lokier, (Tue Jul 7, 7:25 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Boaz Harrosh, (Tue Jul 7, 7:48 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Pavel Machek, (Thu Jul 2, 6:03 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Alan Cox, (Wed Jul 1, 10:41 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, James Bottomley, (Thu Jul 2, 10:56 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, James Bottomley, (Thu Jul 2, 5:25 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Theodore Tso, (Thu Jul 2, 11:27 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Alan Cox, (Thu Jul 2, 12:56 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, OGAWA Hirofumi, (Thu Jul 2, 10:50 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Rusty Russell, (Wed Jul 1, 6:49 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Pavel Machek, (Thu Jul 2, 5:46 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Rusty Russell, (Thu Jul 2, 7:55 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Pavel Machek, (Thu Jul 2, 6:33 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Pavel Machek, (Thu Jul 2, 6:44 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Rusty Russell, (Thu Jul 2, 8:03 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Pavel Machek, (Wed Jul 8, 5:21 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, H. Peter Anvin, (Wed Jul 8, 12:46 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Paul E. McKenney, (Wed Jul 8, 10:25 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Paul E. McKenney, (Wed Jul 8, 6:14 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Paul E. McKenney, (Wed Jul 8, 7:59 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Rusty Russell, (Wed Jul 1, 8:34 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Theodore Tso, (Wed Jul 1, 10:05 am)
Re: CONFIG_VFAT_FS_DUALNAMES regression, Jan Engelhardt, (Thu Jul 2, 7:17 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Thu Jul 2, 7:46 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, , (Thu Jul 2, 8:14 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, OGAWA Hirofumi, (Thu Jul 2, 8:58 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Thu Jul 2, 9:50 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Thu Jul 2, 10:09 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Fri Jul 3, 5:40 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jamie Lokier, (Mon Jul 6, 4:04 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jamie Lokier, (Fri Jul 10, 5:44 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Mon Jul 6, 7:40 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Pavel Machek, (Tue Jul 7, 3:51 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Pavel Machek, (Fri Jul 10, 3:35 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Wed Jul 8, 6:27 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Thu Jul 9, 4:24 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Mon Jul 6, 2:55 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Mon Jul 6, 4:36 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Martin Steigerwald, (Tue Jul 7, 5:56 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Martin Steigerwald, (Wed Jul 8, 7:39 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jamie Lokier, (Fri Jul 10, 5:28 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jamie Lokier, (Fri Jul 10, 10:03 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Pavel Machek, (Fri Jul 10, 3:36 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jamie Lokier, (Fri Jul 10, 5:12 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Martin Steigerwald, (Thu Jul 9, 3:47 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Martin Steigerwald, (Thu Jul 9, 9:34 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jamie Lokier, (Wed Jul 8, 9:53 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jeremy Allison, (Wed Jul 8, 1:12 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Alan Cox, (Wed Jul 8, 6:04 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, James Bottomley, (Wed Jul 8, 11:27 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Alan Cox, (Wed Jul 8, 11:37 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, James Bottomley, (Wed Jul 8, 12:06 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Alan Cox, (Wed Jul 8, 12:18 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, OGAWA Hirofumi, (Thu Jul 9, 12:53 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, OGAWA Hirofumi, (Sun Jul 12, 3:39 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Pavel Machek, (Tue Jul 21, 6:31 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Pavel Machek, (Sat Aug 8, 8:19 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Boaz Harrosh, (Tue Jul 21, 5:16 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, John Lanza, (Tue Jul 21, 11:06 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, John Lanza, (Tue Jul 21, 3:38 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Tue Jul 21, 6:44 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Pavel Machek, (Tue Jul 21, 6:31 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Pavel Machek, (Tue Jul 21, 5:37 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, John Lanza, (Tue Jul 21, 11:08 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, John Lanza, (Tue Jul 21, 3:36 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Alan Cox, (Thu Jul 9, 5:53 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Alan Cox, (Thu Jul 9, 5:51 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, OGAWA Hirofumi, (Thu Jul 9, 1:27 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Pavel Machek, (Thu Jul 9, 3:21 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, David Newall, (Thu Jul 9, 3:34 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Wed Jul 8, 8:23 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Alan Cox, (Wed Jul 8, 8:02 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Alan Cox, (Wed Jul 8, 9:25 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Alan Cox, (Thu Jul 9, 5:42 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, James Bottomley, (Thu Jul 9, 9:59 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Alan Cox, (Thu Jul 9, 10:10 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Theodore Tso, (Thu Jul 9, 11:25 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Alan Cox, (Thu Jul 9, 8:09 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Christoph Hellwig, (Thu Jul 9, 1:15 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, David Newall, (Thu Jul 9, 4:57 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Martin Steigerwald, (Thu Jul 9, 6:23 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, David Newall, (Thu Jul 9, 9:45 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Martin Steigerwald, (Fri Jul 10, 2:49 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, David Newall, (Sun Jul 12, 4:52 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jonathan Corbet, (Fri Jul 10, 3:31 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Alan Cox, (Fri Jul 10, 5:14 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Bartlomiej Zolnierkiewicz..., (Fri Jul 10, 4:40 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jörn, (Sun Jul 12, 7:21 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Sun Jul 12, 7:27 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jamie Lokier, (Mon Jul 13, 6:20 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Mon Jul 13, 6:32 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Martin Steigerwald, (Tue Jul 7, 6:09 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jamie Lokier, (Mon Jul 6, 4:58 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Mon Jul 6, 5:08 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jamie Lokier, (Mon Jul 6, 6:24 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Tue Jul 7, 5:36 am)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jan Engelhardt, (Mon Jul 6, 4:42 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, David Newall, (Mon Jul 6, 12:17 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, Jamie Lokier, (Mon Jul 6, 3:33 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, OGAWA Hirofumi, (Fri Jul 3, 11:09 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regressions, OGAWA Hirofumi, (Fri Jul 3, 2:46 am)
Re: CONFIG_VFAT_FS_DUALNAMES regression, Jan Engelhardt, (Thu Jul 2, 8:11 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regression, Jan Engelhardt, (Thu Jul 2, 9:10 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regression, Jan Engelhardt, (Thu Jul 2, 9:58 pm)
Re: CONFIG_VFAT_FS_DUALNAMES regression, Jamie Lokier, (Fri Jul 10, 8:14 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Stefan Richter, (Wed Jul 1, 12:18 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Alan Cox, (Wed Jul 1, 10:17 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Jamie Lokier, (Mon Jul 6, 3:57 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Pavel Machek, (Thu Jul 2, 5:49 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Eric W. Biederman, (Sat Jun 27, 9:54 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Eric W. Biederman, (Sun Jun 28, 12:10 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, OGAWA Hirofumi, (Sun Jun 28, 2:25 am)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, OGAWA Hirofumi, (Mon Jun 29, 7:36 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, OGAWA Hirofumi, (Mon Jun 29, 8:55 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Eric W. Biederman, (Sun Jun 28, 3:51 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, James Bottomley, (Sun Jun 28, 4:13 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Eric W. Biederman, (Sun Jun 28, 4:45 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, James Bottomley, (Sun Jun 28, 5:45 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Jan Engelhardt, (Sat Jun 27, 1:21 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Jamie Lokier, (Sun Jun 28, 5:57 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Jan Engelhardt, (Sun Jun 28, 6:02 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Jamie Lokier, (Sun Jun 28, 6:05 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, OGAWA Hirofumi, (Fri Jun 26, 9:56 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, H. Peter Anvin, (Fri Jun 26, 5:40 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, OGAWA Hirofumi, (Fri Jun 26, 9:48 pm)
Re: [PATCH] Added CONFIG_VFAT_FS_DUALNAMES option, Jan Engelhardt, (Sat Jun 27, 1:26 pm)