[PATCH] percpu: Fix trivial typos in comments

[PATCH] percpu: Fix trivial typos in comments

[Yannick Guerrini]

Change 'iff' to 'if'
Change 'tranlated' to 'translated'
Change 'mutliples' to 'multiples'

Signed-off-by: Yannick Guerrini <yguerrini@tomshardware.fr>
---
 mm/percpu.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/mm/percpu.c b/mm/percpu.c
index 73c97a5..6e6dcdb 100644
--- a/mm/percpu.c
+++ b/mm/percpu.c
@@ -324,7 +324,7 @@ static void pcpu_mem_free(void *ptr, size_t size)
  *
  * Count the number of pages chunk's @i'th area occupies.  When the area's
  * start and/or end address isn't aligned to page boundary, the straddled
- * page is included in the count iff the rest of the page is free.
+ * page is included in the count if the rest of the page is free.
  */
 static int pcpu_count_occupied_pages(struct pcpu_chunk *chunk, int i)
 {
@@ -963,7 +963,7 @@ restart:
 
 	/*
 	 * No space left.  Create a new chunk.  We don't want multiple
-	 * tasks to create chunks simultaneously.  Serialize and create iff
+	 * tasks to create chunks simultaneously.  Serialize and create if
 	 * there's still no empty chunk after grabbing the mutex.
 	 */
 	if (is_atomic)
@@ -1310,7 +1310,7 @@ bool is_kernel_percpu_address(unsigned long addr)
  * and, from the second one, the backing allocator (currently either vm or
  * km) provides translation.
  *
- * The addr can be tranlated simply without checking if it falls into the
+ * The addr can be translated simply without checking if it falls into the
  * first chunk. But the current code reflects better how percpu allocator
  * actually works, and the verification can discover both bugs in percpu
  * allocator itself and per_cpu_ptr_to_phys() callers. So we keep current
@@ -1744,7 +1744,7 @@ early_param("percpu_alloc", percpu_alloc_setup);
 #define BUILD_EMBED_FIRST_CHUNK
 #endif
 
-/* build pcpu_page_first_chunk() iff needed by the arch config */
+/* build pcpu_page_first_chunk() if needed by the arch config */
 #if defined(CONFIG_NEED_PER_CPU_PAGE_FIRST_CHUNK)
 #define BUILD_PAGE_FIRST_CHUNK
 #endif
@@ -1762,7 +1762,7 @@ early_param("percpu_alloc", percpu_alloc_setup);
  * and other parameters considering needed percpu size, allocation
  * atom size and distances between CPUs.
  *
- * Groups are always mutliples of atom size and CPUs which are of
+ * Groups are always multiples of atom size and CPUs which are of
  * LOCAL_DISTANCE both ways are grouped together and share space for
  * units in the same group.  The returned configuration is guaranteed
  * to have CPUs on different nodes on different groups and >=75% usage
-- 
1.9.5.msysgit.0

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

Re: [PATCH] percpu: Fix trivial typos in comments

[Tejun Heo]

On Fri, Mar 06, 2015 at 10:52:28PM +0100, Yannick Guerrini wrote:
> Change 'iff' to 'if'

iff is intentional.

 http://en.wikipedia.org/wiki/If_and_only_if

> Change 'tranlated' to 'translated'
> Change 'mutliples' to 'multiples'

Can you refresh the patch with just the above two?

Thanks.

-- 
tejun

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

Re: [PATCH] percpu: Fix trivial typos in comments

[Yannick Guerrini]

Le 06/03/2015 23:02, Tejun Heo a ecrit :
> On Fri, Mar 06, 2015 at 10:52:28PM +0100, Yannick Guerrini wrote:
>> Change 'iff' to 'if'
> iff is intentional.
>
>   http://en.wikipedia.org/wiki/If_and_only_if
Oops, sorry ! I will check better next time !
>> Change 'tranlated' to 'translated'
>> Change 'mutliples' to 'multiples'
> Can you refresh the patch with just the above two?
>
> Thanks.
>
Patch v2 is coming...

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

[PATCHv2] percpu: Fix trivial typos in comments

[Yannick Guerrini]

Change 'tranlated' to 'translated'
Change 'mutliples' to 'multiples'

Signed-off-by: Yannick Guerrini <yguerrini@tomshardware.fr>
---
v2: don't replace 'iff' by 'if' !
    as suggested by Tejun Heo <tj@kernel.org>


 mm/percpu.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/mm/percpu.c b/mm/percpu.c
index 73c97a5..dfd0248 100644
--- a/mm/percpu.c
+++ b/mm/percpu.c
@@ -1310,7 +1310,7 @@ bool is_kernel_percpu_address(unsigned long addr)
  * and, from the second one, the backing allocator (currently either vm or
  * km) provides translation.
  *
- * The addr can be tranlated simply without checking if it falls into the
+ * The addr can be translated simply without checking if it falls into the
  * first chunk. But the current code reflects better how percpu allocator
  * actually works, and the verification can discover both bugs in percpu
  * allocator itself and per_cpu_ptr_to_phys() callers. So we keep current
@@ -1762,7 +1762,7 @@ early_param("percpu_alloc", percpu_alloc_setup);
  * and other parameters considering needed percpu size, allocation
  * atom size and distances between CPUs.
  *
- * Groups are always mutliples of atom size and CPUs which are of
+ * Groups are always multiples of atom size and CPUs which are of
  * LOCAL_DISTANCE both ways are grouped together and share space for
  * units in the same group.  The returned configuration is guaranteed
  * to have CPUs on different nodes on different groups and >=75% usage
-- 
1.9.5.msysgit.0


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

Re: [PATCHv2] percpu: Fix trivial typos in comments

[Tejun Heo]

On Fri, Mar 06, 2015 at 11:30:42PM +0100, Yannick Guerrini wrote:
> Change 'tranlated' to 'translated'
> Change 'mutliples' to 'multiples'
> 
> Signed-off-by: Yannick Guerrini <yguerrini@tomshardware.fr>

Applied to wq/for-4.1.

Thanks.

-- 
tejun

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