| To: | Jan Kara <jack@xxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] ext4: fix race aio-dio vs freeze_fs |
| From: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
| Date: | Tue, 24 Nov 2015 08:07:23 -0800 |
| Cc: | Dmitry Monakhov <dmonakhov@xxxxxxxxxx>, linux-ext4@xxxxxxxxxxxxxxx, tytso@xxxxxxx, xfs@xxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20151124132421.GG25232@xxxxxxxxxxxxx> |
| References: | <1448294568-20892-1-git-send-email-dmonakhov@xxxxxxxxxx> <20151124132421.GG25232@xxxxxxxxxxxxx> |
| User-agent: | Mutt/1.5.23 (2014-03-12) |
On Tue, Nov 24, 2015 at 02:24:21PM +0100, Jan Kara wrote: > Well, this problem seems to suggest that we have the freeze protection for > AIO writes wrong. We should call file_end_write() from aio_complete() and > not from aio_run_iocb()... I believe XFS and other filesystems may have > problems with this as well (CCed). Attached patch (so far only compile > tested since my test machine is pondering on something else) should fix > this. Sounds like one way to do it, but we'd really want a vfs_* helper for this so that it doesn't have to duplicated in other write_iter users like the loop driver, which seems to be missing file file_start_write/file_end_write entirely. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH] ext4: fix race aio-dio vs freeze_fs, Jan Kara |
|---|---|
| Next by Date: | Re: [PATCH] ext4: fix race aio-dio vs freeze_fs, Dmitry Monakhov |
| Previous by Thread: | Re: [PATCH] ext4: fix race aio-dio vs freeze_fs, Jan Kara |
| Next by Thread: | Re: [PATCH] ext4: fix race aio-dio vs freeze_fs, Jan Kara |
| Indexes: | [Date] [Thread] [Top] [All Lists] |