on 08/10/2010 10:46 Alan Cox said the following: > Andriy Gapon wrote: >> Here's an updated patch: >> http://people.freebsd.org/~avg/amd64-minidump.3.diff > > The kernel part of the patch looks good. That said, I have one suggestion. The > current generation of AMD and Intel processors has support for 1GB pages. If you > want to make sure that this change will last us a long time, I would suggest > translating the old trick of generating a fake page table page for 2MB pages into > generating a fake page directory page for 1GB pages, rather than disposing of this > code. Let me double-check if I understand your suggestion correctly. So, if a 1GB page is used, then there will be a PDP entry for it with PS bit set, and PD entries for the range covered by the page will not be correctly set up? That is, I assumed that even if 1GB pages are used, then the corresponding PD entries are still correctly set up. But you say that this may not be the case in reality? So, I have to check PDP entry first and only if it's valid and it doesn't point to a 1GB page then I should examine corresponding PD entries. Correct? P.S. is there a macro for extracting frame address from PDPE? -- Andriy GaponReceived on Wed Nov 03 2010 - 17:30:42 UTC
This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:08 UTC