IDE Interface für Kernel 2.6

[jojo]

Ich denke, daß ich nun das Problem gefunden habe.
Der Treiber hatte an einigen Stellen "inb/outb" anstelle von dboxide_inb/dboxide_outb verwendet.
Dies führt beim "inb" dazu, das nur 0xff gelesen wird und falsche Informationen im taskfile stehen.

Hier mein Auszug aus dem Bootlog:

• [ 21.981419] Uniform Multi-Platform E-IDE driver
  [ 22.057366] ide: Assuming 50MHz system bus speed for PIO modes; override with idebus=xx
  [ 22.493234] dboxide: $Id: main.c,v 1.4.2.6 2007/10/09 01:04:06 carjay Exp $
  [ 22.573070] dboxide: activating cs2
  [ 22.614482] dboxide: address space of DBox2 IDE CPLD is at: 0xc3050000
  [ 22.692674] dboxide: IDE-Interface detected, e000e000, Vendor: Unknown
  [ 23.420753] dboxide: sreset succeeded
  [ 23.800867] hda: ST9160821AS, ATA DISK drive
  [ 24.585819] DBox2 at 0x010-0x017,0x04e on irq 47
  [ 25.032328] hda: max request size: 512KiB
  [ 27.049769] hda: 312581808 sectors (160041 MB) w/8192KiB Cache, CHS=19457/255/63
  [ 27.135234] hda: cache flushes supported
  [ 27.183290] hda: hda1 hda2
  [ 30.621964] SGI XFS with security attributes, no debug enabled
  mount: using /etc/fstab
  mount: mounting /dev/hda1 on none failed: No such file or directory
  [ 30.958419] XFS mounting filesystem hda2
  [ 32.393819] Starting XFS recovery on filesystem: hda2 (logdev: internal)
  [ 32.743008] Ending XFS recovery on filesystem: hda2 (logdev: internal)
  Detected STB:
  Vendor: Nokia
  Model: D-BOX2

Um den dboxide Trace zu verwenden, muß der Watchdog-Reset exportiert werden. (Warum wird eigentlich der m8xx_wdt anstatt des m83x_wdt verwendet? Der Watchdog läßt sich definitiv nicht enablen/disablen [siehe auch Kommentar im m83xx_wdt])

Code: Alles auswählen

--- ./linux-2.6.26.4/drivers/watchdog/mpc8xx_wdt.c.orig	2009-02-02 20:59:22.332917521 +0100
+++ ./linux-2.6.26.4/drivers/watchdog/mpc8xx_wdt.c	2009-02-02 20:59:35.644015860 +0100
@@ -23,6 +23,11 @@
 static unsigned long wdt_opened;
 static int wdt_status;
 
+void mpc8xx_wdt_reset(void)
+{
+	m8xx_wdt_reset();
+}
+
 static void mpc8xx_wdt_handler_disable(void)
 {
 	volatile uint __iomem *piscr;
@@ -152,8 +157,7 @@
 	return misc_register(&mpc8xx_wdt_miscdev);
 }
 
-static void __exit 
-exit(void)
+static void __exit mpc8xx_wdt_exit(void)
 {
 	misc_deregister(&mpc8xx_wdt_miscdev);
 
@@ -168,3 +172,5 @@
 MODULE_DESCRIPTION("MPC8xx watchdog driver");
 MODULE_LICENSE("GPL");
 MODULE_ALIAS_MISCDEV(WATCHDOG_MINOR);
+
+EXPORT_SYMBOL(mpc8xx_wdt_reset);

Hier nun die Änderungen des Trace-Modules. Ich habe ein Conditional eingebaut, so daß bei nicht definiertem "TRACE_ENABLE" nur leere Funktionsaufrufe generiert werden. Spart ein paar Byte und einige Mikrosekunden...

Code: Alles auswählen

Index: driver/ide/trace.c
===================================================================
RCS file: /cvs/tuxbox/driver/ide/trace.c,v
retrieving revision 1.1.2.3
diff -u -r1.1.2.3 trace.c
--- a/driver/ide/trace.c	11 Sep 2006 17:34:10 -0000	1.1.2.3
+++ b/driver/ide/trace.c	2 Feb 2009 19:43:56 -0000
@@ -2,6 +2,7 @@
 #include <linux/version.h>
 #include <asm/time.h>
 #include <asm/uaccess.h>
+#include <syslib/m8xx_wdt.h>
 
 #define TRACE_WITH_CALLSTACK 1
 
@@ -11,11 +12,15 @@
 #define STACK_DEPTH 4
 #endif
 
-#if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0)
-extern void m8xx_wdt_reset(void);
+/* enabel debug trace if TRACE_ENABLE is defined */
+#undef TRACE_ENABLE
+
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,0)
+extern void mpc8xx_wdt_reset(void);
 #endif
 
 extern char *dboxide_trace_msg[];
+extern void mpc8xx_wdt_reset();
 
 typedef struct IDETraceData {
 	unsigned long time;
@@ -29,11 +34,14 @@
 
 } IDETraceData;
 
+#ifdef TRACE_ENABLE
 static int traceIdx;
 static IDETraceData trace[TRACE_DEPTH];
+#endif
 
 static void log_backtrace(IDETraceData * td)
 {
+#ifdef TRACE_ENABLE
 #ifdef TRACE_WITH_CALLSTACK
 	int cnt = 0;
 	unsigned long i;
@@ -55,10 +63,12 @@
 		sp = *(unsigned long **)sp;
 	}
 #endif
+#endif
 }
 
 static void print_callstack(IDETraceData * t)
 {
+#ifdef TRACE_ENABLE
 #ifdef TRACE_WITH_CALLSTACK
 	int j;
 	for (j = 0; j < STACK_DEPTH; j++) {
@@ -71,10 +81,12 @@
 	}
 	printk("\n");
 #endif
+#endif
 }
 
 void dboxide_log_trace(unsigned int typ, unsigned int a, unsigned int b)
 {
+#ifdef TRACE_ENABLE
 	IDETraceData *t = &trace[traceIdx];
 	traceIdx = (traceIdx + 1) & ((TRACE_DEPTH) - 1);
 
@@ -84,13 +96,16 @@
 	t->b = b;
 
 	log_backtrace(t);
+#endif
 }
 
 void dboxide_print_trace(void)
 {
+#ifdef TRACE_ENABLE
 	int i;
 	IDETraceData *t;
 	printk("dboxide: trace\n");
+
 	for (i = TRACE_DEPTH - 1; i > 0; i--) {
 		int idx = (traceIdx + i) & ((TRACE_DEPTH) - 1);
 		t = &trace[idx];
@@ -104,14 +119,16 @@
 		print_callstack(t);
 
 #if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0)
-		/* this can take a while and when not printing to console the watchdog is not served 
-           FIXME: symbol is not (yet) exposed in 2.6 */
+		/* this can take a while and when not printing to console the watchdog is not served */
 		if ((i & 7) == 0)
 			m8xx_wdt_reset();
+#else
+		/* requires modified linux/driver/watchdog/mpc8xx_wdt.c */
+		if ((i & 7) == 0)
+			mpc8xx_wdt_reset();
 #endif
-
 		t->typ = 0;
 	}
-
 	printk("trace end.\n");
+#endif
 }

Und hier ist der eigentliche Patch für den IDE Treiber. Habe die letzte Version des 2.4er Kernel gekapert und mit dem 2.6er aus dem CVS gemischt. Zusätzlich ein paar Byte wegoptimiert.

Code: Alles auswählen

Index: driver/ide/main.c
===================================================================
RCS file: /cvs/tuxbox/driver/ide/main.c,v
retrieving revision 1.4.2.6
diff -u -r1.4.2.6 main.c
--- a/driver/ide/main.c	9 Oct 2007 01:04:06 -0000	1.4.2.6
+++ b/driver/ide/main.c	2 Feb 2009 19:44:19 -0000
@@ -29,11 +29,19 @@
 #include <asm/errno.h>
 #include <asm/irq.h>
 #include <asm/8xx_immap.h>
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,25)
+#include <asm/cpm1.h>
+#else
 #include <asm/commproc.h>
+#endif
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(2,5,0)
 #include <linux/device.h>
 #include <linux/platform_device.h>
 #endif
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,25)
+#include <linux/delay.h>
+#endif
+#include <syslib/m8xx_wdt.h>
 
 static uint idebase = 0;
 #if LINUX_VERSION_CODE >= KERNEL_VERSION(2,5,0)
@@ -41,6 +49,7 @@
 #else
 static int ideindex = -1;
 #endif
+static int irq6 = 0;
 
 /* address-offsets of features in the CPLD 
   (we can't call them registers...) */
@@ -76,6 +85,9 @@
 extern void dboxide_outsw_loop(uint ctrl_address,
 			       uint data_address, void *src, int count);
 
+/* enable/disable low level traces */
+#undef TRACE_DBOXIDE
+
 /* trace routines */
 extern void dboxide_log_trace(unsigned int typ, unsigned int a, unsigned int b);
 extern void dboxide_print_trace(void);
@@ -87,6 +99,17 @@
 #define TRACE_LOG_OUTW  0x05
 #define TRACE_LOG_OUTSW 0x06
 
+/* replace fixed printk strings with char * */
+const char *modname = "dboxide:";
+#define DBOXIDE_PRINTK(txt, a...) printk( "%s "txt, modname, ##a)
+
+/* knwon DBox 2 IDE vendors  */
+const char *dboxide_vendors[] = {
+	"Unknown",
+	"Gurgel",
+	"DboxBaer or kpt.ahab/Stingray"
+}; 
+
 const char *dboxide_trace_msg[] = {
 	NULL,
 	"INB  ",
@@ -99,13 +122,13 @@
 
 /* some functions are not implemented and I don't expect we ever
    need them. But if one of them is called, we can work on that. */
-#define NOT_IMPL(txt, a...) printk( "dboxide: NOT IMPLEMENTED: "txt, ##a )
+#define NOT_IMPL(txt, a...) printk( "%s NOT IMPLEMENTED: "txt, modname, ##a )
 
 /* whenever a something didn't work as expected: print everything
    that might be interesting for the developers what has happened */
 void dboxide_problem(const char *msg)
 {
-	printk("dboxide: %s\n", msg);
+	printk("%s %s\n", modname, msg);
 	printk("CPLD Status is %08x\n", CPLD_IN(CPLD_READ_CTRL));
 	dboxide_print_trace();
 }
@@ -160,9 +183,9 @@
 
 	val >>= 8;
 	val &= 0xFF;
-
+#ifdef TRACE_DBOXIDE
 	dboxide_log_trace(TRACE_LOG_INB, port, val);
-
+#endif
 	return val;
 }
 
@@ -185,9 +208,9 @@
 	val = CPLD_IN(CPLD_READ_FIFO);
 
 	val &= 0xFFFF;
-
+#ifdef TRACE_DBOXIDE
 	dboxide_log_trace(TRACE_LOG_INW, port, val);
-
+#endif
 	return val;
 }
 
@@ -197,12 +220,14 @@
 static void dboxide_insw(unsigned long port, void *addr, u32 count)
 {
 	uint *dest = addr;
+	register uint a;
+	register uint b;
 
 	/* special for ATAPI: the kernel calls insw with count=1?! */
 	if (count<4)
 	{
 		short * sdest = addr;
-		/*printk("dboxide: short insw %d\n", (int)count);*/
+		/*printk("%s short insw %d\n", modname, (int)count);*/
 		for (;count>0;count--)
 		{
 			*sdest++ = dboxide_inw( port ); 
@@ -211,49 +236,51 @@
 	}
 
 
+	/* outcommented for better performance, errorchecking is not needed
+
 	if (CPLD_FIFO_LEVEL() != 0)
 		dboxide_problem("insw: fifo not empty?!");
+	*/
+#ifdef TRACE_DBOXIDE
 	dboxide_log_trace(TRACE_LOG_INSW, port, count);
+#endif
 
 	/* activate reading to fifo with auto repeat */
 	CPLD_OUT(CPLD_WRITE_CTRL, port);
 	CPLD_OUT(CPLD_WRITE_CTRL, port | CPLD_CTRL_ENABLE | CPLD_CTRL_REPEAT);
 
 	/* todo: replace the code below by an assembler implementation in this
-	   routine */
+	   routine 
+
 	dboxide_insw_loop(idebase + CPLD_READ_CTRL, idebase + CPLD_READ_FIFO,
 			  dest, count);
+	*/
 
 	{
-		register uint a;
-		register uint b;
-		register uint c;
-		register uint d;
 
 		while (count > 16) {
-			while (CPLD_FIFO_LEVEL() != 0xF) {
-			};
+			while (CPLD_FIFO_LEVEL() != 0xF) {};
 			a = CPLD_IN(CPLD_READ_FIFO);	/* read 2 16 bit words */
 			b = CPLD_IN(CPLD_READ_FIFO);	/* read 2 16 bit words */
-			while (CPLD_FIFO_LEVEL() != 0xF) {
-			};
-			c = CPLD_IN(CPLD_READ_FIFO);	/* read 2 16 bit words */
-			d = CPLD_IN(CPLD_READ_FIFO);	/* read 2 16 bit words */
 			dest[0] = a;
 			dest[1] = b;
-			dest[2] = c;
-			dest[3] = d;
+			while (CPLD_FIFO_LEVEL() != 0xF) {};
+			a = CPLD_IN(CPLD_READ_FIFO);	/* read 2 16 bit words */
+			b = CPLD_IN(CPLD_READ_FIFO);	/* read 2 16 bit words */
+			dest[2] = a;
+			dest[3] = b;
+
 			count -= 8;
 			dest += 4;
 		}
 
 		while (count > 4) {
-			while (CPLD_FIFO_LEVEL() != 0xF) {
-			};
+			while (CPLD_FIFO_LEVEL() != 0xF) {};
 			a = CPLD_IN(CPLD_READ_FIFO);	/* read 2 16 bit words */
 			b = CPLD_IN(CPLD_READ_FIFO);	/* read 2 16 bit words */
 			dest[0] = a;
 			dest[1] = b;
+
 			count -= 4;
 			dest += 2;
 		}
@@ -261,9 +288,7 @@
 	}
 
 	if (count != 4)
-		printk
-		    ("dboxide: oops: insw: something has gone wrong: count is %d\n",
-		     count);
+		printk ("%s oops: insw: something has gone wrong: count is %d\n", modname, count);
 
 	/* wait until fifo is full = 4 Words */
 	while (CPLD_FIFO_LEVEL() != 0xF) {
@@ -273,8 +298,10 @@
 	CPLD_OUT(CPLD_WRITE_CTRL, port);
 
 	/* and read the final 4 16 bit words */
-	dest[0] = CPLD_IN(CPLD_READ_FIFO);
-	dest[1] = CPLD_IN(CPLD_READ_FIFO);
+	a = CPLD_IN(CPLD_READ_FIFO);	/* read 2 16 bit words */
+	b = CPLD_IN(CPLD_READ_FIFO);	/* read 2 16 bit words */
+	dest[0] = a;
+	dest[1] = b;
 }
 
 /* insl reads several 32 bit words from an IDE register.
@@ -302,9 +329,9 @@
 {
 	if (CPLD_FIFO_LEVEL() != 0)
 		dboxide_problem("outb: fifo not empty?!");
-
+#ifdef TRACE_DBOXIDE
 	dboxide_log_trace(TRACE_LOG_OUTB, port, value);
-
+#endif
 	CPLD_OUT(CPLD_WRITE_CTRL, port | CPLD_CTRL_ENABLE | CPLD_CTRL_WRITING);
 	CPLD_OUT(CPLD_WRITE_FIFO_LOW, value << 8);
 
@@ -345,24 +372,28 @@
 {
 	uint *src = addr;
 
+/*
 	if (CPLD_FIFO_LEVEL() != 0)
 		dboxide_problem("outsw: fifo not empty?!");
-
+*/
+#ifdef TRACE_DBOXIDE
 	dboxide_log_trace(TRACE_LOG_OUTSW, port, count);
-
+#endif
 	/* activate writing to fifo with auto repeat */
 	CPLD_OUT(CPLD_WRITE_CTRL,
 		 port | CPLD_CTRL_WRITING | CPLD_CTRL_ENABLE |
 		 CPLD_CTRL_REPEAT);
 
+/*
 	dboxide_outsw_loop(idebase + CPLD_READ_CTRL, idebase + CPLD_READ_FIFO,
 			   src, count);
+*/
 
 	{
 		register int a;
 		register int b;
 
-		while (count > 2) {
+		while (count > 0) {
 			a = *src++;
 			b = *src++;
 			while (CPLD_FIFO_LEVEL() != 0) {
@@ -396,13 +427,151 @@
 	dboxide_outsw(port, addr, count * 2);
 }
 
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,25)
+static void dboxide_tf_load(ide_drive_t *drive, ide_task_t *task)
+{
+ide_hwif_t *hwif = drive->hwif;
+	struct ide_io_ports *io_ports = &hwif->io_ports;
+	struct ide_taskfile *tf = &task->tf;
+	u8 HIHI = (task->tf_flags & IDE_TFLAG_LBA48) ? 0xE0 : 0xEF;
+
+	if (task->tf_flags & IDE_TFLAG_FLAGGED)
+		HIHI = 0xFF;
+
+	//ide_set_irq(drive, 1);
+
+	if (task->tf_flags & IDE_TFLAG_OUT_DATA)
+		dboxide_outw((tf->hob_data << 8) | tf->data, io_ports->data_addr);
+	if (task->tf_flags & IDE_TFLAG_OUT_HOB_FEATURE)
+		dboxide_outb(tf->hob_feature, io_ports->feature_addr);
+	if (task->tf_flags & IDE_TFLAG_OUT_HOB_NSECT)
+		dboxide_outb(tf->hob_nsect, io_ports->nsect_addr);
+	if (task->tf_flags & IDE_TFLAG_OUT_HOB_LBAL)
+		dboxide_outb(tf->hob_lbal, io_ports->lbal_addr);
+	if (task->tf_flags & IDE_TFLAG_OUT_HOB_LBAM)
+		dboxide_outb(tf->hob_lbam, io_ports->lbam_addr);
+	if (task->tf_flags & IDE_TFLAG_OUT_HOB_LBAH)
+		dboxide_outb(tf->hob_lbah, io_ports->lbah_addr);
+	if (task->tf_flags & IDE_TFLAG_OUT_FEATURE)
+		dboxide_outb(tf->feature, io_ports->feature_addr);
+	if (task->tf_flags & IDE_TFLAG_OUT_NSECT)
+		dboxide_outb(tf->nsect, io_ports->nsect_addr);
+	if (task->tf_flags & IDE_TFLAG_OUT_LBAL)
+		dboxide_outb(tf->lbal, io_ports->lbal_addr);
+	if (task->tf_flags & IDE_TFLAG_OUT_LBAM)
+		dboxide_outb(tf->lbam, io_ports->lbam_addr);
+	if (task->tf_flags & IDE_TFLAG_OUT_LBAH)
+		dboxide_outb(tf->lbah, io_ports->lbah_addr);
+	if (task->tf_flags & IDE_TFLAG_OUT_DEVICE)
+		dboxide_outb((tf->device & HIHI) | drive->select.all,
+		     io_ports->device_addr);
+#ifdef TRACE_DBOXIDE
+	/* show what happened so far... */
+	dboxide_problem("dboxide_tf_load");
+#endif
+}
+
+static void dboxide_tf_read(ide_drive_t *drive, ide_task_t *task)
+{
+	ide_hwif_t *hwif = drive->hwif;
+	struct ide_io_ports *io_ports = &hwif->io_ports;
+	struct ide_taskfile *tf = &task->tf;
+
+	if (task->tf_flags & IDE_TFLAG_IN_DATA) {
+		u16 data = dboxide_inw(io_ports->data_addr);
+
+		tf->data = data & 0xff;
+		tf->hob_data = (data >> 8) & 0xff;
+	}
+
+	/* be sure we're looking at the low order bits */
+	dboxide_outb(drive->ctl & ~0x80, io_ports->ctl_addr);
+
+	if (task->tf_flags & IDE_TFLAG_IN_NSECT)
+		tf->nsect  = dboxide_inb(io_ports->nsect_addr);
+	if (task->tf_flags & IDE_TFLAG_IN_LBAL)
+		tf->lbal   = dboxide_inb(io_ports->lbal_addr);
+	if (task->tf_flags & IDE_TFLAG_IN_LBAM)
+		tf->lbam   = dboxide_inb(io_ports->lbam_addr);
+	if (task->tf_flags & IDE_TFLAG_IN_LBAH)
+		tf->lbah   = dboxide_inb(io_ports->lbah_addr);
+	if (task->tf_flags & IDE_TFLAG_IN_DEVICE)
+		tf->device = dboxide_inb(io_ports->device_addr);
+
+	if (task->tf_flags & IDE_TFLAG_LBA48) {
+		dboxide_outb(drive->ctl | 0x80, io_ports->ctl_addr);
+
+		if (task->tf_flags & IDE_TFLAG_IN_HOB_FEATURE)
+			tf->hob_feature = dboxide_inb(io_ports->feature_addr);
+		if (task->tf_flags & IDE_TFLAG_IN_HOB_NSECT)
+			tf->hob_nsect   = dboxide_inb(io_ports->nsect_addr);
+		if (task->tf_flags & IDE_TFLAG_IN_HOB_LBAL)
+			tf->hob_lbal    = dboxide_inb(io_ports->lbal_addr);
+		if (task->tf_flags & IDE_TFLAG_IN_HOB_LBAM)
+			tf->hob_lbam    = dboxide_inb(io_ports->lbam_addr);
+		if (task->tf_flags & IDE_TFLAG_IN_HOB_LBAH)
+			tf->hob_lbah    = dboxide_inb(io_ports->lbah_addr);
+	}
+#ifdef TRACE_DBOXIDE
+	/* show what happened so far... */
+	dboxide_problem("dboxide_tf_read");
+#endif
+
+}
+
+
+static void dboxide_input_data(ide_drive_t *drive, struct request *rq,
+			   void *buf, unsigned int len)
+{
+	unsigned long data_addr = drive->hwif->io_ports.data_addr;
+
+	len++;
+
+	if (drive->io_32bit) {
+		dboxide_insl(data_addr, buf, len / 4);
+
+		if ((len & 3) >= 2)
+			dboxide_insw(data_addr, (u8 *)buf + (len & ~3), 1);
+	} else
+		dboxide_insw(data_addr, buf, len / 2);
+#ifdef TRACE_DBOXIDE
+	/* show what happened so far... */
+	dboxide_problem("dboxide_input_data %8x %8x", data_addr, len);
+#endif
+
+}
+
+static void dboxide_output_data(ide_drive_t *drive,  struct request *rq,
+			    void *buf, unsigned int len)
+{
+	unsigned long data_addr = drive->hwif->io_ports.data_addr;
+
+	len++;
+
+	if (drive->io_32bit) {
+		dboxide_outsl(data_addr, buf, len / 4);
+
+		if ((len & 3) >= 2)
+			dboxide_outsw(data_addr, (u8 *)buf + (len & ~3), 1);
+	} else
+		dboxide_outsw(data_addr, buf, len / 2);
+#ifdef TRACE_DBOXIDE
+	/* show what happened so far... */
+	dboxide_problem("dboxide_output_data %8x %8x", data_addr, len);
+#endif
+
+}
+
+
+#endif
+
 /*---------------------------------------------------------*/
 /* acknowledge the interrupt? */
 /*---------------------------------------------------------*/
 
 int dboxide_ack_intr(ide_hwif_t * hwif)
 {
-	printk("dboxide: ack irq\n");
+	printk("%s ack irq\n", modname);
 	return 1;
 }
 
@@ -411,10 +580,12 @@
 /* also works without them                                 */
 /*---------------------------------------------------------*/
 
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,25)
 void dboxide_tuneproc(ide_drive_t * drive, u8 pio)
 {
-	printk("dboxide: tuneproc called: %d\n", pio);
+	printk("%s tuneproc called: %d\n", modname, pio);
 }
+#endif
 
 /*---------------------------------------------------------*/
 /* end of functions called via function pointer */
@@ -432,20 +603,22 @@
 	/* As this routine is the only one that needs to know about
 	   wich interrupt is used in this code, it returns the number
 	   so it can be given to the kernel */
-	return CPM_IRQ_OFFSET + CPMVEC_PIO_PC15;
+
+
+	return CPM_IRQ_OFFSET + ( irq6 ? -4 : CPMVEC_PIO_PC15);
 }
 
 #if LINUX_VERSION_CODE < KERNEL_VERSION(2,5,0)
 static void reset_function_pointer(ide_hwif_t *hwif)
 {
 	int i;
+	printk("%s reset_function_pointer called\n", modname);
 	for (i = 0; i < ARRAY_SIZE(hwif->hw.io_ports); i++) {
 		uint addr;
 
 		addr = hwif->hw.io_ports[i];
 		if ((addr > idebase) && (addr <= idebase + 0x100))
 			hwif->hw.io_ports[i] = addr - idebase;
-
 		addr = hwif->io_ports[i];
 		if ((addr > idebase) && (addr <= idebase + 0x100))
 			hwif->io_ports[i] = addr - idebase;
@@ -457,22 +630,39 @@
    functions */
 static void set_access_functions(ide_hwif_t * hwif)
 {
+#ifdef TRACE_DBOXIDE
+	printk("%s set_access_functions called\n", modname);
+#endif
 	hwif->mmio = 2;
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,26)
 	hwif->OUTB = dboxide_outb;
 	hwif->OUTBSYNC = dboxide_outbsync;
 	hwif->OUTW = dboxide_outw;
 #if LINUX_VERSION_CODE <= KERNEL_VERSION(2,6,20)
 	hwif->OUTL = dboxide_outl;
-	hwif->INL = dboxide_inl;
 #endif
 	hwif->OUTSW = dboxide_outsw;
 	hwif->OUTSL = dboxide_outsl;
 	hwif->INB = dboxide_inb;
 	hwif->INW = dboxide_inw;
+#if LINUX_VERSION_CODE <= KERNEL_VERSION(2,6,20)
+	hwif->INL = dboxide_inl;
+#endif
 	hwif->INSW = dboxide_insw;
 	hwif->INSL = dboxide_insl;
+#else
+	hwif->INB         = dboxide_inb;
+	hwif->OUTB        = dboxide_outb;
+	hwif->OUTBSYNC    = dboxide_outbsync;
+	hwif->tf_load     = dboxide_tf_load;
+	hwif->tf_read     = dboxide_tf_read;
+	hwif->input_data  = dboxide_input_data;
+	hwif->output_data = dboxide_output_data;
+#endif
 
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,25)
 	hwif->tuneproc = dboxide_tuneproc;
+#endif
 
 #if 0
 	hwif->ack_intr = dboxide_ack_intr;
@@ -492,15 +682,18 @@
    initial address is different for 2.4 and 2.6. */
 static void init_hw_struct( ide_hwif_t *hwif, unsigned long idebase)
 {
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,25)
 	hw_regs_t *hw;
 
-	hwif->chipset = ide_unknown;
 	hwif->tuneproc = NULL;
+#endif
+	hwif->chipset = ide_unknown;
 	hwif->mate = NULL;
 	hwif->channel = 0;
 
-	strncpy (hwif->name, "dbox2 ide", sizeof(hwif->name));
+	strncpy (hwif->name, "DBox2 IDE", sizeof(hwif->name));
 
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,25)
 	hw = &hwif->hw;
 	memset(hw, 0, sizeof(hw_regs_t));
 
@@ -519,6 +712,38 @@
 	hwif->irq = hw->irq = configure_interrupt();
 
 	memcpy(hwif->io_ports, hw->io_ports, sizeof(hw->io_ports));
+#else
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,26)
+
+	hwif->io_ports[IDE_DATA_OFFSET] 	= idebase + 0x0010;
+	hwif->io_ports[IDE_ERROR_OFFSET] 	= idebase + 0x0011;
+	hwif->io_ports[IDE_NSECTOR_OFFSET] 	= idebase + 0x0012;
+	hwif->io_ports[IDE_SECTOR_OFFSET] 	= idebase + 0x0013;
+	hwif->io_ports[IDE_LCYL_OFFSET] 	= idebase + 0x0014;
+	hwif->io_ports[IDE_HCYL_OFFSET] 	= idebase + 0x0015;
+	hwif->io_ports[IDE_SELECT_OFFSET] 	= idebase + 0x0016;
+	hwif->io_ports[IDE_STATUS_OFFSET] 	= idebase + 0x0017;
+
+	hwif->io_ports[IDE_CONTROL_OFFSET] 	= idebase + 0x004E;
+	hwif->io_ports[IDE_IRQ_OFFSET] 	  	= idebase + 0x004E;
+
+	hwif->irq = configure_interrupt();
+#else
+	hwif->io_ports.data_addr 	= idebase + 0x0010;
+	hwif->io_ports.error_addr 	= idebase + 0x0011;
+	hwif->io_ports.nsect_addr 	= idebase + 0x0012;
+	hwif->io_ports.lbal_addr  	= idebase + 0x0013;
+	hwif->io_ports.lbam_addr 	= idebase + 0x0014;
+	hwif->io_ports.lbah_addr 	= idebase + 0x0015;
+	hwif->io_ports.device_addr 	= idebase + 0x0016;
+	hwif->io_ports.status_addr 	= idebase + 0x0017;
+
+	hwif->io_ports.ctl_addr 	= idebase + 0x004E;
+	hwif->io_ports.irq_addr	  	= idebase + 0x004E;
+	hwif->irq = configure_interrupt();
+#endif
+
+#endif
 }
 
 /* the CPLD is connected to CS2, which should be inactive.
@@ -531,18 +756,18 @@
 	uint br2 = memctl->memc_br2;
 
 	if (br2 & 0x1) {
-		printk("dboxide: cs2 already activated\n");
+		printk("%s cs2 already activated\n", modname);
 		return 0;
 	}
 
 	if (br2 != 0x02000080) {
-		printk("dboxide: cs2: unexpected value for br2: %08x\n", br2);
+		printk("%s cs2: unexpected value for br2: %08x\n", modname, br2);
 		return 0;
 	}
 
 	br2 |= 0x1;
 
-	printk("dboxide: activating cs2\n");
+	printk("%s activating cs2\n", modname );
 	memctl->memc_br2 = br2;
 
 	return 1;
@@ -556,28 +781,77 @@
 	uint br2 = memctl->memc_br2;
 
 	if (br2 != 0x02000081) {
-		printk("dboxide: cs2 configuration unexpected: %08x\n", br2);
+		printk("%s cs2 configuration unexpected: %08x\n", modname, br2);
 		return 0;
 	}
 
+	printk("%s deactivating cs2\n", modname);
 	br2 &= ~1;
-
-	printk("dboxide: deactivating cs2\n");
 	memctl->memc_br2 = br2;
 
 	return 1;
 }
 
+/*read out id-code, this value can only get on first read!*/
+static unsigned int read_if_idcode(void) 
+{
+	static unsigned int idcode;
+	static int alreadyread = 0;
+	if (!alreadyread) {
+		idcode = CPLD_IN(CPLD_READ_FIFO);
+		alreadyread = 1;
+	}
+	return idcode;
+}
+
+static int write_if_idcodeback(unsigned int idcode) 
+{
+	CPLD_OUT(CPLD_WRITE_FIFO, idcode);
+}
+
+static int ide_software_reset_drives(void) 
+{
+        int i, j, tmp;
+
+	for (i = 0; i<10; i++) {
+		dboxide_outb(0x0E, 0x4E		/*IDE_CONTROL_REG*/);
+		udelay(10);
+		dboxide_outb(0x0A, 0x4E		/*IDE_CONTROL_REG*/);
+		udelay(10);
+		for (j = 0; j < 200; j++) {
+			tmp = dboxide_inb(0x17	/*IDE_STATUS_REG*/);
+			if (!(tmp & 0x80))
+				break;
+			IDE_DELAY();
+	        }
+        	if (tmp & 0x80)
+			printk("%s timeout, drive is still busy\n", modname);
+		tmp = dboxide_inb(0x11 		/*IDE_ERROR_REG*/);
+
+        	if (tmp == 1) {
+			printk("%s sreset succeeded\n", modname);
+			return 1;
+		}
+	}
+	printk("%s sreset failed, status: 0x%04x\n", modname, tmp);
+	return 0;
+}
+
 /* detect_cpld: Check that the CPLD really works */
 static int detect_cpld(void)
 {
 	int i;
-	uint check, back;
+	int vendor_idx = 0;	/* index into vendor table */
+
+	uint check, back, idcode;
 	uint patterns[2] = { 0xCAFEFEED, 0xBEEFC0DE };
 
 	/* This detection code not only checks that there is a CPLD,
 	   but also that it does work more or less as expected.  */
 
+	/*read out identifynumber*/
+	idcode = read_if_idcode();
+
 	/* first perform a walking bit test via data register:
 	   this checks that there is a data register and
 	   that the data bus is correctly connected */
@@ -589,7 +863,7 @@
 		back = CPLD_IN(CPLD_READ_DATA);
 		if (check != back) {
 			printk
-			    ("dboxide: probing dbox2 IDE CPLD: walking bit test failed: %08x != %08x\n",
+			    ("%s probing DBox2 IDE CPLD: walking bit test failed: %08x != %08x\n", modname, 
 			     check, back);
 			return 0;
 		}
@@ -600,20 +874,20 @@
 		back = CPLD_IN(CPLD_READ_DATA);
 		if (check != back) {
 			printk
-			    ("dboxide: probing dbox2 IDE CPLD: walking bit test failed: %08x != %08x\n",
+			    ("%s probing DBox2 IDE CPLD: walking bit test failed: %08x != %08x\n", modname,
 			     check, back);
 			return 0;
 		}
 	}
 
-	/* second: check ctrl register.
-	   this also activates the IDE Reset. */
-	check = 0x00FF0007;
+	/* second: check ctrl register.*/
+	check = 0x0012001F; /* Activate PIO Mode 4 timing and remove IDE Reset */
+
 	CPLD_OUT(CPLD_WRITE_CTRL_TIMING, check);
 	back = CPLD_IN(CPLD_READ_CTRL);
 	if ((back & check) != check) {
 		printk
-		    ("dboxide: probing dbox2 IDE CPLD: ctrl register not valid: %08x != %08x\n",
+		    ("%s probing DBox2 IDE CPLD: ctrl register not valid: %08x != %08x\n", modname,
 		     check, back & check);
 		return 0;
 	}
@@ -627,7 +901,7 @@
 
 	if (i == 0) {
 		printk
-		    ("dboxide: fifo seems to have data but clearing did not succeed\n");
+		    ("%s fifo seems to have data but clearing did not succeed\n", modname);
 		return 0;
 	}
 
@@ -638,19 +912,19 @@
 	/* and read them back */
 	back = CPLD_IN(CPLD_READ_FIFO);
 	if (back != patterns[0]) {
-		printk("dboxide: fifo did not store first test pattern\n");
+		printk("%s fifo did not store first test pattern\n", modname);
 		return 0;
 	}
 	back = CPLD_IN(CPLD_READ_FIFO);
 	if (back != patterns[1]) {
-		printk("dboxide: fifo did not store second test pattern\n");
+		printk("%s fifo did not store second test pattern\n", modname);
 		return 0;
 	}
 
 	/* now the fifo must be empty again */
 	back = CPLD_IN(CPLD_READ_CTRL);
 	if ((back & 0xF0000000) != 0) {
-		printk("dboxide: fifo not empty after test\n");
+		printk("%s fifo not empty after test\n", modname);
 		return 0;
 	}
 
@@ -659,8 +933,8 @@
 	CPLD_OUT(CPLD_WRITE_FIFO, check);
 	back = CPLD_IN(CPLD_READ_FIFO);
 	if (back != check) {
-		printk("dboxide: final fifo clear did not work: %x!=%x\n", back,
-		       check);
+		printk("%s final fifo clear did not work: %x!=%x\n", modname, 
+			back, check);
 		return 0;
 	}
 
@@ -670,12 +944,25 @@
 	   will probe for drives etc, so this will check a lot
 	 */
 
+	switch (idcode) {
+		case 0:
+		case 0x50505050:
+			vendor_idx = 1;		/* Gurgel */
+			break;
+		case 0x556c6954:
+			vendor_idx = 2;		/* DboxBaer or kpt.ahab/Stingray */
+			break;
+	}
+	printk("%s IDE-Interface detected, %08x, Vendor: %s\n", modname, 
+		idcode, dboxide_vendors[vendor_idx]);
+
+
 	/* before going releasing IDE Reset, wait some time... */
 	for (i = 0; i < 10; i++)
 		IDE_DELAY();
 
-	/* Activate PIO Mode 4 timing and remove IDE Reset */
-	CPLD_OUT(CPLD_WRITE_CTRL_TIMING, 0x0012001F);
+	/*Reset Drives via IDE-Device-Control-Register (SRST)*/
+	ide_software_reset_drives();
 
 	/* finally set all bits in data register, so nothing
 	   useful is read when the CPLD is accessed by the
@@ -716,14 +1003,14 @@
 	map_memory();
 
 	if (idebase == 0) {
-		printk(KERN_ERR "dboxide: address space of dbox2 IDE CPLD not mapped to kernel address space\n");
+		printk(KERN_ERR "%s address space of DBox2 IDE CPLD not mapped to kernel address space\n", modname);
 		return -ENOMEM;
 	}
 
-	printk(KERN_INFO "dboxide: address space of DBox2 IDE CPLD is at: 0x%08x\n", idebase);
+	printk(KERN_INFO "%s address space of DBox2 IDE CPLD is at: 0x%08x\n", modname, idebase);
 
 	if (detect_cpld() == 0) {
-		printk(KERN_ERR "dboxide: not a valid dbox2 IDE CPLD detected\n");
+		printk(KERN_ERR "%s not a valid DBox2 IDE CPLD detected\n", modname);
 		unmap_memory();
 		return -ENODEV;
 	}
@@ -790,7 +1077,7 @@
 		ide_probe_module(1);
 
 	} else {
-		printk("dboxide: no hwif was given\n");
+		printk("%s no hwif was given\n", modname);
 	}
 }
 
@@ -822,7 +1109,7 @@
 	/* register driver will call the scan function above, maybe immediately 
 	   when we are a module, or later when it thinks it is time to do so */
 	printk(KERN_INFO
-	       "dboxide: $Id: main.c,v 1.4.2.6 2007/10/09 01:04:06 carjay Exp $\n");
+	       "%s $Id: main.c,v 1.4.2.6 2007/10/09 01:04:06 carjay Exp $\n", modname);
 
 	ide_register_driver(dboxide_scan);
 
@@ -834,6 +1121,7 @@
 {
 	if (idebase != 0) {
 		CPLD_OUT(CPLD_WRITE_CTRL_TIMING, 0x00FF0007);
+		write_if_idcodeback(read_if_idcode());
 	}
 
 	idebase = 0;
@@ -849,6 +1137,9 @@
 #else
 static int dbox2_ide_probe(struct platform_device *pdev)
 {
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,25)
+	u8 idx[4] = { 0xff, 0xff, 0xff, 0xff };
+#endif
 	ide_hwif_t *hwif = &ide_hwifs[pdev->id];
 
 	init_hw_struct(hwif, 0);
@@ -859,8 +1150,18 @@
 
 	/* tell kernel to the interface
 		(the default is not to probe) */
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,26)
 	hwif->noprobe = 0;
+#else
+	hwif->drives[0].noprobe=0;
+	hwif->drives[1].noprobe=0;
+#endif
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,25)
 	probe_hwif_init(hwif);
+#else
+	idx[0] = hwif->index;
+	ide_device_add(idx, NULL);
+#endif
 
 	platform_set_drvdata(pdev, hwif);
 	
@@ -871,7 +1172,15 @@
 {
 	ide_hwif_t *hwif = platform_get_drvdata(pdev);
 
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,25)
 	ide_unregister(hwif - ide_hwifs);
+#else
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,26)
+	ide_unregister(hwif - ide_hwifs, 0, 0);
+#else	
+	ide_unregister(hwif);
+#endif
+#endif
 	
 	return 0;
 }
@@ -889,7 +1198,7 @@
 	int ret;
 
 	printk(KERN_INFO
-	       "dboxide: $Id: main.c,v 1.4.2.6 2007/10/09 01:04:06 carjay Exp $\n");
+	       "%s $Id: main.c,v 1.4.2.6 2007/10/09 01:04:06 carjay Exp $\n", modname);
 
 	ret = setup_cpld();
 	if (ret < 0)
@@ -913,9 +1222,10 @@
 /* dbox_ide_exit is called when the module is unloaded */
 static void __exit dboxide_exit(void)
 {
-	if (idebase != 0)
+	if (idebase != 0) {
 		CPLD_OUT(CPLD_WRITE_CTRL_TIMING, 0x00FF0007);
-
+		write_if_idcodeback(read_if_idcode());
+	}
 	if (ide_dev)
 		platform_device_unregister(ide_dev);
 
@@ -924,13 +1234,14 @@
 	unmap_memory();
 	deactivate_cs2();
 
-	printk("dboxide: driver unloaded\n");
+	printk("%s driver unloaded\n", modname);
 }
 #endif
 
 module_init(dboxide_init);
 module_exit(dboxide_exit);
 
+module_param(irq6,int,0);
 MODULE_AUTHOR("Uli Tessel <utessel@gmx.de>");
-MODULE_DESCRIPTION("DBOX IDE CPLD Interface driver");
+MODULE_DESCRIPTION("DBox2 IDE CPLD Interface driver");
 MODULE_LICENSE("GPL");

Das Einzige was mich momentan noch nervt ist, dass das Auslesen des CPLD-ID nicht funktioniert. Habe aber den (hoffentlich kosmetischen) Fehler bisher nicht gefunden.

[seife]

Das ist doch mal ein erster post!

Ich habe keine IDE-Interfaces, drum kann ich es nicht testen.
Wegen der ID: irgendwas war da, dass die nur einmal ausgelesen werden kann, die ist dann weg. Evtl. ist da ja irgendwas faul.

[jojo]

Das ist doch mal ein erster post!

Ich habe keine IDE-Interfaces, drum kann ich es nicht testen.
Wegen der ID: irgendwas war da, dass die nur einmal ausgelesen werden kann, die ist dann weg. Evtl. ist da ja irgendwas faul.

Nun, das Auslesen erfolgt direkt nach dem Boot - und ausschließlich im dboxide Treiber. Das differiert ein wenig vom 2.4er, da wird
der Test 2x durchgeführt - einmal in tuxinfo, das 2. Mal in dboxide.
Kann da der spekulative Read der CPU reinfunken oder verhält sich das remap unter 2.6 genau wie bei 2.4?

Was mich momentan auch noch gewaltig nervt - die Kiste glüht beim Aufnehmen - 80% CPU-Last...
Der Aufbau des Favoriten-Menüs hat 5min (300sec) gedauert...

• top - 22:42:22 up 2:39, 0 users, load average: 3.78, 3.97, 3.78
  Tasks: 43 total, 1 running, 42 sleeping, 0 stopped, 0 zombie
  Cpu(s): 3.9%us, 82.9%sy, 0.0%ni, 0.0%id, 0.0%wa, 12.9%hi, 0.3%si, 0.0%st
  Mem: 30324k total, 29300k used, 1024k free, 144k buffers
  Swap: 0k total, 0k used, 0k free, 1944k cached
  
  PID USER PR NI VIRT RES SHR S %CPU %MEM TIME+ COMMAND
  394 root 20 0 62780 10m 336 D 50.4 35.0 27:30.42 neutrino
  267 root 15 -5 0 0 0 S 18.9 0.0 32:19.38 dmx_wq/0

Bei ARD ist das ganze noch übler:

• top - 20:02:00 up 4 min, 0 users, load average: 2.76, 1.52, 0.62
  Tasks: 43 total, 2 running, 41 sleeping, 0 stopped, 0 zombie
  Cpu(s): 51.1%us, 43.3%sy, 0.0%ni, 0.0%id, 0.0%wa, 5.4%hi, 0.2%si, 0.0%st
  Mem: 30324k total, 29328k used, 996k free, 2344k buffers
  Swap: 0k total, 0k used, 0k free, 9892k cached
  
  PID USER PR NI VIRT RES SHR S %CPU %MEM TIME+ COMMAND
  374 root 20 0 74732 6376 952 S 51.4 21.0 1:37.28 sectionsd
  267 root 15 -5 0 0 0 R 45.1 0.0 1:24.12 dmx_wq/0
  408 root 20 0 1788 832 672 R 5.8 2.7 0:07.99 top
  395 root 20 0 29744 4396 2500 S 1.0 14.5 0:22.72 neutrino

Warum frisst der dmx_wq die halbe CPU?

[wolgade]

Bei ARD ist das ganze noch übler:
[..]
Warum frisst der dmx_wq die halbe CPU?

Wenn du die Frage in deinem dritten Posting beantwortest, dann machst du hier eine Menge Leute glücklich.

Laß mich raten: Du hast eine Nokia-Box mit AVIA 600? Da läuft bei ARD-Aufnahmen nämlich regelmäßig der Demux-Puffer über mit der Konsequenz, daß man die Aufnahme wegschmeissen kann.

Sorry, daß mein Posting nicht wirklich weiterhilft. Aber da du hier treffsicher im Bermuda-Dreieck gelandet bist, konnte ich nicht widerstehen.

[jojo]

Laß mich raten: Du hast eine Nokia-Box mit AVIA 600? Da läuft bei ARD-Aufnahmen nämlich regelmäßig der Demux-Puffer über mit der Konsequenz, daß man die Aufnahme wegschmeissen kann.

Treffer
Passiert das auch mit dem 2.4er? Habe bisher noch nicht so drauf geachtet bzw keine Aufnahmen von ARD durchgeführt.
Mir fiel bisher beim 2.4er nur auf, das Aufnahmen teilweise extrem zerstückelt werden.
Aber das gehört wohl eher in die Ecke AVIA / sectionsd?

[wolgade]

Passiert das auch mit dem 2.4er?

Ja, ich hatte noch nie einen 2.6 Kernel auf der Box.

Aber das gehört wohl eher in die Ecke AVIA / sectionsd?

Auf jeden Fall ist es kein spezifisches 2.6 Problem.

[rhabarber1848]

Könntest Du die Patches als .diff irgendwo hochladen?
Die code-Boxen zerstören das Dateilayout, z.B. die Leerzeichen am
Beginn der nicht gepatchten Zeilen.

[rhabarber1848]

Habe den Patch mittlerweile per Email erhalten - Danke! -, aber noch nicht kompiliert.
Einen HW-Test kann ich ohnehin nicht machen, mangels HDD-Interface.

Hier gibt es den Patch zum allgemeinen Testen: EDIT: Patch ist im CVS

[rhabarber1848]

Kompilieren tut der Code, ein warning ist aufgetreten:

driver/ide/trace.c:23: warning: function declaration isn't a prototype

Das bezieht sich auf diese neue Zeile:

Code: Alles auswählen

extern void mpc8xx_wdt_reset();

Kann das zum Problem werden?

[rhabarber1848]

Booten funktioniert auch, das IDE-Modul erkennt korrekt, dass ich kein IDE-Interface habe
yadd-neutrino funktioniert dann weiterhin normal, wenn noch die warning bearbeitet wurde,
kann der Code ins CVS. Gerne würde ich aber noch den positiven Testbericht eines Users mit
HDD-Interface sehen.

[JtG-Riker]

Auch wenns OT ist - wegen dem Nokia - Avia600 - ARD/ZDF teilweise auch Problem mit den hohen Bitraten, wer hat da Lust mal nach zu sehen der bekomme von mir ne Box mit Platte zum debuggen/testen, ich kann ja schlecht schon wieder Houdini dafür werben oder doch?

Gruß Riker

[seife]

Kompilieren tut der Code, ein warning ist aufgetreten:

driver/ide/trace.c:23: warning: function declaration isn't a prototype

Das bezieht sich auf diese neue Zeile:

Code: Alles auswählen

extern void mpc8xx_wdt_reset();

Kann das zum Problem werden?

Ich glaube nicht. Aber mach mal

Code: Alles auswählen

extern void mpc8xx_wdt_reset(void);

daraus. Evtl. reicht das schon.

[jojo]

Kompilieren tut der Code, ein warning ist aufgetreten:

driver/ide/trace.c:23: warning: function declaration isn't a prototype

Das bezieht sich auf diese neue Zeile:

Code: Alles auswählen

extern void mpc8xx_wdt_reset();

Kann das zum Problem werden?

Sollte nicht

Da ist eh' ein Flüchtigkeitsfehler drin:

Code: Alles auswählen

#undef TRACE_ENABLE

#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,0)
extern void mpc8xx_wdt_reset(void);
#endif

extern char *dboxide_trace_msg[];
extern void mpc8xx_wdt_reset();

Ich nehme an, daß wir auf das conditional verzichten können, da ja nach dem linux-2.6.26.4-dbox2.diff Patch alle Versionen mpc8xx_wdt_reset exportieren.
Somit würde folgendes diff die Situation korrigieren:

Code: Alles auswählen

--- ./driver/ide/trace.orig	2009-02-12 08:34:58.658126450 +0100
+++ ./driver/ide/trace.c	2009-02-12 08:35:29.418117174 +0100
@@ -15,12 +15,8 @@
 /* enabel debug trace if TRACE_ENABLE is defined */
 #undef TRACE_ENABLE
 
-#if LINUX_VERSION_CODE > KERNEL_VERSION(2,6,0)
 extern void mpc8xx_wdt_reset(void);
-#endif
-
 extern char *dboxide_trace_msg[];
-// extern void mpc8xx_wdt_reset();
 
 typedef struct IDETraceData {
 	unsigned long time;

[rhabarber1848]

Code: Alles auswählen

[   23.035246] Uniform Multi-Platform E-IDE driver
[   23.097679] ide: Assuming 50MHz system bus speed for PIO modes; override with idebus=xx
[   23.343473] dboxide: $Id: main.c,v 1.4.2.6 2007/10/09 01:04:06 carjay Exp $
[   23.431530] dboxide: cs2 already activated
FATAL: Error inserting dboxide (/lib/modules/2.6.26.4-dbox2/extra/ide/dboxide.ko): No such device

So sieht die Bootmeldung bei mir aus mit dem hier besprochenen Treiber,
inkl. der letzten Änderungen von jojo.

Patch committed:
http://cvs.tuxbox-cvs.sourceforge.net/c ... 1.2&r2=1.3
http://cvs.tuxbox-cvs.sourceforge.net/c ... r1=1.4.2.6
http://cvs.tuxbox-cvs.sourceforge.net/c ... r1=1.1.2.3

[jojo]

Code: Alles auswählen

[   23.035246] Uniform Multi-Platform E-IDE driver
[   23.097679] ide: Assuming 50MHz system bus speed for PIO modes; override with idebus=xx
[   23.343473] dboxide: $Id: main.c,v 1.4.2.6 2007/10/09 01:04:06 carjay Exp $
[   23.431530] dboxide: cs2 already activated
FATAL: Error inserting dboxide (/lib/modules/2.6.26.4-dbox2/extra/ide/dboxide.ko): No such device

So sieht die Bootmeldung bei mir aus mit dem hier besprochenen Treiber,
inkl. der letzten Änderungen von jojo.

Patch committed:
http://cvs.tuxbox-cvs.sourceforge.net/c ... 1.2&r2=1.3
http://cvs.tuxbox-cvs.sourceforge.net/c ... r1=1.4.2.6
http://cvs.tuxbox-cvs.sourceforge.net/c ... r1=1.1.2.3

Und so mit installiertem Interface:

Code: Alles auswählen

[   21.652764] Uniform Multi-Platform E-IDE driver
[   21.708531] ide: Assuming 50MHz system bus speed for PIO modes; override with idebus=xx
[   22.149787] dboxide: $Id: main.c,v 1.4.2.7 2009/02/12 12:41:51 rhabarber1848 Exp $
[   22.237087] dboxide: activating cs2
[   22.278462] dboxide: address space of DBox2 IDE CPLD is at: 0xc3050000
[   22.356641] dboxide: IDE-Interface detected, e000e000, Vendor: Unknown
[   23.090996] dboxide: sreset succeeded
[   23.171315] Probing IDE interface DBox2 ...
[   23.471113] hda: ST9160821AS, ATA DISK drive
[   24.255773] DBox2  at 0x010-0x017,0x04e on irq 47
[   24.715156] hda: max request size: 512KiB
[   24.908462] hda: 312581808 sectors (160041 MB) w/8192KiB Cache, CHS=19457/255/63
[   24.994290] hda: cache flushes supported
[   25.042331]  hda: hda1 hda2
[   28.508369] SGI XFS with security attributes, no debug enabled
[   29.010525] XFS mounting filesystem hda2
[   30.467263] Starting XFS recovery on filesystem: hda2 (logdev: internal)
[   30.806228] Ending XFS recovery on filesystem: hda2 (logdev: internal)

Das "Vendor: Unknown" kommt daher, das ich lediglich ein reboot ausgeführt habe. Bei PowerOn hingegen wird der Interface-Typ korrekt angezeigt. Ist, laut dem Kommentar im Sourcecode, ein Feature.

[Rebel1]

So, ich habe nun auch mal ein 2.6er Image mit IDE-Support gebaut und geflasht.
Das ganze funktioniert soweit auch einwandfrei, was mir aber auffällt, ist das meine Sagem-Box hier, im Gegensatz zu einem 2.4er Image ,
noch wesentlich träger bei der Aufnahme und Wiedergabe reagiert.

Hier mal der Ausschnitt aus dem Log zur Zeit der Aufnahme:

Code: Alles auswählen

[122660.165877] avia_av: setting stream type AVIA_AV_STREAM_TYPE_0/AVIA_AV_STREAM_TYPE_0
[122660.172213] SPTS, queue 0 extended.
[122980.955287] avia_gt_dmx: queue 0 overflow (count: 1)
[122990.119411] avia_gt_dmx: queue 0 overflow (count: 1)
[123404.250014] avia_gt_dmx: queue 0 overflow (count: 1)
[123687.760167] avia_gt_dmx: queue 0 overflow (count: 1)
[124346.748255] avia_gt_dmx: queue 0 overflow (count: 1)
[125272.051175] avia_gt_dmx: queue 0 overflow (count: 2)
[125416.725127] avia_gt_dmx: queue 0 overflow (count: 1)
[126242.279281] avia_gt_dmx: queue 0 overflow (count: 1)
[126247.451518] avia_gt_dmx: queue 0 overflow (count: 1)
[126256.592388] avia_gt_dmx: queue 0 overflow (count: 1)
[126256.846394] avia_gt_dmx: queue 0 overflow (count: 1)
[126257.069510] avia_gt_dmx: queue 0 overflow (count: 1)
[126257.339493] avia_gt_dmx: queue 0 overflow (count: 1)
[126260.073342] avia_gt_dmx: queue 0 overflow (count: 1)
[126309.339237] avia_gt_dmx: queue 0 overflow (count: 1)
[126557.889201] avia_gt_dmx: queue 0 overflow (count: 2)
[127252.292607] avia_gt_dmx: queue 0 overflow (count: 1)
[128151.743279] avia_gt_dmx: queue 0 overflow (count: 1)
[128357.736183] avia_av: setting stream type AVIA_AV_STREAM_TYPE_PES/AVIA_AV_STREAM_TYPE_PES
[128357.756517] PES, queue 0 normal.

Ich habe mir mal die Aufnahme angesehen, und bemerke ab und an ein kurzes zwitschern, das vermutlich vom avia_gt_dmx: queue 0 overflow (count: 1) kommt.
Das kenne ich von meinem 2.4er CVS-Image nicht.

[jojo]

So, ich habe nun auch mal ein 2.6er Image mit IDE-Support gebaut und geflasht.
Das ganze funktioniert soweit auch einwandfrei, was mir aber auffällt, ist das meine Sagem-Box hier, im Gegensatz zu einem 2.4er Image ,
noch wesentlich träger bei der Aufnahme und Wiedergabe reagiert.
....

Ich habe mir mal die Aufnahme angesehen, und bemerke ab und an ein kurzes zwitschern, das vermutlich vom avia_gt_dmx: queue 0 overflow (count: 1) kommt.
Das kenne ich von meinem 2.4er CVS-Image nicht.

Jep... kein Vergleich zum 2.4er Kernel...
Sogar bei "langsamen" Sendern wie Tele5 bekomme ich ein 3-Sekunden Stakatto...
Das Abspielen (von 2.4er Aufnahmen) hingegen läuft ohne jegliche Hänger ab.

Hatte mal aus Jux und Tollerei in den Busy-Waits einen Counter eingebaut, um zu sehen wie häufig der Treiber auf die FIFO warten muß.
Sind je Block (512Byte) zwischen 63 und 70 Warteschleifen.Das bedeutet, das dboxide hier als Ursache komplett wegfällt.

Eine Stichprobe mit "top" (auf ARD) zeigt, das sich die 95% CPU-Last 60/40 auf dmux und Neutrino verteilen.
Hier scheint wirklich die Optimierung des dmux gefragt zu sein.

[rhabarber1848]

Hier scheint wirklich die Optimierung des dmux gefragt zu sein.

Ich teste seit langer Zeit mal wieder ein 2.6er-Image, der Prozess
dmx_wq nutzt, je nach Sender, bis zu 15% CPU, meine dBox2 ist
dabei sozusagen im Leerlauf, ich schaue nur TV.

[GetAway]

@rhabarber1848

Ist geplant das IDE-Setup Menü für den 2.6er mitbauen zu lassen?

#TODO: k26 support for ENABLE_DRIVE_GUI, it's disabled with -enable-kernel26 yet

[dbt]

Könnte man machen. Das hatte ich schon mal angefangen, aber der Kernel hatte damals ganz verdrehte Ausgaben in die virtuellen Dataisysteme geschickt, die bis dato wahrscheinlich nicht gefixt sind, dass ich es habe sein lassen. Deswegen extra Würgarounds reinzufrimeln, war mir zu blöd, auch wenn es mal eine Gelegenheit wäre meinen Stall etwas aufzuräumen, was ich schon lange machen wollte. Dafür fehlt mir aber momentan die Motivation und die Zeit dazu.