On 18 Sep 2002, Michel [ISO-8859-1] Dänzer wrote:

> On Mit, 2002-09-18 at 07:40, Vladimir Dergachev wrote:
> >
> >
> > On 18 Sep 2002, Michel [ISO-8859-1] Dänzer wrote:
> >
> > > On Die, 2002-09-17 at 23:39, Krzysztof Halasa wrote:
> > > >
> > > > 1. Cloning the display and using XVideo halts the system when moving
> > > > the viewport. I've traced that to the fact XVideo routines change
> > > > pScrn->AdjustFrame to their internal function, which isn't prepared
> > > > to be called for "clone display".
> > > > I've changed all calls of pScrn->AdjustFrame() for "clone display"
> > > > to just RADEONAdjustFrame() - clone display has nothing to do with
> > > > XVideo anyway.
> > >
> > > I don't see anything about AdjustFrame in radeon_video.c, must be a
> > > GATOS thing? Anyway, wouldn't it be better to fix that other AdjustFrame
> > > function for clone?
> >
> > Judging by the patch this is not anywhere near GATOS modifications and
> > looks to be pretty reasonable..
>
> I just don't understand what the changes are trying to achieve, much
> less what they have to do with XVideo. I'd appreciate some insights.
>

Most likely what happens is that generic Xvideo layer wraps some functions
in order to trace certain events. So calling pScrn->AdjustFrame results in
calling generic functions (and not whatever was assigned to
pScrn->AdjustFrame during initialization) resulting in conflict.

                           Vladimir Dergachev

>
> --
> Earthling Michel Dänzer (MrCooper)/ Debian GNU/Linux (powerpc) developer
> XFree86 and DRI project member   /  CS student, Free Software enthusiast
>
> _______________________________________________
> Xpert mailing list
> [EMAIL PROTECTED]
> http://XFree86.Org/mailman/listinfo/xpert
>

_______________________________________________
Xpert mailing list
[EMAIL PROTECTED]
http://XFree86.Org/mailman/listinfo/xpert

Reply via email to