Re: [PATCH 0/5] IB/hfi1: Fine-tuning for three function implementations

From: Doug Ledford
Date: Thu Apr 20 2017 - 16:30:04 EST


On Fri, 2017-02-10 at 22:00 +0100, SF Markus Elfring wrote:
> From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
> Date: Fri, 10 Feb 2017 21:53:21 +0100
>
> A few update suggestions were taken into account
> from static source code analysis.
>
> Markus Elfring (5):
> Â Use kcalloc() in hfi1_user_exp_rcv_init()
> Â Use kcalloc() in hfi1_user_sdma_alloc_queues()
> Â Adjust another size determination in hfi1_user_sdma_alloc_queues()
> Â Use memdup_user() rather than duplicating its implementation in
> hfi1_user_sdma_process_request()
> Â Improve another size determination in
> hfi1_user_sdma_process_request()
>
> Âdrivers/infiniband/hw/hfi1/user_exp_rcv.c |ÂÂ5 +++--
> Âdrivers/infiniband/hw/hfi1/user_sdma.cÂÂÂÂ| 35 +++++++++++++++----
> ------------
> Â2 files changed, 20 insertions(+), 20 deletions(-)

I took 4 of the 5, leaving out the one that was addressed separately by
an Intel patch.

--
Doug Ledford <dledford@xxxxxxxxxx>
  GPG KeyID: B826A3330E572FDD
 Â
Key fingerprint = AE6B 1BDA 122B 23B4 265B Â1274 B826 A333 0E57 2FDD