Re: [PATCH] drm/amdgpu: Actually check flags for all context ops.

2024-08-07 Thread Bas Nieuwenhuizen
Yeah, I also found that one. Will send it out this evening if you don't get to it. On Wed, Aug 7, 2024, 5:03 PM Michel Dänzer wrote: > On 2024-08-06 22:27, Bas Nieuwenhuizen wrote: > > Missing validation ... > > > > Checked libdrm and it clears all the structs, so we should be > > safe to just c

Re: [PATCH] drm/amdgpu: Actually check flags for all context ops.

2024-08-07 Thread Michel Dänzer
On 2024-08-06 22:27, Bas Nieuwenhuizen wrote: > Missing validation ... > > Checked libdrm and it clears all the structs, so we should be > safe to just check everything. Thanks for fixing this. FWIW, amdgpu_cs_ioctl has the same issue AFAICT. Haven't checked any others, there might be more. -

Re: [PATCH] drm/amdgpu: Actually check flags for all context ops.

2024-08-06 Thread Alex Deucher
On Tue, Aug 6, 2024 at 4:35 PM Bas Nieuwenhuizen wrote: > > Missing validation ... > > Checked libdrm and it clears all the structs, so we should be > safe to just check everything. > > Signed-off-by: Bas Nieuwenhuizen Applied. Thanks! > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 8 +

[PATCH] drm/amdgpu: Actually check flags for all context ops.

2024-08-06 Thread Bas Nieuwenhuizen
Missing validation ... Checked libdrm and it clears all the structs, so we should be safe to just check everything. Signed-off-by: Bas Nieuwenhuizen --- drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c b