Quoting Mika Kuoppala (2019-07-10 17:21:15) > Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > > + pd = alloc_pd(vm); > > + if (IS_ERR(pd)) > > return pd; > > - } > > - > > - /* 3lvl */ > > - pd = __alloc_pd(); > > - if (!pd) > > - return ERR_PTR(-ENOMEM); > > - > > - pd->entry[GEN8_3LVL_PDPES] = NULL; > > Ok you dont like the sentry. Perhaps you could write > a few soothing words how noisily we crash if we > run long on this runway. If the tower sees and > sends firetrucks, it is fine. It's a tight allocation (or becomes one a patch or two down); you have to enable slab_debug to see fireworks in the redzone. Or kasan to detect the out-of-bounds write. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
