>>> "Jan Beulich" <jbeulich@xxxxxxxxxx> 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@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
|