public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: linux-kernel@vger.kernel.org
Subject: Re: ISOFS corrupt filesizes
Date: 14 Sep 2001 17:54:55 -0700	[thread overview]
Message-ID: <9nu8sv$fnj$1@cesium.transmeta.com> (raw)
In-Reply-To: <20010914145352.A9952@stud.tu-muenchen.de>

Followup to:  <20010914145352.A9952@stud.tu-muenchen.de>
By author:    Matthias Kramm <matthias.kramm@stud.tu-muenchen.de>
In newsgroup: linux.dev.kernel
> According to the (2.4.9) MAINTAINERS-File,  ISOFS doesn't have a maintainer,
> so this probably best fits in this list.
> 
> I came across a (commercial) DVD with an ISOFS Filesystem on it and filesizes
> bigger than 1M.
>

1 GB (not MB) you mean...

> My personal guess would be that the assumption that iso files can't be
> bigger than 1M unless the CD-ROM is defective is wrong.
> (I don't know where the 1M comes from. 2M sounds more logical to me,
>  however)

1 GB comes from the fact that some old CD's actually put garbage in
the upper byte of the file size, so the test triggers if the size is
larger than any CD can be.  Unfortunately, DVDs are a lot bigger than
CDs and that assumption is no longer correct.

> After removing the "indode->i_size > 1073741824" test, I got the correct
> output for ls. Also was I able to cat (css-cat, actually) the whole 1201278976 
> bytes without an error, which may lead to the assumption the file
> is actually that big.
> If I'm wrong and the dvd is actually broken, however, I'd like to 
> suggest making the automatic cruft mount optional.

The automatic cruft mount option should probably be conditionalized
not on a fixed size, but on the actual size of the filesystem.

	-hpa

-- 
<hpa@transmeta.com> at work, <hpa@zytor.com> in private!
"Unix gives you enough rope to shoot yourself in the foot."
http://www.zytor.com/~hpa/puzzle.txt	<amsp@zytor.com>

  parent reply	other threads:[~2001-09-15  0:55 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-09-14 12:54 ISOFS corrupt filesizes Matthias Kramm
2001-09-14 16:06 ` Matthias Kramm
2001-09-15  0:54 ` H. Peter Anvin [this message]
2001-09-15  1:28   ` Alan Cox
2001-09-15  1:45     ` H. Peter Anvin

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='9nu8sv$fnj$1@cesium.transmeta.com' \
    --to=hpa@zytor.com \
    --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