From mboxrd@z Thu Jan 1 00:00:00 1970 From: kbuild test robot Subject: [PATCH] qed: fix kzalloc-simple.cocci warnings Date: Fri, 21 Apr 2017 08:20:07 +0800 Message-ID: <20170421002007.GA8379@lkp-sbx04> References: <201704210853.tUPLU4Mh%fengguang.wu@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: kbuild-all@01.org, netdev@vger.kernel.org, Sudarsana Reddy Kalluru , Yuval Mintz , Ariel Elior , everest-linux-l2@cavium.com, linux-kernel@vger.kernel.org To: "sudarsana.kalluru@cavium.com" Return-path: Received: from mga06.intel.com ([134.134.136.31]:50743 "EHLO mga06.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1031188AbdDUAUf (ORCPT ); Thu, 20 Apr 2017 20:20:35 -0400 Content-Disposition: inline In-Reply-To: <201704210853.tUPLU4Mh%fengguang.wu@intel.com> Sender: netdev-owner@vger.kernel.org List-ID: drivers/net/ethernet/qlogic/qed/qed_dcbx.c:1267:13-20: WARNING: kzalloc should be used for dcbx_info, instead of kmalloc/memset Use kzalloc rather than kmalloc followed by memset with 0 This considers some simple cases that are common and easy to validate Note in particular that there are no ...s in the rule, so all of the matched code has to be contiguous Generated by: scripts/coccinelle/api/alloc/kzalloc-simple.cocci CC: sudarsana.kalluru@cavium.com Signed-off-by: Fengguang Wu --- qed_dcbx.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) --- a/drivers/net/ethernet/qlogic/qed/qed_dcbx.c +++ b/drivers/net/ethernet/qlogic/qed/qed_dcbx.c @@ -1264,11 +1264,10 @@ static struct qed_dcbx_get *qed_dcbnl_ge { struct qed_dcbx_get *dcbx_info; - dcbx_info = kmalloc(sizeof(*dcbx_info), GFP_ATOMIC); + dcbx_info = kzalloc(sizeof(*dcbx_info), GFP_ATOMIC); if (!dcbx_info) return NULL; - memset(dcbx_info, 0, sizeof(*dcbx_info)); if (qed_dcbx_query_params(hwfn, dcbx_info, type)) { kfree(dcbx_info); return NULL;