From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Jan Beulich" <jbeulich@novell.com>
Date: Tue, 19 Dec 2006 11:10:59 +0000
Subject: swiotlb_sync_sg() issue?
Message-Id: <4587D6D3.76E4.0078.0@novell.com>
List-Id: <linux-ia64.vger.kernel.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: linux-ia64@vger.kernel.org

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()?

Thanks, Jan