[PATCH v8 01/24] drm/gpuvm: Pass map arguments through a struct

Danilo Krummrich dakr at kernel.org
Wed Aug 20 12:04:51 UTC 2025


On Wed Aug 20, 2025 at 1:56 PM CEST, Boris Brezillon wrote:
> On Tue, 19 Aug 2025 03:27:30 +0530
> Himal Prasad Ghimiray <himal.prasad.ghimiray at intel.com> wrote:
>> +/**
>> + * struct drm_gpuvm_map_req - arguments passed to drm_gpuvm_sm_map[_ops_create]()
>> + */
>> +struct drm_gpuvm_map_req {
>> +	/**
>> +	 * @op_map: struct drm_gpuva_op_map
>
> Doc name doesn't match the field name.
>
>> +	 */
>> +	struct drm_gpuva_op_map map;
>
> Nit: not convinced map is a good name for that field, given this is a
> map_req, so we now those are map-related arguments. I would probably
> go for 'op'.

Indeed, thanks for catching this! I thought I was suggesting this name in a
previous version too.


More information about the Intel-xe mailing list