----- Original Message ----- | From: Markus Elfring <elfring@xxxxxxxxxxxxxxxxxxxxx> | Date: Sat, 6 May 2017 10:15:25 +0200 | | Some update suggestions were taken into account | from static source code analysis. | | Markus Elfring (10): | Replace six seq_puts() calls by seq_putc() | Add spaces for better code readability | Improve a size determination in table_seq_start() | Use kcalloc() in dlm_scan_waiters() | Improve a size determination in dlm_recover_waiters_pre() | Delete an error message for a failed memory allocation in | dlm_recover_waiters_pre() | Use kmalloc_array() in make_member_array() | Use kcalloc() in two functions | Improve a size determination in two functions | Delete an unnecessary variable initialisation in dlm_ls_start() | | fs/dlm/debug_fs.c | 25 ++++++++++++------------- | fs/dlm/lock.c | 8 +++----- | fs/dlm/member.c | 15 ++++++--------- | 3 files changed, 21 insertions(+), 27 deletions(-) | | -- | 2.12.2 | | Hi Markus, I'm not a DLM maintainer, but I've reviewed your patch set and ACK them. They seem to fit the spirit of: https://www.kernel.org/doc/html/v4.10/process/coding-style.html#allocating-memory I've actually got a bunch of similar cleanups myself, but I don't think there's any overlap. Regards, Bob Peterson Red Hat File Systems -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
