[Skiboot] [PATCH v2] System reset IPI facility and Mambo implementation

[ppaidipe]

On 2017-02-08 14:04, Nicholas Piggin wrote:
> Add an opal call OPAL_SIGNAL_SYSTEM_RESET which allows system reset
> exceptions to be raised on other CPUs and act as an NMI IPI. There
> is an initial simple Mambo implementation, but allowances are made
> for a more complex hardware implementation.
> 
> This API is based on the POWER8 implementation from Alistair Popple.
> 
> Signed-off-by: Nicholas Piggin <npiggin at gmail.com>
> ---
> Changes since v1:
> 
> - Moved to .rst.
> 
> - Mambo broadcast change to signal our CPU last, to ensure all CPUs
>   get system reset interrupts independently.
> 
> - OPAL_PARTIAL error can be kept for platform specific restrictions
>   without putting any restrictions on implmentations. We may or may
>   not used it for the POWER8 implementation.
> 
>  doc/opal-api/opal-signal-system-reset-145.rst | 43 
> ++++++++++++++++++++++
>  include/opal-api.h                            |  3 +-
>  platforms/mambo/mambo.c                       | 52 
> +++++++++++++++++++++++++++
>  3 files changed, 97 insertions(+), 1 deletion(-)
>  create mode 100644 doc/opal-api/opal-signal-system-reset-145.rst
> 
> diff --git a/doc/opal-api/opal-signal-system-reset-145.rst
> b/doc/opal-api/opal-signal-system-reset-145.rst
> new file mode 100644
> index 00000000..e5b7177c
> --- /dev/null
> +++ b/doc/opal-api/opal-signal-system-reset-145.rst
> @@ -0,0 +1,42 @@
> +OPAL_SIGNAL_SYSTEM_RESET
> +========================
> +::
> +   int64_t signal_system_reset(int32_t cpu_nr);

I think it should be like below, as it implements only for mambo 
platform.
int64_t mambo_signal_system_reset(int32_t cpu_nr);

Thanks
Pridhiviraj


> +
> +This OPAL call causes the specified cpu(s) to be reset to the system
> +reset exception handler (0x100).
> +
> +The exact contents of system registers (e.g., SRR1 wakeup causes) may
> +vary depending on implementation and should not be relied upon.
> +
> +Resetting active threads on the same core as this call is run may
> +not be supported by some platforms. In that case, OPAL_PARTIAL will be
> +returned and NONE of the interrupts will be delivered.
> +
> +Arguments
> +---------
> +::
> +  int32_t cpu_nr
> +    cpu_nr >= 0        The cpu server number of the target cpu to 
> reset.
> +    SYS_RESET_ALL (-1) All cpus should be reset.
> +    SYS_RESET_ALL_OTHERS (-2) All but the current cpu should be reset.
> +
> +Returns
> +-------
> +OPAL_SUCCESS
> +  The power down was updated successful.
> +
> +OPAL_PARAMETER
> +  A parameter was incorrect.
> +
> +OPAL_HARDWARE
> +  Hardware indicated failure during reset.
> +
> +OPAL_PARTIAL
> +  Platform can not reset all requested CPUs at this time. This 
> requires
> +  platform-specific code to work around, otherwise to be treated as
> +  failure. No CPUs are reset.
> +
> +OPAL_UNSUPPORTED
> +  This processor/platform is not supported.
> +
> diff --git a/include/opal-api.h b/include/opal-api.h
> index c891db97..c32a2fde 100644
> --- a/include/opal-api.h
> +++ b/include/opal-api.h
> @@ -200,7 +200,8 @@
>  #define OPAL_XIVE_RESERVED2			142
>  #define OPAL_XIVE_RESERVED3			143
>  #define OPAL_XIVE_RESERVED4			144
> -#define OPAL_LAST				144
> +#define OPAL_SIGNAL_SYSTEM_RESET		145
> +#define OPAL_LAST				145
> 
>  /* Device tree flags */
> 
> diff --git a/platforms/mambo/mambo.c b/platforms/mambo/mambo.c
> index 976efeac..cb6e103c 100644
> --- a/platforms/mambo/mambo.c
> +++ b/platforms/mambo/mambo.c
> @@ -19,6 +19,7 @@
>  #include <device.h>
>  #include <console.h>
>  #include <chip.h>
> +#include <cpu.h>
>  #include <opal-api.h>
>  #include <opal-internal.h>
>  #include <time-utils.h>
> @@ -211,8 +212,59 @@ static void mambo_rtc_init(void)
>  	opal_register(OPAL_RTC_READ, mambo_rtc_read, 2);
>  }
> 
> +static void mambo_system_reset_cpu(struct cpu_thread *cpu)
> +{
> +	uint32_t core_id;
> +	uint32_t thread_id;
> +	char tcl_cmd[50];
> +
> +	core_id = pir_to_core_id(cpu->pir);
> +	thread_id = pir_to_thread_id(cpu->pir);
> +
> +	snprintf(tcl_cmd, sizeof(tcl_cmd), "mysim cpu %i:%i interrupt
> SystemReset", core_id, thread_id);
> +	callthru_tcl(tcl_cmd, strlen(tcl_cmd));
> +}
> +
> +#define SYS_RESET_ALL		-1
> +#define SYS_RESET_ALL_OTHERS	-2
> +
> +static int64_t mambo_signal_system_reset(int32_t cpu_nr)
> +{
> +	struct cpu_thread *cpu;
> +
> +	if (cpu_nr < 0) {
> +		if (cpu_nr < SYS_RESET_ALL_OTHERS)
> +			return OPAL_PARAMETER;
> +
> +		for_each_cpu(cpu) {
> +			if (cpu == this_cpu())
> +				continue;
> +			mambo_system_reset_cpu(cpu);
> +
> +		}
> +		if (cpu_nr == SYS_RESET_ALL)
> +			mambo_system_reset_cpu(this_cpu());
> +
> +		return OPAL_SUCCESS;
> +
> +	} else {
> +		cpu = find_cpu_by_server(cpu_nr);
> +		if (!cpu)
> +			return OPAL_PARAMETER;
> +
> +		mambo_system_reset_cpu(cpu);
> +		return OPAL_SUCCESS;
> +	}
> +}
> +
> +static void mambo_sreset_init(void)
> +{
> +	opal_register(OPAL_SIGNAL_SYSTEM_RESET, mambo_signal_system_reset, 
> 1);
> +}
> +
>  static void mambo_platform_init(void)
>  {
> +	mambo_sreset_init();
>  	mambo_rtc_init();
>  	bogus_disk_flash_init();
>  }