On Thu, Oct 24, 2002 at 02:00:48PM +0200, Michel Dänzer wrote:
> On Don, 2002-10-24 at 13:52, Alan Hourihane wrote:
> > On Thu, Oct 24, 2002 at 01:47:01PM +0200, Michel Dänzer wrote:
> > > > That certainly shouldn't be needed for shared entities. Where exactly
> > > > was it crashing ?
> > > 
> > > XAA dereferences pScrn->pScreen to track the state of shared entities.
> > > RADEONScreenInit() calls other radeon driver functions which call
> > > info->accel->Sync(), but the driver independent code only sets pScrn->pScreen
> > > after ScreenInit(). Boom.
> >  
> > A backtrace would be useful.
> 
> If you don't believe me, why don't you just try yourself? As noted in
> the comment for that CVS commit, I tracked this down together with Kevin
> Puetz, maybe he still has a backtrace somewhere.

It's not that I don't believe you Michel, I don't have a dual head
radeon card to test.

> What's the problem with setting this pointer, anyway?

If it's done in the driver independent code then we're duplicating something.
But, if it is the only solution to put it there, maybe we should
be doing it earlier in the driver independent code so that other drivers
don't have to go through the same pain.

I'm not criticizing you Michel, I'm just trying to understand the issue.

Alan. 


-------------------------------------------------------
This sf.net email is sponsored by: Influence the future
of Java(TM) technology. Join the Java Community
Process(SM) (JCP(SM)) program now.
http://ad.doubleclick.net/clk;4729346;7592162;s?http://www.sun.com/javavote
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to