On Tue, Jan 13, 2015 at 04:12:29PM +0800, Li, ZhenHua wrote: > Current status: > 1. Use old root entry table, > 2. Copy old context entry tables. > 3. Copy old page tables. > 4. Allocate new page table when device driver is loading. > > I remember the progress we discussed was this. Checking the old mails, > founding we did not clearly discuss about the old page tables, only > agrred with allocating new page tables when device driver is loading. > > To me, both of these two options are fine, copying old page tables, or > re-use the old ones. My debugging shows both of them works fine. > > If we use the old page tables, the patchset will use less code. > Functions copy_page_addr, copy_page_table are no longer needed, and > functions copy_context_entry and copy_context_entry_table will be > reduced to several lines. The patchset will be much simpler. Yes, please do it this way. There is no point in copying the old page-tables. We never make changes to them, so they can stay where they are. The problem with the backup region still exists, but it is not specific to IOMMUs. This problem and should be addressed seperatly. Joerg -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
