Re: [PATCH 0/5] A couple more present fixes

2013-12-02 Thread Michel Dänzer
On Don, 2013-11-21 at 10:16 -0500, Alex Deucher wrote: On Thu, Nov 21, 2013 at 1:12 AM, Keith Packard kei...@keithp.com wrote: Alex Deucher alexdeuc...@gmail.com writes: For anyone interested in fixing this for pre-present xserver, Ilija fixed it similarly in the ddx for radeon:

Re: [PATCH 0/5] A couple more present fixes

2013-12-02 Thread Keith Packard
Michel Dänzer mic...@daenzer.net writes: IIRC changing the MSC frequency would e.g. break some piglit tests if DPMS happened to be active when they were running. There could be similar issues with real apps. I'm open to suggestions, but given that DRI2 is already broken in one direction

Re: [PATCH 0/5] A couple more present fixes

2013-11-21 Thread Alex Deucher
On Thu, Nov 21, 2013 at 1:12 AM, Keith Packard kei...@keithp.com wrote: Alex Deucher alexdeuc...@gmail.com writes: For anyone interested in fixing this for pre-present xserver, Ilija fixed it similarly in the ddx for radeon:

Re: [PATCH 0/5] A couple more present fixes

2013-11-20 Thread Adam Jackson
On Tue, 2013-11-19 at 22:51 -0800, Keith Packard wrote: I've gotten a couple more bugs in Present resolved, and I'd love to have some review of the current set of five tiny patches before I merge them to master. Three of these are a repost, the last two are new Again, I'm soliciting any

Re: [PATCH 0/5] A couple more present fixes

2013-11-20 Thread Keith Packard
Adam Jackson a...@nwnk.net writes: I've not had the chance to test these, but they all appear to do what they say on the box, so: Reviewed-by: Adam Jackson a...@redhat.com Thanks muchly! Merged. da9997f..6403cbb master - master -- keith.pack...@intel.com pgpp3GeZOS6au.pgp

Re: [PATCH 0/5] A couple more present fixes

2013-11-20 Thread Alex Deucher
On Wed, Nov 20, 2013 at 1:51 AM, Keith Packard kei...@keithp.com wrote: I've gotten a couple more bugs in Present resolved, and I'd love to have some review of the current set of five tiny patches before I merge them to master. Three of these are a repost, the last two are new Again, I'm

Re: [PATCH 0/5] A couple more present fixes

2013-11-20 Thread Keith Packard
Alex Deucher alexdeuc...@gmail.com writes: For anyone interested in fixing this for pre-present xserver, Ilija fixed it similarly in the ddx for radeon: http://cgit.freedesktop.org/xorg/driver/xf86-video-ati/commit/?id=6981a5c087165b126c15ba0025cffdba218ab652 Nice that this avoids the current

[PATCH 0/5] A couple more present fixes

2013-11-19 Thread Keith Packard
I've gotten a couple more bugs in Present resolved, and I'd love to have some review of the current set of five tiny patches before I merge them to master. Three of these are a repost, the last two are new Again, I'm soliciting any kind of comments as I'd like to get these into the server before