grub-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] bug fix for ntfs


From: Bean
Subject: Re: [PATCH] bug fix for ntfs
Date: Tue, 24 Apr 2012 00:03:14 +0800

Hi,

2012/4/23 Vladimir 'φ-coder/phcoder' Serbinenko <address@hidden>:
> Have you tried with ntfs-3g?
> Also if the problem is with only one directory, it's better if it stays
> the way it is. The only quirk associated with false assumption about
> case-sensitivity is the need to reference the name in correct case which
> should be done anyway. If you wrongly assume case insensitivity some
> files may become inaccessible with is more serious.
> OS may forbid some file names without it being FS limitation (conversely
> OS may fail to enforce FS limitation like an old trick with ".." in the
> root directory)
>

I did some testing and found more serious issue, it seems directories
like Windows, Users has disappeared. I need more testing to find the
problem.

>>>> 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.
>> MFT in ntfs is only 1024 bytes, and it must be sector aligned, so if
>> this test fails, there is serious problem with the fs (or the driver).
> Is it true for 4K sector NTFS?

Yeah, I've tried 64K block size and MFT is still 1024 bytes, it just
pack multiple MFT in a single allocation unit.

-- 
Best wishes
Bean



reply via email to

[Prev in Thread] Current Thread [Next in Thread]