SeaBIOS October 2012

seabios@seabios.org

27 participants
89 discussions

Re: [SeaBIOS] great things are coming your way :-)
by ron minnich 15 Oct '12

15 Oct '12

next problem: Compilation complete. 0 Errors, 31 Warnings, 0 Remarks, 3 Optimizations Compiling whole program /root/coreboot/build/seabios/out/ccode32flat.o In file included from /root/coreboot/build/seabios/out/ccode32flat.o.tmp.c:42:0: /root/coreboot/build/seabios/out/../src/acpi.c: In function ‘build_ssdt’: /root/coreboot/build/seabios/out/../src/acpi.c:416:34: error: ‘ssdt_proc_end’ undeclared (first use in this function) /root/coreboot/build/seabios/out/../src/acpi.c:416:34: note: each undeclared identifier is reported only once for each function it appears in /root/coreboot/build/seabios/out/../src/acpi.c:416:34: error: ‘ssdt_proc_start’ undeclared (first use in this function) /root/coreboot/build/seabios/out/../src/acpi.c:437:26: error: ‘ssdp_proc_aml’ undeclared (first use in this function) /root/coreboot/build/seabios/out/../src/acpi.c:438:18: error: ‘ssdt_proc_name’ undeclared (first use in this function) /root/coreboot/build/seabios/out/../src/acpi.c:441:18: error: ‘ssdt_proc_id’ undeclared (first use in this function) In file included from /root/coreboot/build/seabios/out/ccode32flat.o.tmp.c:42:0: /root/coreboot/build/seabios/out/../src/acpi.c: In function ‘build_pcihp’: /root/coreboot/build/seabios/out/../src/acpi.c:498:35: error: ‘ssdp_pcihp_aml’ undeclared (first use in this function) /root/coreboot/build/seabios/out/../src/acpi.c:507:9: error: ‘aml_ej0_name’ undeclared (first use in this function) /root/coreboot/build/seabios/out/../src/acpi.c:507:37: error: ‘aml_adr_dword’ undeclared (first use in this function) /root/coreboot/build/seabios/out/../src/acpi.c: In function ‘acpi_bios_init’: /root/coreboot/build/seabios/out/../src/acpi.c:737:41: error: ‘AmlCode’ undeclared (first use in this function) make[2]: *** [/root/coreboot/build/seabios/out/ccode32flat.o] Error 1 make[1]: *** [build] Error 2 make: *** [seabios] Error 2

1 0

Re: [SeaBIOS] great things are coming your way :-)
by ron minnich 15 Oct '12

15 Oct '12

next problem: If the seabios build fails, and you do a make again, it fails with this kind of error CC cbfs/fallback/coreboot_ram.debug OBJCOPY cbfs/fallback/coreboot_ram.elf Checking out SeaBIOS revision 385a7d0dec28841a05531cba96c62138c3959fef error: Your local changes to the following files would be overwritten by checkout: src/acpi-dsdt.hex Please, commit your changes or stash them before you can switch branches. Aborting fatal: A branch named 'coreboot' already exists. make[1]: *** [checkout] Error 128 make: *** [seabios] Error 2

1 0

Re: [SeaBIOS] great things are coming your way :-)
by Kevin O'Connor 15 Oct '12

15 Oct '12

On Sun, Oct 14, 2012 at 05:45:33PM -0700, ron minnich wrote: > I just did a pacman -S iasl and noticed that I got the aug2012 version > And, now, I can't get seabios to build for coreboot. We've had a few things pop up with iasl - can you post the error message? Technically, for coreboot the acpi stuff isn't needed at all. -Kevin

2 1

great things are coming your way :-)
by ron minnich 15 Oct '12

15 Oct '12

I just did a pacman -S iasl and noticed that I got the aug2012 version And, now, I can't get seabios to build for coreboot. Just an FYI, great things are coming your way. ron

1 0

Re: [SeaBIOS] USB Problems geode lx800
by Christian Gmeiner 14 Oct '12

14 Oct '12

On 11 Oct 2012 16:42, "Christian Gmeiner" <christian.gmeiner(a)gmail.com> wrote: > > Christian Gmeiner, MSc > > > 2012/10/11 Christian Gmeiner <christian.gmeiner(a)gmail.com>: > > 2012/10/11 Kevin O'Connor <kevin(a)koconnor.net>: > >> On Tue, Oct 09, 2012 at 02:31:05PM +0200, Christian Gmeiner wrote: > >>> 2012/10/9 Kevin O'Connor <kevin(a)koconnor.net>: > >>> > On Mon, Oct 08, 2012 at 02:14:03PM +0200, Christian Gmeiner wrote: > >>> >> HI all > >>> >> > >>> >> I am running into some usb problems with coreboot & seabios: > >>> > Can you set the debug level to 8 and post the whole log? Also, for > >>> > timeout issues, having timestamps (via tools/readserial.py tool) > >>> > sometimes helps. > >>> > >>> Attached > >> > >> Hrmm, I guess you've disabled threads in your config? > >> > > yep... > > > > > >> From your log, I see: > >> > >> [14:23:00.5] ehci_send_bulk qh=0x000ef680 dir=0 data=0x00006d55 size=31^M > >> [14:23:00.5] ehci_send_bulk qh=0x000ef700 dir=128 data=0x00006e3c size=36^M > >> [14:23:00.5] ehci_send_bulk qh=0x000ef700 dir=128 data=0x00006d8f size=13^M > >> [14:23:05.5] WARNING - Timeout at ehci_wait_td:542!^M > >> [14:23:05.5] ehci pipe=0x000ef700 cur=00000000 tok=00000000 next=6b80 td=0x00006b80 > >> +status=d0d80^M > >> [14:23:05.5] USB transmission failed^M > >> [14:23:05.5] Unable to configure USB MSC drive.^M > >> > >> This looks like the transfer is stalling in the cdb_get_inquiry() call > >> as part of scsi_init_drive(). The drive is identified okay and at > >> least some transfers look okay, so it doesn't appear to be a > >> controller issue. > >> > >> I've been told some USB sticks are very picky about init order, but > >> I'm surprised to see that this is occuring for you on a variety of USB > >> sticks. > >> > >> It's possible to verify that a stall has occurred - print out > >> tds[i].token for all tds in ehci_send_bulk() on the error path. > >> > >> Does this device still have problems if you soft-reset after a failed > >> boot? > >> > > > > Here are the results with some more debug: > > > > Cold Reboot > > ------------ > > > > |0f7a5000| pmm_malloc zone=0x0f7afe87 handle=ffffffff size=192 > > align=40 ret=0x0f7a7bc0 (detail=0x0f7a7d70) > > |0f7a5000| ehci_control: td at phys(f7a7bc0): status: 80 : token 80e80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_control: td at phys(f7a7c00): status: 0 : token d00 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_control: td at phys(f7a7c40): status: 0 : token 0 > > |0f7a5000| - cerr: 0, total_len: 0 > > |0f7a5000| pmm_free 0x0f7a7bc0 (detail=0x0f7a7d70) > > |0f7a5000| ehci_alloc_async_pipe 0x0f7a9090 2 > > |0f7a5000| pmm_malloc zone=0x0f7afe7f handle=ffffffff size=92 align=80 > > ret=0x000ef700 (detail=0x0f7a7d70) > > |0f7a5000| ehci_alloc_async_pipe 0x0f7a9090 2 > > |0f7a5000| pmm_malloc zone=0x0f7afe7f handle=ffffffff size=92 align=80 > > ret=0x000ef680 (detail=0x0f7a7d10) > > |0f7a5000| ehci_control 0x0f7a7cd0 (dir=128 cmd=8 data=1) > > |0f7a5000| pmm_malloc zone=0x0f7afe87 handle=ffffffff size=192 > > align=40 ret=0x0f7a7bc0 (detail=0x0f7a7ce0) > > |0f7a5000| ehci_control: td at phys(f7a7bc0): status: 80 : token 80e80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_control: td at phys(f7a7c00): status: 0 : token d00 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_control: td at phys(f7a7c40): status: 0 : token 1c00 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| pmm_free 0x0f7a7bc0 (detail=0x0f7a7ce0) > > |0f7a5000| pmm_malloc zone=0x0f7afe83 handle=ffffffff size=48 align=10 > > ret=0x000fdae0 (detail=0x0f7a7ce0) > > |0f7a5000| usb_cmd_data id=0x000fdae0 write=0 count=1 bs=36 buf=0x0f7a5f3c > > |0f7a5000| cbw to device > > |0f7a5000| ehci_send_bulk qh=0x000ef680 dir=0 data=0x0f7a5e65 size=31 > > |0f7a5000| before setup: td at phys(f7a5c80): status: 0 : token 0 > > |0f7a5000| - cerr: 0, total_len: 0 > > |0f7a5000| ehci_send_bulk: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| Data-In from the device to the host - 36 bytes > > |0f7a5000| ehci_send_bulk qh=0x000ef700 dir=128 data=0x0f7a5f3c size=36 > > |0f7a5000| before setup: td at phys(f7a5c80): status: 0 : token 0 > > |0f7a5000| - cerr: 0, total_len: 0 > > |0f7a5000| ehci_send_bulk: td at phys(f7a5c80): status: 80 : token d80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > > |0f7a5000| - cerr: 3, total_len: 0 > > > > > > Warm Reboot > > ----------- > > > > 0f7a5000| pmm_free 0x0f7a7bc0 (detail=0x0f7a7d70) > > |0f7a5000| ehci_alloc_async_pipe 0x0f7a9090 2 > > |0f7a5000| pmm_malloc zone=0x0f7afe7f handle=ffffffff size=92 align=80 > > ret=0x000ef700 (detail=0x0f7a7d70) > > |0f7a5000| ehci_alloc_async_pipe 0x0f7a9090 2 > > |0f7a5000| pmm_malloc zone=0x0f7afe7f handle=ffffffff size=92 align=80 > > ret=0x000ef680 (detail=0x0f7a7d10) > > |0f7a5000| ehci_control 0x0f7a7cd0 (dir=128 cmd=8 data=1) > > |0f7a5000| pmm_malloc zone=0x0f7afe87 handle=ffffffff size=192 > > align=40 ret=0x0f7a7bc0 (detail=0x0f7a7ce0) > > |0f7a5000| ehci_control: td at phys(f7a7bc0): status: 80 : token 80e80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_control: td at phys(f7a7c00): status: 0 : token d00 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_control: td at phys(f7a7c40): status: 0 : token 1c00 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| pmm_free 0x0f7a7bc0 (detail=0x0f7a7ce0) > > |0f7a5000| pmm_malloc zone=0x0f7afe83 handle=ffffffff size=48 align=10 > > ret=0x000fdae0 (detail=0x0f7a7ce0) > > |0f7a5000| usb_cmd_data id=0x000fdae0 write=0 count=1 bs=36 buf=0x0f7a5f3c > > |0f7a5000| cbw to device > > |0f7a5000| ehci_send_bulk qh=0x000ef680 dir=0 data=0x0f7a5e65 size=31 > > |0f7a5000| before setup: td at phys(f7a5c80): status: 0 : token 0 > > |0f7a5000| - cerr: 0, total_len: 0 > > |0f7a5000| ehci_send_bulk: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > > |0f7a5000| - cerr: 3, total_len: 0 > > > > > > This is the used patch: > > > > diff --git a/src/usb-ehci.c b/src/usb-ehci.c > > index 2676615..92a9c60 100644 > > --- a/src/usb-ehci.c > > +++ b/src/usb-ehci.c > > @@ -178,6 +178,14 @@ check_ehci_ports(struct usb_ehci_s *cntl) > > * Setup > > ****************************************************************/ > > > > +static void dump_td(struct ehci_qtd *td, const char *func) > > +{ > > + u32 addr = (u32)td; > > + dprintf(1, "%s: td at phys(%x): status: %x : token %x\n", func, > > addr, td->token & QTD_STS_MASK, td->token); > > + dprintf(1, "- cerr: %x, total_len: %x\n", (td->token & > > QTD_CERR_MASK) >> QTD_CERR_SHIFT, > > + (td->token & QTD_LENGTH_MASK) >> QTD_LENGTH_SHIFT); > > +} > > + > > // Wait for next USB async frame to start - for ensuring safe memory release. > > static void > > ehci_waittick(struct usb_ehci_s *cntl) > > @@ -535,6 +543,9 @@ ehci_wait_td(struct ehci_pipe *pipe, struct > > ehci_qtd *td, int timeout) > > status = td->token; > > if (!(status & QTD_STS_ACTIVE)) > > break; > > + > > + dump_td(td, __func__); > > + > > if (check_tsc(end)) { > > u32 cur = GET_LOWFLAT(pipe->qh.current); > > u32 tok = GET_LOWFLAT(pipe->qh.token); > > @@ -633,6 +644,7 @@ ehci_control(struct usb_pipe *p, int dir, const > > void *cmd, int cmdsize > > int i, ret=0; > > for (i=0; i<3; i++) { > > struct ehci_qtd *td = &tds[i]; > > + dump_td(td, __func__); > > ret = ehci_wait_td(pipe, td, 500); > > if (ret) > > break; > > @@ -665,18 +677,25 @@ ehci_send_bulk(struct usb_pipe *p, int dir, void > > *data, int datasize) > > struct ehci_qtd *td = &tds[tdpos++ % STACKQTDS]; > > int ret = ehci_wait_td(pipe, td, 5000); > > if (ret) > > + { > > + dump_td(td, "ehci_send_bulk ERROR2"); > > return -1; > > + } > > > > struct ehci_qtd *nexttd_fl = MAKE_FLATPTR(GET_SEG(SS) > > , &tds[tdpos % STACKQTDS]); > > > > + dump_td(td, "before setup"); > > + > > int transfer = fillTDbuffer(td, maxpacket, data, datasize); > > td->qtd_next = (transfer==datasize ? EHCI_PTR_TERM : (u32)nexttd_fl); > > td->alt_next = EHCI_PTR_TERM; > > barrier(); > > - td->token = (ehci_explen(transfer) | QTD_STS_ACTIVE > > + td->token = (/*ehci_explen(transfer) | */QTD_STS_ACTIVE > > | (dir ? QTD_PID_IN : QTD_PID_OUT) | ehci_maxerr(3)); > > > > + dump_td(td, __func__); > > + > > data += transfer; > > datasize -= transfer; > > } > > @@ -685,7 +704,10 @@ ehci_send_bulk(struct usb_pipe *p, int dir, void > > *data, int datasize) > > struct ehci_qtd *td = &tds[tdpos++ % STACKQTDS]; > > int ret = ehci_wait_td(pipe, td, 5000); > > if (ret) > > + { > > + dump_td(td, "ehci_send_bulk ERROR2"); > > return -1; > > + } > > } > > > > return 0; > > > > > > > > greets > > > Here is the log of filo using libpayload > > [16:40:24.5] FILO version 0.6.0 (christian@chgm-pc) Thu Oct 11 > 16:12:26 CEST 2012 > [16:40:24.5] 00:0f.5 2095:1022.5 EHCI controller > [16:40:24.7] highspeed device > [16:40:24.7] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:24.8] - cerr: 3, total_len: 0 > > [16:40:24.8] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:24.8] - cerr: 3, total_len: 0 > > > [16:40:24.8] moving from 148260 to 1482a0 > [16:40:24.8] wait_for_tds: td at phys(1f6b22a0): status: 0 token: 100 > > [16:40:24.8] - cerr: 0, total_len: 0 > > [16:40:24.8] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:24.8] - cerr: 3, total_len: 0 > > [16:40:24.8] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:24.9] - cerr: 3, total_len: 0 > > > [16:40:24.9] moving from 148260 to 1482a0 > [16:40:24.9] wait_for_tds: td at phys(1f6b22a0): status: 0 token: d00 > > [16:40:25.0] - cerr: 3, total_len: 0 > > [16:40:25.0] wait_for_tds done: td at phys(1f6b22a0): status: 0 token: d00 > > [16:40:25.0] - cerr: 3, total_len: 0 > > > [16:40:25.1] moving from 1482a0 to 1482e0 > [16:40:25.1] wait_for_tds: td at phys(1f6b22e0): status: 0 token: 1000 > > [16:40:25.2] - cerr: 0, total_len: 0 > > [16:40:25.2] maxPacketSize0: 40 > [16:40:25.2] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:25.2] - cerr: 3, total_len: 0 > > [16:40:25.3] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:25.3] - cerr: 3, total_len: 0 > > > [16:40:25.4] moving from 148260 to 1482a0 > [16:40:25.4] wait_for_tds: td at phys(1f6b22a0): status: 0 token: d00 > > [16:40:25.5] - cerr: 3, total_len: 0 > > [16:40:25.5] wait_for_tds done: td at phys(1f6b22a0): status: 0 token: d00 > > [16:40:25.5] - cerr: 3, total_len: 0 > > > [16:40:25.6] moving from 1482a0 to 1482e0 > [16:40:25.6] wait_for_tds: td at phys(1f6b22e0): status: 0 token: 1000 > > [16:40:25.7] - cerr: 0, total_len: 0 > > [16:40:25.7] * found device (0x1370:0x3252, USB 2.0) > [16:40:25.7] device has 1 configurations > [16:40:25.7] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:25.8] - cerr: 3, total_len: 0 > > [16:40:25.8] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:25.9] - cerr: 3, total_len: 0 > > > [16:40:25.9] moving from 148260 to 1482a0 > [16:40:26.0] wait_for_tds: td at phys(1f6b22a0): status: 0 token: d00 > > [16:40:26.0] - cerr: 3, total_len: 0 > > [16:40:26.0] wait_for_tds done: td at phys(1f6b22a0): status: 0 token: d00 > > [16:40:26.1] - cerr: 3, total_len: 0 > > > [16:40:26.1] moving from 1482a0 to 1482e0 > [16:40:26.1] wait_for_tds: td at phys(1f6b22e0): status: 0 token: 1000 > > [16:40:26.2] - cerr: 0, total_len: 0 > > [16:40:26.2] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:26.3] - cerr: 3, total_len: 0 > > [16:40:26.3] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:26.4] - cerr: 3, total_len: 0 > > > [16:40:26.4] moving from 148260 to 1482a0 > [16:40:26.4] wait_for_tds: td at phys(1f6b22a0): status: 0 token: d00 > > [16:40:26.5] - cerr: 3, total_len: 0 > > [16:40:26.5] wait_for_tds done: td at phys(1f6b22a0): status: 0 token: d00 > > [16:40:26.6] - cerr: 3, total_len: 0 > > > [16:40:26.6] moving from 1482a0 to 1482e0 > [16:40:26.7] wait_for_tds: td at phys(1f6b22e0): status: 0 token: 1000 > > [16:40:26.7] - cerr: 0, total_len: 0 > > [16:40:26.7] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:26.8] - cerr: 3, total_len: 0 > > [16:40:26.8] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 > > [16:40:26.8] - cerr: 3, total_len: 0 > > > [16:40:26.9] moving from 148260 to 1482a0 > [16:40:26.9] wait_for_tds: td at phys(1f6b22a0): status: 0 token: 100 > > [16:40:27.0] - cerr: 0, total_len: 0 > > [16:40:27.0] device has 1 interfaces > [16:40:27.0] #0 has 2 endpoints, interface 8:6, protocol 50 > [16:40:27.0] #1: Endpoint 1 (in), max packet size 200, type bulk > [16:40:27.1] #2: Endpoint 2 (out), max packet size 200, type bulk > [16:40:27.1] , class: MSC > [16:40:27.1] Waiting for device to become ready...ehci_bulk: setup: > td at phys(148260): status: 0 token: 0 > > [16:40:27.2] - cerr: 0, total_len: 0 > > [16:40:27.2] wait_for_tds: td at phys(1f6b2260): status: 1 token: 80000001 > > [16:40:27.3] - cerr: 0, total_len: 0 > > [16:40:27.3] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 > > [16:40:27.3] - cerr: 0, total_len: 0 > > [16:40:27.4] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000100 > > [16:40:27.4] - cerr: 0, total_len: 0 > > [16:40:27.5] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 > > [16:40:27.5] - cerr: 0, total_len: 0 > > [16:40:27.5] wait_for_tds: td at phys(1f6b2260): status: 1 token: 1 > > [16:40:27.6] - cerr: 0, total_len: 0 > > [16:40:27.6] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 > > [16:40:27.7] - cerr: 0, total_len: 0 > > [16:40:27.7] wait_for_tds: td at phys(1f6b2260): status: 0 token: 100 > > [16:40:27.7] - cerr: 0, total_len: 0 > > [16:40:27.8] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 > > [16:40:27.8] - cerr: 0, total_len: 0 > > [16:40:27.9] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000100 > > [16:40:27.9] - cerr: 0, total_len: 0 > > [16:40:28.1] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 > > [16:40:28.1] - cerr: 0, total_len: 0 > > [16:40:28.2] wait_for_tds: td at phys(1f6b2260): status: 1 token: 80000001 > > [16:40:28.2] - cerr: 0, total_len: 0 > > [16:40:28.2] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 > > [16:40:28.3] - cerr: 0, total_len: 0 > > [16:40:28.3] wait_for_tds: td at phys(1f6b2260): status: 0 token: 100 > > [16:40:28.3] - cerr: 0, total_len: 0 > > [16:40:28.4] ok. > [16:40:28.4] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 > > [16:40:28.4] - cerr: 0, total_len: 0 > > [16:40:28.5] wait_for_tds: td at phys(1f6b2260): status: 1 token: 1 > > [16:40:28.5] - cerr: 0, total_len: 0 > > [16:40:28.6] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 > > [16:40:28.6] - cerr: 0, total_len: 0 > > [16:40:28.6] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000100 > > [16:40:28.7] - cerr: 0, total_len: 0 > > [16:40:28.7] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 > > [16:40:28.8] - cerr: 0, total_len: 0 > > [16:40:28.8] wait_for_tds: td at phys(1f6b2260): status: 1 token: 80000001 > > [16:40:28.9] - cerr: 0, total_len: 0 > > [16:40:28.9] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 > > [16:40:28.9] - cerr: 0, total_len: 0 > > [16:40:29.0] wait_for_tds: td at phys(1f6b2260): status: 0 token: 100 > > [16:40:29.0] - cerr: 0, total_len: 0 > > [16:40:29.1] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 > > [16:40:29.1] - cerr: 0, total_len: 0 > > [16:40:29.2] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000100 > > [16:40:29.2] - cerr: 0, total_len: 0 > > [16:40:29.2] 15663104 512-byte sectors (8019 MB) > [16:40:29.3] 00:0f.4 2094:1022.4 OHCI controller > [16:40:29.3] OHCI Version 1.0 > [16:40:29.4] ERROR: No such CMOS option (boot_devices) > [16:40:29.4] menu: hda3:/boot/filo/menu.lst > Kevin do you have an idea what could be wrong? --- Christian Gmeiner, MSc

1 0

Re: [SeaBIOS] Help needed with ASSERT32FLAT()
by Kevin O'Connor 13 Oct '12

13 Oct '12

On Fri, Oct 12, 2012 at 12:43:02PM +0200, Christian Gmeiner wrote: > During my usb debug session I want to malloc ehci usb bulk tds like > for control messages, > but I can not compile it... here is an example: The usb_send_bulk() function is called in both 32bit mode and 16bit mode. The ASSERT32FLAT() macro is a compile-time check that verifies the given code is only ever called in 32bit mode. So, stopping the compile is exactly what it is supposed to do. One can't call malloc from 16bit mode (or indeed any time after the "post" phase completes). That's why the existing bulk transfer code has all the weird stack allocation stuff. -Kevin

2 1

Re: [SeaBIOS] [RFC PATCH v3 06/19] Implement "-dimm" command line option
by Blue Swirl 13 Oct '12

13 Oct '12

On Tue, Oct 9, 2012 at 5:04 PM, Vasilis Liaskovitis <vasilis.liaskovitis(a)profitbricks.com> wrote: > Hi, > > sorry for the delayed answer. > > On Sat, Sep 29, 2012 at 11:13:04AM +0000, Blue Swirl wrote: >> > >> > The "-dimm" option is supposed to specify the dimm/memory layout, and not create >> > any devices. >> > >> > If we don't want this new option, I have a question: >> > >> > A "-device/device_add" means we create a new qdev device at startup or as a >> > hotplug operation respectively. So, the semantics of >> > "-device dimm,id=dimm0,size=512M,node=0,populated=on" are clear to me. >> > >> > What does "-device dimm,populated=off" mean from a qdev perspective? There are 2 >> > alternatives: >> > >> > - The device is created on the dimmbus, but is not used/populated yet. Than the >> > activation/acpi-hotplug of the dimm may require a separate command (we used to have >> > "dimm_add" in versions < 3). "device_add" handling always hotplugs a new qdev >> > device, so this wouldn't fit this usecase, because the device already exists. In >> > this case, the actual "acpi hotplug" operation is decoupled from qdev device >> > creation. >> >> The bus exists but the devices do not, device_add would add DIMMs to >> the bus. This matches PCI bus created by the host bridge and PCI >> device hotplug. >> >> A more complex setup would be dimm bus, dimm slot devices and DIMM >> devices. The intermediate slot device would contain one DIMM device if >> plugged. > > interesting, I haven't thought about this alternative. It does sounds overly > complex, but a dimmslot / dimmdevice splitup could consolidate hotplug semantic > differences between populated=on/off. Something similar to the dimmslot device > is already present in v3 (dimmcfg structure), but it's not a qdev visible device. > I 'd rather avoid the complication, but i might revisit this idea. The memory controller could be able to also enable and disable slots independently to their population state. > >> >> > >> > - The dimmdevice is not created when "-device dimm,populated=off" (this would >> > require some ugly checking in normal -device argument handling). Only the dimm >> > layout is saved. The hotplug is triggered from a normal device_add later. So in >> > this case, the "acpi hotplug" happens at the same time as the qdev hotplug. >> > >> > Do you see a simpler alternative without introducing a new option? >> > >> > Using the "-dimm" option follows the second semantic and avoids changing the "-device" >> > semantics. Dimm layout description is decoupled from dimmdevice creation, and qdev >> > hotplug coincides with acpi hotplug. >> >> Maybe even the dimmbus device shouldn't exist by itself after all, or >> it should be pretty much invisible to users. On real HW, the memory >> controller or south bridge handles the memory. For i440fx, it's part >> of the same chipset. So I think we should just add qdev properties to >> i440fx to specify the sizes, nodes etc. Then i440fx should create the >> dimmbus device unconditionally using the properties. The default >> properties should create a sane configuration, otherwise -global >> i440fx.dimm_size=512M etc. could be used. Then the bus would be >> populated as before or with device_add. > > hmm the problem with using only i440fx properties, is that size/nodes look > dimm specific to me, not chipset-memcontroller specific. Unless we only allow > uniform size dimms. Is it possible to have a dynamic list of sizes/nodes pairs as > properties of a qdev device? I don't think so, but probably there's a limit of DIMMs that real controllers have, something like 8 max. > > Also if there is no dimmbus, and instead we have only links<> from i440fx to dimm-devices, > would the current qdev hotplug API be enough? I'd just disable hotplug if there is no dimmbus (ISA PC?). > > I am currently leaning towards this: i440fx unconditionally creates the dimmbus. Users > don't have to specify the bus (i assume this is what you mean by "dimmbus should > be invisible to the users") > > We only use "-device dimm" to describe dimms. With "-device dimm,populated=off", only > the dimm config layout will be saved in the dimmbus. The hotplug is triggered from a normal > device_add later (same as pci hotplug). OK. > > thanks, > > - Vasilis

1 0

Help needed with ASSERT32FLAT()
by Christian Gmeiner 12 Oct '12

12 Oct '12

During my usb debug session I want to malloc ehci usb bulk tds like for control messages, but I can not compile it... here is an example: http://dpaste.com/812830/ Hope somebody could help me -- Christian Gmeiner, MSc

1 0

Re: [SeaBIOS] USB Problems geode lx800
by Christian Gmeiner 11 Oct '12

11 Oct '12

Christian Gmeiner, MSc 2012/10/11 Christian Gmeiner <christian.gmeiner(a)gmail.com>: > 2012/10/11 Kevin O'Connor <kevin(a)koconnor.net>: >> On Tue, Oct 09, 2012 at 02:31:05PM +0200, Christian Gmeiner wrote: >>> 2012/10/9 Kevin O'Connor <kevin(a)koconnor.net>: >>> > On Mon, Oct 08, 2012 at 02:14:03PM +0200, Christian Gmeiner wrote: >>> >> HI all >>> >> >>> >> I am running into some usb problems with coreboot & seabios: >>> > Can you set the debug level to 8 and post the whole log? Also, for >>> > timeout issues, having timestamps (via tools/readserial.py tool) >>> > sometimes helps. >>> >>> Attached >> >> Hrmm, I guess you've disabled threads in your config? >> > yep... > > >> From your log, I see: >> >> [14:23:00.5] ehci_send_bulk qh=0x000ef680 dir=0 data=0x00006d55 size=31^M >> [14:23:00.5] ehci_send_bulk qh=0x000ef700 dir=128 data=0x00006e3c size=36^M >> [14:23:00.5] ehci_send_bulk qh=0x000ef700 dir=128 data=0x00006d8f size=13^M >> [14:23:05.5] WARNING - Timeout at ehci_wait_td:542!^M >> [14:23:05.5] ehci pipe=0x000ef700 cur=00000000 tok=00000000 next=6b80 td=0x00006b80 >> +status=d0d80^M >> [14:23:05.5] USB transmission failed^M >> [14:23:05.5] Unable to configure USB MSC drive.^M >> >> This looks like the transfer is stalling in the cdb_get_inquiry() call >> as part of scsi_init_drive(). The drive is identified okay and at >> least some transfers look okay, so it doesn't appear to be a >> controller issue. >> >> I've been told some USB sticks are very picky about init order, but >> I'm surprised to see that this is occuring for you on a variety of USB >> sticks. >> >> It's possible to verify that a stall has occurred - print out >> tds[i].token for all tds in ehci_send_bulk() on the error path. >> >> Does this device still have problems if you soft-reset after a failed >> boot? >> > > Here are the results with some more debug: > > Cold Reboot > ------------ > > |0f7a5000| pmm_malloc zone=0x0f7afe87 handle=ffffffff size=192 > align=40 ret=0x0f7a7bc0 (detail=0x0f7a7d70) > |0f7a5000| ehci_control: td at phys(f7a7bc0): status: 80 : token 80e80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_control: td at phys(f7a7c00): status: 0 : token d00 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_control: td at phys(f7a7c40): status: 0 : token 0 > |0f7a5000| - cerr: 0, total_len: 0 > |0f7a5000| pmm_free 0x0f7a7bc0 (detail=0x0f7a7d70) > |0f7a5000| ehci_alloc_async_pipe 0x0f7a9090 2 > |0f7a5000| pmm_malloc zone=0x0f7afe7f handle=ffffffff size=92 align=80 > ret=0x000ef700 (detail=0x0f7a7d70) > |0f7a5000| ehci_alloc_async_pipe 0x0f7a9090 2 > |0f7a5000| pmm_malloc zone=0x0f7afe7f handle=ffffffff size=92 align=80 > ret=0x000ef680 (detail=0x0f7a7d10) > |0f7a5000| ehci_control 0x0f7a7cd0 (dir=128 cmd=8 data=1) > |0f7a5000| pmm_malloc zone=0x0f7afe87 handle=ffffffff size=192 > align=40 ret=0x0f7a7bc0 (detail=0x0f7a7ce0) > |0f7a5000| ehci_control: td at phys(f7a7bc0): status: 80 : token 80e80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_control: td at phys(f7a7c00): status: 0 : token d00 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_control: td at phys(f7a7c40): status: 0 : token 1c00 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| pmm_free 0x0f7a7bc0 (detail=0x0f7a7ce0) > |0f7a5000| pmm_malloc zone=0x0f7afe83 handle=ffffffff size=48 align=10 > ret=0x000fdae0 (detail=0x0f7a7ce0) > |0f7a5000| usb_cmd_data id=0x000fdae0 write=0 count=1 bs=36 buf=0x0f7a5f3c > |0f7a5000| cbw to device > |0f7a5000| ehci_send_bulk qh=0x000ef680 dir=0 data=0x0f7a5e65 size=31 > |0f7a5000| before setup: td at phys(f7a5c80): status: 0 : token 0 > |0f7a5000| - cerr: 0, total_len: 0 > |0f7a5000| ehci_send_bulk: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| Data-In from the device to the host - 36 bytes > |0f7a5000| ehci_send_bulk qh=0x000ef700 dir=128 data=0x0f7a5f3c size=36 > |0f7a5000| before setup: td at phys(f7a5c80): status: 0 : token 0 > |0f7a5000| - cerr: 0, total_len: 0 > |0f7a5000| ehci_send_bulk: td at phys(f7a5c80): status: 80 : token d80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token d80 > |0f7a5000| - cerr: 3, total_len: 0 > > > Warm Reboot > ----------- > > 0f7a5000| pmm_free 0x0f7a7bc0 (detail=0x0f7a7d70) > |0f7a5000| ehci_alloc_async_pipe 0x0f7a9090 2 > |0f7a5000| pmm_malloc zone=0x0f7afe7f handle=ffffffff size=92 align=80 > ret=0x000ef700 (detail=0x0f7a7d70) > |0f7a5000| ehci_alloc_async_pipe 0x0f7a9090 2 > |0f7a5000| pmm_malloc zone=0x0f7afe7f handle=ffffffff size=92 align=80 > ret=0x000ef680 (detail=0x0f7a7d10) > |0f7a5000| ehci_control 0x0f7a7cd0 (dir=128 cmd=8 data=1) > |0f7a5000| pmm_malloc zone=0x0f7afe87 handle=ffffffff size=192 > align=40 ret=0x0f7a7bc0 (detail=0x0f7a7ce0) > |0f7a5000| ehci_control: td at phys(f7a7bc0): status: 80 : token 80e80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_control: td at phys(f7a7c00): status: 0 : token d00 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_control: td at phys(f7a7c40): status: 0 : token 1c00 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| pmm_free 0x0f7a7bc0 (detail=0x0f7a7ce0) > |0f7a5000| pmm_malloc zone=0x0f7afe83 handle=ffffffff size=48 align=10 > ret=0x000fdae0 (detail=0x0f7a7ce0) > |0f7a5000| usb_cmd_data id=0x000fdae0 write=0 count=1 bs=36 buf=0x0f7a5f3c > |0f7a5000| cbw to device > |0f7a5000| ehci_send_bulk qh=0x000ef680 dir=0 data=0x0f7a5e65 size=31 > |0f7a5000| before setup: td at phys(f7a5c80): status: 0 : token 0 > |0f7a5000| - cerr: 0, total_len: 0 > |0f7a5000| ehci_send_bulk: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > |0f7a5000| ehci_wait_td: td at phys(f7a5c80): status: 80 : token c80 > |0f7a5000| - cerr: 3, total_len: 0 > > > This is the used patch: > > diff --git a/src/usb-ehci.c b/src/usb-ehci.c > index 2676615..92a9c60 100644 > --- a/src/usb-ehci.c > +++ b/src/usb-ehci.c > @@ -178,6 +178,14 @@ check_ehci_ports(struct usb_ehci_s *cntl) > * Setup > ****************************************************************/ > > +static void dump_td(struct ehci_qtd *td, const char *func) > +{ > + u32 addr = (u32)td; > + dprintf(1, "%s: td at phys(%x): status: %x : token %x\n", func, > addr, td->token & QTD_STS_MASK, td->token); > + dprintf(1, "- cerr: %x, total_len: %x\n", (td->token & > QTD_CERR_MASK) >> QTD_CERR_SHIFT, > + (td->token & QTD_LENGTH_MASK) >> QTD_LENGTH_SHIFT); > +} > + > // Wait for next USB async frame to start - for ensuring safe memory release. > static void > ehci_waittick(struct usb_ehci_s *cntl) > @@ -535,6 +543,9 @@ ehci_wait_td(struct ehci_pipe *pipe, struct > ehci_qtd *td, int timeout) > status = td->token; > if (!(status & QTD_STS_ACTIVE)) > break; > + > + dump_td(td, __func__); > + > if (check_tsc(end)) { > u32 cur = GET_LOWFLAT(pipe->qh.current); > u32 tok = GET_LOWFLAT(pipe->qh.token); > @@ -633,6 +644,7 @@ ehci_control(struct usb_pipe *p, int dir, const > void *cmd, int cmdsize > int i, ret=0; > for (i=0; i<3; i++) { > struct ehci_qtd *td = &tds[i]; > + dump_td(td, __func__); > ret = ehci_wait_td(pipe, td, 500); > if (ret) > break; > @@ -665,18 +677,25 @@ ehci_send_bulk(struct usb_pipe *p, int dir, void > *data, int datasize) > struct ehci_qtd *td = &tds[tdpos++ % STACKQTDS]; > int ret = ehci_wait_td(pipe, td, 5000); > if (ret) > + { > + dump_td(td, "ehci_send_bulk ERROR2"); > return -1; > + } > > struct ehci_qtd *nexttd_fl = MAKE_FLATPTR(GET_SEG(SS) > , &tds[tdpos % STACKQTDS]); > > + dump_td(td, "before setup"); > + > int transfer = fillTDbuffer(td, maxpacket, data, datasize); > td->qtd_next = (transfer==datasize ? EHCI_PTR_TERM : (u32)nexttd_fl); > td->alt_next = EHCI_PTR_TERM; > barrier(); > - td->token = (ehci_explen(transfer) | QTD_STS_ACTIVE > + td->token = (/*ehci_explen(transfer) | */QTD_STS_ACTIVE > | (dir ? QTD_PID_IN : QTD_PID_OUT) | ehci_maxerr(3)); > > + dump_td(td, __func__); > + > data += transfer; > datasize -= transfer; > } > @@ -685,7 +704,10 @@ ehci_send_bulk(struct usb_pipe *p, int dir, void > *data, int datasize) > struct ehci_qtd *td = &tds[tdpos++ % STACKQTDS]; > int ret = ehci_wait_td(pipe, td, 5000); > if (ret) > + { > + dump_td(td, "ehci_send_bulk ERROR2"); > return -1; > + } > } > > return 0; > > > > greets Here is the log of filo using libpayload [16:40:24.5] FILO version 0.6.0 (christian@chgm-pc) Thu Oct 11 16:12:26 CEST 2012 [16:40:24.5] 00:0f.5 2095:1022.5 EHCI controller [16:40:24.7] highspeed device [16:40:24.7] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:24.8] - cerr: 3, total_len: 0 [16:40:24.8] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:24.8] - cerr: 3, total_len: 0 [16:40:24.8] moving from 148260 to 1482a0 [16:40:24.8] wait_for_tds: td at phys(1f6b22a0): status: 0 token: 100 [16:40:24.8] - cerr: 0, total_len: 0 [16:40:24.8] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:24.8] - cerr: 3, total_len: 0 [16:40:24.8] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:24.9] - cerr: 3, total_len: 0 [16:40:24.9] moving from 148260 to 1482a0 [16:40:24.9] wait_for_tds: td at phys(1f6b22a0): status: 0 token: d00 [16:40:25.0] - cerr: 3, total_len: 0 [16:40:25.0] wait_for_tds done: td at phys(1f6b22a0): status: 0 token: d00 [16:40:25.0] - cerr: 3, total_len: 0 [16:40:25.1] moving from 1482a0 to 1482e0 [16:40:25.1] wait_for_tds: td at phys(1f6b22e0): status: 0 token: 1000 [16:40:25.2] - cerr: 0, total_len: 0 [16:40:25.2] maxPacketSize0: 40 [16:40:25.2] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:25.2] - cerr: 3, total_len: 0 [16:40:25.3] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:25.3] - cerr: 3, total_len: 0 [16:40:25.4] moving from 148260 to 1482a0 [16:40:25.4] wait_for_tds: td at phys(1f6b22a0): status: 0 token: d00 [16:40:25.5] - cerr: 3, total_len: 0 [16:40:25.5] wait_for_tds done: td at phys(1f6b22a0): status: 0 token: d00 [16:40:25.5] - cerr: 3, total_len: 0 [16:40:25.6] moving from 1482a0 to 1482e0 [16:40:25.6] wait_for_tds: td at phys(1f6b22e0): status: 0 token: 1000 [16:40:25.7] - cerr: 0, total_len: 0 [16:40:25.7] * found device (0x1370:0x3252, USB 2.0) [16:40:25.7] device has 1 configurations [16:40:25.7] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:25.8] - cerr: 3, total_len: 0 [16:40:25.8] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:25.9] - cerr: 3, total_len: 0 [16:40:25.9] moving from 148260 to 1482a0 [16:40:26.0] wait_for_tds: td at phys(1f6b22a0): status: 0 token: d00 [16:40:26.0] - cerr: 3, total_len: 0 [16:40:26.0] wait_for_tds done: td at phys(1f6b22a0): status: 0 token: d00 [16:40:26.1] - cerr: 3, total_len: 0 [16:40:26.1] moving from 1482a0 to 1482e0 [16:40:26.1] wait_for_tds: td at phys(1f6b22e0): status: 0 token: 1000 [16:40:26.2] - cerr: 0, total_len: 0 [16:40:26.2] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:26.3] - cerr: 3, total_len: 0 [16:40:26.3] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:26.4] - cerr: 3, total_len: 0 [16:40:26.4] moving from 148260 to 1482a0 [16:40:26.4] wait_for_tds: td at phys(1f6b22a0): status: 0 token: d00 [16:40:26.5] - cerr: 3, total_len: 0 [16:40:26.5] wait_for_tds done: td at phys(1f6b22a0): status: 0 token: d00 [16:40:26.6] - cerr: 3, total_len: 0 [16:40:26.6] moving from 1482a0 to 1482e0 [16:40:26.7] wait_for_tds: td at phys(1f6b22e0): status: 0 token: 1000 [16:40:26.7] - cerr: 0, total_len: 0 [16:40:26.7] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:26.8] - cerr: 3, total_len: 0 [16:40:26.8] wait_for_tds done: td at phys(1f6b2260): status: 0 token: 80000e00 [16:40:26.8] - cerr: 3, total_len: 0 [16:40:26.9] moving from 148260 to 1482a0 [16:40:26.9] wait_for_tds: td at phys(1f6b22a0): status: 0 token: 100 [16:40:27.0] - cerr: 0, total_len: 0 [16:40:27.0] device has 1 interfaces [16:40:27.0] #0 has 2 endpoints, interface 8:6, protocol 50 [16:40:27.0] #1: Endpoint 1 (in), max packet size 200, type bulk [16:40:27.1] #2: Endpoint 2 (out), max packet size 200, type bulk [16:40:27.1] , class: MSC [16:40:27.1] Waiting for device to become ready...ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:27.2] - cerr: 0, total_len: 0 [16:40:27.2] wait_for_tds: td at phys(1f6b2260): status: 1 token: 80000001 [16:40:27.3] - cerr: 0, total_len: 0 [16:40:27.3] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:27.3] - cerr: 0, total_len: 0 [16:40:27.4] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000100 [16:40:27.4] - cerr: 0, total_len: 0 [16:40:27.5] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:27.5] - cerr: 0, total_len: 0 [16:40:27.5] wait_for_tds: td at phys(1f6b2260): status: 1 token: 1 [16:40:27.6] - cerr: 0, total_len: 0 [16:40:27.6] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:27.7] - cerr: 0, total_len: 0 [16:40:27.7] wait_for_tds: td at phys(1f6b2260): status: 0 token: 100 [16:40:27.7] - cerr: 0, total_len: 0 [16:40:27.8] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:27.8] - cerr: 0, total_len: 0 [16:40:27.9] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000100 [16:40:27.9] - cerr: 0, total_len: 0 [16:40:28.1] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:28.1] - cerr: 0, total_len: 0 [16:40:28.2] wait_for_tds: td at phys(1f6b2260): status: 1 token: 80000001 [16:40:28.2] - cerr: 0, total_len: 0 [16:40:28.2] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:28.3] - cerr: 0, total_len: 0 [16:40:28.3] wait_for_tds: td at phys(1f6b2260): status: 0 token: 100 [16:40:28.3] - cerr: 0, total_len: 0 [16:40:28.4] ok. [16:40:28.4] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:28.4] - cerr: 0, total_len: 0 [16:40:28.5] wait_for_tds: td at phys(1f6b2260): status: 1 token: 1 [16:40:28.5] - cerr: 0, total_len: 0 [16:40:28.6] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:28.6] - cerr: 0, total_len: 0 [16:40:28.6] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000100 [16:40:28.7] - cerr: 0, total_len: 0 [16:40:28.7] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:28.8] - cerr: 0, total_len: 0 [16:40:28.8] wait_for_tds: td at phys(1f6b2260): status: 1 token: 80000001 [16:40:28.9] - cerr: 0, total_len: 0 [16:40:28.9] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:28.9] - cerr: 0, total_len: 0 [16:40:29.0] wait_for_tds: td at phys(1f6b2260): status: 0 token: 100 [16:40:29.0] - cerr: 0, total_len: 0 [16:40:29.1] ehci_bulk: setup: td at phys(148260): status: 0 token: 0 [16:40:29.1] - cerr: 0, total_len: 0 [16:40:29.2] wait_for_tds: td at phys(1f6b2260): status: 0 token: 80000100 [16:40:29.2] - cerr: 0, total_len: 0 [16:40:29.2] 15663104 512-byte sectors (8019 MB) [16:40:29.3] 00:0f.4 2094:1022.4 OHCI controller [16:40:29.3] OHCI Version 1.0 [16:40:29.4] ERROR: No such CMOS option (boot_devices) [16:40:29.4] menu: hda3:/boot/filo/menu.lst --- Christian Gmeiner, MSc

1 0

Re: [SeaBIOS] USB Problems geode lx800
by Kevin O'Connor 11 Oct '12

11 Oct '12

On Tue, Oct 09, 2012 at 02:31:05PM +0200, Christian Gmeiner wrote: > 2012/10/9 Kevin O'Connor <kevin(a)koconnor.net>: > > On Mon, Oct 08, 2012 at 02:14:03PM +0200, Christian Gmeiner wrote: > >> HI all > >> > >> I am running into some usb problems with coreboot & seabios: > > Can you set the debug level to 8 and post the whole log? Also, for > > timeout issues, having timestamps (via tools/readserial.py tool) > > sometimes helps. > > Attached Hrmm, I guess you've disabled threads in your config? >From your log, I see: [14:23:00.5] ehci_send_bulk qh=0x000ef680 dir=0 data=0x00006d55 size=31^M [14:23:00.5] ehci_send_bulk qh=0x000ef700 dir=128 data=0x00006e3c size=36^M [14:23:00.5] ehci_send_bulk qh=0x000ef700 dir=128 data=0x00006d8f size=13^M [14:23:05.5] WARNING - Timeout at ehci_wait_td:542!^M [14:23:05.5] ehci pipe=0x000ef700 cur=00000000 tok=00000000 next=6b80 td=0x00006b80 +status=d0d80^M [14:23:05.5] USB transmission failed^M [14:23:05.5] Unable to configure USB MSC drive.^M This looks like the transfer is stalling in the cdb_get_inquiry() call as part of scsi_init_drive(). The drive is identified okay and at least some transfers look okay, so it doesn't appear to be a controller issue. I've been told some USB sticks are very picky about init order, but I'm surprised to see that this is occuring for you on a variety of USB sticks. It's possible to verify that a stall has occurred - print out tds[i].token for all tds in ehci_send_bulk() on the error path. Does this device still have problems if you soft-reset after a failed boot? -Kevin

2 1

← Newer
1
2
3
4
5
6
7
8
9
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SeaBIOS October 2012