linux-f2fs-devel.lists.sourceforge.net archive mirror
 help / color / mirror / Atom feed
* [PATCH v2] f2fs: Add f2fs_balance_fs for direct IO
@ 2014-07-12 12:10 Huang Ying
  2014-07-26  8:03 ` huang ying
  0 siblings, 1 reply; 3+ messages in thread
From: Huang Ying @ 2014-07-12 12:10 UTC (permalink / raw)
  To: Jaegeuk Kim, Changman Lee; +Cc: Huang Ying, linux-kernel, linux-f2fs-devel

Otherwise, if a large amount of direct IO writes were done, the
segment allocation may be failed because no enough segments are gced.

Changes:

v2: add f2fs_balance_fs into __get_data_block instead of f2fs_direct_IO.

Signed-off-by: Huang, Ying <ying.huang@intel.com>
---
 fs/f2fs/data.c |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -626,8 +626,10 @@ static int __get_data_block(struct inode
 	if (check_extent_cache(inode, pgofs, bh_result))
 		goto out;
 
-	if (create)
+	if (create) {
+		f2fs_balance_fs(sbi);
 		f2fs_lock_op(sbi);
+	}
 
 	/* When reading holes, we need its node page */
 	set_new_dnode(&dn, inode, NULL, NULL, 0);

------------------------------------------------------------------------------

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] f2fs: Add f2fs_balance_fs for direct IO
  2014-07-12 12:10 [PATCH v2] f2fs: Add f2fs_balance_fs for direct IO Huang Ying
@ 2014-07-26  8:03 ` huang ying
  2014-07-26 22:51   ` Jaegeuk Kim
  0 siblings, 1 reply; 3+ messages in thread
From: huang ying @ 2014-07-26  8:03 UTC (permalink / raw)
  To: Huang Ying; +Cc: Jaegeuk Kim, LKML, linux-f2fs-devel


[-- Attachment #1.1: Type: text/plain, Size: 88 bytes --]

Hi, Jaegeuk,

Do you have time to take a look at this patch?

Best Regards,
Huang, Ying

[-- Attachment #1.2: Type: text/html, Size: 143 bytes --]

[-- Attachment #2: Type: text/plain, Size: 362 bytes --]

------------------------------------------------------------------------------
Want fast and easy access to all the code in your enterprise? Index and
search up to 200,000 lines of code with a free copy of Black Duck
Code Sight - the same software that powers the world's largest code
search on Ohloh, the Black Duck Open Hub! Try it now.
http://p.sf.net/sfu/bds

[-- Attachment #3: Type: text/plain, Size: 179 bytes --]

_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v2] f2fs: Add f2fs_balance_fs for direct IO
  2014-07-26  8:03 ` huang ying
@ 2014-07-26 22:51   ` Jaegeuk Kim
  0 siblings, 0 replies; 3+ messages in thread
From: Jaegeuk Kim @ 2014-07-26 22:51 UTC (permalink / raw)
  To: huang ying; +Cc: linux-f2fs-devel, LKML, Huang Ying

Hi Huang,

On Sat, Jul 26, 2014 at 04:03:34PM +0800, huang ying wrote:
> Hi, Jaegeuk,
> 
> Do you have time to take a look at this patch?

I merged this in my local tree. :)
Thanks,

> 
> Best Regards,
> Huang, Ying

------------------------------------------------------------------------------
Want fast and easy access to all the code in your enterprise? Index and
search up to 200,000 lines of code with a free copy of Black Duck
Code Sight - the same software that powers the world's largest code
search on Ohloh, the Black Duck Open Hub! Try it now.
http://p.sf.net/sfu/bds

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-07-26 22:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-12 12:10 [PATCH v2] f2fs: Add f2fs_balance_fs for direct IO Huang Ying
2014-07-26  8:03 ` huang ying
2014-07-26 22:51   ` Jaegeuk Kim

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).