Re: [PATCH V3 5/5] accel/amdxdna: Add zero check for pad in ioctl input structures

2024-12-16 Thread Mario Limonciello
This change actually meant here?  It seems like it should be it's own change. Agree. Hopefully, the patch 1 - 4 are good to go and I do not need to resend them. :) Yeah those are fine at this point. They've gotten good feedback, no complaints from LKP robot. I think you can just send th

Re: [PATCH V3 5/5] accel/amdxdna: Add zero check for pad in ioctl input structures

2024-12-16 Thread Lizhi Hou
On 12/16/24 12:38, Mario Limonciello wrote: On 12/13/2024 17:29, Lizhi Hou wrote: For input structures, it is better to check if the pad is zero. Thus, the pad bytes might be usable in the future. IIRC you should pick up: Suggested-by: Jeffrey Hugo Sure. Will add it. Signed-off-by: Liz

Re: [PATCH V3 5/5] accel/amdxdna: Add zero check for pad in ioctl input structures

2024-12-16 Thread Mario Limonciello
On 12/13/2024 17:29, Lizhi Hou wrote: For input structures, it is better to check if the pad is zero. Thus, the pad bytes might be usable in the future. IIRC you should pick up: Suggested-by: Jeffrey Hugo Signed-off-by: Lizhi Hou --- drivers/accel/amdxdna/aie2_ctx.c | 3 +++ driver

[PATCH V3 5/5] accel/amdxdna: Add zero check for pad in ioctl input structures

2024-12-13 Thread Lizhi Hou
For input structures, it is better to check if the pad is zero. Thus, the pad bytes might be usable in the future. Signed-off-by: Lizhi Hou --- drivers/accel/amdxdna/aie2_ctx.c | 3 +++ drivers/accel/amdxdna/aie2_message.c | 3 +++ drivers/accel/amdxdna/amdxdna_ctx.c | 6 ++ drivers/