Re: [PATCH 15/20] glamor: Provide a placeholder for glamor_pushpixels

2014-03-22 Thread Keith Packard
Eric Anholt e...@anholt.net writes: Keith Packard kei...@keithp.com writes: This just calls miPushPixels until glamor uses FBOs for bitmaps The only cases that PushPixels() is called that I can find: - miglblt.c We're doing hand-accelerated glyph blits, so that's gone. - miarc.c I

Re: [PATCH 15/20] glamor: Provide a placeholder for glamor_pushpixels

2014-03-22 Thread Keith Packard
Jasper St. Pierre jstpie...@mecheye.net writes: I'd imagine that's hit by anything using XDrawArc. Surprisingly, there's not many applications using it out there. Only if you use a non-idempotent rasterop. Otherwise, the arc code just takes the generated spans and draws them all. --

Re: [PATCH 15/20] glamor: Provide a placeholder for glamor_pushpixels

2014-03-20 Thread Eric Anholt
Keith Packard kei...@keithp.com writes: This just calls miPushPixels until glamor uses FBOs for bitmaps The only cases that PushPixels() is called that I can find: - miglblt.c We're doing hand-accelerated glyph blits, so that's gone. - miarc.c I tried a few likely x11perfs and nothing hit

Re: [PATCH 15/20] glamor: Provide a placeholder for glamor_pushpixels

2014-03-20 Thread Jasper St. Pierre
On Thu, Mar 20, 2014 at 7:23 PM, Eric Anholt e...@anholt.net wrote: Keith Packard kei...@keithp.com writes: This just calls miPushPixels until glamor uses FBOs for bitmaps The only cases that PushPixels() is called that I can find: - miglblt.c We're doing hand-accelerated glyph blits,

[PATCH 15/20] glamor: Provide a placeholder for glamor_pushpixels

2014-03-18 Thread Keith Packard
This just calls miPushPixels until glamor uses FBOs for bitmaps Signed-off-by: Keith Packard kei...@keithp.com --- glamor/glamor_copy.c | 8 glamor/glamor_core.c | 2 +- glamor/glamor_priv.h | 4 3 files changed, 13 insertions(+), 1 deletion(-) diff --git a/glamor/glamor_copy.c