[PATCH 2/6] randr/provider: only allow slave gpu to be offload sources.

Alexander E. Patrakov patrakov at gmail.com
Fri Apr 29 17:20:06 UTC 2016


29.04.2016 09:01, Dave Airlie wrote:
> The other way around makes no sense.

Not sure.

My old (and now broken) Sony VAIO Z23A4R laptop had outputs attached to 
both Intel and Radeon GPUs. I don't know whether the Radeon card had 
this isGPU flag.

-- 
Alexander E. Patrakov

>
> Signed-off-by: Dave Airlie <airlied at redhat.com>
> ---
>  randr/rrprovider.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/randr/rrprovider.c b/randr/rrprovider.c
> index bbb8e51..5329f41 100644
> --- a/randr/rrprovider.c
> +++ b/randr/rrprovider.c
> @@ -325,6 +325,8 @@ ProcRRSetProviderOffloadSink(ClientPtr client)
>      VERIFY_RR_PROVIDER(stuff->provider, provider, DixReadAccess);
>      if (!(provider->capabilities & RR_Capability_SourceOffload))
>          return BadValue;
> +    if (!provider->pScreen->isGPU)
> +        return BadValue;
>
>      if (stuff->sink_provider) {
>          VERIFY_RR_PROVIDER(stuff->sink_provider, sink_provider, DixReadAccess);
>



More information about the xorg-devel mailing list