Re: [PATCH 6/7] anv: add VK_EXT_display_control to anv driver [v2]

From: "Keith Packard" <keithp@keithp.com>
To: Jason Ekstrand <jason@jlekstrand.net>
Cc: ML mesa-dev <mesa-dev@lists.freedesktop.org>,
	Maling list - DRI developers <dri-devel@lists.freedesktop.org>
Subject: Re: [PATCH 6/7] anv: add VK_EXT_display_control to anv driver [v2]
Date: Wed, 20 Jun 2018 15:45:21 -0700	[thread overview]
Message-ID: <874lhxxeke.fsf@keithp.com> (raw)
In-Reply-To: <CAOFGe964euzZC17S5tjwfnXZpYOa2hLhoxRq=hCnssjRF5BNfA@mail.gmail.com>

[-- Attachment #1.1: Type: text/plain, Size: 629 bytes --]

Jason Ekstrand <jason@jlekstrand.net> writes:

> I believe that the WSI common code should be capable of fishing the
> instance allocator out of the wsi_display so we need only pass the
> allocator argument unmodified through to the core WSI code.  Make sense?

Thanks, I think I've sorted it out. I've pushed an updated series with
this change.

> Yeah, Vulkan allocator fishing is weird.

Allowing custom allocators is one of the bad parts of the Vulkan spec;
it will "never" get used, and the chances of it working correctly in any
driver are pretty small. But, we do what we can to implement it.

-- 
-keith

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

[-- Attachment #2: Type: text/plain, Size: 157 bytes --]

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev