The i810 driver uses a lot of i830 driver code but does not by default define the necessary symbols for the i830 driver to function. The attached patch below adds the necessary define to Makefile.am in xf86-video-i810/src/ to enable the relevant defines. After i830_shadow.c is added (previous bug) and this patch is applied the i810 driver appears to function normally.
Created attachment 3338 [details] Screenshot of the corruption Adds -DBUILD_FOR_I830 required for the defines in i810_io.c to resolve externs in i830.h used in i830_driver.c .
Josh, could you reattach the patch? Bugzilla crashed and lost it.
Created attachment 2999 [details] [review] xf86-video-i810-1.4.0-Makefile.am.patch Sorry for the delay, had a busy weekend. Thanks for the heads-up.
I think that patch will break the driver for i810, but the fix that's in cvs should make it work for both i810 and i830. Could you give that a try, and close the bug if it works for you? Thanks.
Actually my patch works fine on the 810 (what I'm using). The 810 appears to make extensive reference to 830 calls and this might be why it works. I'll try CVS now and let you know.
Ah, I see this now depends on XvMC. Will XvMC support be optional in the future?
Confirmed that this is now fixed. An answer to comment #6 would still be appreciated ;D
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.