Re: [PATCH 00/29] IB/mlx: Fine-tuning for several function implementations

[Doug Ledford]

On Sat, 2017-02-18 at 21:45 +0100, SF Markus Elfring wrote:
> From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx>
> Date: Sat, 18 Feb 2017 21:34:32 +0100
> 
> Several update suggestions were taken into account
> from static source code analysis.

I've dropped this entire series. ÂIf you want me to consider it, you
need to respin it with the following changes:

1) Put all similar corrections in a single patch. ÂIf you are replacing
a sizeof() usage with a safer sizeof() usage, then I don't want one
patch per replacement, I want one patch for all replacements.

2) Use reasonable git commit messages. Â"Improve another" is never a
reasonable commit message because it implicitly refereces another git
commit message. ÂUnless this is a fix for a previous commit, all commit
messages should be standalone.

3) Keep your git commit message subject lines within the recommended
size. ÂYou have many that are too long. ÂAs trivial as all of these
changes are, I'm not going to go fixup 29 commit messages to take them.

4) Make sure to preserve the reviewed-by tags on the resubmission.

> Markus Elfring (29):
> Â Use kcalloc() in mlx4_ib_alloc_pv_bufs()
> Â Improve another size determination in mlx4_ib_alloc_pv_bufs()
> Â Improve another size determination in mlx4_ib_alloc_demux_ctx()
> Â Improve another size determination in alloc_pv_object()
> Â Fix a typo in a comment line
> Â Delete three unnecessary return statements
> Â Split a condition check in handle_slaves_guid_change()
> Â Delete an unnecessary check before the function call "kfree" in
> free_pv_object()
> Â Move an assignment out of a check in forward_trap()
> Â Enclose 15 expressions for the sizeof operator by parentheses
> Â Use kmalloc_array() in three functions
> Â Enclose 17 expressions for the sizeof operator by parentheses
> Â Split a condition check in five functions
> Â Delete an unnecessary variable in __mlx4_ib_query_gid()
> Â Delete an unnecessary return statement in do_slave_init()
> Â Improve another size determination in do_slave_init()
> Â Improve another size determination in mlx4_ib_add()
> Â Delete an unnecessary variable initialisation in mlx4_ib_add()
> Â Delete an unnecessary variable assignment in mlx4_ib_add()
> Â Delete an error message for a failed memory allocation in
> mlx4_ib_add()
> Â Delete unnecessary braces in mlx4_ib_add()
> Â Use kmalloc_array() in alloc_proxy_bufs()
> Â Improve size determinations in create_qp_common()
> Â Delete unwanted spaces behind usages of the sizeof operator
> Â Add spaces for better code readability
> Â Enclose 14 expressions for the sizeof operator by parentheses
> Â Use kmalloc_array() in create_kernel_qp()
> Â Less function calls in create_kernel_qp() after error detection
> Â Use kmalloc_array() in create_srq_kernel()
> 
> Âdrivers/infiniband/hw/mlx4/mad.cÂÂ|ÂÂ72 ++++++++--------
> Âdrivers/infiniband/hw/mlx4/main.c | 164 ++++++++++++++++++++------
> ----------
> Âdrivers/infiniband/hw/mlx4/qp.cÂÂÂ| 173 +++++++++++++++++++---------
> ----------
> Âdrivers/infiniband/hw/mlx5/qp.cÂÂÂ|ÂÂ66 ++++++++++-----
> Âdrivers/infiniband/hw/mlx5/srq.cÂÂ|ÂÂÂ5 +-
> Â5 files changed, 261 insertions(+), 219 deletions(-)
> 
-- 
Doug Ledford <dledford@xxxxxxxxxx>
  GPG KeyID: B826A3330E572FDD
 Â
Key fingerprint = AE6B 1BDA 122B 23B4 265B Â1274 B826 A333 0E57 2FDD