On Mon, 2008-10-06 at 18:51 +0200, Johannes Berg wrote: > > > * 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. > > > > Great job, however do not apply this before I review it I had strong > > feeling this will not > > work with aggregation flows. > > I cannot imagine why you think that, care to explain? Of course, I would very much appreciate you review the actual bug fix in iwl_hcmd_queue_reclaim, which consist of the addition of the line + bd = &txq->bd[index]; johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
