Re: [Intel-gfx] [PATCH] drm/i915: add back the avail tracking

2021-06-24 Thread Daniel Vetter
On Wed, Jun 23, 2021 at 08:35:30PM +0100, Matthew Auld wrote: > On Wed, 23 Jun 2021 at 20:15, Daniel Vetter wrote: > > > > On Wed, Jun 23, 2021 at 06:27:06PM +0100, Matthew Auld wrote: > > > Looks like it got lost along the way, so add it back. This is needed for > > > the region query uAPI where

Re: [Intel-gfx] [PATCH] drm/i915: add back the avail tracking

2021-06-23 Thread kernel test robot
Hi Matthew, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [cannot apply to drm-intel/for-linux-next drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next drm/drm-next v5.13-rc7 next-20210623] [If your patch is applied to the wrong git tree,

Re: [Intel-gfx] [PATCH] drm/i915: add back the avail tracking

2021-06-23 Thread kernel test robot
Hi Matthew, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on drm-tip/drm-tip] [cannot apply to drm-intel/for-linux-next drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next drm/drm-next v5.13-rc7 next-20210623] [If your patch is applied to the wrong git tree,

Re: [Intel-gfx] [PATCH] drm/i915: add back the avail tracking

2021-06-23 Thread Matthew Auld
On Wed, 23 Jun 2021 at 20:15, Daniel Vetter wrote: > > On Wed, Jun 23, 2021 at 06:27:06PM +0100, Matthew Auld wrote: > > Looks like it got lost along the way, so add it back. This is needed for > > the region query uAPI where we want to report a snapshot of how much > > lmem is available. > > > >

Re: [Intel-gfx] [PATCH] drm/i915: add back the avail tracking

2021-06-23 Thread Daniel Vetter
On Wed, Jun 23, 2021 at 06:27:06PM +0100, Matthew Auld wrote: > Looks like it got lost along the way, so add it back. This is needed for > the region query uAPI where we want to report a snapshot of how much > lmem is available. > > This time around let's push it directly into the allocator, which

[Intel-gfx] [PATCH] drm/i915: add back the avail tracking

2021-06-23 Thread Matthew Auld
Looks like it got lost along the way, so add it back. This is needed for the region query uAPI where we want to report a snapshot of how much lmem is available. This time around let's push it directly into the allocator, which simplifies things, like not having to care about internal fragmentation