[LU-11919] cl_max_dirty_pages not zeroed on startup Created: 04/Feb/19  Updated: 08/Apr/19  Resolved: 19/Feb/19

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: None
Fix Version/s: Lustre 2.13.0, Lustre 2.12.1

Type: Bug Priority: Minor
Reporter: Patrick Farrell (Inactive) Assignee: Patrick Farrell (Inactive)
Resolution: Fixed Votes: 0
Labels: None

Issue Links:
Related
Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

cl_dirty_max_pages (the internal representation of max_dirty_mb) is not zeroed at initialization.  The function client_adjust_max_dirty is called, but it only sets it if the value is <= 0, which will be true most of the time for random memory...  But not all of the time.

So sometimes cl_max_dirty_pages ends up at some random positive value.  Easy fix.



 Comments   
Comment by Patrick Farrell (Inactive) [ 04/Feb/19 ]

This is the issue causing the odd max_dirty_mb values.  (The grant request issue is separate and should still be fixed with that patch.)

Comment by Gerrit Updater [ 04/Feb/19 ]

Patrick Farrell (pfarrell@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/34173
Subject: LU-11919 llite: Initialize cl_dirty_max_pages
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 6f7ceb99574ddf136f18ba77c3a3c41887464529

Comment by Gerrit Updater [ 18/Feb/19 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34173/
Subject: LU-11919 llite: Initialize cl_dirty_max_pages
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 2e9c896dec6d5c582c5dd24d3026e8324dfa807c

Comment by Peter Jones [ 19/Feb/19 ]

Landed for 2.13

Comment by Gerrit Updater [ 19/Mar/19 ]

Minh Diep (mdiep@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/34458
Subject: LU-11919 llite: Initialize cl_dirty_max_pages
Project: fs/lustre-release
Branch: b2_12
Current Patch Set: 1
Commit: f5c0d51c8e372de438a2c469e9cfbfc96d888ccd

Comment by Gerrit Updater [ 08/Apr/19 ]

Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/34458/
Subject: LU-11919 llite: Initialize cl_dirty_max_pages
Project: fs/lustre-release
Branch: b2_12
Current Patch Set:
Commit: 7c8d60106acd5a781660a05fef7612a7db5b243f

Generated at Sat Feb 10 02:48:04 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.