Re: CONFIG_VFAT_FS_DUALNAMES regressions

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

Hi Jan,

> No-no, as previously mentioned it ignores filenames not in the
> format of dscf????.jpg. But if I interpret the hexdump right,
> standard Linux creates a long name *anyway*, even for names
> fitting in 8.3, if it's lowercase:
>
> $,2 "00bce510 79 39 79 39 03 00 f5 5b-79 39 95 4b 00 00 00 00 |y9y9? ?[y9?K |
> $,2 "00bce520 41 67 00 6f 00 64 00 64-00 61 00 0f 00 27 6d 00 |Ag o d d a ? 'm |
> $,2 "00bce530 6e 00 2e 00 7a 00 69 00-70 00 00 00 00 00 ff ff |n . z i p ??|
> $,2 "00bce540 47 4f 44 44 41 4d 4e 20-5a 49 50 20 00 64 d2 20 |GODDAMN ZIP d? |

right, that is not unexpected. It creates both the long name and the
short name in this case as the short name is constrained to not be
case preserving, and the filesystem normally tries to be both
case-insensitive and case-preserving. As the long name has no
constraints on the case of characters it creates both so that devices
(like your camera I think) that only look at the 8.3 entries will see
a file.

That is why setting shortname=winnt is desirable when using the "no
dualnames" patch I posted. That tells Linux to behave more like what
current windows versions behave, which is to try and put it in the 8.3
entry if it can, even if it is all lowercase.

What I would like to see us do for maximum compatibility with devices
like this, while still maximising the legal safety, is to take mixed
case prefixes or suffixes that fit in 8.3 and uppercase them and set
the CASE_LOWER_BASE and CASE_LOWER_EXT bits appropriately in the flags
field, and not store a long name. That would makes us really nicely
compatible with devices like your camera, at the price of losing mixed
case preservation for 8.3 names.

It would be another departure from previous behaviour however, so I
want to get some feedback from Hirofumi-san first.

btw, if you still think your camera really is looking at the long
filename entries, then try creating a file like DsCf2000.jpg with a
unpatched kernel, mounted with shortname=winnt. That will create two
directory entries for the file, one being DSCF2000.JPG and the other
being DsCf2000.jpg.

Then view the list of photos on the LCD back panel of your
camera. Assuming it has a way to see the filename, does it display the
mixed case name DsCf2000.jpg, or does it display a non-mixed name like
dscf2000.jpg or DSCF2000.JPG.

If it displays a mixed name (ie. DsCf) then your camera can indeed
read long filename entries. If it displays a all uppercase or all
lowercase name then I think it is only capable of reading short names
from FAT filesystems. That is what I think you will find.

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, 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, , (Thu Jul 2, 11:25 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)