Messages in this thread | | | Date | Mon, 5 Apr 2010 11:12:54 +0900 | Subject | Re: [PATCH] swiotlb v0.6: seperation of physical/virtual address translation | From | FUJITA Tomonori <> |
| |
On Fri, 19 Mar 2010 11:04:17 -0400 Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> wrote:
> Fujita-san et al. > > Attached is a set of patches that separate the address translation > (virt_to_phys, virt_to_bus, etc) from the SWIOTLB library. > > Since the last posting I've: > - Made the exported functions/variables have the 'swiotlb_bk' prefix instead > of the 'do_[map|unmap]*' and 'io_tlb_*' combination.
Why can't we use more simpler names such as 'swiotlb_tbl_index'?
Why do we need to add the prefix to static things like 'swiotlb_bk_list', 'swiotlb_bk_index', etc? Please let them alone.
> - dropped the checkpatches/other reworks patches. > > I had not addressed the question of removing the 'overflow' buffer. There are over > ~300 instances of the the DMA operations not being checked which plan on addressing > in a seperate set of patches that will slowly roll out the checks and then > finally the removal of the 'overflow' buffer.
Except for swiotlb, no IOMMU implementations has the mechanism of overflow buffer. So drivers that don't check a DMA mapping error are broken anyway. Also the size of the overflow is 32K by default. We often see larger request than that. Even with the overflow mechanism, we see data corruption anyway.
| |