[Mesa-dev] [PATCH] anv: Return correct result in EnumeratePhysicalDevices

[Jason Ekstrand]

Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

I'll push it.

On Tue, Oct 11, 2016 at 6:08 PM, Nicolas Koch <nioko1337 at gmail.com> wrote:

> It's been a few days. Is anybody willing to push this?
>
> On Thu, Oct 6, 2016 at 11:25 PM, Anuj Phogat <anuj.phogat at gmail.com>
> wrote:
> > On Thu, Oct 6, 2016 at 12:21 PM, Nicolas Koch <nioko1337 at gmail.com>
> wrote:
> >> If pPhysicalDevices is too small for all physical devices,
> >> the driver must return VK_INCOMPLETE.
> >> Since only a single physical device is supported, this is only the case
> >> when pPhysicalDeviceCount == 0 && pPhysicalDevices != NULL.
> >> ---
> >>  src/intel/vulkan/anv_device.c | 2 ++
> >>  1 file changed, 2 insertions(+)
> >>
> >> diff --git a/src/intel/vulkan/anv_device.c
> b/src/intel/vulkan/anv_device.c
> >> index c7b9979..76cbb69 100644
> >> --- a/src/intel/vulkan/anv_device.c
> >> +++ b/src/intel/vulkan/anv_device.c
> >> @@ -385,6 +385,8 @@ VkResult anv_EnumeratePhysicalDevices(
> >>     } else if (*pPhysicalDeviceCount >= 1) {
> >>        pPhysicalDevices[0] = anv_physical_device_to_handle(
> &instance->physicalDevice);
> >>        *pPhysicalDeviceCount = 1;
> >> +   } else if (*pPhysicalDeviceCount < instance->physicalDeviceCount) {
> >> +      return VK_INCOMPLETE;
> >>     } else {
> >>        *pPhysicalDeviceCount = 0;
> >>     }
> >> --
> >> 2.10.0
> >>
> >> _______________________________________________
> >> mesa-dev mailing list
> >> mesa-dev at lists.freedesktop.org
> >> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
> >
> > Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20161011/cf3a44ae/attachment.html>