Commit bdda3b93 authored by Thomas Gleixner's avatar Thomas Gleixner
Browse files

x86/apic: Move no_ipi_broadcast() out of 32bit 



For the upcoming shorthand support for all APIC incarnations the command
line option needs to be available for 64 bit as well.

While at it, rename the control variable, make it static and mark it
__ro_after_init.

Signed-off-by: default avatarThomas Gleixner <tglx@linutronix.de>
Acked-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20190722105220.278327940@linutronix.de
parent bd82dba2

arch/x86/kernel/apic/ipi.c

+27 −2
Original line number Diff line number Diff line
@@ -5,6 +5,31 @@ 


#include "local.h"



#ifdef CONFIG_SMP

#ifdef CONFIG_HOTPLUG_CPU

#define DEFAULT_SEND_IPI	(1)

#else

#define DEFAULT_SEND_IPI	(0)

#endif



static int apic_ipi_shorthand_off __ro_after_init = DEFAULT_SEND_IPI;



static __init int apic_ipi_shorthand(char *str)

{

	get_option(&str, &apic_ipi_shorthand_off);

	return 1;

}

__setup("no_ipi_broadcast=", apic_ipi_shorthand);



static int __init print_ipi_mode(void)

{

	pr_info("IPI shorthand broadcast: %s\n",

		apic_ipi_shorthand_off ? "disabled" : "enabled");

	return 0;

}

late_initcall(print_ipi_mode);

#endif



static inline int __prepare_ICR2(unsigned int mask)

{

	return SET_APIC_DEST_FIELD(mask);
@@ -203,7 +228,7 @@ void default_send_IPI_allbutself(int vector) 
	if (num_online_cpus() < 2)

		return;



	if (no_broadcast || vector == NMI_VECTOR) {

	if (apic_ipi_shorthand_off || vector == NMI_VECTOR) {

		apic->send_IPI_mask_allbutself(cpu_online_mask, vector);

	} else {

		__default_send_IPI_shortcut(APIC_DEST_ALLBUT, vector);
@@ -212,7 +237,7 @@ void default_send_IPI_allbutself(int vector) 


void default_send_IPI_all(int vector)

{

	if (no_broadcast || vector == NMI_VECTOR) {

	if (apic_ipi_shorthand_off || vector == NMI_VECTOR) {

		apic->send_IPI_mask(cpu_online_mask, vector);

	} else {

		__default_send_IPI_shortcut(APIC_DEST_ALLINC, vector);

arch/x86/kernel/apic/local.h

+0 −2
Original line number Diff line number Diff line
@@ -51,8 +51,6 @@ void default_send_IPI_single_phys(int cpu, int vector); 
void default_send_IPI_mask_sequence_phys(const struct cpumask *mask, int vector);

void default_send_IPI_mask_allbutself_phys(const struct cpumask *mask, int vector);



extern int no_broadcast;



#ifdef CONFIG_X86_32

void default_send_IPI_mask_sequence_logical(const struct cpumask *mask, int vector);

void default_send_IPI_mask_allbutself_logical(const struct cpumask *mask, int vector);

arch/x86/kernel/apic/probe_32.c

+0 −25
Original line number Diff line number Diff line
@@ -15,31 +15,6 @@ 


#include "local.h"



#ifdef CONFIG_HOTPLUG_CPU

#define DEFAULT_SEND_IPI	(1)

#else

#define DEFAULT_SEND_IPI	(0)

#endif



int no_broadcast = DEFAULT_SEND_IPI;



static __init int no_ipi_broadcast(char *str)

{

	get_option(&str, &no_broadcast);

	pr_info("Using %s mode\n",

		no_broadcast ? "No IPI Broadcast" : "IPI Broadcast");

	return 1;

}

__setup("no_ipi_broadcast=", no_ipi_broadcast);



static int __init print_ipi_mode(void)

{

	pr_info("Using IPI %s mode\n",

		no_broadcast ? "No-Shortcut" : "Shortcut");

	return 0;

}

late_initcall(print_ipi_mode);



static int default_x86_32_early_logical_apicid(int cpu)

{

	return 1 << cpu;

CRA Git | Maintained and supported by SUSTech CRA and CCSE