diff for duplicates of <20090730020922.GD7326@localhost> diff --git a/a/1.txt b/N1/1.txt index 643db66..7c2b18a 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -3,10 +3,10 @@ On Thu, Jul 30, 2009 at 09:28:07AM +0800, Martin Bligh wrote: > > BTW, can you explain this code at the bottom of generic_sync_sb_inodes > > for me? > > -> > if (wbc->nr_to_write <= 0) { -> > wbc->more_io = 1; -> > break; -> > } +> > A A A A A A A A if (wbc->nr_to_write <= 0) { +> > A A A A A A A A A A A A wbc->more_io = 1; +> > A A A A A A A A A A A A break; +> > A A A A A A A A } > > > > I don't understand why we are setting more_io here? AFAICS, more_io > > means there's more stuff to write ... I would think we'd set this if @@ -16,7 +16,7 @@ On Thu, Jul 30, 2009 at 09:28:07AM +0800, Martin Bligh wrote: > > break check and do: > > > > if (!list_empty(&sb->s_more_io) || !list_empty(&sb->s_io)) -> > wbc->more_io = 1; +> > A A A A wbc->more_io = 1; > > > > Am I just misunderstanding the intent of more_io ? > @@ -52,3 +52,9 @@ Fengguang > + wbc->more_io = 1; > return; /* Leave any unwritten inodes on s_io */ > } + +-- +To unsubscribe, send a message with 'unsubscribe linux-mm' in +the body to majordomo@kvack.org. For more info on Linux MM, +see: http://www.linux-mm.org/ . +Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> diff --git a/a/content_digest b/N1/content_digest index 54e2423..b5b02d8 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -22,10 +22,10 @@ "> > BTW, can you explain this code at the bottom of generic_sync_sb_inodes\n" "> > for me?\n" "> >\n" - "> > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240if (wbc->nr_to_write <= 0) {\n" - "> > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240wbc->more_io = 1;\n" - "> > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240break;\n" - "> > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240}\n" + "> > A A A A A A A A if (wbc->nr_to_write <= 0) {\n" + "> > A A A A A A A A A A A A wbc->more_io = 1;\n" + "> > A A A A A A A A A A A A break;\n" + "> > A A A A A A A A }\n" "> >\n" "> > I don't understand why we are setting more_io here? AFAICS, more_io\n" "> > means there's more stuff to write ... I would think we'd set this if\n" @@ -35,7 +35,7 @@ "> > break check and do:\n" "> >\n" "> > if (!list_empty(&sb->s_more_io) || !list_empty(&sb->s_io))\n" - "> > \302\240 \302\240 \302\240 \302\240wbc->more_io = 1;\n" + "> > A A A A wbc->more_io = 1;\n" "> >\n" "> > Am I just misunderstanding the intent of more_io ?\n" "> \n" @@ -70,6 +70,12 @@ "> + if (!list_empty(&sb->s_more_io) || !list_empty(&sb->s_io)\n" "> + wbc->more_io = 1;\n" "> return; /* Leave any unwritten inodes on s_io */\n" - > } + "> }\n" + "\n" + "--\n" + "To unsubscribe, send a message with 'unsubscribe linux-mm' in\n" + "the body to majordomo@kvack.org. For more info on Linux MM,\n" + "see: http://www.linux-mm.org/ .\n" + "Don't email: <a href=mailto:\"dont@kvack.org\"> email@kvack.org </a>" -989c9d91a56d860e63a621bddeb88ab5fc28bc0444e178318726fc6d0a29c89e +20439e7dda6576fe933f40ec8bcfc93c12bb12008e8e1bc89083362ac2fec67d
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.