linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] mm: remove obsolete comments about page table lock
@ 2013-09-04 16:50 Naoya Horiguchi
  2013-09-05  2:23 ` Wanpeng Li
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Naoya Horiguchi @ 2013-09-04 16:50 UTC (permalink / raw)
  To: Andrew Morton; +Cc: linux-mm, Aneesh Kumar K.V, linux-kernel

The callers of free_pgd_range() and hugetlb_free_pgd_range() don't hold
page table locks. The comments seems to be obsolete, so let's remove them.

Signed-off-by: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
---
 arch/powerpc/mm/hugetlbpage.c | 2 --
 mm/memory.c                   | 2 --
 2 files changed, 4 deletions(-)

diff --git a/arch/powerpc/mm/hugetlbpage.c b/arch/powerpc/mm/hugetlbpage.c
index 7e56cb7..31c20e2 100644
--- a/arch/powerpc/mm/hugetlbpage.c
+++ b/arch/powerpc/mm/hugetlbpage.c
@@ -635,8 +635,6 @@ static void hugetlb_free_pud_range(struct mmu_gather *tlb, pgd_t *pgd,
 
 /*
  * This function frees user-level page tables of a process.
- *
- * Must be called with pagetable lock held.
  */
 void hugetlb_free_pgd_range(struct mmu_gather *tlb,
 			    unsigned long addr, unsigned long end,
diff --git a/mm/memory.c b/mm/memory.c
index 6827a35..8c97ef0 100644
--- a/mm/memory.c
+++ b/mm/memory.c
@@ -478,8 +478,6 @@ static inline void free_pud_range(struct mmu_gather *tlb, pgd_t *pgd,
 
 /*
  * This function frees user-level page tables of a process.
- *
- * Must be called with pagetable lock held.
  */
 void free_pgd_range(struct mmu_gather *tlb,
 			unsigned long addr, unsigned long end,
-- 
1.8.3.1

--
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>

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] mm: remove obsolete comments about page table lock
  2013-09-04 16:50 [PATCH] mm: remove obsolete comments about page table lock Naoya Horiguchi
@ 2013-09-05  2:23 ` Wanpeng Li
  2013-09-05  2:23 ` Wanpeng Li
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Wanpeng Li @ 2013-09-05  2:23 UTC (permalink / raw)
  To: Naoya Horiguchi; +Cc: Andrew Morton, linux-mm, Aneesh Kumar K.V, linux-kernel

Cced David Gibson who add this comment on powerpc part.
On Wed, Sep 04, 2013 at 12:50:23PM -0400, Naoya Horiguchi wrote:
>The callers of free_pgd_range() and hugetlb_free_pgd_range() don't hold
>page table locks. The comments seems to be obsolete, so let's remove them.
>
>Signed-off-by: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
>---
> arch/powerpc/mm/hugetlbpage.c | 2 --
> mm/memory.c                   | 2 --
> 2 files changed, 4 deletions(-)
>
>diff --git a/arch/powerpc/mm/hugetlbpage.c b/arch/powerpc/mm/hugetlbpage.c
>index 7e56cb7..31c20e2 100644
>--- a/arch/powerpc/mm/hugetlbpage.c
>+++ b/arch/powerpc/mm/hugetlbpage.c
>@@ -635,8 +635,6 @@ static void hugetlb_free_pud_range(struct mmu_gather *tlb, pgd_t *pgd,
>
> /*
>  * This function frees user-level page tables of a process.
>- *
>- * Must be called with pagetable lock held.
>  */
> void hugetlb_free_pgd_range(struct mmu_gather *tlb,
> 			    unsigned long addr, unsigned long end,
>diff --git a/mm/memory.c b/mm/memory.c
>index 6827a35..8c97ef0 100644
>--- a/mm/memory.c
>+++ b/mm/memory.c
>@@ -478,8 +478,6 @@ static inline void free_pud_range(struct mmu_gather *tlb, pgd_t *pgd,
>
> /*
>  * This function frees user-level page tables of a process.
>- *
>- * Must be called with pagetable lock held.
>  */
> void free_pgd_range(struct mmu_gather *tlb,
> 			unsigned long addr, unsigned long end,
>-- 
>1.8.3.1
>
>--
>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>

--
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>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] mm: remove obsolete comments about page table lock
  2013-09-04 16:50 [PATCH] mm: remove obsolete comments about page table lock Naoya Horiguchi
  2013-09-05  2:23 ` Wanpeng Li
@ 2013-09-05  2:23 ` Wanpeng Li
  2013-09-05  2:27 ` Wanpeng Li
  2013-09-05  2:27 ` Wanpeng Li
  3 siblings, 0 replies; 5+ messages in thread
From: Wanpeng Li @ 2013-09-05  2:23 UTC (permalink / raw)
  To: Naoya Horiguchi; +Cc: Andrew Morton, linux-mm, Aneesh Kumar K.V, linux-kernel

Cced David Gibson who add this comment on powerpc part.
On Wed, Sep 04, 2013 at 12:50:23PM -0400, Naoya Horiguchi wrote:
>The callers of free_pgd_range() and hugetlb_free_pgd_range() don't hold
>page table locks. The comments seems to be obsolete, so let's remove them.
>
>Signed-off-by: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
>---
> arch/powerpc/mm/hugetlbpage.c | 2 --
> mm/memory.c                   | 2 --
> 2 files changed, 4 deletions(-)
>
>diff --git a/arch/powerpc/mm/hugetlbpage.c b/arch/powerpc/mm/hugetlbpage.c
>index 7e56cb7..31c20e2 100644
>--- a/arch/powerpc/mm/hugetlbpage.c
>+++ b/arch/powerpc/mm/hugetlbpage.c
>@@ -635,8 +635,6 @@ static void hugetlb_free_pud_range(struct mmu_gather *tlb, pgd_t *pgd,
>
> /*
>  * This function frees user-level page tables of a process.
>- *
>- * Must be called with pagetable lock held.
>  */
> void hugetlb_free_pgd_range(struct mmu_gather *tlb,
> 			    unsigned long addr, unsigned long end,
>diff --git a/mm/memory.c b/mm/memory.c
>index 6827a35..8c97ef0 100644
>--- a/mm/memory.c
>+++ b/mm/memory.c
>@@ -478,8 +478,6 @@ static inline void free_pud_range(struct mmu_gather *tlb, pgd_t *pgd,
>
> /*
>  * This function frees user-level page tables of a process.
>- *
>- * Must be called with pagetable lock held.
>  */
> void free_pgd_range(struct mmu_gather *tlb,
> 			unsigned long addr, unsigned long end,
>-- 
>1.8.3.1
>
>--
>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>

--
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>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] mm: remove obsolete comments about page table lock
  2013-09-04 16:50 [PATCH] mm: remove obsolete comments about page table lock Naoya Horiguchi
                   ` (2 preceding siblings ...)
  2013-09-05  2:27 ` Wanpeng Li
@ 2013-09-05  2:27 ` Wanpeng Li
  3 siblings, 0 replies; 5+ messages in thread
From: Wanpeng Li @ 2013-09-05  2:27 UTC (permalink / raw)
  To: Naoya Horiguchi
  Cc: David Gibson, Andrew Morton, linux-mm, Aneesh Kumar K.V,
	linux-kernel

Cced David Gibson who add this comment on powerpc part.
On Wed, Sep 04, 2013 at 12:50:23PM -0400, Naoya Horiguchi wrote:
>The callers of free_pgd_range() and hugetlb_free_pgd_range() don't hold
>page table locks. The comments seems to be obsolete, so let's remove them.
>
>Signed-off-by: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
>---
> arch/powerpc/mm/hugetlbpage.c | 2 --
> mm/memory.c                   | 2 --
> 2 files changed, 4 deletions(-)
>
>diff --git a/arch/powerpc/mm/hugetlbpage.c b/arch/powerpc/mm/hugetlbpage.c
>index 7e56cb7..31c20e2 100644
>--- a/arch/powerpc/mm/hugetlbpage.c
>+++ b/arch/powerpc/mm/hugetlbpage.c
>@@ -635,8 +635,6 @@ static void hugetlb_free_pud_range(struct mmu_gather *tlb, pgd_t *pgd,
>
> /*
>  * This function frees user-level page tables of a process.
>- *
>- * Must be called with pagetable lock held.
>  */
> void hugetlb_free_pgd_range(struct mmu_gather *tlb,
> 			    unsigned long addr, unsigned long end,
>diff --git a/mm/memory.c b/mm/memory.c
>index 6827a35..8c97ef0 100644
>--- a/mm/memory.c
>+++ b/mm/memory.c
>@@ -478,8 +478,6 @@ static inline void free_pud_range(struct mmu_gather *tlb, pgd_t *pgd,
>
> /*
>  * This function frees user-level page tables of a process.
>- *
>- * Must be called with pagetable lock held.
>  */
> void free_pgd_range(struct mmu_gather *tlb,
> 			unsigned long addr, unsigned long end,
>-- 
>1.8.3.1
>
>--
>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>

--
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>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] mm: remove obsolete comments about page table lock
  2013-09-04 16:50 [PATCH] mm: remove obsolete comments about page table lock Naoya Horiguchi
  2013-09-05  2:23 ` Wanpeng Li
  2013-09-05  2:23 ` Wanpeng Li
@ 2013-09-05  2:27 ` Wanpeng Li
  2013-09-05  2:27 ` Wanpeng Li
  3 siblings, 0 replies; 5+ messages in thread
From: Wanpeng Li @ 2013-09-05  2:27 UTC (permalink / raw)
  To: Naoya Horiguchi
  Cc: David Gibson, Andrew Morton, linux-mm, Aneesh Kumar K.V,
	linux-kernel

Cced David Gibson who add this comment on powerpc part.
On Wed, Sep 04, 2013 at 12:50:23PM -0400, Naoya Horiguchi wrote:
>The callers of free_pgd_range() and hugetlb_free_pgd_range() don't hold
>page table locks. The comments seems to be obsolete, so let's remove them.
>
>Signed-off-by: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
>---
> arch/powerpc/mm/hugetlbpage.c | 2 --
> mm/memory.c                   | 2 --
> 2 files changed, 4 deletions(-)
>
>diff --git a/arch/powerpc/mm/hugetlbpage.c b/arch/powerpc/mm/hugetlbpage.c
>index 7e56cb7..31c20e2 100644
>--- a/arch/powerpc/mm/hugetlbpage.c
>+++ b/arch/powerpc/mm/hugetlbpage.c
>@@ -635,8 +635,6 @@ static void hugetlb_free_pud_range(struct mmu_gather *tlb, pgd_t *pgd,
>
> /*
>  * This function frees user-level page tables of a process.
>- *
>- * Must be called with pagetable lock held.
>  */
> void hugetlb_free_pgd_range(struct mmu_gather *tlb,
> 			    unsigned long addr, unsigned long end,
>diff --git a/mm/memory.c b/mm/memory.c
>index 6827a35..8c97ef0 100644
>--- a/mm/memory.c
>+++ b/mm/memory.c
>@@ -478,8 +478,6 @@ static inline void free_pud_range(struct mmu_gather *tlb, pgd_t *pgd,
>
> /*
>  * This function frees user-level page tables of a process.
>- *
>- * Must be called with pagetable lock held.
>  */
> void free_pgd_range(struct mmu_gather *tlb,
> 			unsigned long addr, unsigned long end,
>-- 
>1.8.3.1
>
>--
>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>

--
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>

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2013-09-05  2:27 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-04 16:50 [PATCH] mm: remove obsolete comments about page table lock Naoya Horiguchi
2013-09-05  2:23 ` Wanpeng Li
2013-09-05  2:23 ` Wanpeng Li
2013-09-05  2:27 ` Wanpeng Li
2013-09-05  2:27 ` Wanpeng Li

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).