Re: [discuss] swiotlb_sync_sg() issue?

From: Jan Beulich <jbeulich_at_novell.com>
Date: 2006-12-20 00:26:47
>>> "Jan Beulich" <jbeulich@novell.com> 19.12.06 12:10 >>>
>Due to its apparent obviousness, I'm afraid I'm missing something, but isn't it
>incorrect that lib/swiotlb.c:swiotlb_sync_sg() calls sync_single() with
>sg->dma_address simply cast to void* rather than going through phys_to_virt()?
>
>Also, for ia64, isn't mark_clean() violating its own comment, i.e. to mark only
>*complete* pages? And then, for completeness, isn't swiotlb_sync_sg() missing
>the call to mark_clean()?

Okay, I see I mis-read mark_clean(), it's not violating what it claims to do. But
the other two questions remain.

Thanks, Jan
-
To unsubscribe from this list: send the line "unsubscribe linux-ia64" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Received on Wed Dec 20 00:26:25 2006

This archive was generated by hypermail 2.1.8 : 2006-12-20 00:26:46 EST