linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/4] sparc64: Set cpu state to offline when stopped
@ 2016-11-10 22:43 Vijay Kumar
  2016-11-10 22:43 ` [PATCH 2/4] sparc64: Migrate hvcons irq to panicked cpu Vijay Kumar
                   ` (3 more replies)
  0 siblings, 4 replies; 5+ messages in thread
From: Vijay Kumar @ 2016-11-10 22:43 UTC (permalink / raw)
  To: davem, sparclinux; +Cc: karl.volz, rob.gardner, linux-kernel

CPU needs to be marked offline before stopping it. When not marked
offline, the xcall receives HV_EWOULDBLOCK and so assumes that not all
CPUs received the message, and retries. After 10000 retries, it finally
fails with fatal mondo timeout.

Signed-off-by: Vijay Kumar <vijay.ac.kumar@oracle.com>
---
 arch/sparc/kernel/smp_64.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/arch/sparc/kernel/smp_64.c b/arch/sparc/kernel/smp_64.c
index d3035ba..14138ad 100644
--- a/arch/sparc/kernel/smp_64.c
+++ b/arch/sparc/kernel/smp_64.c
@@ -1435,6 +1435,7 @@ void __irq_entry smp_receive_signal_client(int irq, struct pt_regs *regs)
 
 static void stop_this_cpu(void *dummy)
 {
+	set_cpu_online(smp_processor_id(), false);
 	prom_stopself();
 }
 
@@ -1446,6 +1447,8 @@ void smp_send_stop(void)
 		for_each_online_cpu(cpu) {
 			if (cpu == smp_processor_id())
 				continue;
+
+			set_cpu_online(cpu, false);
 #ifdef CONFIG_SUN_LDOMS
 			if (ldom_domaining_enabled) {
 				unsigned long hv_err;
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 2/4] sparc64: Migrate hvcons irq to panicked cpu
  2016-11-10 22:43 [PATCH 1/4] sparc64: Set cpu state to offline when stopped Vijay Kumar
@ 2016-11-10 22:43 ` Vijay Kumar
  2016-11-10 22:43 ` [PATCH 3/4] sparc64: Send break twice from console to return to boot prom Vijay Kumar
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 5+ messages in thread
From: Vijay Kumar @ 2016-11-10 22:43 UTC (permalink / raw)
  To: davem, sparclinux; +Cc: karl.volz, rob.gardner, linux-kernel

On panic, all other CPUs are stopped except the one which had
hit panic. To keep console alive, we need to migrate hvcons irq
to panicked CPU.

Signed-off-by: Vijay Kumar <vijay.ac.kumar@oracle.com>
---
 arch/sparc/include/asm/setup.h |    1 +
 arch/sparc/kernel/smp_64.c     |    6 +++++-
 drivers/tty/serial/sunhv.c     |    6 ++++++
 3 files changed, 12 insertions(+), 1 deletions(-)

diff --git a/arch/sparc/include/asm/setup.h b/arch/sparc/include/asm/setup.h
index 29d64b1..41691a5 100644
--- a/arch/sparc/include/asm/setup.h
+++ b/arch/sparc/include/asm/setup.h
@@ -61,6 +61,7 @@ extern atomic_t dcpage_flushes_xcall;
 extern int sysctl_tsb_ratio;
 #endif
 
+void sunhv_migrate_hvcons_irq(int cpu);
 void sun_do_break(void);
 extern int stop_a_enabled;
 extern int scons_pwroff;
diff --git a/arch/sparc/kernel/smp_64.c b/arch/sparc/kernel/smp_64.c
index 14138ad..52dc4b7 100644
--- a/arch/sparc/kernel/smp_64.c
+++ b/arch/sparc/kernel/smp_64.c
@@ -1444,8 +1444,12 @@ void smp_send_stop(void)
 	int cpu;
 
 	if (tlb_type == hypervisor) {
+		int this_cpu = smp_processor_id();
+
+		sunhv_migrate_hvcons_irq(this_cpu);
+
 		for_each_online_cpu(cpu) {
-			if (cpu == smp_processor_id())
+			if (cpu == this_cpu)
 				continue;
 
 			set_cpu_online(cpu, false);
diff --git a/drivers/tty/serial/sunhv.c b/drivers/tty/serial/sunhv.c
index 4e603d0..59828d8 100644
--- a/drivers/tty/serial/sunhv.c
+++ b/drivers/tty/serial/sunhv.c
@@ -398,6 +398,12 @@ static struct uart_driver sunhv_reg = {
 
 static struct uart_port *sunhv_port;
 
+void sunhv_migrate_hvcons_irq(int cpu)
+{
+	/* Migrate hvcons irq to param cpu */
+	irq_force_affinity(sunhv_port->irq, cpumask_of(cpu));
+}
+
 /* Copy 's' into the con_write_page, decoding "\n" into
  * "\r\n" along the way.  We have to return two lengths
  * because the caller needs to know how much to advance
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 3/4] sparc64: Send break twice from console to return to boot prom
  2016-11-10 22:43 [PATCH 1/4] sparc64: Set cpu state to offline when stopped Vijay Kumar
  2016-11-10 22:43 ` [PATCH 2/4] sparc64: Migrate hvcons irq to panicked cpu Vijay Kumar
@ 2016-11-10 22:43 ` Vijay Kumar
  2016-11-10 22:43 ` [PATCH 4/4] Documentation/sparc: Steps for sending break on sunhv console Vijay Kumar
  2016-11-10 23:27 ` [PATCH 1/4] sparc64: Set cpu state to offline when stopped David Miller
  3 siblings, 0 replies; 5+ messages in thread
From: Vijay Kumar @ 2016-11-10 22:43 UTC (permalink / raw)
  To: davem, sparclinux; +Cc: karl.volz, rob.gardner, linux-kernel

Now we can also jump to boot prom from sunhv console by sending
break twice on console for both running and panicked kernel
cases.

Signed-off-by: Vijay Kumar <vijay.ac.kumar@oracle.com>
---
 drivers/tty/serial/sunhv.c |    6 +++++-
 kernel/panic.c             |    3 ++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/drivers/tty/serial/sunhv.c b/drivers/tty/serial/sunhv.c
index 59828d8..33c35b4 100644
--- a/drivers/tty/serial/sunhv.c
+++ b/drivers/tty/serial/sunhv.c
@@ -116,7 +116,7 @@ static int receive_chars_getchar(struct uart_port *port)
 
 static int receive_chars_read(struct uart_port *port)
 {
-	int saw_console_brk = 0;
+	static int saw_console_brk;
 	int limit = 10000;
 
 	while (limit-- > 0) {
@@ -128,6 +128,9 @@ static int receive_chars_read(struct uart_port *port)
 			bytes_read = 0;
 
 			if (stat == CON_BREAK) {
+				if (saw_console_brk)
+					sun_do_break();
+
 				if (uart_handle_break(port))
 					continue;
 				saw_console_brk = 1;
@@ -151,6 +154,7 @@ static int receive_chars_read(struct uart_port *port)
 		if (port->sysrq != 0 &&  *con_read_page) {
 			for (i = 0; i < bytes_read; i++)
 				uart_handle_sysrq_char(port, con_read_page[i]);
+			saw_console_brk = 0;
 		}
 
 		if (port->state == NULL)
diff --git a/kernel/panic.c b/kernel/panic.c
index ca8cea1..4fe3b28 100644
--- a/kernel/panic.c
+++ b/kernel/panic.c
@@ -240,7 +240,8 @@ void panic(const char *fmt, ...)
 		extern int stop_a_enabled;
 		/* Make sure the user can actually press Stop-A (L1-A) */
 		stop_a_enabled = 1;
-		pr_emerg("Press Stop-A (L1-A) to return to the boot prom\n");
+		pr_emerg("Press Stop-A (L1-A) from sun keyboard or send break\n"
+			 "twice on console to return to the boot prom\n");
 	}
 #endif
 #if defined(CONFIG_S390)
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [PATCH 4/4] Documentation/sparc: Steps for sending break on sunhv console
  2016-11-10 22:43 [PATCH 1/4] sparc64: Set cpu state to offline when stopped Vijay Kumar
  2016-11-10 22:43 ` [PATCH 2/4] sparc64: Migrate hvcons irq to panicked cpu Vijay Kumar
  2016-11-10 22:43 ` [PATCH 3/4] sparc64: Send break twice from console to return to boot prom Vijay Kumar
@ 2016-11-10 22:43 ` Vijay Kumar
  2016-11-10 23:27 ` [PATCH 1/4] sparc64: Set cpu state to offline when stopped David Miller
  3 siblings, 0 replies; 5+ messages in thread
From: Vijay Kumar @ 2016-11-10 22:43 UTC (permalink / raw)
  To: davem, sparclinux; +Cc: karl.volz, rob.gardner, linux-kernel

Documented the steps for sending break on sunhv console.

Signed-off-by: Vijay Kumar <vijay.ac.kumar@oracle.com>
---
 Documentation/sparc/console.txt |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/Documentation/sparc/console.txt b/Documentation/sparc/console.txt
new file mode 100644
index 0000000..ab55353
--- /dev/null
+++ b/Documentation/sparc/console.txt
@@ -0,0 +1,9 @@
+Steps for sending 'break' on sunhv console:
+===========================================
+
+On Baremetal:
+   1. press   Esc + 'B'
+
+On LDOM:
+   1. press    Ctrl + ']'
+   2. telnet> send  break	
-- 
1.7.1

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH 1/4] sparc64: Set cpu state to offline when stopped
  2016-11-10 22:43 [PATCH 1/4] sparc64: Set cpu state to offline when stopped Vijay Kumar
                   ` (2 preceding siblings ...)
  2016-11-10 22:43 ` [PATCH 4/4] Documentation/sparc: Steps for sending break on sunhv console Vijay Kumar
@ 2016-11-10 23:27 ` David Miller
  3 siblings, 0 replies; 5+ messages in thread
From: David Miller @ 2016-11-10 23:27 UTC (permalink / raw)
  To: vijay.ac.kumar; +Cc: sparclinux, karl.volz, rob.gardner, linux-kernel


When submitting a series of patches, you must always provide
a cover posting.

This posting has Subject of the form:

	Subject: [PATCH 0/NN] Toplevel description.

And the body of the message explains what the patch series as
a whole does, why it is doing it, and how it is doing it.

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-11-10 23:27 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-10 22:43 [PATCH 1/4] sparc64: Set cpu state to offline when stopped Vijay Kumar
2016-11-10 22:43 ` [PATCH 2/4] sparc64: Migrate hvcons irq to panicked cpu Vijay Kumar
2016-11-10 22:43 ` [PATCH 3/4] sparc64: Send break twice from console to return to boot prom Vijay Kumar
2016-11-10 22:43 ` [PATCH 4/4] Documentation/sparc: Steps for sending break on sunhv console Vijay Kumar
2016-11-10 23:27 ` [PATCH 1/4] sparc64: Set cpu state to offline when stopped David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).