Skip to contentSkip to conversation

ia64 kgdb

PATCH In review Robert Picco 2 participants 3 messages Jun 28, 2004

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ia64 kgdb

Hi Andrew:

This fixes the broken kgdb patch.

thanks,

Bob

drivers/firmware/pcdp.c +0 -0
diff -ruN -X /home/picco/losl/dontdiff linux-2.6.7-mm3-orig/drivers/firmware/pcdp.c linux-2.6.7-mm3-kgdb/drivers/firmware/pcdp.c
@@ -51,13 +51,20 @@

}

static void __init
+#ifndef CONFIG_KGDB_EARLY
setup_serial_console(int rev, struct pcdp_uart *uart) +#else
+setup_serial_console(int rev, struct pcdp_uart *uart, int line) +#endif
{

#ifdef CONFIG_SERIAL_8250_CONSOLE
	struct uart_port port;
	static char options[16];

	memset(&port, 0, sizeof(port));
+#ifdef	CONFIG_KGDB_EARLY
+	port.line = line;
+#endif
	port.uartclk = uart->clock_rate;
	if (!port.uartclk)	/* some FW doesn't supply this */
		port.uartclk = BASE_BAUD * 16;

@@ -99,6 +106,9 @@

	snprintf(options, sizeof(options), "%lun%d", uart->baud,
		uart->bits ? uart->bits : 8);
+#ifdef	CONFIG_KGDB_EARLY
+	if (!line)
+#endif
	add_preferred_console("ttyS", port.line, options);

	printk(KERN_INFO "PCDP: serial console at %s 0x%lx (ttyS%d, options %s)\n",
@@ -145,10 +155,19 @@
	for (i = 0, uart = pcdp->uart; i < pcdp->num_uarts; i++, uart++) {
		if (uart->flags & PCDP_UART_PRIMARY_CONSOLE || serial) {
			if (uart->type == PCDP_CONSOLE_UART) {
+#ifndef	CONFIG_KGDB_EARLY
				setup_serial_console(pcdp->rev, uart);
				return;

+#else
+ setup_serial_console(pcdp->rev, uart, 0);
+ serial = 0;
+#endif

			}
		}

+#ifdef CONFIG_KGDB_EARLY
+ else if (uart->type == PCDP_DEBUG_UART)
+ setup_serial_console(pcdp->rev, uart, 1);
+#endif

	}

	end = (struct pcdp_device *) ((u8 *) pcdp + pcdp->length);
#1 lore raw
Quoted 3 lines · Robert Picco

Hi Andrew:

This fixes the broken kgdb patch.

Hey Bob, thanks for the patch. Does the kgdb for ia64 require a special version of gdb or is the latest one from gnu.org sufficient? And does it work with netconsole?

Thanks,
Jesse

#2 lore raw
Quoted 5 lines · Jesse Barnes
Quoted 2 lines · Robert Picco

Hi Andrew:

Quoted 11 lines
Quoted 3 lines

This fixes the broken kgdb patch.

Hey Bob, thanks for the patch. Does the kgdb for ia64 require a special version of gdb or is the latest one from gnu.org sufficient? And does it work with netconsole?

Thanks,
Jesse

gdb-6.1 with patch at http://kernel.org/pub/linux/kernel/people/akpm/patches/gdb/gdb_patch_for_IA64_kgdb. It works with netconsole and serial.

Bob