diff for duplicates of <20161222081713.GA32480@node.shutemov.name> diff --git a/a/1.txt b/N1/1.txt index 684a25c..693e026 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -74,39 +74,3 @@ false-negative. Bit check was the right thing. And I don't understand why do you need to change code in __handle_mm_fault() at all. -From what I see change to huge_pmd_set_accessed() should be enough. - -> + goto pte_fault; -> return ret; -> - } else { -> - huge_pmd_set_accessed(&vmf, orig_pmd); -> - return 0; -> + } -> } -> + -> + huge_pmd_set_accessed(&vmf, orig_pmd); -> + return 0; -> } -> } -> - -> +pte_fault: -> return handle_pte_fault(&vmf); -> } -> -> -- -> 2.7.4 -> -> -- -> 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> - --- - Kirill A. Shutemov - --- -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 3fa0eaf..b7e3c85 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -90,42 +90,6 @@ "false-negative. Bit check was the right thing.\n" "\n" "And I don't understand why do you need to change code in\n" - "__handle_mm_fault() at all.\n" - "From what I see change to huge_pmd_set_accessed() should be enough.\n" - "\n" - "> +\t\t\t\t\t\tgoto pte_fault;\n" - "> \t\t\t\t\treturn ret;\n" - "> -\t\t\t} else {\n" - "> -\t\t\t\thuge_pmd_set_accessed(&vmf, orig_pmd);\n" - "> -\t\t\t\treturn 0;\n" - "> +\t\t\t\t}\n" - "> \t\t\t}\n" - "> +\n" - "> +\t\t\thuge_pmd_set_accessed(&vmf, orig_pmd);\n" - "> +\t\t\treturn 0;\n" - "> \t\t}\n" - "> \t}\n" - "> -\n" - "> +pte_fault:\n" - "> \treturn handle_pte_fault(&vmf);\n" - "> }\n" - "> \n" - "> -- \n" - "> 2.7.4\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>\n" - "\n" - "-- \n" - " Kirill A. Shutemov\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>" + __handle_mm_fault() at all. -295bc25dd29cd98e7ed95f397c39793a726075822050d041061191c02e3c96a3 +83a57e2094d8ffeba3afb8f6042fe8489bda275419b9704c59e59d24bc7d86fb
diff --git a/a/1.txt b/N2/1.txt index 684a25c..f614944 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -3,7 +3,7 @@ On Thu, Dec 22, 2016 at 08:48:21AM +0900, Minchan Kim wrote: I guess you wanted put b8d3c4c3009d before [1], right? -> http://lkml.kernel.org/r/mvmmvfy37g1.fsf@hawking.suse.de +> http://lkml.kernel.org/r/mvmmvfy37g1.fsf at hawking.suse.de > > The problem is page fault handler supports only accessed flag emulation > for THP page of SW-dirty/accessed architecture. @@ -17,8 +17,8 @@ I guess you wanted put b8d3c4c3009d before [1], right? > Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com> > Cc: Will Deacon <will.deacon@arm.com> > Cc: Catalin Marinas <catalin.marinas@arm.com> -> Cc: linux-arch@vger.kernel.org -> Cc: linux-arm-kernel@lists.infradead.org +> Cc: linux-arch at vger.kernel.org +> Cc: linux-arm-kernel at lists.infradead.org > Cc: <stable@vger.kernel.org> [4.5+] > Fixes: b8d3c4c3009d ("mm/huge_memory.c: don't split THP page when MADV_FREE syscall is called") > Reported-by: Andreas Schwab <schwab@suse.de> @@ -74,7 +74,7 @@ false-negative. Bit check was the right thing. And I don't understand why do you need to change code in __handle_mm_fault() at all. -From what I see change to huge_pmd_set_accessed() should be enough. +>From what I see change to huge_pmd_set_accessed() should be enough. > + goto pte_fault; > return ret; @@ -98,15 +98,9 @@ From what I see change to huge_pmd_set_accessed() should be enough. > > -- > To unsubscribe, send a message with 'unsubscribe linux-mm' in -> the body to majordomo@kvack.org. For more info on Linux MM, +> the body to majordomo at 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> +> Don't email: <a href=mailto:"dont@kvack.org"> email at kvack.org </a> -- Kirill A. Shutemov - --- -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/N2/content_digest index 3fa0eaf..596e998 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,18 +1,8 @@ "ref\01482364101-16204-1-git-send-email-minchan@kernel.org\0" - "From\0Kirill A. Shutemov <kirill@shutemov.name>\0" - "Subject\0Re: [PATCH] mm: pmd dirty emulation in page fault handler\0" + "From\0kirill@shutemov.name (Kirill A. Shutemov)\0" + "Subject\0[PATCH] mm: pmd dirty emulation in page fault handler\0" "Date\0Thu, 22 Dec 2016 11:17:13 +0300\0" - "To\0Minchan Kim <minchan@kernel.org>\0" - "Cc\0Andrew Morton <akpm@linux-foundation.org>" - linux-mm@kvack.org - Jason Evans <je@fb.com> - Kirill A . Shutemov <kirill.shutemov@linux.intel.com> - Will Deacon <will.deacon@arm.com> - Catalin Marinas <catalin.marinas@arm.com> - linux-arch@vger.kernel.org - linux-arm-kernel@lists.infradead.org - [4.5+] <stable@vger.kernel.org> - " Andreas Schwab <schwab@suse.de>\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Thu, Dec 22, 2016 at 08:48:21AM +0900, Minchan Kim wrote:\n" @@ -20,7 +10,7 @@ "\n" "I guess you wanted put b8d3c4c3009d before [1], right?\n" "\n" - "> http://lkml.kernel.org/r/mvmmvfy37g1.fsf@hawking.suse.de\n" + "> http://lkml.kernel.org/r/mvmmvfy37g1.fsf at hawking.suse.de\n" "> \n" "> The problem is page fault handler supports only accessed flag emulation\n" "> for THP page of SW-dirty/accessed architecture.\n" @@ -34,8 +24,8 @@ "> Cc: Kirill A. Shutemov <kirill.shutemov@linux.intel.com>\n" "> Cc: Will Deacon <will.deacon@arm.com>\n" "> Cc: Catalin Marinas <catalin.marinas@arm.com>\n" - "> Cc: linux-arch@vger.kernel.org\n" - "> Cc: linux-arm-kernel@lists.infradead.org\n" + "> Cc: linux-arch at vger.kernel.org\n" + "> Cc: linux-arm-kernel at lists.infradead.org\n" "> Cc: <stable@vger.kernel.org> [4.5+]\n" "> Fixes: b8d3c4c3009d (\"mm/huge_memory.c: don't split THP page when MADV_FREE syscall is called\")\n" "> Reported-by: Andreas Schwab <schwab@suse.de>\n" @@ -91,7 +81,7 @@ "\n" "And I don't understand why do you need to change code in\n" "__handle_mm_fault() at all.\n" - "From what I see change to huge_pmd_set_accessed() should be enough.\n" + ">From what I see change to huge_pmd_set_accessed() should be enough.\n" "\n" "> +\t\t\t\t\t\tgoto pte_fault;\n" "> \t\t\t\t\treturn ret;\n" @@ -115,17 +105,11 @@ "> \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" + "> the body to majordomo at 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>\n" + "> Don't email: <a href=mailto:\"dont@kvack.org\"> email at kvack.org </a>\n" "\n" "-- \n" - " Kirill A. Shutemov\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>" + Kirill A. Shutemov -295bc25dd29cd98e7ed95f397c39793a726075822050d041061191c02e3c96a3 +b53dc32c86070c6ac04061b2f2486ae8c318423d6200d8fe6997fdfd79eeb7d0
diff --git a/a/1.txt b/N3/1.txt index 684a25c..3897e2e 100644 --- a/a/1.txt +++ b/N3/1.txt @@ -74,7 +74,7 @@ false-negative. Bit check was the right thing. And I don't understand why do you need to change code in __handle_mm_fault() at all. -From what I see change to huge_pmd_set_accessed() should be enough. +>From what I see change to huge_pmd_set_accessed() should be enough. > + goto pte_fault; > return ret; diff --git a/a/content_digest b/N3/content_digest index 3fa0eaf..6967347 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -91,7 +91,7 @@ "\n" "And I don't understand why do you need to change code in\n" "__handle_mm_fault() at all.\n" - "From what I see change to huge_pmd_set_accessed() should be enough.\n" + ">From what I see change to huge_pmd_set_accessed() should be enough.\n" "\n" "> +\t\t\t\t\t\tgoto pte_fault;\n" "> \t\t\t\t\treturn ret;\n" @@ -128,4 +128,4 @@ "see: http://www.linux-mm.org/ .\n" "Don't email: <a href=mailto:\"dont@kvack.org\"> email@kvack.org </a>" -295bc25dd29cd98e7ed95f397c39793a726075822050d041061191c02e3c96a3 +217f6671253424b6d2926ce7cdae2012e0d2c6b2b985778ffbb67a3385a7b4aa
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.