Unverified Commit e0313ab4 authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!15382 srcu: Tighten cleanup_srcu_struct() GP checks 

Merge Pull Request from: @ci-robot 
 
PR sync from: Yang Yingliang <yangyingliang@huawei.com>
https://mailweb.openeuler.org/archives/list/kernel@openeuler.org/message/L2GPSILUS7UAPZUWSKPV5UGMHTVRW7WB/ 
 
https://gitee.com/src-openeuler/kernel/issues/IBP415 
 
Link:https://gitee.com/openeuler/kernel/pulls/15382

 

Reviewed-by: default avatarWang ShaoBo <bobo.shaobowang@huawei.com>
Reviewed-by: default avatarXie XiuQi <xiexiuqi@huawei.com>
Reviewed-by: default avatarLi Nan <linan122@huawei.com>
Signed-off-by: default avatarLi Nan <linan122@huawei.com>
parents 4942ceab 82f3c251

kernel/rcu/srcutree.c

+4 −2
Original line number Diff line number Diff line
@@ -390,9 +390,11 @@ void cleanup_srcu_struct(struct srcu_struct *ssp) 
			return; /* Forgot srcu_barrier(), so just leak it! */

	}

	if (WARN_ON(rcu_seq_state(READ_ONCE(ssp->srcu_gp_seq)) != SRCU_STATE_IDLE) ||

	    WARN_ON(rcu_seq_current(&ssp->srcu_gp_seq) != ssp->srcu_gp_seq_needed) ||

	    WARN_ON(srcu_readers_active(ssp))) {

		pr_info("%s: Active srcu_struct %p state: %d\n",

			__func__, ssp, rcu_seq_state(READ_ONCE(ssp->srcu_gp_seq)));

		pr_info("%s: Active srcu_struct %p read state: %d gp state: %lu/%lu\n",

			__func__, ssp, rcu_seq_state(READ_ONCE(ssp->srcu_gp_seq)),

			rcu_seq_current(&ssp->srcu_gp_seq), ssp->srcu_gp_seq_needed);

		return; /* Caller forgot to stop doing call_srcu()? */

	}

	free_percpu(ssp->sda);