On Tue, 2008-10-14 at 17:21 +0200, Johannes Berg wrote: > On Mon, 2008-10-06 at 18:10 +0200, Johannes Berg wrote: > > This patch cleans up the DMA code to be understandable and not > > completely wrong. In particular: > > * there is no need to have a weird iwl_tfd_frame_data struct that is > > used 10 times, just use an address struct 20 times > > * therefore, all the is_odd junk goes away > > * fix a bug in iwl_hcmd_queue_reclaim where it would reclaim all the > > fragments of a descriptor rather than all descriptors (this may be > > the cause of the dma unmapping problem I reported) > > * some more cleanups > > > > Signed-off-by: Johannes Berg <johannes@xxxxxxxxxxxxxxxx> > > --- > > Tested on 5000 hw, please apply. > > > > v2: fixes small issue with getting rid of iwl_get_dma_hi_address. > > John, can you apply this? I keep getting invalid frees, or double-frees > without this patch. Tomas reworked this patch. I will send the new patch upstream soon. Reinette -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html