Re: 2.6.29 git master and PAT problems

[Ingo Molnar <mingo@elte.hu>]

* Arkadiusz Miskiewicz <a.miskiewicz@gmail.com> wrote:

> On Wednesday 08 of April 2009, Pallipadi, Venkatesh wrote:
> > On Tue, Apr 07, 2009 at 02:12:28AM -0700, Arkadiusz Miskiewicz wrote:
> > > On Tuesday 07 of April 2009, Pallipadi, Venkatesh wrote:
> > > > On Thu, 2009-04-02 at 00:12 -0700, Arkadiusz Miskiewicz wrote:
> > > >
> > > > I was finally able to reproduce the problem of "freeing invalid
> > > > memtype" with upstream git kernel (commit 0221c81b1b) + latest xf86
> > > > intel driver. But, with upstream + the patch I had sent you earlier in
> > > > this thread (http://marc.info/?l=linux-kernel&m=123863345520617&w=2) I
> > > > don't see those freeing invalid memtype errors anymore.
> > > >
> > > > Can you please double check with current git and that patch and let me
> > > > know if you are still seeing the problem.
> > >
> > > Latest linus tree + that patch (it's really applied here), xserver 1.6,
> > > libdrm from git master, intel driver from git master, previously mesa 7.4
> > > (and 7.5 snap currently), tremolous.net 1.1.0 game (tremolous-smp
> > > binary), GM45 gpu.
> > >
> > > To reproduce I just need to run tremolous-smp and connect to some map.
> > > When map finishes loading I instantly get:
> [...]
> 
> > OK. One more test patch below, applies over linus's git and you can ignore
> > the earlier patch. The patch below should get rid of the problem and
> > as it removes the track/untrack of vm_insert_pfn completely. This will
> > also eliminate the overhead of hundreds or thousands of entries in
> > pat_memtype_list. Can you please test it.
> 
> With this patch I'm no longer able to reproduce problem. Thanks!

Great, thanks!

Venki, mind sending a patch with a proper changelog, Reported-by, 
Tested-by tags, with Nick and Andrew Cc:-ed for the memory.c bits, 
etc.?

	Ingo