<br>
<br><font size=2><tt>crash-utility-bounces@redhat.com wrote on 11/08/2005
02:54:33 PM:<br>
<br>
> On Tue, 2005-11-08 at 17:32 -0500, Dave Anderson wrote:<br>
> > Badari Pulavarty wrote:<br>
> > <br>
> > > On Tue, 2005-11-08 at 14:42 -0500, Dave Anderson wrote:<br>
> > > > Badari Pulavarty wrote:<br>
> > > ><br>
> > > > ><br>
> > > > ><br>
> > > > > Yep. You are right, those pages are not mapped.
Initially<br>
> > > > > I thought that was the case, but I wasn't really
sure if it<br>
> > > > > can happen with segments from shared libraries.
(since<br>
> > > > > the routine which is telling its not mapped involves<br>
> > > > > going through my code).<br>
> > > > ><br>
> > > > > Anyway, I verified this on x86-64 machine also
- I do<br>
> > > > > get "read" error and those pages are
not mapped.<br>
> > > > ><br>
> > > > > I guess, my changes are good :)<br>
> > > > ><br>
> > > > > Haren, can you review them ?<br>
> > > > ><br>
> > > > > Thanks,<br>
> > > > > Badari<br>
> > > > ><br>
> > > ><br>
> > > > The patch looks just fine to me -- the only minor things
to<br>
> > > > add is a dump of the VM_ORIG/VM_4_LEVEL bits in<br>
> > > > the ppc64_dump_machdep_table() debug function, and<br>
> > > > although the "pud_clear_bad" is in all probability
quite<br>
> > > > suitable as a qualifier, it's probably also worth adding<br>
> > > > an optional "--machdep vm=xxx" command line
argument<br>
> > > > like I put in place for the x86_64 equivalent.  Just
to cover<br>
> > > > our collective arse...<br>
> > > ><br>
> > > > Anyway, upon Haren's blessing, it's good to go.<br>
> > > ><br>
> > > > Again -- really nice work!<br>
> > ><br>
> > > Okay, made recommended changes.<br>
> > ><br>
> > > I still need to figure out a decent symbol to identify<br>
> > > 4-level pagetable layout. I am looking through "nm
-Bn".<br>
> > > I still can't find a decent one. (has to be some symbol<br>
> > > added for 2.6.14).<br>
> > ><br>
> > > Any ideas ?<br>
> > ><br>
> > <br>
> > If you can google out the wholesale patch that introduced the<br>
> > 4-level pagetable support, you might find something added<br>
> > or removed?  That's what I did for the x86_64 qualifier.<br>
> > <br>
> <br>
> Okay. I found new structure "pgtable_cache" got added along<br>
> with 4-level pagetable changes. I am using that symbol now.<br>
> <br>
> Thanks,<br>
> Badari<br>
> </tt></font>
<br><font size=2><tt>Badari, Thanks a lot for helping us on PPC64. </tt></font>
<br>
<br><font size=2><tt>Dave, Is it a problem if we use Linux version to determine
4-level page table changes instead of looking for some symbol which might
get changed later. We need to make some more changes for 64K page support
added in 2.6.14-git11.  </tt></font>
<br>
<br><font size=2><tt>Thanks</tt></font>
<br><font size=2><tt>Haren</tt></font>
<br><font size=2><tt><br>
> <br>
> [attachment "crash-ppc64-4level-latest.patch" deleted by
Haren <br>
> Myneni/Beaverton/IBM] --<br>
> Crash-utility mailing list<br>
> Crash-utility@redhat.com<br>
> https://www.redhat.com/mailman/listinfo/crash-utility<br>
</tt></font>