From: Al Viro <viro@ftp.linux.org.uk>
To: Linus Torvalds <torvalds@osdl.org>
Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org
Subject: Re: minixfs bitmaps and associated lossage
Date: Sun, 7 May 2006 08:37:08 +0100 [thread overview]
Message-ID: <20060507073708.GW27946@ftp.linux.org.uk> (raw)
In-Reply-To: <Pine.LNX.4.64.0605061633020.16343@g5.osdl.org>
On Sat, May 06, 2006 at 04:42:27PM -0700, Linus Torvalds wrote:
> > If somebody wants to play with that code, they could just merge fs/minix
> > into fs/sysv - that might very well turn out to be the right thing and
> > a fun exercise. Codebases are very close - minixfs is a derivative of
> > v7 filesystem, after all, and our fs/minix and fs/sysv had been kept
> > mostly in sync.
>
> Heh. Yes. The physical filesystem layout of minix is close to the old sysv
> one, and the implementation ends up being pretty closely related too,
> although the genealogy there is the other way around.
Actually, some things (e.g. indirect block tree handling and directory
handling via pagecache) went the other way - from fs/sysv to fs/minix.
> However, I thought the direct sysv descendants used linked lists of
> free-block lists, not bitmaps? So while a lot of the _other_ part of the
> filesystem layout is similar, the actual free-block handling is very
> different. No?
Yes and no - keep in mind that details of those lists are different for
various sysvfs flavours, so sysv_new_block() et.al. check sbi->s_type
anyway. And the entry points into [ib]alloc are parallel, so it's not
hard to merge transparently for the rest of code.
Superblock layouts are very different, obviously, but they are just as
different among sysv flavours. Again, no big deal...
BTW, there's a sysv flavour that uses bitmaps (EAFS); we only do it
read-only, so that's not an issue with the current fs/sysv code.
Again, what I'm saying is that figuring out details of doing it clean
way would make a good exercise, not that we can't live without that.
next prev parent reply other threads:[~2006-05-07 7:37 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <44560796.8010700@gmail.com>
[not found] ` <20060506162956.GO27946@ftp.linux.org.uk>
[not found] ` <20060506163737.GP27946@ftp.linux.org.uk>
2006-05-06 22:04 ` minixfs bitmaps and associated lossage Al Viro
2006-05-06 22:25 ` Matthew Wilcox
2006-05-06 22:26 ` Linus Torvalds
2006-05-06 23:10 ` Al Viro
2006-05-06 23:42 ` Linus Torvalds
2006-05-07 7:37 ` Al Viro [this message]
2006-05-07 7:35 ` 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=20060507073708.GW27946@ftp.linux.org.uk \
--to=viro@ftp.linux.org.uk \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@osdl.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).