On Fri, 2006-06-23 at 15:48 +0200, Arjan van de Ven wrote: > > > > + /* Tell HW to xmit */ > > > > + __raw_writeq(cpu_to_be64(mapaddr), elem->hw_desc + C2_TXP_ADDR); > > > > + __raw_writew(cpu_to_be16(maplen), elem->hw_desc + C2_TXP_LEN); > > > > + __raw_writew(cpu_to_be16(TXP_HTXD_READY), elem->hw_desc + > > > > C2_TXP_FLAGS); > > > > > > or here > > > > > > > No need here. This logic submits the packet for transmission. We don't > > assume it is transmitted until we (after a completion interrupt usually) > > read back the HTXD entry and see the TXP_HTXD_DONE bit set (see > > c2_tx_interrupt()). > > ... but will that interrupt happen at all if these 3 writes never hit > the hardware? >
I thought the posted write WILL eventually get to adapter memory. Not stall forever cached in a bridge. I'm wrong? My point is for a given HTXD entry, we write it to post a packet for transmission, then only free the packet memory and reuse this entry _after_ reading the HTXD and seeing the DONE bit set. So I still don't see a problem. But I've been wrong before ;-) Steve. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html