diff for duplicates of <20050104202104.GA28454@muc.de> diff --git a/a/1.txt b/N1/1.txt index 8e1977f..f28f84c 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -13,11 +13,11 @@ if you only tested page faults < 4GB. > > > > +#define pmd_test_and_populate(mm, pmd, pte) \ -> > > + (cmpxchg((int *)pmd, PMD_NONE, _PAGE_TABLE | __pa(pte)) = PMD_NONE) +> > > + (cmpxchg((int *)pmd, PMD_NONE, _PAGE_TABLE | __pa(pte)) == PMD_NONE) > > > +#define pud_test_and_populate(mm, pud, pmd) \ -> > > + (cmpxchg((int *)pgd, PUD_NONE, _PAGE_TABLE | __pa(pmd)) = PUD_NONE) +> > > + (cmpxchg((int *)pgd, PUD_NONE, _PAGE_TABLE | __pa(pmd)) == PUD_NONE) > > > +#define pgd_test_and_populate(mm, pgd, pud) \ -> > > + (cmpxchg((int *)pgd, PGD_NONE, _PAGE_TABLE | __pa(pud)) = PGD_NONE) +> > > + (cmpxchg((int *)pgd, PGD_NONE, _PAGE_TABLE | __pa(pud)) == PGD_NONE) > > > + > > > > Shouldn't this all be (long *)pmd ? page table entries on x86-64 are 64bit. diff --git a/a/content_digest b/N1/content_digest index 8e0bd05..a32dfbd 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -10,7 +10,8 @@ "ref\0Pine.LNX.4.58.0501041154560.1083@schroedinger.engr.sgi.com\0" "From\0Andi Kleen <ak@muc.de>\0" "Subject\0Re: page fault scalability patch V14 [5/7]: x86_64 atomic pte operations\0" - "Date\0Tue, 04 Jan 2005 20:21:04 +0000\0" + "Date\04 Jan 2005 21:21:04 +0100\0" + "Date\0Tue, 4 Jan 2005 21:21:04 +0100\0" "To\0Christoph Lameter <clameter@sgi.com>\0" "Cc\0Hugh Dickins <hugh@veritas.com>" akpm@osdl.org @@ -35,11 +36,11 @@ "\n" "> \n" "> > > +#define pmd_test_and_populate(mm, pmd, pte) \\\n" - "> > > +\t\t(cmpxchg((int *)pmd, PMD_NONE, _PAGE_TABLE | __pa(pte)) = PMD_NONE)\n" + "> > > +\t\t(cmpxchg((int *)pmd, PMD_NONE, _PAGE_TABLE | __pa(pte)) == PMD_NONE)\n" "> > > +#define pud_test_and_populate(mm, pud, pmd) \\\n" - "> > > +\t\t(cmpxchg((int *)pgd, PUD_NONE, _PAGE_TABLE | __pa(pmd)) = PUD_NONE)\n" + "> > > +\t\t(cmpxchg((int *)pgd, PUD_NONE, _PAGE_TABLE | __pa(pmd)) == PUD_NONE)\n" "> > > +#define pgd_test_and_populate(mm, pgd, pud) \\\n" - "> > > +\t\t(cmpxchg((int *)pgd, PGD_NONE, _PAGE_TABLE | __pa(pud)) = PGD_NONE)\n" + "> > > +\t\t(cmpxchg((int *)pgd, PGD_NONE, _PAGE_TABLE | __pa(pud)) == PGD_NONE)\n" "> > > +\n" "> >\n" "> > Shouldn't this all be (long *)pmd ? page table entries on x86-64 are 64bit.\n" @@ -54,4 +55,4 @@ "\n" -Andi -f5ce05ec589775b97345836e923eb3ef3385e9cc3fe8219548c1faa030c49a62 +d4197cd4b00b5d133fe43ff82e0f66762ba61f976bd6139d53d70523df720104
diff --git a/a/1.txt b/N2/1.txt index 8e1977f..4a1bba7 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -13,11 +13,11 @@ if you only tested page faults < 4GB. > > > > +#define pmd_test_and_populate(mm, pmd, pte) \ -> > > + (cmpxchg((int *)pmd, PMD_NONE, _PAGE_TABLE | __pa(pte)) = PMD_NONE) +> > > + (cmpxchg((int *)pmd, PMD_NONE, _PAGE_TABLE | __pa(pte)) == PMD_NONE) > > > +#define pud_test_and_populate(mm, pud, pmd) \ -> > > + (cmpxchg((int *)pgd, PUD_NONE, _PAGE_TABLE | __pa(pmd)) = PUD_NONE) +> > > + (cmpxchg((int *)pgd, PUD_NONE, _PAGE_TABLE | __pa(pmd)) == PUD_NONE) > > > +#define pgd_test_and_populate(mm, pgd, pud) \ -> > > + (cmpxchg((int *)pgd, PGD_NONE, _PAGE_TABLE | __pa(pud)) = PGD_NONE) +> > > + (cmpxchg((int *)pgd, PGD_NONE, _PAGE_TABLE | __pa(pud)) == PGD_NONE) > > > + > > > > Shouldn't this all be (long *)pmd ? page table entries on x86-64 are 64bit. @@ -31,3 +31,9 @@ if you only tested page faults < 4GB. It just checks sizeof, that should be fine. -Andi + +-- +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:"aart@kvack.org"> aart@kvack.org </a> diff --git a/a/content_digest b/N2/content_digest index 8e0bd05..dbcac7a 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -10,7 +10,8 @@ "ref\0Pine.LNX.4.58.0501041154560.1083@schroedinger.engr.sgi.com\0" "From\0Andi Kleen <ak@muc.de>\0" "Subject\0Re: page fault scalability patch V14 [5/7]: x86_64 atomic pte operations\0" - "Date\0Tue, 04 Jan 2005 20:21:04 +0000\0" + "Date\04 Jan 2005 21:21:04 +0100\0" + "Date\0Tue, 4 Jan 2005 21:21:04 +0100\0" "To\0Christoph Lameter <clameter@sgi.com>\0" "Cc\0Hugh Dickins <hugh@veritas.com>" akpm@osdl.org @@ -35,11 +36,11 @@ "\n" "> \n" "> > > +#define pmd_test_and_populate(mm, pmd, pte) \\\n" - "> > > +\t\t(cmpxchg((int *)pmd, PMD_NONE, _PAGE_TABLE | __pa(pte)) = PMD_NONE)\n" + "> > > +\t\t(cmpxchg((int *)pmd, PMD_NONE, _PAGE_TABLE | __pa(pte)) == PMD_NONE)\n" "> > > +#define pud_test_and_populate(mm, pud, pmd) \\\n" - "> > > +\t\t(cmpxchg((int *)pgd, PUD_NONE, _PAGE_TABLE | __pa(pmd)) = PUD_NONE)\n" + "> > > +\t\t(cmpxchg((int *)pgd, PUD_NONE, _PAGE_TABLE | __pa(pmd)) == PUD_NONE)\n" "> > > +#define pgd_test_and_populate(mm, pgd, pud) \\\n" - "> > > +\t\t(cmpxchg((int *)pgd, PGD_NONE, _PAGE_TABLE | __pa(pud)) = PGD_NONE)\n" + "> > > +\t\t(cmpxchg((int *)pgd, PGD_NONE, _PAGE_TABLE | __pa(pud)) == PGD_NONE)\n" "> > > +\n" "> >\n" "> > Shouldn't this all be (long *)pmd ? page table entries on x86-64 are 64bit.\n" @@ -52,6 +53,12 @@ "\n" "It just checks sizeof, that should be fine.\n" "\n" - -Andi + "-Andi\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:\"aart@kvack.org\"> aart@kvack.org </a>" -f5ce05ec589775b97345836e923eb3ef3385e9cc3fe8219548c1faa030c49a62 +5e0a946c57d2dcff88cf949bc3d6ff2ef1ceb4fcac906a6cf6579b0383357e9d
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.