github/open-cas-linux
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

refactor: Optimize cache cleaning policy configuration flow

Browse Source 
Move the common cleaning policy setting outside the conditional block to ensure
it's always set first, regardless of the policy type. This change maintains
the specific parameter settings for 'acp' and 'alru' policies while improving
code structure and execution flow.
This commit is contained in:
td-zhangshun 2025-04-24 10:25:39 +08:00 committed by Netlops
parent 02a209aff8
commit e20cf4f2fe
1 changed files with 3 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
utils/opencas.py
View File

@ -635,15 +635,13 @@ def start_cache(cache, load, force=False):
def configure_cache(cache): def configure_cache(cache):
if "cleaning_policy" in cache.params: if "cleaning_policy" in cache.params:
casadm.set_param(
"cleaning", cache_id=cache.cache_id, policy=cache.params["cleaning_policy"]
)
if cache.params["cleaning_policy"] == "acp" or cache.params["cleaning_policy"] == "alru": if cache.params["cleaning_policy"] == "acp" or cache.params["cleaning_policy"] == "alru":
casadm.set_param_cleaning_policy( casadm.set_param_cleaning_policy(
policy=cache.params["cleaning_policy"], cache_id=cache.cache_id, wake_up=cache.params["wake_up"], flush_max_buffers=cache.params["flush_max_buffers"] policy=cache.params["cleaning_policy"], cache_id=cache.cache_id, wake_up=cache.params["wake_up"], flush_max_buffers=cache.params["flush_max_buffers"]
) )
else:
casadm.set_param(
"cleaning", cache_id=cache.cache_id, policy=cache.params["cleaning_policy"]
)
if "promotion_policy" in cache.params: if "promotion_policy" in cache.params:
casadm.set_param( casadm.set_param(