diff for duplicates of <AANLkTi=aWxekxZ-b59JNOjLArygcxOv1o4HwRt4t_xKh@mail.gmail.com>
diff --git a/a/1.txt b/N1/1.txt
index b7ea519..9d13d04 100644
--- a/a/1.txt
+++ b/N1/1.txt
@@ -1,12 +1,12 @@
On 15 November 2010 18:34, Russell King - ARM Linux
<linux@arm.linux.org.uk> wrote:
> On Fri, Nov 12, 2010 at 06:00:26PM +0000, Catalin Marinas wrote:
->> +#define PMD_TYPE_MASK (_AT(pmd_t, 3) << 0)
->> +#define PMD_TYPE_FAULT (_AT(pmd_t, 0) << 0)
->> +#define PMD_TYPE_TABLE (_AT(pmd_t, 3) << 0)
->> +#define PMD_TYPE_SECT (_AT(pmd_t, 1) << 0)
->> +#define PMD_BIT4 (_AT(pmd_t, 0))
->> +#define PMD_DOMAIN(x) (_AT(pmd_t, 0))
+>> +#define PMD_TYPE_MASK ? ? ? ? ? ? ? ?(_AT(pmd_t, 3) << 0)
+>> +#define PMD_TYPE_FAULT ? ? ? ? ? ? ? (_AT(pmd_t, 0) << 0)
+>> +#define PMD_TYPE_TABLE ? ? ? ? ? ? ? (_AT(pmd_t, 3) << 0)
+>> +#define PMD_TYPE_SECT ? ? ? ? ? ? ? ?(_AT(pmd_t, 1) << 0)
+>> +#define PMD_BIT4 ? ? ? ? ? ? (_AT(pmd_t, 0))
+>> +#define PMD_DOMAIN(x) ? ? ? ? ? ? ? ?(_AT(pmd_t, 0))
>
> It is really not correct to have these constants type'd as pmd_t.
> The idea behind pmd_t et.al. is to detect when normal arithmetic or
@@ -20,19 +20,19 @@ OK, I can add pmdval_t (and pgdval_t) and use it for these definitions.
>> --- a/arch/arm/mm/mm.h
>> +++ b/arch/arm/mm/mm.h
>> @@ -16,10 +16,10 @@ static inline pmd_t *pmd_off_k(unsigned long virt)
->> }
+>> ?}
>>
->> struct mem_type {
->> - unsigned int prot_pte;
->> - unsigned int prot_l1;
->> - unsigned int prot_sect;
->> - unsigned int domain;
->> + pgprot_t prot_pte;
->> + pgprot_t prot_l1;
->> + pgprot_t prot_sect;
->> + pgprot_t domain;
+>> ?struct mem_type {
+>> - ? ? unsigned int prot_pte;
+>> - ? ? unsigned int prot_l1;
+>> - ? ? unsigned int prot_sect;
+>> - ? ? unsigned int domain;
+>> + ? ? pgprot_t prot_pte;
+>> + ? ? pgprot_t prot_l1;
+>> + ? ? pgprot_t prot_sect;
+>> + ? ? pgprot_t domain;
>
-> Again, this is wrong. There's an accessor for pgprot_t typed data. This
+> Again, this is wrong. ?There's an accessor for pgprot_t typed data. ?This
> causes code to violate it.
OK, I'll define pgprotval_t and accessors.
diff --git a/a/content_digest b/N1/content_digest
index f62c186..18a728b 100644
--- a/a/content_digest
+++ b/N1/content_digest
@@ -8,20 +8,16 @@
"ref\00020101115183443.GE31421\@n2100.arm.linux.org.uk\0"
]
[
- "From\0Catalin Marinas <catalin.marinas\@arm.com>\0"
+ "From\0catalin.marinas\@arm.com (Catalin Marinas)\0"
]
[
- "Subject\0Re: [PATCH v2 06/20] ARM: LPAE: Introduce the 3-level page table format definitions\0"
+ "Subject\0[PATCH v2 06/20] ARM: LPAE: Introduce the 3-level page table format definitions\0"
]
[
"Date\0Tue, 16 Nov 2010 09:57:57 +0000\0"
]
[
- "To\0Russell King - ARM Linux <linux\@arm.linux.org.uk>\0"
-]
-[
- "Cc\0linux-arm-kernel\@lists.infradead.org",
- " linux-kernel\@vger.kernel.org\0"
+ "To\0linux-arm-kernel\@lists.infradead.org\0"
]
[
"\0000:1\0"
@@ -33,12 +29,12 @@
"On 15 November 2010 18:34, Russell King - ARM Linux\n",
"<linux\@arm.linux.org.uk> wrote:\n",
"> On Fri, Nov 12, 2010 at 06:00:26PM +0000, Catalin Marinas wrote:\n",
- ">> +#define PMD_TYPE_MASK \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240(_AT(pmd_t, 3) << 0)\n",
- ">> +#define PMD_TYPE_FAULT \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 (_AT(pmd_t, 0) << 0)\n",
- ">> +#define PMD_TYPE_TABLE \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 (_AT(pmd_t, 3) << 0)\n",
- ">> +#define PMD_TYPE_SECT \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240(_AT(pmd_t, 1) << 0)\n",
- ">> +#define PMD_BIT4 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 (_AT(pmd_t, 0))\n",
- ">> +#define PMD_DOMAIN(x) \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240 \302\240(_AT(pmd_t, 0))\n",
+ ">> +#define PMD_TYPE_MASK ? ? ? ? ? ? ? ?(_AT(pmd_t, 3) << 0)\n",
+ ">> +#define PMD_TYPE_FAULT ? ? ? ? ? ? ? (_AT(pmd_t, 0) << 0)\n",
+ ">> +#define PMD_TYPE_TABLE ? ? ? ? ? ? ? (_AT(pmd_t, 3) << 0)\n",
+ ">> +#define PMD_TYPE_SECT ? ? ? ? ? ? ? ?(_AT(pmd_t, 1) << 0)\n",
+ ">> +#define PMD_BIT4 ? ? ? ? ? ? (_AT(pmd_t, 0))\n",
+ ">> +#define PMD_DOMAIN(x) ? ? ? ? ? ? ? ?(_AT(pmd_t, 0))\n",
">\n",
"> It is really not correct to have these constants type'd as pmd_t.\n",
"> The idea behind pmd_t et.al. is to detect when normal arithmetic or\n",
@@ -52,19 +48,19 @@
">> --- a/arch/arm/mm/mm.h\n",
">> +++ b/arch/arm/mm/mm.h\n",
">> \@\@ -16,10 +16,10 \@\@ static inline pmd_t *pmd_off_k(unsigned long virt)\n",
- ">> \302\240}\n",
+ ">> ?}\n",
">>\n",
- ">> \302\240struct mem_type {\n",
- ">> - \302\240 \302\240 unsigned int prot_pte;\n",
- ">> - \302\240 \302\240 unsigned int prot_l1;\n",
- ">> - \302\240 \302\240 unsigned int prot_sect;\n",
- ">> - \302\240 \302\240 unsigned int domain;\n",
- ">> + \302\240 \302\240 pgprot_t prot_pte;\n",
- ">> + \302\240 \302\240 pgprot_t prot_l1;\n",
- ">> + \302\240 \302\240 pgprot_t prot_sect;\n",
- ">> + \302\240 \302\240 pgprot_t domain;\n",
+ ">> ?struct mem_type {\n",
+ ">> - ? ? unsigned int prot_pte;\n",
+ ">> - ? ? unsigned int prot_l1;\n",
+ ">> - ? ? unsigned int prot_sect;\n",
+ ">> - ? ? unsigned int domain;\n",
+ ">> + ? ? pgprot_t prot_pte;\n",
+ ">> + ? ? pgprot_t prot_l1;\n",
+ ">> + ? ? pgprot_t prot_sect;\n",
+ ">> + ? ? pgprot_t domain;\n",
">\n",
- "> Again, this is wrong. \302\240There's an accessor for pgprot_t typed data. \302\240This\n",
+ "> Again, this is wrong. ?There's an accessor for pgprot_t typed data. ?This\n",
"> causes code to violate it.\n",
"\n",
"OK, I'll define pgprotval_t and accessors.\n",
@@ -73,4 +69,4 @@
"Catalin"
]
-df683acf96d13fb4c490d887a203944cfda92a8ab8e374cd468022331a9b0f6f
+69e42ad0dcf3ea67f4a145960ca4b299517c25aa01a87d6a98e7a1d9f6bf3baa
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.