linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Vyacheslav Dubeyko <slava@dubeyko.com>
To: htl10@users.sourceforge.net
Cc: linux-fsdevel@vger.kernel.org,
	Till Kamppeter <till.kamppeter@canonical.com>,
	Naohiro Aota <naota@elisp.net>, Matthew Garrett <mjg@redhat.com>
Subject: Re: hfsplus journalling support revisited, netgear, and fsck.hfsplus's observation.
Date: Wed, 25 Jul 2012 10:34:06 +0400	[thread overview]
Message-ID: <1343198046.2025.38.camel@slavad-ubuntu-11> (raw)
In-Reply-To: <024719A2-2AAE-468A-ADA8-28D0D71C0EE5@dubeyko.com>

Hi Hin-Tak,

Sorry, I can see from the hfsplus code about what you are talking. You
are right. The situation is clear for me. It needs to work. :-)

Thank you for bug report.

With the best regards,
Vyacheslav Dubeyko.

On Mon, 2012-07-23 at 23:35 +0400, Vyacheslav Dubeyko wrote:
> Hi,
> 
> On Jul 22, 2012, at 1:06 AM, Hin-Tak Leung wrote:
> 
> > 
> > Correct - two interesting bugs. I thought I had supplied enough details for others to try to reproduce? The first is simply about deleting files, the latter about deleting files with extended attributes. So, to reproduce:
> > 
> > 1. Have a hfs+ volume (created under Mac OS X would be better). Make sure it passed fsck.hfsplus (on linux).
> > 2. copy the system Fonts folder (I think it is under /Library/Fonts) to it from Mac OS X. Font files have extended attributes, but other system files might do too.
> > 3. Try to delete some of those files from Linux.
> > 4. umount, run fsck.hfsplus on the volume. One would see:
> > 
> >   Executing fsck_hfs (version 540.1-Linux).
> >   ...
> > ** Checking extents overflow file.
> >   Unused node is not erased (node = 1)
> > ** Checking catalog file.
> >   Unused node is not erased (node = 18)
> >   ...
> > ** Checking extended attributes file.
> >   Incorrect number of extended attributes
> >   (It should be 13 instead of 6)
> >   ...
> > ** Repairing volume.
> > 	RepairAttributesCheckABT: No matching catalog record found for id=438
> >   ...
> > ** Rechecking volume.
> >   ...
> > ** The volume journalled was repaired successfully.
> 
> Currently, I can't reproduce these bugs. I think that I have not fully correct reproduction path. So, I have some questions.
> What version of Linux kernel do you use? Or maybe do you have hfsplus file system driver code from special branch?
> 
> I tried to reproduce these bugs on non-journaled hfs+ volume, firstly, but without success. As I can see you use journaled hfs+ volume. Maybe the forced mount of journaled hfs+ was the reason of these bugs? By the way do you use MBR or GPT partitioned disk?
> 
> > 
> > 'Unused node is not erased' & 'Incorrect number of extended attributes' were essentially what I wrote earlier. BTW, I ran fsck.hfsplus with:
> > 
> > fsck.hfsplus -d -D 0x0033 -f 
> > 
> > - i.e. maximum amount of information, and have a look even if it appears to be clean. (there are -l, -y and -n switches to control whether fsck.hfsplus would actually fix anything found or leave it alone and just output info).
> > 
> > The font folder is only about a few hundred MB, so one should be able to experiment with a small pen drive and do byte-level comparisons to see what fsk.hfsplus is not happy about and what does it change - although I am sure looking at diskdev_cmds's source would also be useful. I just happen to want the font folder for other things I do.
> > 
> > 
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> With the best regards,
> Vyacheslav Dubeyko.
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



  reply	other threads:[~2012-07-25  6:34 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-07-19 23:56 hfsplus journalling support revisited, netgear, and fsck.hfsplus's observation Hin-Tak Leung
2012-07-21 10:05 ` Vyacheslav Dubeyko
2012-07-21 21:06   ` Hin-Tak Leung
2012-07-22 19:35     ` Vyacheslav Dubeyko
2012-07-23 19:35     ` Vyacheslav Dubeyko
2012-07-25  6:34       ` Vyacheslav Dubeyko [this message]
2012-07-26  5:53         ` Hin-Tak Leung
2012-08-03  4:21           ` off-spec-ness of HFS+ image from apple, hfsplus 2nd volume header, and mount options Hin-Tak Leung
2012-08-03  5:51             ` Vyacheslav Dubeyko
2012-08-03  8:01               ` Hin-Tak Leung
2012-08-03  8:14                 ` Vyacheslav Dubeyko

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=1343198046.2025.38.camel@slavad-ubuntu-11 \
    --to=slava@dubeyko.com \
    --cc=htl10@users.sourceforge.net \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=mjg@redhat.com \
    --cc=naota@elisp.net \
    --cc=till.kamppeter@canonical.com \
    /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).