From: "Vladimir 'φ-coder/phcoder' Serbinenko" <phcoder@gmail.com>
To: grub-devel@gnu.org
Subject: Re: [PATCH] bug fix for ntfs
Date: Mon, 23 Apr 2012 17:04:54 +0200 [thread overview]
Message-ID: <4F956F96.6030502@gmail.com> (raw)
In-Reply-To: <CAF-6-Q2=O+waSt9jaWvEzMHj5iHRBFP7TejncQ8n-S6X4Weuqw@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1842 bytes --]
On 23.04.2012 16:41, Bean wrote:
> Hi all,
>
> This patch fix three issues:
>
> 1, In ntfsdoc, it says namespace = 0 means POSIX and it's case
> sensitive. However, i found out this is not sure in Windows 7 as some
> system directory use this value as well. Now i ignore this flag and
> treat all files in ntfs as case insensitive.
This would be wrong. It's certаinly possible to create files differing
only in case in POSIX namespace, so it's case-sensitive. If some system
directory uses this namespace it's case-sensitive
> 2, Previous version doesn't return blocklist information for small
> files embed in MFT, this patch fixes it. For example, create a
> 512-byte file test in ntfs and try this command:
>
> grub-fstest /ntfs.img blocklist /test
It looks like this part of patch has issues. Like that it doesn't handle
the case when the read is split across 2 sectors or if MFT entry is at
offset >=1024. Or that it adds some checks (like "invalid mft offset")
which weren't there previously and which would make GRUB bail out on
weird FS even if user doesn't want blocklists.
AFAIR MFT is duplicated and so we can't just overwrite it, also MFT can
more easily change place. In such cases we intentionally skip the
blocklists (like we do for ZFS and BtrFS). In any case I feel like this
part shouldn't go it until after 2.00 as it doesn't fix any end-user
problem (blocklists for embed files would be rejected by any of
blocklist users, other than blocklist command) and it may cause new
problems.
> 3, Missing break in switch statement in grub-fstest
Will commit this part shortly.
>
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel
--
Regards
Vladimir 'φ-coder/phcoder' Serbinenko
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 294 bytes --]
next prev parent reply other threads:[~2012-04-23 15:05 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-23 14:41 [PATCH] bug fix for ntfs Bean
2012-04-23 15:04 ` Vladimir 'φ-coder/phcoder' Serbinenko [this message]
2012-04-23 15:26 ` Bean
2012-04-23 15:41 ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-04-23 16:03 ` Bean
2012-04-23 16:29 ` Bean
2012-04-23 16:31 ` Vladimir 'φ-coder/phcoder' Serbinenko
2012-04-23 16:54 ` Bean
2012-04-23 17:00 ` Bean
2012-04-23 20:02 ` Jeffrey Sheinberg
2012-04-23 20:02 ` jeff
2012-04-24 15:36 ` Thomas Frauendorfer
2012-04-25 3:01 ` Bean
2012-04-25 6:11 ` Thomas Frauendorfer
2012-04-25 8:34 ` Vladimir 'φ-coder/phcoder' Serbinenko
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=4F956F96.6030502@gmail.com \
--to=phcoder@gmail.com \
--cc=grub-devel@gnu.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.