Re: [PATCH xserver 1/5] modesetting: Don't call xf86HandleColorMaps() at screen depth 30.

2018-02-09 Thread Mario Kleiner
On 02/09/2018 10:45 AM, Michel Dänzer wrote: On 2018-02-09 07:24 AM, Mario Kleiner wrote: On 02/08/2018 03:55 PM, Michel Dänzer wrote: On 2018-02-08 12:14 PM, Mario Kleiner wrote: As it turns out, doing so will make any gamma table updates silently fail, because xf86HandleColorMaps() hooks

Re: [PATCH xserver 1/5] modesetting: Don't call xf86HandleColorMaps() at screen depth 30.

2018-02-09 Thread Michel Dänzer
On 2018-02-09 07:24 AM, Mario Kleiner wrote: > On 02/08/2018 03:55 PM, Michel Dänzer wrote: >> On 2018-02-08 12:14 PM, Mario Kleiner wrote: >>> As it turns out, doing so will make any gamma table updates >>> silently fail, because xf86HandleColorMaps() hooks the >>> .LoadPalette function to

Re: [PATCH xserver 1/5] modesetting: Don't call xf86HandleColorMaps() at screen depth 30.

2018-02-08 Thread Mario Kleiner
On 02/08/2018 03:55 PM, Michel Dänzer wrote: On 2018-02-08 12:14 PM, Mario Kleiner wrote: As it turns out, doing so will make any gamma table updates silently fail, because xf86HandleColorMaps() hooks the .LoadPalette function to xf86RandR12LoadPalette() if the .gamma_set function is supported

Re: [PATCH xserver 1/5] modesetting: Don't call xf86HandleColorMaps() at screen depth 30.

2018-02-08 Thread Mario Kleiner
On 02/08/2018 10:09 PM, Adam Jackson wrote: On Thu, 2018-02-08 at 12:14 +0100, Mario Kleiner wrote: That in turn is a problem, because the size of the hw lut crtc->gamma_size is fixed to 256 slots on all kms-drivers when using the legacy gamma_set ioctl, What exactly is the non-legacy ioctl?

Re: [PATCH xserver 1/5] modesetting: Don't call xf86HandleColorMaps() at screen depth 30.

2018-02-08 Thread Adam Jackson
On Thu, 2018-02-08 at 12:14 +0100, Mario Kleiner wrote: > That in turn is a problem, because the size of the hw lut > crtc->gamma_size is fixed to 256 slots on all kms-drivers > when using the legacy gamma_set ioctl, What exactly is the non-legacy ioctl? - ajax

Re: [PATCH xserver 1/5] modesetting: Don't call xf86HandleColorMaps() at screen depth 30.

2018-02-08 Thread Michel Dänzer
On 2018-02-08 12:14 PM, Mario Kleiner wrote: > As it turns out, doing so will make any gamma table updates > silently fail, because xf86HandleColorMaps() hooks the > .LoadPalette function to xf86RandR12LoadPalette() if the > .gamma_set function is supported by the ddx, as is in our > case. > >

[PATCH xserver 1/5] modesetting: Don't call xf86HandleColorMaps() at screen depth 30.

2018-02-08 Thread Mario Kleiner
As it turns out, doing so will make any gamma table updates silently fail, because xf86HandleColorMaps() hooks the .LoadPalette function to xf86RandR12LoadPalette() if the .gamma_set function is supported by the ddx, as is in our case. Once xf86RandR12LoadPalette() has been called during server