diff for duplicates of <4F9A5368.5000904@gmail.com> diff --git a/a/1.txt b/N1/1.txt index 11ee470..7038f1d 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,13 +1,10 @@ -=E4=BA=8E 2012=E5=B9=B404=E6=9C=8826=E6=97=A5 23:12, Josef Bacik =E5=86= -=99=E9=81=93: +于 2012年04月26日 23:12, Josef Bacik 写道: > On Thu, Apr 26, 2012 at 03:55:52PM +0800, Xie Miao wrote: ->> On Thu, Apr 26, 2012 at 11:11 AM, Dave Chinner <david@fromorbit.com>= - wrote: +>> On Thu, Apr 26, 2012 at 11:11 AM, Dave Chinner <david@fromorbit.com> wrote: >>>> writeback_inodes_sb(_nr) grabs s_umount lock when it want to start >>>> writeback, ->>>> it may bring us deadlock problem when doing umount. So we introduc= -e new +>>>> it may bring us deadlock problem when doing umount. So we introduce new >>>> functions -- try_to_writeback_inodes_sb(_nr) -- which use >>>> down_read_trylock() >>>> instead of down_read() to avoid that deadlock problem. @@ -18,22 +15,16 @@ e new >>> This just re-implements writeback_inodes_[nr]_sb_if_idle() with a >>> trylock instead of a blocking lock. >>> ->>> Just replace the blocking lock in writeback_inodes_[nr]_sb_if_idle(= -) +>>> Just replace the blocking lock in writeback_inodes_[nr]_sb_if_idle() >>> with a trylock and use that. >> ->> The change of these two functions is relative to three modules, so = -I think ->> the patch set now is easy to be reviewed by the developers of each m= -odule. +>> The change of these two functions is relative to three modules, so I think +>> the patch set now is easy to be reviewed by the developers of each module. >> ->=20 -> I agree with David, there's no sense in making something completely s= -eperate, -> this function was introduced soley to kick off background writeout if= - we could -> with no garuntees, if the other users suddenly don't like the behavio= -r they can +> +> I agree with David, there's no sense in making something completely seperate, +> this function was introduced soley to kick off background writeout if we could +> with no garuntees, if the other users suddenly don't like the behavior they can > creating something different for themselves. Thanks, @@ -42,7 +33,6 @@ OK, I'll make them together. Thanks Miao -- -To unsubscribe from this list: send the line "unsubscribe linux-ext4" i= -n +To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html diff --git a/a/content_digest b/N1/content_digest index 52df794..27d00ac 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -17,16 +17,13 @@ " Linux Ext4 <linux-ext4@vger.kernel.org>\0" "\00:1\0" "b\0" - "=E4=BA=8E 2012=E5=B9=B404=E6=9C=8826=E6=97=A5 23:12, Josef Bacik =E5=86=\n" - "=99=E9=81=93:\n" + "\344\272\216 2012\345\271\26404\346\234\21026\346\227\245 23:12, Josef Bacik \345\206\231\351\201\223:\n" "\n" "> On Thu, Apr 26, 2012 at 03:55:52PM +0800, Xie Miao wrote:\n" - ">> On Thu, Apr 26, 2012 at 11:11 AM, Dave Chinner <david@fromorbit.com>=\n" - " wrote:\n" + ">> On Thu, Apr 26, 2012 at 11:11 AM, Dave Chinner <david@fromorbit.com> wrote:\n" ">>>> writeback_inodes_sb(_nr) grabs s_umount lock when it want to start\n" ">>>> writeback,\n" - ">>>> it may bring us deadlock problem when doing umount. So we introduc=\n" - "e new\n" + ">>>> it may bring us deadlock problem when doing umount. So we introduce new\n" ">>>> functions -- try_to_writeback_inodes_sb(_nr) -- which use\n" ">>>> down_read_trylock()\n" ">>>> instead of down_read() to avoid that deadlock problem.\n" @@ -37,22 +34,16 @@ ">>> This just re-implements writeback_inodes_[nr]_sb_if_idle() with a\n" ">>> trylock instead of a blocking lock.\n" ">>>\n" - ">>> Just replace the blocking lock in writeback_inodes_[nr]_sb_if_idle(=\n" - ")\n" + ">>> Just replace the blocking lock in writeback_inodes_[nr]_sb_if_idle()\n" ">>> with a trylock and use that.\n" ">>\n" - ">> The change of these two functions is relative to three modules, so =\n" - "I think\n" - ">> the patch set now is easy to be reviewed by the developers of each m=\n" - "odule.\n" + ">> The change of these two functions is relative to three modules, so I think\n" + ">> the patch set now is easy to be reviewed by the developers of each module.\n" ">>\n" - ">=20\n" - "> I agree with David, there's no sense in making something completely s=\n" - "eperate,\n" - "> this function was introduced soley to kick off background writeout if=\n" - " we could\n" - "> with no garuntees, if the other users suddenly don't like the behavio=\n" - "r they can\n" + "> \n" + "> I agree with David, there's no sense in making something completely seperate,\n" + "> this function was introduced soley to kick off background writeout if we could\n" + "> with no garuntees, if the other users suddenly don't like the behavior they can\n" "> creating something different for themselves. Thanks,\n" "\n" "\n" @@ -61,9 +52,8 @@ "Thanks\n" "Miao\n" "--\n" - "To unsubscribe from this list: send the line \"unsubscribe linux-ext4\" i=\n" - "n\n" + "To unsubscribe from this list: send the line \"unsubscribe linux-ext4\" in\n" "the body of a message to majordomo@vger.kernel.org\n" More majordomo info at http://vger.kernel.org/majordomo-info.html -1503a876b24b471254d136638d0701fbd65c02541d5fb0f68cc305f36e8c32fd +8d9be607a306b5184f158b41df95a323525ffb869c3a984177c79d95ba8b1ab9
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.