mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Adam J. Richter" <adam@yggdrasil.com>
To: jaharkes@cs.cmu.edu
Cc: linux-kernel@vger.kernel.org
Subject: Re: Loop devices under NTFS
Date: Tue, 27 Aug 2002 18:49:17 -0700	[thread overview]
Message-ID: <200208280149.SAA07234@baldur.yggdrasil.com> (raw)

On Tue, 27 Aug 2002 at 13:26:44 -0400, Jan Harkes wrote:
>Not all filesystems use generic_read/generic_write. If they did we
>wouldn't need those calls in the fops structure.

	My loop.c patch supports files that do not provide
aops->{prepare,commit}_write (derived from changes by Jari Ruusu
and Andrew Morton).

	Christoph was arguing that even if the file provides
aops->{prepare,commit}_write, that there could be a problem using it.
I am looking for a clear example of that.  I don't see the problem
with using this facility if you first check that it is provided.

	However, thank you for correcting me about Coda.  I missed
that in the list of file system that do not appear to provide
{prepare,commit}_write for plain files.  I actually grepped around and
discussed this by email with Andrew Morton and Hugh Dickens on August
16th.  Looking back at that email now, the list of file systems that
my grepping around suggested lacked {prepare,commot}_write for
writable files was:

	       tmpfs
	       coda
	       intermezzo
	       ncpfs

Side note:
>Ofcourse the prepar_write/commit_write were introduced later on and
>perhaps it is possible to modify all filesystems to put all their
>custom functionality in these functions. Then we can simply remove the
>read and write (and mmap?) fops [...]

	You still need read and write methods at least for files that
are not seekable (e.g., serial devices, network sockets, pipes), but I
think you could conceivably have everything else use generic page
cache routines.

Adam J. Richter     __     ______________   575 Oroville Road
adam@yggdrasil.com     \ /                  Milpitas, California 95035
+1 408 309-6081         | g g d r a s i l   United States of America
                         "Free Software For The Rest Of Us."

             reply	other threads:[~2002-08-28  1:45 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-08-28  1:49 Adam J. Richter [this message]
2002-08-28  8:36 ` Urban Widmark
  -- strict thread matches above, loose matches on Subject: below --
2002-08-29 11:00 Adam J. Richter
2002-08-29 11:27 ` Anton Altaparmakov
2002-08-29 15:16 ` Jari Ruusu
2002-08-28  9:17 Adam J. Richter
2002-08-28  1:06 Adam J. Richter
2002-08-28 15:50 ` Andre Bonin
2002-08-28 16:41 ` Christoph Hellwig
2002-08-27 23:42 Adam J. Richter
2002-08-27 23:59 ` Christoph Hellwig
2002-08-27 13:53 Adam J. Richter
2002-08-27 17:04 ` Christoph Hellwig
2002-08-27 17:26 ` Jan Harkes
2002-08-27 13:23 Adam J. Richter
2002-08-27 13:27 ` Christoph Hellwig
2002-08-27 12:40 Adam J. Richter
2002-08-27 12:46 ` Anton Altaparmakov
2002-08-27 13:15 ` Christoph Hellwig
2002-08-27  4:48 Andre Bonin
2002-08-27  9:17 ` Anton Altaparmakov

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=200208280149.SAA07234@baldur.yggdrasil.com \
    --to=adam@yggdrasil.com \
    --cc=jaharkes@cs.cmu.edu \
    --cc=linux-kernel@vger.kernel.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

all inboxes | Powered by JetHome®