By the way... > +int ipath_user_sdma_pkt_sent(const struct ipath_user_sdma_queue *pq, > + u32 counter) > +{ > + const u32 scounter = ipath_user_sdma_complete_counter(pq); > + const s32 dcounter = scounter - counter; > + > + return dcounter >= 0; > +} I don't see this called anywhere... should I just delete it?