[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090408081711.GA4938@elte.hu>
Date: Wed, 8 Apr 2009 10:17:11 +0200
From: Ingo Molnar <mingo@...e.hu>
To: Arkadiusz Miskiewicz <a.miskiewicz@...il.com>
Cc: "Pallipadi, Venkatesh" <venkatesh.pallipadi@...el.com>,
"Siddha, Suresh B" <suresh.b.siddha@...el.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Jesse Barnes <jbarnes@...tuousgeek.org>
Subject: Re: 2.6.29 git master and PAT problems
* Arkadiusz Miskiewicz <a.miskiewicz@...il.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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists