linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Steve French <smfrench@gmail.com>
To: Dave Kleikamp <shaggy@linux.vnet.ibm.com>,
	Christoph Hellwig <hch@infradead.org>
Cc: Ogawa Hirofumi <hirofumi@mail.parknet.co.jp>,
	linux-fsdevel <linux-fsdevel@vger.kernel.org>,
	Michael Tokarev <mjt@tls.msk.ru>,
	Paul McKenney <paulmck@linux.vnet.ibm.com>,
	Andrew Tridgell <tridge@samba.org>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] Add CONFIG_VFAT_NO_CREATE_WITH_LONGNAMES option
Date: Fri, 1 May 2009 15:18:20 -0500	[thread overview]
Message-ID: <524f69650905011318m34e0027dt57877d225b3fe2da@mail.gmail.com> (raw)

Dave Kleikamp <shaggy@linux.vnet.ibm.com> wrote on 05/01/2009 02:09:33 PM:
> On Fri, 2009-05-01 at 22:19 +0400, Michael Tokarev wrote:
> > Dave Kleikamp wrote:
> > > On Fri, 2009-05-01 at 13:47 -0400, Christoph Hellwig wrote:
> > >> On Fri, May 01, 2009 at 12:41:29PM -0500, Dave Kleikamp wrote:
> > >>> From: Andrew Tridgell <tridge@samba.org>
> > >>> Subject: [PATCH] Add CONFIG_VFAT_NO_CREATE_WITH_LONGNAMES option
> > >>>
> > >>> When this option is enabled the VFAT filesystem will refuse to create
> > >>> new files with long names. Accessing existing files with long names
> > >>> will continue to work.
> > >>>
> > >>> File names to be created must conform to the 8.3 format.  Mixed case is
> > >>> not allowed in either the prefix or the suffix.
> > >> This doesn't make any sense as a compile time option. Might make sense
> > >> as a mount option, but I'd like to hear a rationale for it first.
> > >
> > > Some linux-based devices would be happy not to contain code to create
> > > the long name at all.
> >
> > Well, is that a rationale per se?  Which devices they are and why?
>
> Could be anything.  cameras, phones, etc.  Anything that might be
> mountable by a host computer in order to share files, or to write to a
> device that can be shared by other computers or devices.
>
> > But besides, why `msdos' filesystem is not sufficient?
> > It contains no code to create long file names and no code to
> > read such names either.
>
> An example, an mp3 player wants to read files with long mixed-case
> names, which can be manipulated on a host computer.  But it may not need
> to create files that don't fit the 8.3 syntax.
>
> Of course, msdos might be a good option for other devices.

msdos file system does not support other features that vfat does
(there are probably more than maximum volume and file size).
For those manufacturers who who would like to disable
creation of long file names, but allow reading long file names,
and handle FAT32 on disk format and maximum sizes, it seems
reasonable to give them a simple configure option for it.  It is
harder, and less effective, to make the corresponding change
to modify the mount helper and kernel code to add
a new mount option, because it can be bypassed trivially
at the command line (ie having to "force" mount to pass a "nolongfilename"
mount option, would be harder than a simple kernel configure option)

-- 
Thanks,

Steve

             reply	other threads:[~2009-05-01 20:18 UTC|newest]

Thread overview: 90+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-01 20:18 Steve French [this message]
2009-05-01 21:01 ` [PATCH] Add CONFIG_VFAT_NO_CREATE_WITH_LONGNAMES option Christoph Hellwig
2009-05-02  1:37   ` Paul E. McKenney
2009-05-02  1:59     ` Matthew Wilcox
2009-05-02  3:02       ` Steve French
2009-05-02  4:51       ` Paul E. McKenney
2009-05-02  9:15       ` tridge
2009-05-02  9:22         ` Christoph Hellwig
2009-05-02  9:30           ` tridge
2009-05-02 12:44             ` Christoph Hellwig
2009-05-03 21:57             ` Pavel Machek
2009-05-03 22:25               ` tridge
2009-05-03 22:56                 ` Al Viro
2009-05-03 23:15                   ` tridge
2009-05-04  5:42                     ` Eric W. Biederman
2009-05-04  6:34                       ` Paul E. McKenney
2009-05-04  6:49                         ` Eric W. Biederman
2009-05-04 12:41                           ` Paul E. McKenney
2009-05-04 12:44                             ` Matthew Wilcox
2009-05-04 13:06                               ` Paul E. McKenney
2009-05-04 13:21                                 ` Matthew Wilcox
2009-05-04 14:39                                   ` Paul E. McKenney
2009-05-04 15:08                                     ` Matthew Wilcox
2009-05-04 15:36                                       ` Dave Kleikamp
2009-05-04 15:59                                         ` Eric W. Biederman
2009-05-04 16:07                                           ` Dave Kleikamp
2009-05-04 16:30                                             ` Eric W. Biederman
2009-05-04 16:42                                               ` Paul E. McKenney
2009-05-04 17:18                                                 ` Eric W. Biederman
2009-05-04 17:49                                                   ` Paul E. McKenney
2009-05-04 17:54                                                     ` Matthew Wilcox
2009-05-04 18:14                                                       ` Paul E. McKenney
2009-05-04 18:17                                                 ` Al Viro
2009-05-04 20:18                                                   ` Paul E. McKenney
2009-05-04 17:06                                               ` Olivier Galibert
2009-05-04 17:27                                                 ` Christoph Hellwig
2009-05-04 20:53                                                   ` Chris Friesen
2009-05-04 23:03                                                     ` Theodore Tso
2009-05-05 11:09                                                       ` David Newall
2009-05-05 20:56                                                         ` Valdis.Kletnieks
2009-05-05 21:04                                                           ` Christoph Hellwig
2009-05-05 22:29                                                             ` Steve French
2009-05-05  8:31                                               ` Zero-day exploit details (Was: Add CONFIG_VFAT_NO_CREATE_WITH_LONGNAMES option) David Newall
2009-05-04 16:11                                           ` [PATCH] Add CONFIG_VFAT_NO_CREATE_WITH_LONGNAMES option Paul E. McKenney
2009-05-04 15:38                                       ` Paul E. McKenney
2009-05-04 15:55                                         ` Matthew Wilcox
2009-05-04 16:10                                           ` Paul E. McKenney
2009-05-04 16:22                                             ` Matthew Wilcox
2009-05-04 22:12                                               ` Greg KH
2009-05-05  2:01                                                 ` Matthew Wilcox
2009-05-05  2:11                                                   ` Paul E. McKenney
2009-05-05  2:18                                                     ` Matthew Wilcox
2009-05-05  3:34                                                       ` Paul E. McKenney
2009-05-05  8:05                                                         ` Valdis.Kletnieks
2009-05-05 15:35                                                           ` Paul E. McKenney
2009-05-05 21:00                                                             ` Valdis.Kletnieks
2009-05-05 21:56                                                               ` Paul E. McKenney
2009-05-05  3:08                                                     ` Valdis.Kletnieks
2009-05-04 15:55                                         ` Christoph Hellwig
2009-05-04 16:11                                           ` Paul E. McKenney
2009-05-04 15:40                 ` Valdis.Kletnieks
2009-05-02  6:33     ` Christoph Hellwig
2009-05-02  2:12   ` Theodore Tso
2009-05-02  6:38     ` Christoph Hellwig
  -- strict thread matches above, loose matches on Subject: below --
2009-05-01 17:41 Dave Kleikamp
2009-05-01 17:47 ` Christoph Hellwig
2009-05-01 18:12   ` Dave Kleikamp
2009-05-01 18:19     ` Michael Tokarev
2009-05-01 19:09       ` Dave Kleikamp
2009-05-01 18:30     ` Christoph Hellwig
2009-05-02 10:09 ` OGAWA Hirofumi
2009-05-02 10:14   ` OGAWA Hirofumi
2009-05-02 10:26     ` OGAWA Hirofumi
2009-05-02 10:41       ` tridge
2009-05-02 11:03         ` OGAWA Hirofumi
2009-05-02 11:13           ` tridge
2009-05-02 11:29             ` OGAWA Hirofumi
2009-05-02 11:41               ` tridge
2009-05-02 11:59                 ` OGAWA Hirofumi
2009-05-02 12:15                   ` tridge
2009-05-02 12:48                     ` OGAWA Hirofumi
2009-05-02 13:06                       ` tridge
2009-05-02 14:01                         ` OGAWA Hirofumi
2009-05-27 12:05                   ` vimal singh
2009-05-27 23:57                     ` tridge
2009-06-04 10:26                       ` vimal singh
2009-06-04 21:33                         ` tridge
2009-05-02 10:20   ` tridge
2009-05-02 10:32     ` OGAWA Hirofumi
2009-05-03 21:56 ` Pavel Machek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=524f69650905011318m34e0027dt57877d225b3fe2da@mail.gmail.com \
    --to=smfrench@gmail.com \
    --cc=hch@infradead.org \
    --cc=hirofumi@mail.parknet.co.jp \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mjt@tls.msk.ru \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=shaggy@linux.vnet.ibm.com \
    --cc=tridge@samba.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).