From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] ethdev: fix wrong memset Date: Fri, 20 Jan 2017 11:20:06 +0100 Message-ID: <2005966.sVKUI0LeEs@xps13> References: <1484899493-11051-1-git-send-email-yuanhan.liu@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, Ferruh Yigit To: Yuanhan Liu Return-path: Received: from mail-wm0-f46.google.com (mail-wm0-f46.google.com [74.125.82.46]) by dpdk.org (Postfix) with ESMTP id B5CD22BA7 for ; Fri, 20 Jan 2017 11:20:07 +0100 (CET) Received: by mail-wm0-f46.google.com with SMTP id r126so31046930wmr.0 for ; Fri, 20 Jan 2017 02:20:07 -0800 (PST) In-Reply-To: <1484899493-11051-1-git-send-email-yuanhan.liu@linux.intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2017-01-20 16:04, Yuanhan Liu: > Fix an silly error by auto-complete while managing the merge conflicts. > It's the eth_dev_data (but not eth_dev) entry should be memset. > > Fixes: d948f596fee2 ("ethdev: fix port data mismatched in multiple process model") You should describe the impact on this bug. It will be helpful for those testing the RC1. > - memset(&rte_eth_devices[port_id], 0, sizeof(*eth_dev->data)); > + memset(&rte_eth_dev_data[port_id], 0, sizeof(struct rte_eth_dev_data)); The title should be contain the scope of the bug. I suggest "fix data reset when allocating port".