]> git.baikalelectronics.ru Git - kernel.git/commit
f2fs: remove set_page_dirty for atomic f2fs_end_io_write
authorJaegeuk Kim <jaegeuk.kim@samsung.com>
Wed, 12 Dec 2012 00:47:39 +0000 (09:47 +0900)
committerJaegeuk Kim <jaegeuk.kim@samsung.com>
Wed, 26 Dec 2012 01:39:51 +0000 (10:39 +0900)
commitb0eb339ca6c3070c3151461c114ad4484e54a165
tree562e1bd2ee29b43b1b1c2bd0f5ce30f3c09064d9
parent4f322f5ad6b5f32d8d7ce66f2a6b689b51c56073
f2fs: remove set_page_dirty for atomic f2fs_end_io_write

We should guarantee not to do *scheduling while atomic*.
I found, in atomic f2fs_end_io_write(), there is a set_page_dirty() call
to deal with IO errors.

But, set_page_dirty() calls:
 -> f2fs_set_data_page_dirty()
   -> set_dirty_dir_page()
      -> cond_resched() which results in scheduling.

In order to avoid this, I'd like to remove simply set_page_dirty(),
since the page is already marked as ERROR and f2fs will be operated
as the read-only mode as well.
So, there is no recovery issue with this.

Signed-off-by: Jaegeuk Kim <jaegeuk.kim@samsung.com>
fs/f2fs/segment.c