Re: [PATCH v1 1/4] misc: fastrpc: Add CRC support using invokeV2 request

[Ekansh Gupta]

On 10/7/2024 2:23 PM, Greg KH wrote:
> On Mon, Oct 07, 2024 at 02:15:15PM +0530, Ekansh Gupta wrote:
>> +struct fastrpc_invoke_v2 {
>> +    struct fastrpc_invoke inv;
>> +    __u64 crc;
>> +    __u32 reserved[16];
> I'm sure we have been over this before, but if you have a reserved
> field, you HAVE to check that it is zero if you ever want to use it in
> the future.  As you did not check it, just remove it from this structure
> as obviously it can never be used for anything.
>
> Your internal review should have caught this, if not, please go fix it
> AND get some internal review and signed-off-by before submitting this
> series again.
Thanks for your insights. I'll fix this in the next patch. My apologies for 
missing it.

--ekansh
>
> thanks,
>
> greg k-h