Skip to contentSkip to conversation

ARM: pgtable: Fix typo in the comment

PATCH In review Hyuk Myeong 2 participants 3 messages Jun 9, 2015 – Jun 10, 2015 Arm (32-bit)

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ARM: pgtable: Fix typo in the comment

This patch fix a spelling typo in the comment in pgtable-2level.h.

---
 arch/arm/include/asm/pgtable-2level.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
(quoted diff) +0 -0

b/arch/arm/include/asm/pgtable-2level.h
index bfd662e..49f91be 100644

arch/arm/include/asm/pgtable-2level.h +1 -1
@@ -66,7 +66,7 @@
  *
  * However, when the "young" bit is cleared, we deny access to the page
  * by clearing the hardware PTE.  Currently Linux does not flush the TLB
- * for us in this case, which means the TLB will retain the transation
+ * for us in this case, which means the TLB will retain the transaction
  * until either the TLB entry is evicted under pressure, or a context
  * switch which changes the user space mapping occurs.
  */

--
1.9.3

#1 lore raw
Quoted 2 lines · Hyuk Myeong

This patch fix a spelling typo in the comment in pgtable-2level.h.

Hi Hyuk,

Quoted 11 lines
---
 arch/arm/include/asm/pgtable-2level.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
arch/arm/include/asm/pgtable-2level.h +0 -0
index bfd662e..49f91be 100644

arch/arm/include/asm/pgtable-2level.h line 69

Quoted 5 lines
arch/arm/include/asm/pgtable-2level.h +1 -1
@@ -66,7 +66,7 @@
  *
  * However, when the "young" bit is cleared, we deny access to the page
  * by clearing the hardware PTE.  Currently Linux does not flush the TLB
- * for us in this case, which means the TLB will retain the transation
+ * for us in this case, which means the TLB will retain the transaction

Don't you mean "translation" rather than "transaction"?

Cheers,
--
Steve

Quoted 6 lines
arch/arm/include/asm/pgtable-2level.h +0 -1
  * until either the TLB entry is evicted under pressure, or a context
  * switch which changes the user space mapping occurs.
  */
--

1.9.3

#2 lore raw

Hello Steve,

Quoted 11 lines · Steve Capper
Quoted 2 lines · Hyuk Myeong

This patch fix a spelling typo in the comment in pgtable-2level.h.

Hi Hyuk,

Quoted 5 lines
---
 arch/arm/include/asm/pgtable-2level.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Quoted 14 lines

arch/arm/include/asm/pgtable-2level.h line 69

Quoted 11 lines
arch/arm/include/asm/pgtable-2level.h +1 -1
index bfd662e..49f91be 100644
@@ -66,7 +66,7 @@
  *
  * However, when the "young" bit is cleared, we deny access to the page
  * by clearing the hardware PTE.  Currently Linux does not flush the TLB
- * for us in this case, which means the TLB will retain the transation
+ * for us in this case, which means the TLB will retain the transaction

Don't you mean "translation" rather than "transaction"? Cheers,

Thank you for the advice
I had been confused between the two because I assumed that "transaction" is not that bad in this context (as a set of the translation)

Like you said, if "translation" is better here than I will send a new patch