linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Linus Torvalds <torvalds@osdl.org>
To: Al Viro <viro@ftp.linux.org.uk>
Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org
Subject: Re: minixfs bitmaps and associated lossage
Date: Sat, 6 May 2006 16:42:27 -0700 (PDT)	[thread overview]
Message-ID: <Pine.LNX.4.64.0605061633020.16343@g5.osdl.org> (raw)
In-Reply-To: <20060506231054.GR27946@ftp.linux.org.uk>



On Sun, 7 May 2006, Al Viro wrote:
> 
> FWIW, the only way to really deal with such structure would be to treat
> on-disk values as "fs-endian" and make the conversion to and from
> host-endian check the superblock.  That would _really_ consolidate
> minix_..._bit() (turning them into __test_bit(nr ^ sbi->mangle, p), etc.)

Yeah, especially for bitmaps, it really _should_ be pretty simple, since 
it's literally a bitwise xor of the bit number. It's actually worse for 
things that truly have byte order dependencies where the values span bytes 
and need re-ordering. For bits, that obviously will never be the case.

> 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.

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?

So there are things that are very similar (directory layout, inode 
format), and could probably be share, while other things (free block and 
inode handling) are fundamentally different, no?

			Linus

  reply	other threads:[~2006-05-06 23:42 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 [this message]
2006-05-07  7:37             ` Al Viro
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=Pine.LNX.4.64.0605061633020.16343@g5.osdl.org \
    --to=torvalds@osdl.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=viro@ftp.linux.org.uk \
    /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).