From 975ec6f32a3173b0171178e62f1964d1907b49ee Mon Sep 17 00:00:00 2001 From: Michal Mielewczyk Date: Mon, 15 Jun 2020 10:26:26 -0400 Subject: [PATCH] Keep pointer to rollback thread in start_finalize If cache is initialized successfully, `struct _cache_mngt_attach_context` might be freed in `cache_mngt_init_instance()`. In such case pointer to rollback thread can't be accessed and thread can't be stopped. To prevent such scenario pointer should be kept in separate variable. Signed-off-by: Michal Mielewczyk --- modules/cas_cache/layer_cache_management.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/modules/cas_cache/layer_cache_management.c b/modules/cas_cache/layer_cache_management.c index 84d01b5..e3d31fc 100644 --- a/modules/cas_cache/layer_cache_management.c +++ b/modules/cas_cache/layer_cache_management.c @@ -1848,6 +1848,7 @@ static void cache_start_finalize(struct work_struct *work) { struct cache_priv *cache_priv = container_of(work, struct cache_priv, start_worker); + struct task_struct *rollback_thread; struct _cache_mngt_attach_context *ctx = cache_priv->attach_context; int result; ocf_cache_t cache = ctx->cache; @@ -1870,6 +1871,8 @@ static void cache_start_finalize(struct work_struct *work) init_instance_complete(ctx, cache); + rollback_thread = ctx->rollback_thread; + if (_cache_mngt_async_callee_set_result(&ctx->async, 0)) { /* caller interrupted */ ctx->ocf_start_error = 0; @@ -1878,7 +1881,7 @@ static void cache_start_finalize(struct work_struct *work) return; } - kthread_stop(ctx->rollback_thread); + kthread_stop(rollback_thread); ocf_mngt_cache_unlock(cache); }