linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Giuliano Pochini <pochini@shiny.it>
Cc: <linuxppc-dev@lists.linuxppc.org>, Alan Cox <alan@lxorguk.ukuu.org.uk>
Subject: Re: hfs cdrom broken in 2.4.13pre
Date: Tue, 23 Oct 2001 21:59:16 +0200	[thread overview]
Message-ID: <20011023195916.18893@smtp.wanadoo.fr> (raw)
In-Reply-To: <XFMail.20011023151612.pochini@shiny.it>


>
>> One would hope so. All I remember about CDROMs is they use a different
>> blocksize which causes all sorts of funny side effects in generic code
>> that assumes the only sane blocksize is 512.
>
>No. The point is that filesystems should not make assumptions about
>hardware block size (except it being power of 2 I believe). If there
>was code in block layer to provide broken FS's compatibility and that
>code has been removed, then we must fix the FS. Who's HFS maintainer ?
>
>> So mounting HFS CDROMs no longer panics ?
>
>They always worked fine for me.

I don't know if there's still an active HFS maintainer. That is not
the only bugs in HFS. Locking is broken and HFS can easily deadlock
on an SMP box. I've been reported disk corruption with it, and
experienced corrupted file when written from linux occasionally,
I suspect something must be wrong with btree rebalancing.

I have done some work on locking (replacing most broken spinlocks with
semaphores) but never really finished it. Most people rely on HFS tools,
not on the HFS fs. I don't have the time to work more seriously on it,
but it would be an interesting project for someone motivated, especially
since you now have Apple's implementation to use as a reference ;)

Ben.


** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/

      parent reply	other threads:[~2001-10-23 19:59 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20011022155617.B6673@cpe-24-221-152-185.az.sprintbbd.net>
2001-10-23 10:08 ` hfs cdrom broken in 2.4.13pre Michael Schmitz
2001-10-23 11:04   ` Martin Costabel
2001-10-23 11:26     ` Michael Schmitz
2001-10-23 13:16       ` Giuliano Pochini
2001-10-23 13:26         ` Alan Cox
2001-10-23 15:24         ` Michael Schmitz
2001-10-23 15:52           ` Alan Cox
2001-10-23 16:31             ` Michael Schmitz
2001-10-23 16:51               ` Alan Cox
2001-10-23 20:08                 ` Michael Schmitz
2001-10-23 20:49                   ` Alan Cox
2001-10-24  7:21                     ` Michael Schmitz
2001-10-23 19:59         ` Benjamin Herrenschmidt [this message]

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=20011023195916.18893@smtp.wanadoo.fr \
    --to=benh@kernel.crashing.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linuxppc-dev@lists.linuxppc.org \
    --cc=pochini@shiny.it \
    /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).