Skip to contentSkip to conversation

mm: Fix a comment typo

PATCH In review brookxu 2 participants 3 messages Mar 18, 2020

Subsystems (2)
lore
Share & tools
lore

brookxu

author
#0 lore raw

PATCH 1/1 mm: Fix a comment typo

---
 mm/page-writeback.c | 2 +-

 1 file changed, 1 insertion(+), 1 deletion(-)
mm/page-writeback.c +0 -0
index 2caf780..2acf754 100644
@@ -1271,7 +1271,7 @@ static void wb_update_dirty_ratelimit(struct dirty_throttle_control *dtc,

      */

     /*
-     * dirty_ratelimit will follow balanced_dirty_ratelimit iff +     * dirty_ratelimit will follow balanced_dirty_ratelimit if
      * task_ratelimit is on the same side of dirty_ratelimit, too.
      * For example, when
      * - dirty_ratelimit > balanced_dirty_ratelimit

Signature · 1 line
1.8.3.1
#1 lore raw
Quoted 11 lines · brookxu
---
 mm/page-writeback.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
mm/page-writeback.c +0 -0
index 2caf780..2acf754 100644

mm/page-writeback.c static void wb_update_dirty_ratelimit(struct · line 1275

Quoted 6 lines
mm/page-writeback.c +1 -1
@@ -1271,7 +1271,7 @@ static void wb_update_dirty_ratelimit(struct
 dirty_throttle_control *dtc,
      */
 
     /*
-     * dirty_ratelimit will follow balanced_dirty_ratelimit iff
+     * dirty_ratelimit will follow balanced_dirty_ratelimit if

Are you sure it's a mistake? iff means if and only if which would be reasonable unless the condition is only implies.

James

author
#2 lore raw

Thank you very much for your reply, it should be my misunderstanding, thanks again~

James Bottomley wrote on 2020/3/18 11:59:

Quoted 23 lines · brookxu
Quoted 11 lines
---
 mm/page-writeback.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
mm/page-writeback.c +0 -0
index 2caf780..2acf754 100644

mm/page-writeback.c static void wb_update_dirty_ratelimit(struct · line 1275

Quoted 6 lines
mm/page-writeback.c +1 -1
@@ -1271,7 +1271,7 @@ static void wb_update_dirty_ratelimit(struct
 dirty_throttle_control *dtc,
      */
 
     /*
-     * dirty_ratelimit will follow balanced_dirty_ratelimit iff
+     * dirty_ratelimit will follow balanced_dirty_ratelimit if

Are you sure it's a mistake? iff means if and only if which would be reasonable unless the condition is only implies.

James