All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Matthew Wilcox <matthew@wil.cx>
Cc: Christoph Hellwig <hch@infradead.org>,
	Steve French <smfrench@gmail.com>,
	Dave Kleikamp <shaggy@linux.vnet.ibm.com>,
	Ogawa Hirofumi <hirofumi@mail.parknet.co.jp>,
	linux-fsdevel <linux-fsdevel@vger.kernel.org>,
	Michael Tokarev <mjt@tls.msk.ru>,
	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 21:51:03 -0700	[thread overview]
Message-ID: <20090502045103.GA7141@linux.vnet.ibm.com> (raw)
In-Reply-To: <20090502015927.GJ8822@parisc-linux.org>

On Fri, May 01, 2009 at 07:59:28PM -0600, Matthew Wilcox wrote:
> On Fri, May 01, 2009 at 06:37:29PM -0700, Paul E. McKenney wrote:
> > On Fri, May 01, 2009 at 05:01:09PM -0400, Christoph Hellwig wrote:
> > > >From the complete lack of technical arguments it's pretty obvious that
> > > this seems to be some FUD fallout from the MS vs TomTom patent lawsuite.
> > > 
> > > I'm not a lawyer so I don't know how much of a threat it is.  But either
> > > the case gets shot down by showing prior art and everything is fine, or
> > > we indeed are in deep trouble and should remove it completely.  Given
> > > the Cc list on here IBM seems to have some legal opinion on it, so can
> > > we please see it and discuss what we want to with all cards on the
> > > table?
> > 
> > Hello, Christoph!
> > 
> > Hmmm...  Both Tridge and Dave have Signed-off-by on the original patch,
> > and Steve has Acked-by, Mingming has Cc, and Dave is on the From list
> > rather than the Cc list, so I have to guess that there is a good chance
> > that you are talking about me.  ;-)
> > 
> > However, as far as I know, none of us are lawyers, and LKML is definitely
> > a technical rather than a legal forum, so we really do need to stick to
> > technical topics.  I understand that this might be a bit frustrating
> > to you.  On the other hand, I for one much prefer being in a forum
> > restricted to technical topics than to be in those places designed to
> > handle legal topics!
> 
> So what's the purely technical argument for including this patch?

Pretty much what it says in the config-variable description in the patch
itself.  To recap, with some technical elaboration:

1.	It allows you to read VFAT media, mixed-case long names and all.
	Full compatibility for reading from existing media.

2.	It allows opening existing mixed-case long-named files
	both for reading and writing.

3.	For file creation, only the 8.3-format filename is permitted.
	Such a name can be stored in a single VFAT directory entry.

4.	It allows you to create files with both uppercase and lowercase
	8.3-format names, as long as the 8-character portion is either
	entirely uppercase or entirely lowercase, and as long as
	the 3-character extension is either entirely uppercase
	or entirely lowercase.  This again keeps the name confined
	to a single VFAT directory entry.

>From the original patch, with a couple additional examples added:

o	Creating FILE1234.TXT is allowed.

o	Creating FILE12345.TXT or FILE1234.TEXT is prohibited, as
	neither fits the 8.3 format.

o	Creating FILE.TXT, FILE.txt, file.TXT, and file.txt would all
	be allowed, and would create files by exactly those names.

o	Attempting to create File.TxT will give you a file named FILE.TXT.

There you have it!

							Thanx, Paul

  parent reply	other threads:[~2009-05-02  4:51 UTC|newest]

Thread overview: 98+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-01 20:18 [PATCH] Add CONFIG_VFAT_NO_CREATE_WITH_LONGNAMES option Steve French
2009-05-01 21:01 ` 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  3:02         ` Steve French
2009-05-02  4:51       ` Paul E. McKenney [this message]
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 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 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  2:18                                                       ` Matthew Wilcox
2009-05-05  3:34                                                       ` Paul E. McKenney
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
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 12:05                     ` vimal singh
2009-05-27 23:57                     ` tridge
2009-06-04 10:26                       ` vimal singh
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=20090502045103.GA7141@linux.vnet.ibm.com \
    --to=paulmck@linux.vnet.ibm.com \
    --cc=hch@infradead.org \
    --cc=hirofumi@mail.parknet.co.jp \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matthew@wil.cx \
    --cc=mjt@tls.msk.ru \
    --cc=shaggy@linux.vnet.ibm.com \
    --cc=smfrench@gmail.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.