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

!3426 Revert "hrtimers: Push pending hrtimers away from outgoing CPU earlier" 

Merge Pull Request from: @ci-robot 
 
PR sync from: Yu Liao <liaoyu15@huawei.com>
https://mailweb.openeuler.org/hyperkitty/list/kernel@openeuler.org/message/2ISP54RHZDV6NOZW443ML7V7URXPZBXD/ 
Yu Liao (2):
  Revert "cpu/hotplug: fix kabi breakage in enum cpuhp_state"
  Revert "hrtimers: Push pending hrtimers away from outgoing CPU
    earlier"


-- 
2.25.1
 
https://gitee.com/openeuler/kernel/issues/I8OSNK 
 
Link:https://gitee.com/openeuler/kernel/pulls/3426

 

Reviewed-by: default avatarWei Li <liwei391@huawei.com>
Signed-off-by: default avatarZhang Changzhong <zhangchangzhong@huawei.com>
parents 20197701 12e9b566

include/linux/hrtimer.h

+2 −2
Original line number Diff line number Diff line
@@ -508,9 +508,9 @@ extern void sysrq_timer_list_show(void); 


int hrtimers_prepare_cpu(unsigned int cpu);

#ifdef CONFIG_HOTPLUG_CPU

int hrtimers_cpu_dying(unsigned int cpu);

int hrtimers_dead_cpu(unsigned int cpu);

#else

static inline int hrtimers_cpu_dying(unsigned int cpu) { return 0; }

#define hrtimers_dead_cpu	NULL

#endif



#endif

include/linux/smp.h

+0 −1
Original line number Diff line number Diff line
@@ -220,6 +220,5 @@ int smp_call_on_cpu(unsigned int cpu, int (*func)(void *), void *par, 
int smpcfd_prepare_cpu(unsigned int cpu);

int smpcfd_dead_cpu(unsigned int cpu);

int smpcfd_dying_cpu(unsigned int cpu);

int smpcfd_and_hrtimer_dying_cpu(unsigned int cpu);



#endif /* __LINUX_SMP_H */

kernel/cpu.c

+2 −15
Original line number Diff line number Diff line
@@ -1390,7 +1390,7 @@ static struct cpuhp_step cpuhp_hp_states[] = { 
	[CPUHP_HRTIMERS_PREPARE] = {

		.name			= "hrtimers:prepare",

		.startup.single		= hrtimers_prepare_cpu,

		.teardown.single	= NULL,

		.teardown.single	= hrtimers_dead_cpu,

	},

	[CPUHP_SMPCFD_PREPARE] = {

		.name			= "smpcfd:prepare",
@@ -1452,24 +1452,11 @@ static struct cpuhp_step cpuhp_hp_states[] = { 
		.startup.single		= NULL,

		.teardown.single	= rcutree_dying_cpu,

	},

	/*

	 * In order to fix the kabi breakage, we had to move the hrtimers:dying

	 * step into smpcfd:dying and create a new function smpcfd_and_hrtimer_dying_cpu().

	 * Please ensure that there are no other steps with teardown handler

	 * between smpcfd:dying and cpu:teardown.

	 */

	[CPUHP_AP_SMPCFD_DYING] = {

		.name			= "smpcfd:dying",

		.startup.single		= NULL,

		.teardown.single	= smpcfd_and_hrtimer_dying_cpu,

		.teardown.single	= smpcfd_dying_cpu,

	},



	/*

	 * Attention: Please do not add steps between smpcfd:dying

	 * and ap:online. Please refer to the above for specific

	 * reasons.

	 */



	/* Entry state on starting. Interrupts enabled from here on. Transient

	 * state for synchronsization */

	[CPUHP_AP_ONLINE] = {

kernel/smp.c

+0 −8
Original line number Diff line number Diff line
@@ -71,14 +71,6 @@ int smpcfd_dead_cpu(unsigned int cpu) 
	return 0;

}



int smpcfd_and_hrtimer_dying_cpu(unsigned int cpu)

{

	hrtimers_cpu_dying(cpu);

	smpcfd_dying_cpu(cpu);



	return 0;

}



int smpcfd_dying_cpu(unsigned int cpu)

{

	/*

kernel/time/hrtimer.c

+21 −12
Original line number Diff line number Diff line
@@ -1922,22 +1922,29 @@ static void migrate_hrtimer_list(struct hrtimer_clock_base *old_base, 
	}

}



int hrtimers_cpu_dying(unsigned int dying_cpu)

int hrtimers_dead_cpu(unsigned int scpu)

{

	struct hrtimer_cpu_base *old_base, *new_base;

	int i, ncpu = cpumask_first(cpu_active_mask);



	tick_cancel_sched_timer(dying_cpu);

	int i;



	old_base = this_cpu_ptr(&hrtimer_bases);

	new_base = &per_cpu(hrtimer_bases, ncpu);

	BUG_ON(cpu_online(scpu));

	tick_cancel_sched_timer(scpu);



	/*

	 * this BH disable ensures that raise_softirq_irqoff() does

	 * not wakeup ksoftirqd (and acquire the pi-lock) while

	 * holding the cpu_base lock

	 */

	local_bh_disable();

	local_irq_disable();

	old_base = &per_cpu(hrtimer_bases, scpu);

	new_base = this_cpu_ptr(&hrtimer_bases);

	/*

	 * The caller is globally serialized and nobody else

	 * takes two locks at once, deadlock is not possible.

	 */

	raw_spin_lock(&old_base->lock);

	raw_spin_lock_nested(&new_base->lock, SINGLE_DEPTH_NESTING);

	raw_spin_lock(&new_base->lock);

	raw_spin_lock_nested(&old_base->lock, SINGLE_DEPTH_NESTING);



	for (i = 0; i < HRTIMER_MAX_CLOCK_BASES; i++) {

		migrate_hrtimer_list(&old_base->clock_base[i],
@@ -1948,13 +1955,15 @@ int hrtimers_cpu_dying(unsigned int dying_cpu) 
	 * The migration might have changed the first expiring softirq

	 * timer on this CPU. Update it.

	 */

	__hrtimer_get_next_event(new_base, HRTIMER_ACTIVE_SOFT);

	/* Tell the other CPU to retrigger the next event */

	smp_call_function_single(ncpu, retrigger_next_event, NULL, 0);

	hrtimer_update_softirq_timer(new_base, false);



	raw_spin_unlock(&new_base->lock);

	raw_spin_unlock(&old_base->lock);

	raw_spin_unlock(&new_base->lock);



	/* Check, if we got expired work to do */

	__hrtimer_peek_ahead_timers();

	local_irq_enable();

	local_bh_enable();

	return 0;

}