Re: [PATCH V1 09/13] accel/amdxdna: Use unsigned long for nr_pages in amdxdna_hmm_register()

[Max Zhen <max.zhen@amd.com>]

On 6/15/2026 Mon 23:15, Lizhi Hou wrote:
> nr_pages is declared as u32 in amdxdna_hmm_register(), which may not be
> large enough to represent the number of pages for large mappings.
> 
> Use unsigned long for nr_pages to avoid potential overflow.
> 
> Fixes: ac49797c1815 ("accel/amdxdna: Add GEM buffer object management")
> Signed-off-by: Lizhi Hou <lizhi.hou@amd.com>
Reviewed-by: Max Zhen <max.zhen@amd.com>> ---
>   drivers/accel/amdxdna/amdxdna_gem.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/accel/amdxdna/amdxdna_gem.c b/drivers/accel/amdxdna/amdxdna_gem.c
> index aef4c23a1a66..201536aff440 100644
> --- a/drivers/accel/amdxdna/amdxdna_gem.c
> +++ b/drivers/accel/amdxdna/amdxdna_gem.c
> @@ -351,7 +351,7 @@ static int amdxdna_hmm_register(struct amdxdna_gem_obj *abo,
>   	unsigned long len = vma->vm_end - vma->vm_start;
>   	unsigned long addr = vma->vm_start;
>   	struct amdxdna_umap *mapp;
> -	u32 nr_pages;
> +	unsigned long nr_pages;
>   	int ret;
>   
>   	if (!amdxdna_pasid_on(abo->client)) {