> -----Original Message----- > From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On > Behalf Of Xu, Randy > Sent: Monday, March 20, 2017 2:41 PM > To: Palli, Tapani <tapani.pa...@intel.com>; Emil Velikov > <emil.l.veli...@gmail.com>; Landwerlin, Lionel G > <lionel.g.landwer...@intel.com> > Cc: ML mesa-dev <mesa-dev@lists.freedesktop.org>; x...@freedesktop.org > Subject: Re: [Mesa-dev] [PATCH] Vulkan: Solve the vkCmdClearColorImage > crash issue > > > > > Thanks, > Randy > > > > -----Original Message----- > > From: Palli, Tapani > > Sent: Monday, March 20, 2017 2:34 PM > > To: Xu, Randy <randy...@intel.com>; Emil Velikov > > <emil.l.veli...@gmail.com>; Landwerlin, Lionel G > > <lionel.g.landwer...@intel.com> > > Cc: ML mesa-dev <mesa-dev@lists.freedesktop.org>; x...@freedesktop.org > > Subject: Re: [Mesa-dev] [PATCH] Vulkan: Solve the vkCmdClearColorImage > > crash issue > > > > > > > > On 03/20/2017 02:35 AM, Xu, Randy wrote: > > > > > >> -----Original Message----- > > >> From: Emil Velikov [mailto:emil.l.veli...@gmail.com] > > >> Sent: Sunday, March 19, 2017 11:39 PM > > >> To: Landwerlin, Lionel G <lionel.g.landwer...@intel.com> > > >> Cc: Xu, Randy <randy...@intel.com>; ML mesa-dev <mesa- > > >> d...@lists.freedesktop.org>; x...@freedesktop.org > > >> Subject: Re: [Mesa-dev] [PATCH] Vulkan: Solve the > > >> vkCmdClearColorImage crash issue > > >> > > >> On 18 March 2017 at 12:21, Lionel Landwerlin > > >> <lionel.g.landwer...@intel.com> wrote: > > >>> Also, we want this in Cc for stable :) > > >>> > > >> That would be a good idea, imho. > > >> > > >> Randy, for future patches please check with `git log' for the patch > > >> prefix - > > >> "anv: " is the one you want here. > > > Sure, I will follow this rule, should I revise the recent two patches? > > > > I pushed your first patch in and changed the prefix as anv/genX for that > one. > > > > As for the second one, I'm not sure if all layerCount accesses should > > be changed at once or is it ok to push this one separately (?) > > Thanks, Tapani > I am checking other layerCount access, and plan to submit another patch > soon. > But for vkCmdClearColorImage crash, this one is enough.
Sorry, please wait a minute, I will submit a new patch set, which will handle another case in anv_CmdClearDepthStencilImage also. The VK_REMAINING_ARRAY_LAYERS only occurs in VkImageSubresourceRange. We don’t need call anv_get_layerCount for VkImageSubresourceLayers > > > > > > > > > > >> If you can drop the non-existing x...@freedesktop.org email that > > >> would be even better. > > > Sorry, it's due to my git configure, and I have solved it. > > > > > >> > > >> Thanks > > >> Emil > > > _______________________________________________ > > > mesa-dev mailing list > > > mesa-dev@lists.freedesktop.org > > > https://lists.freedesktop.org/mailman/listinfo/mesa-dev > > > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
