Re: [PATCH 27/27] IB/hfi1: Code reuse with memdup_copy

From: SF Markus Elfring
Date: Mon Feb 13 2017 - 04:51:24 EST

Next message: Geert Uytterhoeven: "Re: Build regressions/improvements in v4.10-rc8"
Previous message: Geert Uytterhoeven: "Build regressions/improvements in v4.10-rc8"
In reply to: Dennis Dalessandro: "Re: [PATCH 4/5] IB/hfi1: Use memdup_user() rather than duplicating its implementation in hfi1_user_sdma_process_request()"
Next in thread: Dan Carpenter: "Re: [PATCH 4/5] IB/hfi1: Use memdup_user() rather than duplicating its implementation in hfi1_user_sdma_process_request()"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

> Thanks for the patch, but this one is already taken care of along with other similar uses of kmalloc/copy:
> http://marc.info/?l=linux-rdma&m=148656088729538&w=2

Thanks for your information.

The shown source code is reasonable in the update step â[PATCH 27/27] IB/hfi1:
Code reuse with memdup_copyâ.
https://patchwork.kernel.org/patch/9562565/
https://lkml.kernel.org/r/<20170208132830.16442.93943.stgit@xxxxxxxxxxxxxxxxxxx>

I find the commit subject and message partly inappropriate.
How do you think about to mention other function names there?

Regards,
Markus

Next message: Geert Uytterhoeven: "Re: Build regressions/improvements in v4.10-rc8"
Previous message: Geert Uytterhoeven: "Build regressions/improvements in v4.10-rc8"
In reply to: Dennis Dalessandro: "Re: [PATCH 4/5] IB/hfi1: Use memdup_user() rather than duplicating its implementation in hfi1_user_sdma_process_request()"
Next in thread: Dan Carpenter: "Re: [PATCH 4/5] IB/hfi1: Use memdup_user() rather than duplicating its implementation in hfi1_user_sdma_process_request()"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]