From: Marcelo B. <mo...@mo...> - 2005-09-10 01:22:01
|
Ok, now I have logs with debug=0x4020 and the proposed changes. They are here: http://mosca.yi.org/~mosca/acx.log Or, hopefully, attached here. On Fri, Sep 09, 2005 at 09:33:29AM +0300, Denis Vlasenko wrote: > > Ah, yes. txdescs really are look botched. We see one bogus 0x00 byte, > but there may be others! We need to see that. > > Add this after struct txdesc definition in acx_struct.h: > > static inline void dump_txdesc111(txdesc_t* d) { > printk( "txdesc @%p: " > "pNextDesc=%X " > "HostMemPtr=%X " > "AcxMemPtr=%X " > "tx_time=%X " > "total_length=%X " > "Reserved=%X " > "dummy[16]=%X,%X,%X,%X " > "Ctl_8=%X " > "Ctl2_8=%X " > "error=%X " > "ack_failures=%X " > "rts_failures=%X " > "rts_ok=%X " > "u.r2.rate111=%X " > "queue_info=%X " > "acx111_tail=%X " > "\n", > d, > acx2cpu(d->pNextDesc), > acx2cpu(d->HostMemPtr), > acx2cpu(d->AcxMemPtr), > d->tx_time, > d->total_length, > d->Reserved, > d->fixed_size.dummy[0],d->fixed_size.dummy[1],d->fixed_size.dummy[2],d->fixed_size.dummy[3], > d->Ctl_8, > d->Ctl2_8, > d->error, > d->ack_failures, > d->rts_failures, > d->rts_ok, > d->u.r2.rate111, > d->queue_info, > *(u32*)(d + 1) /* acx111 only */ > ); > } > > and insert calls to it in interesting places: in acxpci_l_get_txbuf, > in acx_l_clean_tx_desc (two places: after "pTxDesc = GET_TX_DESC_PTR(priv, finger)" > and right after "next:" label). Etc. > > Let's see where it start to look wrong. > > Thanks! > -- > vda > |