On 01/03/2018 01:27 PM, Jason Ekstrand wrote: > This patch looks good in isolation and you can have my rb. However, > this all a bit concerning and it makes me wonder if we're papering over > the real bug.
It may be a bit of both. There's also some debate in the bugzilla about whether or not this fixes the original bug or if there are actually two problems or ... I'm going to wait a bit to push this, and I'm going to try to investigate the other crash that has been observed. > On January 3, 2018 12:41:56 "Ian Romanick" <i...@freedesktop.org> wrote: > >> From: Ian Romanick <ian.d.roman...@intel.com> >> >> Every other caller of make_surface does something sensible when NULL is >> returned. >> >> Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> >> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104214 >> Tested-by: Cyril <cyril.d...@gmail.com> >> Cc: "17.3" <mesa-sta...@lists.freedesktop.org> >> --- >> src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c >> b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c >> index ead0c35..0079a08 100644 >> --- a/src/mesa/drivers/dri/i965/intel_mipmap_tree.c >> +++ b/src/mesa/drivers/dri/i965/intel_mipmap_tree.c >> @@ -719,6 +719,9 @@ miptree_create(struct brw_context *brw, >> ISL_SURF_USAGE_DEPTH_BIT | ISL_SURF_USAGE_TEXTURE_BIT, >> BO_ALLOC_BUSY, 0, NULL); >> >> + if (mt == NULL) >> + return NULL; >> + >> if (needs_separate_stencil(brw, mt, format) && >> !make_separate_stencil_surface(brw, mt)) { >> intel_miptree_release(&mt); >> -- >> 2.9.5 >> >> _______________________________________________ >> 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
