RE: [PATCH] drm/i915/gt: Report full vm address range

2024-03-20 Thread Mrozek, Michal
> If we provide the total GTT size we will have one page that will be contended > between kernel and userspace and, if userspace is unaware that the page > belongs to the > kernel, we might step on each other toe. That's fine, Compute needs to know total GTT size. Not available GTT size.

Re: [PATCH] drm/i915/gt: Report full vm address range

2024-03-20 Thread Andi Shyti
Hi Michal, On Mon, Mar 18, 2024 at 05:21:54AM +, Mrozek, Michal wrote: > > > Lionel, Michal, thoughts? > Compute UMD needs to know exact GTT total size. the problem is that we cannot apply the workaround without reserving one page from the GTT total size and we need to apply the workaround.

RE: [PATCH] drm/i915/gt: Report full vm address range

2024-03-17 Thread Mrozek, Michal
> > Lionel, Michal, thoughts? Compute UMD needs to know exact GTT total size.

Re: [PATCH] drm/i915/gt: Report full vm address range

2024-03-15 Thread Andi Shyti
Hi Nirmoy, > > In Mesa we've been relying on I915_CONTEXT_PARAM_GTT_SIZE so as long as > > that is adjusted by the kernel > > What do you mean by adjusted by, should it be a aligned size? > > I915_CONTEXT_PARAM_GTT_SIZE ioctl is returning vm->total which is > adjusted(reduced by a page). > >

Re: [PATCH] drm/i915/gt: Report full vm address range

2024-03-14 Thread Nirmoy Das
On 3/14/2024 3:04 PM, Lionel Landwerlin wrote: Hi Andi, In Mesa we've been relying on I915_CONTEXT_PARAM_GTT_SIZE so as long as that is adjusted by the kernel What do you mean by adjusted by, should it be a aligned size? I915_CONTEXT_PARAM_GTT_SIZE ioctl is returning vm->total which is

Re: [PATCH] drm/i915/gt: Report full vm address range

2024-03-14 Thread Lionel Landwerlin
Hi Andi, In Mesa we've been relying on I915_CONTEXT_PARAM_GTT_SIZE so as long as that is adjusted by the kernel, we should be able to continue working without issues. Acked-by: Lionel Landwerlin Thanks, -Lionel On 13/03/2024 21:39, Andi Shyti wrote: Commit 9bb66c179f50 ("drm/i915:

RE: [PATCH] drm/i915/gt: Report full vm address range

2024-03-13 Thread Mrozek, Michal
Commit 9bb66c179f50 ("drm/i915: Reserve some kernel space per vm") has reserved an object for kernel space usage. Userspace, though, needs to know the full address range. Fixes: 9bb66c179f50 ("drm/i915: Reserve some kernel space per vm") Signed-off-by: Andi Shyti Cc: Andrzej Hajda Cc: Chris