diff for duplicates of <20110421160057.GA28712@suse.de> diff --git a/a/1.txt b/N1/1.txt index 5125b0f..d21d790 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -21,14 +21,14 @@ On Thu, Apr 21, 2011 at 11:26:36PM +0900, Minchan Kim wrote: > > > > lock. The effect is noticable in page_test from aim9. > > > > > > > > AIM9 -> > > > 2.6.38-vanilla 2.6.38-checkptenone -> > > > creat-clo 446.10 ( 0.00%) 424.47 (-5.10%) -> > > > page_test 38.10 ( 0.00%) 42.04 ( 9.37%) -> > > > brk_test 52.45 ( 0.00%) 51.57 (-1.71%) -> > > > exec_test 382.00 ( 0.00%) 456.90 (16.39%) -> > > > fork_test 60.11 ( 0.00%) 67.79 (11.34%) +> > > > 2.6.38-vanilla 2.6.38-checkptenone +> > > > creat-clo 446.10 ( 0.00%) 424.47 (-5.10%) +> > > > page_test 38.10 ( 0.00%) 42.04 ( 9.37%) +> > > > brk_test 52.45 ( 0.00%) 51.57 (-1.71%) +> > > > exec_test 382.00 ( 0.00%) 456.90 (16.39%) +> > > > fork_test 60.11 ( 0.00%) 67.79 (11.34%) > > > > MMTests Statistics: duration -> > > > Total Elapsed Time (seconds) 611.90 612.22 +> > > > Total Elapsed Time (seconds) 611.90 612.22 > > > > > > > > (While this affects 2.6.38, it is a performance rather than a > > > > functional bug and normally outside the rules -stable. While the big @@ -39,22 +39,22 @@ On Thu, Apr 21, 2011 at 11:26:36PM +0900, Minchan Kim wrote: > > > > Reported-by: Raz Ben Yehuda <raziebe@gmail.com> > > > > Signed-off-by: Mel Gorman <mgorman@suse.de> > > > > -- -> > > > mm/memory.c | 2 +- -> > > > 1 file changed, 1 insertion(+), 1 deletion(-) +> > > > mm/memory.c | 2 +- +> > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > > > diff --git a/mm/memory.c b/mm/memory.c > > > > index 5823698..1659574 100644 > > > > --- a/mm/memory.c > > > > +++ b/mm/memory.c > > > > @@ -3322,7 +3322,7 @@ int handle_mm_fault(struct mm_struct *mm, struct vm_area_struct *vma, -> > > > * run pte_offset_map on the pmd, if an huge pmd could -> > > > * materialize from under us from a different thread. -> > > > */ -> > > > - if (unlikely(__pte_alloc(mm, vma, pmd, address))) -> > > > + if (unlikely(pmd_none(*pmd)) && __pte_alloc(mm, vma, pmd, address)) -> > > > return VM_FAULT_OOM; -> > > > /* if an huge pmd materialized from under us just retry later */ -> > > > if (unlikely(pmd_trans_huge(*pmd))) +> > > > * run pte_offset_map on the pmd, if an huge pmd could +> > > > * materialize from under us from a different thread. +> > > > */ +> > > > - if (unlikely(__pte_alloc(mm, vma, pmd, address))) +> > > > + if (unlikely(pmd_none(*pmd)) && __pte_alloc(mm, vma, pmd, address)) +> > > > return VM_FAULT_OOM; +> > > > /* if an huge pmd materialized from under us just retry later */ +> > > > if (unlikely(pmd_trans_huge(*pmd))) > > > > > > > > > > Reviewed-by: Minchan Kim <minchan.kim@gmail.com> @@ -131,3 +131,10 @@ Thanks -- Mel Gorman SUSE Labs + +-- +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/ . +Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ +Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a> diff --git a/a/content_digest b/N1/content_digest index 3df2ad3..6ccd8b7 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -39,14 +39,14 @@ "> > > > lock. The effect is noticable in page_test from aim9.\n" "> > > >\n" "> > > > AIM9\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\2402.6.38-vanilla 2.6.38-checkptenone\n" - "> > > > creat-clo \302\240 \302\240 \302\240446.10 ( 0.00%) \302\240 424.47 (-5.10%)\n" - "> > > > page_test \302\240 \302\240 \302\240 38.10 ( 0.00%) \302\240 \302\24042.04 ( 9.37%)\n" - "> > > > brk_test \302\240 \302\240 \302\240 \302\24052.45 ( 0.00%) \302\240 \302\24051.57 (-1.71%)\n" - "> > > > exec_test \302\240 \302\240 \302\240382.00 ( 0.00%) \302\240 456.90 (16.39%)\n" - "> > > > fork_test \302\240 \302\240 \302\240 60.11 ( 0.00%) \302\240 \302\24067.79 (11.34%)\n" + "> > > > 2.6.38-vanilla 2.6.38-checkptenone\n" + "> > > > creat-clo 446.10 ( 0.00%) 424.47 (-5.10%)\n" + "> > > > page_test 38.10 ( 0.00%) 42.04 ( 9.37%)\n" + "> > > > brk_test 52.45 ( 0.00%) 51.57 (-1.71%)\n" + "> > > > exec_test 382.00 ( 0.00%) 456.90 (16.39%)\n" + "> > > > fork_test 60.11 ( 0.00%) 67.79 (11.34%)\n" "> > > > MMTests Statistics: duration\n" - "> > > > Total Elapsed Time (seconds) \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240611.90 \302\240 \302\240612.22\n" + "> > > > Total Elapsed Time (seconds) 611.90 612.22\n" "> > > >\n" "> > > > (While this affects 2.6.38, it is a performance rather than a\n" "> > > > functional bug and normally outside the rules -stable. While the big\n" @@ -57,22 +57,22 @@ "> > > > Reported-by: Raz Ben Yehuda <raziebe@gmail.com>\n" "> > > > Signed-off-by: Mel Gorman <mgorman@suse.de>\n" "> > > > --\n" - "> > > > \302\240mm/memory.c | \302\240 \302\2402 +-\n" - "> > > > \302\2401 file changed, 1 insertion(+), 1 deletion(-)\n" + "> > > > mm/memory.c | 2 +-\n" + "> > > > 1 file changed, 1 insertion(+), 1 deletion(-)\n" "> > > >\n" "> > > > diff --git a/mm/memory.c b/mm/memory.c\n" "> > > > index 5823698..1659574 100644\n" "> > > > --- a/mm/memory.c\n" "> > > > +++ b/mm/memory.c\n" "> > > > @@ -3322,7 +3322,7 @@ int handle_mm_fault(struct mm_struct *mm, struct vm_area_struct *vma,\n" - "> > > > \302\240 \302\240 \302\240 \302\240 * run pte_offset_map on the pmd, if an huge pmd could\n" - "> > > > \302\240 \302\240 \302\240 \302\240 * materialize from under us from a different thread.\n" - "> > > > \302\240 \302\240 \302\240 \302\240 */\n" - "> > > > - \302\240 \302\240 \302\240 if (unlikely(__pte_alloc(mm, vma, pmd, address)))\n" - "> > > > + \302\240 \302\240 \302\240 if (unlikely(pmd_none(*pmd)) && __pte_alloc(mm, vma, pmd, address))\n" - "> > > > \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240return VM_FAULT_OOM;\n" - "> > > > \302\240 \302\240 \302\240 \302\240/* if an huge pmd materialized from under us just retry later */\n" - "> > > > \302\240 \302\240 \302\240 \302\240if (unlikely(pmd_trans_huge(*pmd)))\n" + "> > > > * run pte_offset_map on the pmd, if an huge pmd could\n" + "> > > > * materialize from under us from a different thread.\n" + "> > > > */\n" + "> > > > - if (unlikely(__pte_alloc(mm, vma, pmd, address)))\n" + "> > > > + if (unlikely(pmd_none(*pmd)) && __pte_alloc(mm, vma, pmd, address))\n" + "> > > > return VM_FAULT_OOM;\n" + "> > > > /* if an huge pmd materialized from under us just retry later */\n" + "> > > > if (unlikely(pmd_trans_huge(*pmd)))\n" "> > > >\n" "> > > \n" "> > > Reviewed-by: Minchan Kim <minchan.kim@gmail.com>\n" @@ -148,6 +148,13 @@ "\n" "-- \n" "Mel Gorman\n" - SUSE Labs + "SUSE Labs\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" + "Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/\n" + "Don't email: <a href=mailto:\"dont@kvack.org\"> email@kvack.org </a>" -91fe5ccda7a6fa79a32269c61ffaf9d0948e2097a33ac43c0009247bd100d052 +aa338c1d02d88fd1eefa32c188990460b8367a158d08a42143371267cafae3c9
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.