Re: [Intel-gfx] [PATCH 1/3] drm/i915: Stop holding a ref to the ppgtt from each vma

2018-08-16 Thread Chris Wilson
Quoting Mika Kuoppala (2018-08-16 13:14:37) > Chris Wilson writes: > > > Quoting Mika Kuoppala (2018-08-16 12:42:08) > >> Chris Wilson writes: > >> > >> > The context owns both the ppgtt and the vma within it, and our activity > >> > tracking on the context ensures that we do not release

Re: [Intel-gfx] [PATCH 1/3] drm/i915: Stop holding a ref to the ppgtt from each vma

2018-08-16 Thread Chris Wilson
Quoting Mika Kuoppala (2018-08-16 13:14:37) > Chris Wilson writes: > > > Quoting Mika Kuoppala (2018-08-16 12:42:08) > >> Chris Wilson writes: > >> > >> > The context owns both the ppgtt and the vma within it, and our activity > >> > tracking on the context ensures that we do not release

Re: [Intel-gfx] [PATCH 1/3] drm/i915: Stop holding a ref to the ppgtt from each vma

2018-08-16 Thread Mika Kuoppala
Chris Wilson writes: > Quoting Mika Kuoppala (2018-08-16 12:42:08) >> Chris Wilson writes: >> >> > The context owns both the ppgtt and the vma within it, and our activity >> > tracking on the context ensures that we do not release active ppgtt. As >> > the context fulfils our obligations for

Re: [Intel-gfx] [PATCH 1/3] drm/i915: Stop holding a ref to the ppgtt from each vma

2018-08-16 Thread Chris Wilson
Quoting Mika Kuoppala (2018-08-16 12:42:08) > Chris Wilson writes: > > > The context owns both the ppgtt and the vma within it, and our activity > > tracking on the context ensures that we do not release active ppgtt. As > > the context fulfils our obligations for active memory tracking, we can

Re: [Intel-gfx] [PATCH 1/3] drm/i915: Stop holding a ref to the ppgtt from each vma

2018-08-16 Thread Mika Kuoppala
Chris Wilson writes: > The context owns both the ppgtt and the vma within it, and our activity > tracking on the context ensures that we do not release active ppgtt. As > the context fulfils our obligations for active memory tracking, we can > relinquish the reference from the vma. > The part

[Intel-gfx] [PATCH 1/3] drm/i915: Stop holding a ref to the ppgtt from each vma

2018-08-16 Thread Chris Wilson
The context owns both the ppgtt and the vma within it, and our activity tracking on the context ensures that we do not release active ppgtt. As the context fulfils our obligations for active memory tracking, we can relinquish the reference from the vma. This fixes a silly transient refleak from