> On Nov 3, 2022, at 8:49 AM, Trond Myklebust <trondmy@xxxxxxxxxx> wrote:
>
> On Thu, 2022-11-03 at 09:35 +0800, Li zeming wrote:
>> The iov_base pointer does not need to cast the type.
>>
>> Signed-off-by: Li zeming <zeming@xxxxxxxxxxxx>
>> ---
>> include/linux/sunrpc/svc.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/linux/sunrpc/svc.h b/include/linux/sunrpc/svc.h
>> index daecb009c05b..4b0eead91c94 100644
>> --- a/include/linux/sunrpc/svc.h
>> +++ b/include/linux/sunrpc/svc.h
>> @@ -222,7 +222,7 @@ static inline __be32 svc_getu32(struct kvec *iov)
>>
>> static inline void svc_ungetu32(struct kvec *iov)
>> {
>> - __be32 *vp = (__be32 *)iov->iov_base;
>> + __be32 *vp = iov->iov_base;
>> iov->iov_base = (void *)(vp - 1);
>> iov->iov_len += sizeof(*vp);
>> }
>
> If you're going to submit a patch just in order to strip unnecessary
> casts, then why are you leaving the equally unnecessary cast in the
> next line?
>
> That said, why waste time "fixing" an inlined function that is not
> actually called anywhere in the kernel? svc_ungetu32() should just be
> removed.
Sorry for the delay, I was expecting an electrical service outage
here at home yesterday, so the lab was powered off.
So, Li, can you drop this one, and instead send a patch that removes
the definition of svc_ungetu32 ?
--
Chuck Lever
