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?