On 05/12/15 at 01:57pm, Dave Young wrote: > On 05/11/15 at 12:11pm, Joerg Roedel wrote: > > On Thu, May 07, 2015 at 09:56:00PM +0800, Dave Young wrote: > > > Joreg, I can not find the last reply from you, so just reply here about > > > my worries here. > > > > > > I said that the patchset will cause more problems, let me explain about > > > it more here: > > > > > > Suppose page table was corrupted, ie. original mapping iova1 -> page 1 > > > it was changed to iova1 -> page 2 accidently while crash happening, > > > thus future dma will read/write page 2 instead page 1, right? > > > > When the page-table is corrupted then it is a left-over from the old > > kernel. When the kdump kernel boots the situation can at least not get > > worse. For the page tables it is also hard to detect wrong mappings (if > > this would be possible the hardware could already do it), so any checks > > we could do there are of limited use anyway. > > Joerg, since both of you guys do not think it is a problem I will object it s/will object/will not object > any more though I still do not like reusing the old page tables. So let's > leave it as a future issue. > > Thanks > Dave -- 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