Re: [PATCH] nouveau/gsp: use correct size for registry rpc.

[Danilo Krummrich <me@xxxxxxxx>]

On 1/30/24 04:26, Dave Airlie wrote:
> From: Dave Airlie <airlied@xxxxxxxxxx>
>
> Timur pointed this out before, and it just slipped my mind,
> but this might help some things work better, around pcie power
> management.
>
> Fixes: 8d55b0a940bb ("nouveau/gsp: add some basic registry entries.")
> Signed-off-by: Dave Airlie <airlied@xxxxxxxxxx>

Added stable CC for v6.7 and applied it to drm-misc-fixes.

> ---
>   drivers/gpu/drm/nouveau/nvkm/subdev/gsp/r535.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/gsp/r535.c b/drivers/gpu/drm/nouveau/nvkm/subdev/gsp/r535.c
> index 9ee58e2a0eb2..5e1fa176aac4 100644
> --- a/drivers/gpu/drm/nouveau/nvkm/subdev/gsp/r535.c
> +++ b/drivers/gpu/drm/nouveau/nvkm/subdev/gsp/r535.c
> @@ -1078,7 +1078,6 @@ r535_gsp_rpc_set_registry(struct nvkm_gsp *gsp)
>   	if (IS_ERR(rpc))
>   		return PTR_ERR(rpc);
>   
> -	rpc->size = sizeof(*rpc);
>   	rpc->numEntries = NV_GSP_REG_NUM_ENTRIES;
>   
>   	str_offset = offsetof(typeof(*rpc), entries[NV_GSP_REG_NUM_ENTRIES]);
> @@ -1094,6 +1093,7 @@ r535_gsp_rpc_set_registry(struct nvkm_gsp *gsp)
>   		strings += name_len;
>   		str_offset += name_len;
>   	}
> +	rpc->size = str_offset;
>   
>   	return nvkm_gsp_rpc_wr(gsp, rpc, false);
>   }