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

!6965 v2 memcg: fix incorrect value of sysctl_memcg_swap_qos_stat 

Merge Pull Request from: @ci-robot 
 
PR sync from: Liu Shixin <liushixin2@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/Z6NLXRZRRGXOA2CMEZ5CJ5YOGBF5LXKK/ 
 
https://gitee.com/openeuler/kernel/issues/I9NC0J 
 
Link:https://gitee.com/openeuler/kernel/pulls/6965

 

Reviewed-by: default avatarJialin Zhang <zhangjialin11@huawei.com>
Signed-off-by: default avatarJialin Zhang <zhangjialin11@huawei.com>
parents f689f421 54fc2dfe

mm/memcontrol.c

+20 −11
Original line number Diff line number Diff line
@@ -4095,22 +4095,21 @@ static int sysctl_memcg_swap_qos_handler(struct ctl_table *table, int write, 
			void __user *buffer, size_t *length, loff_t *ppos)

{

	int ret;

	int qos_stat_old = sysctl_memcg_swap_qos_stat;

	int qos_stat_old;

	int swap_type;

	static DEFINE_MUTEX(sysctl_mutex);



	mutex_lock(&sysctl_mutex);

	qos_stat_old = sysctl_memcg_swap_qos_stat;

	ret = proc_dointvec_minmax(table, write, buffer, length, ppos);

	if (ret)

		return ret;



	if (write && !ret) {

		if (qos_stat_old == sysctl_memcg_swap_qos_stat)

		return 0;



	if (write) {

			goto unlock;



		switch (sysctl_memcg_swap_qos_stat) {

		case MEMCG_SWAP_STAT_DISABLE:

			static_branch_disable(&memcg_swap_qos_key);

			return 0;

			goto unlock;

		case MEMCG_SWAP_STAT_ALL:

			swap_type = SWAP_TYPE_ALL;

			break;
@@ -4119,16 +4118,26 @@ static int sysctl_memcg_swap_qos_handler(struct ctl_table *table, int write, 
			break;

		}



		/*

		 * Enable the feature when it is in disabled state.

		 * If it is already in enabled state, don't allowed

		 * to switch it to other state directly since it is

		 * dangerous that will impact all memory cgroups.

		 */

		if (!qos_stat_old) {

			memcg_swap_qos_reset(swap_type);

			static_branch_enable(&memcg_swap_qos_key);

			enable_swap_slots_cache_max();

		} else {

			return -EINVAL;

			sysctl_memcg_swap_qos_stat = qos_stat_old;

			ret = -EINVAL;

		}

	}



	return 0;

unlock:

	mutex_unlock(&sysctl_mutex);



	return ret;

}



static struct ctl_table memcg_swap_qos_sysctls[] = {