On Wed, 2007-03-07 at 19:02 +0200, Panagiotis Papadakos wrote: > On Friday 02 March 2007 16:41, Michel Dänzer wrote: > > On Mon, 2007-02-26 at 07:09 +0200, Panagiotis Papadakos wrote: > > > Well I think I have the correct patch for this. > > > > I'm afraid not; the test modification is incorrect. The test is an > > optimization to avoid doing unnecessary work when the context is already > > up to date wrt the drawables passed in and doesn't have anything to do > > with the stamps. > > Well I think we should do this, because __driUtilUpdateDrawableInfo, > is called when lastStamp != stamp, and although the drawables have not > changed, we get new cliprects. So we have to call radeonSetCliprects, else > we point to memory freed in DoBindContext.
Right, just your test modification was wrong. This turned out to also fix issues with apps that don't call glViewport by default, so I pushed something based on it in the meantime, see http://bugs.freedesktop.org/show_bug.cgi?id=9876 . Thanks. -- Earthling Michel Dänzer | http://tungstengraphics.com Libre software enthusiast | Debian, X and DRI developer ------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys-and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV -- _______________________________________________ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel
