Re: [PATCH xserver 2/2] glamor: Avoid software fallback for planemasked ZPixmap GetImage

2017-03-20 Thread Adam Jackson
On Mon, 2017-03-20 at 10:42 -0700, Eric Anholt wrote: > > Adam Jackson writes: > > @@ -128,6 +128,16 @@ glamor_get_image_gl(DrawablePtr drawable, int x, int > > y, int w, int h, > >    drawable->x + off_x, drawable->y + off_y, > >  

Re: [PATCH xserver 2/2] glamor: Avoid software fallback for planemasked ZPixmap GetImage

2017-03-20 Thread Eric Anholt
Adam Jackson writes: > Same trick as in fb: just do a normal GetImage and deal with the > planemask on the CPU if you have to. Since the software fallback hit for > glamor is pretty brutal, this is a much more impressive win for glamor > than it was for fb: > > 11100.0

Re: [PATCH xserver 2/2] glamor: Avoid software fallback for planemasked ZPixmap GetImage

2017-03-20 Thread Adam Jackson
On Wed, 2016-09-28 at 13:05 -0400, Adam Jackson wrote: > Same trick as in fb: just do a normal GetImage and deal with the > planemask on the CPU if you have to. Since the software fallback hit for > glamor is pretty brutal, this is a much more impressive win for glamor > than it was for fb: > >  

[PATCH xserver 2/2] glamor: Avoid software fallback for planemasked ZPixmap GetImage

2016-09-28 Thread Adam Jackson
Same trick as in fb: just do a normal GetImage and deal with the planemask on the CPU if you have to. Since the software fallback hit for glamor is pretty brutal, this is a much more impressive win for glamor than it was for fb: 11100.0 87700.0 (7.901) (copy 0x) ShmGetImage 10x10