On 18-08-2025 23:02, Danilo Krummrich wrote:
On Thu Aug 14, 2025 at 12:59 PM CEST, Himal Prasad Ghimiray wrote:
This ops is used to iterate over GPUVA's in the user-provided range
and split the existing sparse VMA's if the start or end of the input
range lies within it. The operations can create up to 2 REMAPS and 2 MAPs.

The primary use case is for drivers to assign attributes to GPU VAs in
the specified range without performing unmaps or merging mappings,
supporting fine-grained control over sparse va's.

v2
- use drm_gpuvm_sm_map_ops_create with flags instead of defining new
   ops_create (Danilo)
- Add doc (Danilo)

v3
- Fix doc
- Fix unmapping check

v4
- Fix mapping for non madvise ops

v5
- Fix mapping (Matthew Brost)
- Rebase on top of struct changes

v6
- flag moved to map_req

v7 (Danilo)
- Use different functions
- Add kernel-doc
- Modify op_unmap_cb and op_map_cb to handle madvise and NULL ptr
- use gem_obj check in single place

(Same comment as in patch 1.)

Cc: Danilo Krummrich <d...@kernel.org>
Cc: Matthew Brost <matthew.br...@intel.com>
Cc: Boris Brezillon <bbrezil...@kernel.org>
Cc: <dri-devel@lists.freedesktop.org>
Signed-off-by: Himal Prasad Ghimiray<himal.prasad.ghimi...@intel.com>
Reviewed-by: Matthew Brost <matthew.br...@intel.com> #v6

This looks pretty clean now, and thanks for adding the documentation.

Acked-by: Danilo Krummrich <d...@kernel.org>

Feel free to take this through the Xe tree if necessary.

Thanks


Reply via email to