Bug 6046

an additional detail is while moving the window in any direction causes the
initial "snap to left" to happen, the gears will properly follow the window
vertically, it's only the horizontal offset that stays locked to the left.

In fact, more than just moving the window around causes the "snap to left". For
example here, just covering partially the window will cause it to happen... I
would appreciate some pointers to where the various window offsets are
calculated and used in the DRI to try to track down what's going on wrong... I
suspect something we aren't properly re-initializing when swapping contexts but
I'm not too sure...

Ok, by randomly poking around, I found a workaround which may help somebody who
understand that code better find out precisely what is wrong:

In r300InvalidateState(), if I comment out the #ifndef CB_DPATH so that it does
call r300ResetHwState(), then the bug is gone.

I suspect there is some state we properly init when doing that reset thing but
that we don't properly restore later on, but what preciesly is beyond my
understanding at this point.

(In reply to comment #3)
> Ok, by randomly poking around, I found a workaround which may help somebody who
> understand that code better find out precisely what is wrong:
> 
> In r300InvalidateState(), if I comment out the #ifndef CB_DPATH so that it does
> call r300ResetHwState(), then the bug is gone.

Try calling r300UpdateWindow in r300InvalidateState.

If that helps, try:
	R300_FIREVERTICES(rmesa);
	R300_STATECHANGE(rmesa, vpt);

Ok, so:

 - Adding r300UpdateWindow(ctx); instead of r300ResetHwState(r300); (at the same
location) inside r300InvalidateState() does fix the problem. Good.

 - However, just doing 

       R300_FIREVERTICES(r300);
       R300_STATECHANGE(r300, vpt);

instead does _NOT_ fix it.

 - Now, if I duplicate r300UpdateWindow(), calling the new one
r300UpdateWindow2() which does exactly as r300UpdateWindow() does _without_ the
2 lines above (that is basically keep only the part that updates the various
rmesa->hw.vpt.cmd[] entries). The problem is fixed too.

 - I finally went all the way down to isolating which bit actually fixes it and it's

	rmesa->hw.vpt.cmd[R300_VPT_XOFFSET] = r300PackFloat32(tx);

Just that line fixes the problem.

So for some reason, that XOFFSET thing gets corrupt.... I don't know enough of
what's going on in there, maybe it's normal that some or all of these gets
clobbered and they indeed should be restored... or maybe not, I'll let you guys
decide on the proper fix. That routine seems to be called fairly often, at least
in a windowed environment, so it seems performance critical enough to try to
optimize it.

While I was there, I also noticed an unrelated bit=:

r300Viewport() does

	R300_FIREVERTICES(R300_CONTEXT(ctx));
	r300UpdateWindow(ctx);

Isn't the first line redundant ? (That is, r300UpdateWindow does the
FIREVERTICES thing already).

So I added some more debug and it seems that R300_VPT_XOFFSET is regulary
getting clobbered with the value 0xf729860 (always the same) when I move the
window, or cover it, or whatever ...

I'll try to investigate what is causing this later.

Ok, that's getting fun :)

So building with -O1 -> no bug

Rebuilding with -O3 -> a different bug is back (the wheel completely disappear
when moving the window, calling r300UpdateWindow still fixes it, thus I think
another field gets corrupted in there).

Smells like either a compiler bug or a bug in the driver or Mesa vs. strict
aliasing. I'll investigate more.

(In reply to comment #7)
> Ok, that's getting fun :)
> 
> So building with -O1 -> no bug
> 
> Rebuilding with -O3 -> a different bug is back (the wheel completely disappear
> when moving the window, calling r300UpdateWindow still fixes it, thus I think
> another field gets corrupted in there).
> 
> Smells like either a compiler bug or a bug in the driver or Mesa vs. strict
> aliasing. I'll investigate more.

I'm building the binary snapshots with -fno-strict-aliasing because there were
problems, I think in mach64, related to clipping. This is probably a different
issue, though.

OK, I verified that indeed, just rebuilding r300_state.c with
-fno-strict-aliasing fixes it. I'll have to dig through that file now to figure
out where the aliasing bug is though, probably a dodgy cast.

Created attachment 4784 [details] [review]
Fix aliasing bug

Fixed in CVS, I leave the bug open until we have audited radeon, r200 and mach64
for similar bugs

FYI, I have exactly the same "half screen" problem with glxgears (and others) 
here on a ThinkPad A30 / Radeon Mobility M6 LY / Xorg 7.0 / xf86-video-ati 
6.5.7.3 / Gentoo

Additionally, I found an effect described (and showed) for XGL at 
http://gentoo-wiki.com/HOWTO_XGL:Troubleshooting#Half-screen_problem that looks 
quite similar.

(In reply to comment #12)
> FYI, I have exactly the same "half screen" problem with glxgears (and others) 
> here on a ThinkPad A30 / Radeon Mobility M6 LY / Xorg 7.0 / xf86-video-ati 
> 6.5.7.3 / Gentoo
I have fixed the dodgy casts in the viewport updates for radeon and r200, so if
it's caused by that the problem should no longer be present with Mesa CVS (or
the 6.5 release). That said, there are still potential aliasing problems in the
code, I'd suggest you compile with -fno-strict-aliasing for the time being.