Re: [Mesa-dev] [PATCH] anv: don't crash when creating a huge image

2017-11-10 Thread Jason Ekstrand
On Fri, Nov 10, 2017 at 10:48 AM, Chad Versace wrote: > On Thu 09 Nov 2017, Jason Ekstrand wrote: > > On Thu, Nov 9, 2017 at 4:23 PM, Chad Versace <[1] > chadvers...@chromium.org> > > wrote: > > > > On Wed 08 Nov 2017, Jason Ekstrand wrote: > > > On Wed, Nov 8, 2017 at 1:34 AM, Samuel Igl

Re: [Mesa-dev] [PATCH] anv: don't crash when creating a huge image

2017-11-10 Thread Chad Versace
On Thu 09 Nov 2017, Jason Ekstrand wrote: > On Thu, Nov 9, 2017 at 4:23 PM, Chad Versace <[1]chadvers...@chromium.org> > wrote: > > On Wed 08 Nov 2017, Jason Ekstrand wrote: > > On Wed, Nov 8, 2017 at 1:34 AM, Samuel Iglesias Gonsálvez <[1] > > [2]sigles...@igalia.com> wrote: > > >

Re: [Mesa-dev] [PATCH] anv: don't crash when creating a huge image

2017-11-09 Thread Samuel Iglesias Gonsálvez
On Thu, 2017-11-09 at 16:34 -0800, Jason Ekstrand wrote: > On Thu, Nov 9, 2017 at 4:23 PM, Chad Versace g> > wrote: > > > On Wed 08 Nov 2017, Jason Ekstrand wrote: > > > On Wed, Nov 8, 2017 at 1:34 AM, Samuel Iglesias Gonsálvez <[1] > > > sigles...@igalia.com> wrote: > > > > > > The HW has s

Re: [Mesa-dev] [PATCH] anv: don't crash when creating a huge image

2017-11-09 Thread Jason Ekstrand
On Thu, Nov 9, 2017 at 4:23 PM, Chad Versace wrote: > On Wed 08 Nov 2017, Jason Ekstrand wrote: > > On Wed, Nov 8, 2017 at 1:34 AM, Samuel Iglesias Gonsálvez <[1] > > sigles...@igalia.com> wrote: > > > > The HW has some limits but, according to the spec, we can create > > the image as it

Re: [Mesa-dev] [PATCH] anv: don't crash when creating a huge image

2017-11-09 Thread Chad Versace
On Wed 08 Nov 2017, Jason Ekstrand wrote: > On Wed, Nov 8, 2017 at 1:34 AM, Samuel Iglesias Gonsálvez <[1] > sigles...@igalia.com> wrote: > > The HW has some limits but, according to the spec, we can create > the image as it has not yet any memory backing it. When we allocate > that me

Re: [Mesa-dev] [PATCH] anv: don't crash when creating a huge image

2017-11-08 Thread Jason Ekstrand
On Wed, Nov 8, 2017 at 1:34 AM, Samuel Iglesias Gonsálvez < sigles...@igalia.com> wrote: > The HW has some limits but, according to the spec, we can create > the image as it has not yet any memory backing it. When we allocate > that memory, then we fail following what Vulkan spec, "10.2. Device >

[Mesa-dev] [PATCH] anv: don't crash when creating a huge image

2017-11-08 Thread Samuel Iglesias Gonsálvez
The HW has some limits but, according to the spec, we can create the image as it has not yet any memory backing it. When we allocate that memory, then we fail following what Vulkan spec, "10.2. Device Memory" says when talking about vkAllocateMemory(): "Some platforms may have a limit on the maxim