From: Andre Bonin <Bonin@bonin.ca>
To: "Adam J. Richter" <adam@yggdrasil.com>
Cc: hch@infradead.org, aia21@cantab.net, kernel@bonin.ca,
linux-kernel@vger.kernel.org
Subject: Re: Loop devices under NTFS
Date: Wed, 28 Aug 2002 11:50:10 -0400 [thread overview]
Message-ID: <3D6CF132.4090205@bonin.ca> (raw)
In-Reply-To: 200208280106.SAA05492@adam.yggdrasil.com
Adam J. Richter wrote:
>On Wed, 28 Aug 2002 at 00:59:55AM +0100, Christoph Hellwig wrote:
>
>
>>On Tue, Aug 27, 2002 at 04:42:56PM -0700, Adam J. Richter wrote:
>>
>>
>>>>On Tue, Aug 27, 2002 at 06:53:19AM -0700, Adam J. Richter wrote:
>>>>
>>>>
>>>>> Why?
>>>>>
>>>>> According to linux-2.5.31/Documentation/Locking,
>>>>>"->prepare_write(), ->commit_write(), ->sync_page() and ->readpage()
>>>>>may be called from the request handler (/dev/loop)."
>>>>>
>>>>>
>>>>Just because it's present in current code it doesn't mean it's right.
>>>>Calling aops directly from generic code is a layering violation and
>>>>it will not survive 2.5.
>>>>
>>>>
>>> Only according your own proclamation. You are arguing
>>>circular logic, and I am aruging a concrete benefit: we can avoid an
>>>extra copying of all data in the input and output paths going through
>>>an encrypted device.
>>>
I'me new to kernel development and i've never fooled around with drivers
before (I do have a course in it this september though, Wohoo!).
Why are you saying it would copy the data? Couldn't you just make some
sort of shared memory system that would let you unencript/uncompress the
data without having to do a copy? The way i see it you can read the
block, pass it through the necessary mods using the same data. You
could get a wierd race condition if your uncompress and your unencript
work on the same data at the same time but that can easily be avoided
The way i see it, the NTFS driver should be able to read the file and
uncompress. The loop driver should have access to that block without
having to do a copy to present it to a third party driver. Which then
reads the data read by the driver and rpesents it as a filesystem.
Maby even do away with loop.c, there should really be no loop.c . A
normal mount (/dev/hda1 for example) is a first level mount. If you let
/mnt/foo/myfile.iso be from /dev/hda1, then the chain should be
/dev/hda1->ISO9660 module-->presentation.
I think the filesystem drivers should be written in such a way that they
are totally pluggable with eachother. That it doesn't matter where the
blocks are comming from and going to. You could have a mount of
/dev/hda1 of an iso containing an ext2 image within it etc etc etc.
But like i said i'me new to kernel development so I think i might have
the wrong perspective.
-----------------------------------
Andre Bonin
Student in Software Engineering
Lakehad University
Thunder Bay,
Canada
-----------------------------------
>-
>To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
>the body of a message to majordomo@vger.kernel.org
>More majordomo info at http://vger.kernel.org/majordomo-info.html
>Please read the FAQ at http://www.tux.org/lkml/
>
>
>
>
next prev parent reply other threads:[~2002-08-28 15:46 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-08-28 1:06 Loop devices under NTFS Adam J. Richter
2002-08-28 15:50 ` Andre Bonin [this message]
2002-08-28 16:41 ` Christoph Hellwig
-- 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:49 Adam J. Richter
2002-08-28 8:36 ` Urban Widmark
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=3D6CF132.4090205@bonin.ca \
--to=bonin@bonin.ca \
--cc=adam@yggdrasil.com \
--cc=aia21@cantab.net \
--cc=hch@infradead.org \
--cc=kernel@bonin.ca \
--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