From: Yan Pas <yanp.bugz@gmail.com>
To: Konstantin Komarov <almaz.alexandrovich@paragon-software.com>,
linux-fsdevel@vger.kernel.org
Cc: viro@zeniv.linux.org.uk, linux-kernel@vger.kernel.org,
pali@kernel.org, dsterba@suse.cz, aaptel@suse.com,
willy@infradead.org, rdunlap@infradead.org, joe@perches.com,
mark@harmstone.com, nborisov@suse.com,
linux-ntfs-dev@lists.sourceforge.net, anton@tuxera.com,
dan.carpenter@oracle.com, hch@lst.de, ebiggers@kernel.org,
andy.lavr@gmail.com, kari.argillander@gmail.com,
oleksandr@natalenko.name
Subject: Re: [PATCH v26 00/10] NTFS read-write driver GPL implementation by Paragon Software
Date: Sun, 25 Jul 2021 20:02:18 +0300 [thread overview]
Message-ID: <cc64ac69-f4e5-3fc4-1362-ced7cf68119a@gmail.com> (raw)
In-Reply-To: <20210402155347.64594-1-almaz.alexandrovich@paragon-software.com>
Hi everyone!
I'm regular Arch-linux user and have been using your driver for a while
(https://aur.archlinux.org/packages/ntfs3-dkms/). There are a lot of
users who tested this patch and some of them, like me, have complaints.
I faced stalling of a kthread with some ntfs functions in a stacktrace
while rsyncing one NTFS partition (SATA) to another (USB HDD)
(https://aur.archlinux.org/packages/ntfs3-dkms/#comment-818819):
[ 5529.507567] INFO: task kworker/0:1:18 blocked for more than 1105 seconds.
[ 5529.507580] Tainted: P OE 5.13.4-arch1-1 #1
[ 5529.507584] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs"
disables this message.
[ 5529.507586] task:kworker/0:1 state:D stack: 0 pid: 18
ppid: 2 flags:0x00004000
[ 5529.507598] Workqueue: usb_hub_wq hub_event
[ 5529.507612] Call Trace:
[ 5529.507615] ? out_of_line_wait_on_bit_lock+0xb0/0xb0
[ 5529.507631] __schedule+0x310/0x930
[ 5529.507641] ? out_of_line_wait_on_bit_lock+0xb0/0xb0
[ 5529.507648] schedule+0x5b/0xc0
[ 5529.507654] bit_wait+0xd/0x60
[ 5529.507661] __wait_on_bit+0x2a/0x90
[ 5529.507669] __inode_wait_for_writeback+0xb0/0xe0
[ 5529.507680] ? var_wake_function+0x20/0x20
[ 5529.507689] writeback_single_inode+0x64/0x140
[ 5529.507699] sync_inode_metadata+0x3d/0x60
[ 5529.507712] ntfs_set_state+0x126/0x1a0 [ntfs3]
[ 5529.507738] ni_write_inode+0x244/0xef0 [ntfs3]
[ 5529.507764] ? pagevec_lookup_range_tag+0x24/0x30
[ 5529.507772] ? __filemap_fdatawait_range+0x6f/0xf0
[ 5529.507785] __writeback_single_inode+0x260/0x310
[ 5529.507795] writeback_single_inode+0xa7/0x140
[ 5529.507803] sync_inode_metadata+0x3d/0x60
[ 5529.507814] ntfs_set_state+0x126/0x1a0 [ntfs3]
[ 5529.507834] ntfs_sync_fs+0xf9/0x100 [ntfs3]
[ 5529.507857] sync_filesystem+0x40/0x90
[ 5529.507868] fsync_bdev+0x21/0x60
[ 5529.507874] delete_partition+0x13/0x80
[ 5529.507882] blk_drop_partitions+0x5b/0xa0
[ 5529.507889] del_gendisk+0xa5/0x220
...
Also adding ntfs3 partition (1.4TB) to fstab makes booting much slower:
I see two lines in console (systemd version and some other line) and HDD
led constantly flickering (seems like ntfs3 is traversing entire
partition)
(https://aur.archlinux.org/pkgbase/ntfs3-dkms/edit-comment/?comment_id=818673).
Regards, Yan.
next prev parent reply other threads:[~2021-07-25 17:02 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-04-02 15:53 [PATCH v26 00/10] NTFS read-write driver GPL implementation by Paragon Software Konstantin Komarov
2021-04-02 15:53 ` [PATCH v26 01/10] fs/ntfs3: Add headers and misc files Konstantin Komarov
2021-04-02 15:53 ` [PATCH v26 02/10] fs/ntfs3: Add initialization of super block Konstantin Komarov
2021-05-20 16:51 ` Darrick J. Wong
2021-04-02 15:53 ` [PATCH v26 03/10] fs/ntfs3: Add bitmap Konstantin Komarov
2021-04-02 15:53 ` [PATCH v26 04/10] fs/ntfs3: Add file operations and implementation Konstantin Komarov
2021-07-12 8:28 ` Oleksandr Natalenko
2021-04-02 15:53 ` [PATCH v26 05/10] fs/ntfs3: Add attrib operations Konstantin Komarov
2021-04-02 15:53 ` [PATCH v26 06/10] fs/ntfs3: Add compression Konstantin Komarov
2021-04-02 15:53 ` [PATCH v26 07/10] fs/ntfs3: Add NTFS journal Konstantin Komarov
2021-04-02 15:53 ` [PATCH v26 08/10] fs/ntfs3: Add Kconfig, Makefile and doc Konstantin Komarov
2021-04-02 15:53 ` [PATCH v26 09/10] fs/ntfs3: Add NTFS3 in fs/Kconfig and fs/Makefile Konstantin Komarov
2021-04-02 15:53 ` [PATCH v26 10/10] fs/ntfs3: Add MAINTAINERS Konstantin Komarov
2021-05-19 3:47 ` [PATCH v26 00/10] NTFS read-write driver GPL implementation by Paragon Software Neal Gompa
2021-05-19 3:51 ` Neal Gompa
2021-05-20 16:13 ` Darrick J. Wong
2021-07-17 16:36 ` Pali Rohár
2021-07-25 17:02 ` Yan Pas [this message]
-- strict thread matches above, loose matches on Subject: below --
2021-06-07 9:55 Marius Cirsta
2021-07-28 16:23 Mohammad Rasim
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=cc64ac69-f4e5-3fc4-1362-ced7cf68119a@gmail.com \
--to=yanp.bugz@gmail.com \
--cc=aaptel@suse.com \
--cc=almaz.alexandrovich@paragon-software.com \
--cc=andy.lavr@gmail.com \
--cc=anton@tuxera.com \
--cc=dan.carpenter@oracle.com \
--cc=dsterba@suse.cz \
--cc=ebiggers@kernel.org \
--cc=hch@lst.de \
--cc=joe@perches.com \
--cc=kari.argillander@gmail.com \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-ntfs-dev@lists.sourceforge.net \
--cc=mark@harmstone.com \
--cc=nborisov@suse.com \
--cc=oleksandr@natalenko.name \
--cc=pali@kernel.org \
--cc=rdunlap@infradead.org \
--cc=viro@zeniv.linux.org.uk \
--cc=willy@infradead.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.