Re: [9front] BUG: thinclient not booting with usbehci,usbohci

[cinap_lenrek@felloff.net]

of course, please try.

i want to fix this for everyone, not just your personal machine :)

there might be some problems in the ehci or ohci initialization
that fails to recover the controller when busmaster has been
disabled before.

it would be great to investigate here instead of removing
the call to pcireset(), as it provides some safety.

the issue is that pci devices might be in a unknown state,
potentially fetching garbage dma descriptors from main memory
that we think is free to use.

the usual approach in drivers is to first reset the device
and bring it to a known state before enabling busmaster.

the call to pcireset() ensures that this is the case.
turn everything off, and then each driver carefully
reenables it when the device is ready.

maybe the first task would be to figure out which of the
controllers causes the problems (and which does does NOT).
if it is the ohci or ehci controllers.

we can selectively keep the busmaster on by editing
/sys/src/9/port/pci.c:

void
pcireset(void)
{
	Pcidev *p;

	for(p = pcilist; p != nil; p = p->list) {
		/* don't mess with the bridges */
		if(p->ccrb == 0x06)
			continue;

+		if(p->ccrb == Pcibcserial && p->ccru == Pciscusb){
+			switch(p->ccrp){
+			// case 0x30:	/* XHCI */
+			case 0x20:	/* EHCI */
+			// case 0x10:	/* OHCI */
+			// case 0x00:	/* UHCI */
+				continue;
+			}
+		}

		pcidisable(p);
	}
}

this skips the EHCI controllers, keeping its busmaster enable on.

in a second test, comment out the case 0x20 and remove comment
for case 0x10, so that the OHCI controller is kept untouched
instead of the ehci.

and last, try both cases...

that way we might be able to reduce the problem to one specifc
controller type and then start investigating there.

it is possible that when we disable the musmaster flag,
that the host controller gets a transaction abort and
raises an Host error that our driver didnt mask or clear
when our driver enables the device. this is what the
the other code from the preivous mail does.

--
cinap