Commit 5df7731f authored by Chao Yu's avatar Chao Yu Committed by Jaegeuk Kim
Browse files

f2fs: introduce DEFAULT_IO_TIMEOUT 



As Geert Uytterhoeven reported:

for parameter HZ/50 in congestion_wait(BLK_RW_ASYNC, HZ/50);

On some platforms, HZ can be less than 50, then unexpected 0 timeout
jiffies will be set in congestion_wait().

This patch introduces a macro DEFAULT_IO_TIMEOUT to wrap a determinate
value with msecs_to_jiffies(20) to instead HZ/50 to avoid such issue.

Quoted from Geert Uytterhoeven:

"A timeout of HZ means 1 second.
HZ/50 means 20 ms, but has the risk of being zero, if HZ < 50.

If you want to use a timeout of 20 ms, you best use msecs_to_jiffies(20),
as that takes care of the special cases, and never returns 0."

Signed-off-by: default avatarChao Yu <yuchao0@huawei.com>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent 2bac0763

fs/f2fs/checkpoint.c

+1 −1
Original line number Diff line number Diff line
@@ -1260,7 +1260,7 @@ void f2fs_wait_on_all_pages(struct f2fs_sb_info *sbi, int type) 
		if (unlikely(f2fs_cp_error(sbi)))

			break;



		io_schedule_timeout(HZ/50);

		io_schedule_timeout(DEFAULT_IO_TIMEOUT);

	}

	finish_wait(&sbi->cp_wait, &wait);

}

fs/f2fs/compress.c

+2 −1
Original line number Diff line number Diff line
@@ -988,7 +988,8 @@ retry_write: 
			} else if (ret == -EAGAIN) {

				ret = 0;

				cond_resched();

				congestion_wait(BLK_RW_ASYNC, HZ/50);

				congestion_wait(BLK_RW_ASYNC,

						DEFAULT_IO_TIMEOUT);

				lock_page(cc->rpages[i]);

				clear_page_dirty_for_io(cc->rpages[i]);

				goto retry_write;

fs/f2fs/data.c

+2 −2
Original line number Diff line number Diff line
@@ -2317,7 +2317,7 @@ retry_encrypt: 
		/* flush pending IOs and wait for a while in the ENOMEM case */

		if (PTR_ERR(fio->encrypted_page) == -ENOMEM) {

			f2fs_flush_merged_writes(fio->sbi);

			congestion_wait(BLK_RW_ASYNC, HZ/50);

			congestion_wait(BLK_RW_ASYNC, DEFAULT_IO_TIMEOUT);

			gfp_flags |= __GFP_NOFAIL;

			goto retry_encrypt;

		}
@@ -2908,7 +2908,7 @@ result: 
					if (wbc->sync_mode == WB_SYNC_ALL) {

						cond_resched();

						congestion_wait(BLK_RW_ASYNC,

								HZ/50);

							DEFAULT_IO_TIMEOUT);

						goto retry_write;

					}

					goto next;

fs/f2fs/f2fs.h

+3 −0
Original line number Diff line number Diff line
@@ -558,6 +558,9 @@ enum { 


#define DEFAULT_RETRY_IO_COUNT	8	/* maximum retry read IO count */



/* congestion wait timeout value, default: 20ms */

#define	DEFAULT_IO_TIMEOUT	(msecs_to_jiffies(20))



/* maximum retry quota flush count */

#define DEFAULT_RETRY_QUOTA_FLUSH_COUNT		8

fs/f2fs/gc.c

+2 −1
Original line number Diff line number Diff line
@@ -977,7 +977,8 @@ retry: 
		if (err) {

			clear_cold_data(page);

			if (err == -ENOMEM) {

				congestion_wait(BLK_RW_ASYNC, HZ/50);

				congestion_wait(BLK_RW_ASYNC,

						DEFAULT_IO_TIMEOUT);

				goto retry;

			}

			if (is_dirty)

CRA Git | Maintained and supported by SUSTech CRA and CCSE