Re: [PATCH 2/2] ntfs3: remove warning

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

 



Hi, 

Had a look at ntfs3 code and it is corrupting your volume.  Every such message you are seeing is damaging a file or directory on your volume.

I would personally suggest you modify your /etc/fstab to mount read-only.  If it is getting simple things like this wrong who knows what else it is doing incorrect...

Best regards,

Anton

> On 15 Apr 2024, at 11:20, Johan Hovold <johan@xxxxxxxxxx> wrote:
> 
> On Mon, Apr 15, 2024 at 11:54:19AM +0200, Johan Hovold wrote:
>> On Thu, Apr 11, 2024 at 02:03:52PM +0300, Konstantin Komarov wrote:
> 
>>> Messages like this:
>>> 
>>> diff --git a/fs/ntfs3/inode.c b/fs/ntfs3/inode.c
>>> index eb7a8c9fba01..8cc94a6a97ed 100644
>>> --- a/fs/ntfs3/inode.c
>>> +++ b/fs/ntfs3/inode.c
>>> @@ -424,7 +424,6 @@ static struct inode *ntfs_read_mft(struct inode *inode,
>>>     if (names != le16_to_cpu(rec->hard_links)) {
>>>         /* Correct minor error on the fly. Do not mark inode as dirty. */
>>> -        ntfs_inode_warn(inode, "Correct links count -> %u.", names);
>>>         rec->hard_links = cpu_to_le16(names);
>>>         ni->mi.dirty = true;
>>>     }
>>> 
>>> can also be suppressed for the sake of seamless transition from a remote 
>>> NTFS driver.
>>> However, I believe that file system corrections should be reported to 
>>> the user.
>> 
>> A colleague of mine also tracked down a failed boot to the removal of
>> the ntfs driver and reported seeing similar warnings with the ntfs3
>> driver.
>> 
>> We're both accessing an NTFS partition on a Windows on Arm device, but
>> it makes you wonder whether these warnings (corrections) are correct or
>> indicative of a problem in the driver?
> 
> This is what I see with a recursive ls on that partition (I've added
> rec->hard_links in parentheses):
> 
> [   38.287555] ntfs3: nvme0n1p3: ino=2e1e7, Correct links count -> 1 (2).
> [   38.288593] ntfs3: nvme0n1p3: ino=75ff, Correct links count -> 1 (2).
> [   38.289887] ntfs3: nvme0n1p3: ino=1b4e1, Correct links count -> 1 (2).
> [   38.290144] ntfs3: nvme0n1p3: ino=78c6, Correct links count -> 1 (2).
> [   38.291313] ntfs3: nvme0n1p3: ino=8781b, Correct links count -> 1 (2).
> [   38.291823] ntfs3: nvme0n1p3: ino=8781e, Correct links count -> 1 (2).
> [   38.292289] ntfs3: nvme0n1p3: ino=87820, Correct links count -> 1 (2).
> [   38.292978] ntfs3: nvme0n1p3: ino=87823, Correct links count -> 1 (2).
> [   38.300531] ntfs3: nvme0n1p3: ino=a324, Correct links count -> 1 (2).
> [   38.312235] ntfs3: nvme0n1p3: ino=882c3, Correct links count -> 1 (2).
> [   43.286846] ntfs3: 5479 callbacks suppressed
> [   43.286856] ntfs3: nvme0n1p3: ino=14aa, Correct links count -> 1 (2).
> [   43.286998] ntfs3: nvme0n1p3: ino=14ac, Correct links count -> 1 (2).
> [   43.287194] ntfs3: nvme0n1p3: ino=12cc2, Correct links count -> 1 (2).
> [   43.287386] ntfs3: nvme0n1p3: ino=12ccd, Correct links count -> 1 (2).
> [   43.287576] ntfs3: nvme0n1p3: ino=12d15, Correct links count -> 1 (2).
> [   43.287667] ntfs3: nvme0n1p3: ino=12d19, Correct links count -> 1 (2).
> [   43.287877] ntfs3: nvme0n1p3: ino=12d3a, Correct links count -> 1 (2).
> [   43.288051] ntfs3: nvme0n1p3: ino=12d88, Correct links count -> 1 (2).
> [   43.288265] ntfs3: nvme0n1p3: ino=874, Correct links count -> 1 (2).
> [   43.288326] ntfs3: nvme0n1p3: ino=875, Correct links count -> 1 (2).
> [   48.288211] ntfs3: 7735 callbacks suppressed
> [   48.288220] ntfs3: nvme0n1p3: ino=33391, Correct links count -> 1 (2).
> [   48.289115] ntfs3: nvme0n1p3: ino=347a4, Correct links count -> 1 (2).
> [   48.290214] ntfs3: nvme0n1p3: ino=3553f, Correct links count -> 1 (2).
> [   48.291193] ntfs3: nvme0n1p3: ino=35793, Correct links count -> 1 (2).
> [   48.292818] ntfs3: nvme0n1p3: ino=358a0, Correct links count -> 1 (2).
> [   48.293529] ntfs3: nvme0n1p3: ino=38f54, Correct links count -> 1 (2).
> [   48.293901] ntfs3: nvme0n1p3: ino=391f6, Correct links count -> 1 (2).
> [   48.294303] ntfs3: nvme0n1p3: ino=39324, Correct links count -> 1 (2).
> [   48.294729] ntfs3: nvme0n1p3: ino=394a0, Correct links count -> 1 (2).
> [   48.295063] ntfs3: nvme0n1p3: ino=3956a, Correct links count -> 1 (2).
> [   53.289392] ntfs3: 11442 callbacks suppressed
> [   53.289401] ntfs3: nvme0n1p3: ino=6293e, Correct links count -> 1 (2).
> [   53.289972] ntfs3: nvme0n1p3: ino=61df1, Correct links count -> 1 (2).
> [   53.290241] ntfs3: nvme0n1p3: ino=61df3, Correct links count -> 1 (2).
> [   53.290578] ntfs3: nvme0n1p3: ino=61f3b, Correct links count -> 1 (2).
> [   53.290881] ntfs3: nvme0n1p3: ino=62025, Correct links count -> 1 (2).
> [   53.291045] ntfs3: nvme0n1p3: ino=629af, Correct links count -> 1 (2).
> [   53.291181] ntfs3: nvme0n1p3: ino=61e3c, Correct links count -> 1 (2).
> [   53.291463] ntfs3: nvme0n1p3: ino=61e22, Correct links count -> 1 (2).
> [   53.291743] ntfs3: nvme0n1p3: ino=62882, Correct links count -> 1 (2).
> [   53.292099] ntfs3: nvme0n1p3: ino=61b3d, Correct links count -> 1 (2).
> [   58.291790] ntfs3: 5373 callbacks suppressed
> [   58.291799] ntfs3: nvme0n1p3: ino=6d5a5, Correct links count -> 1 (2).
> [   58.292106] ntfs3: nvme0n1p3: ino=6d7f6, Correct links count -> 1 (2).
> [   58.292372] ntfs3: nvme0n1p3: ino=6db43, Correct links count -> 1 (2).
> [   58.292653] ntfs3: nvme0n1p3: ino=72557, Correct links count -> 1 (2).
> [   58.293244] ntfs3: nvme0n1p3: ino=728d8, Correct links count -> 1 (2).
> [   58.294306] ntfs3: nvme0n1p3: ino=72c6e, Correct links count -> 1 (2).
> [   58.294944] ntfs3: nvme0n1p3: ino=72ff7, Correct links count -> 1 (2).
> [   58.295666] ntfs3: nvme0n1p3: ino=73271, Correct links count -> 1 (2).
> [   58.296281] ntfs3: nvme0n1p3: ino=735fd, Correct links count -> 1 (2).
> [   58.296991] ntfs3: nvme0n1p3: ino=73880, Correct links count -> 1 (2).
> [   63.295009] ntfs3: 13921 callbacks suppressed
> [   63.295017] ntfs3: nvme0n1p3: ino=6be65, Correct links count -> 1 (2).
> [   63.295902] ntfs3: nvme0n1p3: ino=6c08e, Correct links count -> 1 (2).
> [   63.296252] ntfs3: nvme0n1p3: ino=6c2e3, Correct links count -> 1 (2).
> [   63.297581] ntfs3: nvme0n1p3: ino=6c610, Correct links count -> 1 (2).
> [   63.298321] ntfs3: nvme0n1p3: ino=6c7f9, Correct links count -> 1 (2).
> [   63.298730] ntfs3: nvme0n1p3: ino=6cb24, Correct links count -> 1 (2).
> [   63.299079] ntfs3: nvme0n1p3: ino=6ceda, Correct links count -> 1 (2).
> [   63.299408] ntfs3: nvme0n1p3: ino=6d288, Correct links count -> 1 (2).
> [   63.299727] ntfs3: nvme0n1p3: ino=6d533, Correct links count -> 1 (2).
> [   63.300080] ntfs3: nvme0n1p3: ino=6d77b, Correct links count -> 1 (2).
> [   68.299457] ntfs3: 8228 callbacks suppressed
> [   68.299467] ntfs3: nvme0n1p3: ino=3e248, Correct links count -> 1 (2).
> [   68.301391] ntfs3: nvme0n1p3: ino=5d7b7, Correct links count -> 1 (2).
> [   68.302440] ntfs3: nvme0n1p3: ino=5853d, Correct links count -> 1 (2).
> [   68.303123] ntfs3: nvme0n1p3: ino=3ca2e, Correct links count -> 1 (2).
> [   68.303722] ntfs3: nvme0n1p3: ino=59a98, Correct links count -> 1 (2).
> [   68.304292] ntfs3: nvme0n1p3: ino=59a9b, Correct links count -> 1 (2).
> [   68.304981] ntfs3: nvme0n1p3: ino=59a9e, Correct links count -> 1 (2).
> [   68.305629] ntfs3: nvme0n1p3: ino=59aa1, Correct links count -> 1 (2).
> [   68.306120] ntfs3: nvme0n1p3: ino=3214f, Correct links count -> 1 (2).
> [   68.306539] ntfs3: nvme0n1p3: ino=2077b, Correct links count -> 1 (2).
> [   73.302727] ntfs3: 8502 callbacks suppressed
> [   73.302736] ntfs3: nvme0n1p3: ino=5aa99, Correct links count -> 1 (2).
> [   73.303992] ntfs3: nvme0n1p3: ino=50ee9, Correct links count -> 1 (2).
> [   73.304744] ntfs3: nvme0n1p3: ino=20420, Correct links count -> 1 (2).
> [   73.305080] ntfs3: nvme0n1p3: ino=258c, Correct links count -> 1 (2).
> [   73.305470] ntfs3: nvme0n1p3: ino=5a30d, Correct links count -> 1 (2).
> [   73.307283] ntfs3: nvme0n1p3: ino=5a54c, Correct links count -> 1 (2).
> [   73.307890] ntfs3: nvme0n1p3: ino=5c9de, Correct links count -> 1 (2).
> [   73.308495] ntfs3: nvme0n1p3: ino=3d82d, Correct links count -> 1 (2).
> [   73.309581] ntfs3: nvme0n1p3: ino=3d839, Correct links count -> 1 (2).
> [   73.310016] ntfs3: nvme0n1p3: ino=25c77, Correct links count -> 1 (2).
> [   78.304861] ntfs3: 11462 callbacks suppressed
> [   78.304868] ntfs3: nvme0n1p3: ino=5c714, Correct links count -> 1 (2).
> [   78.305579] ntfs3: nvme0n1p3: ino=57d11, Correct links count -> 1 (2).
> [   78.306151] ntfs3: nvme0n1p3: ino=5842d, Correct links count -> 1 (2).
> [   78.307412] ntfs3: nvme0n1p3: ino=34e6, Correct links count -> 1 (3).
> [   78.307843] ntfs3: nvme0n1p3: ino=5bb23, Correct links count -> 1 (2).
> [   78.308509] ntfs3: nvme0n1p3: ino=5c722, Correct links count -> 1 (2).
> [   78.310018] ntfs3: nvme0n1p3: ino=5d761, Correct links count -> 1 (2).
> [   78.310717] ntfs3: nvme0n1p3: ino=33d18, Correct links count -> 1 (3).
> [   78.311179] ntfs3: nvme0n1p3: ino=5d75b, Correct links count -> 1 (3).
> [   78.311605] ntfs3: nvme0n1p3: ino=5c708, Correct links count -> 1 (3).
> 
> Johan

-- 
Anton Altaparmakov <anton at tuxera.com> (replace at with @)
Lead in File System Development, Tuxera Inc., http://www.tuxera.com/






[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux