On Tue, Sep 25, 2018 at 03:22:11PM -0700, Jordan Justen wrote: > Signed-off-by: Jordan Justen <[email protected]> > Cc: Nanley Chery <[email protected]> > --- > src/intel/vulkan/anv_device.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c > index 0ea8be052fa..4e446c3280a 100644 > --- a/src/intel/vulkan/anv_device.c > +++ b/src/intel/vulkan/anv_device.c > @@ -1574,6 +1574,15 @@ static void > anv_device_init_hiz_clear_batch(struct anv_device *device) > { > anv_bo_init_new(&device->hiz_clear_bo, device, 4096); > + > + if (device->instance->physicalDevice.has_exec_async) > + device->hiz_clear_bo.flags |= EXEC_OBJECT_ASYNC; > + > + if (device->instance->physicalDevice.use_softpin) > + device->hiz_clear_bo.flags |= EXEC_OBJECT_PINNED; > + > + anv_vma_alloc(device, &device->hiz_clear_bo); > +
Seems like we should handle the return value of this function. Maybe also hook into the block of gotos in anv_CreateDevice()? > uint32_t *map = anv_gem_mmap(device, device->hiz_clear_bo.gem_handle, > 0, 4096, 0); > > -- > 2.18.0 > > _______________________________________________ > mesa-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
