From patchwork Fri Apr 22 21:14:35 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Thomas Monjalon X-Patchwork-Id: 12217 Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 6B69A2BE3; Fri, 22 Apr 2016 23:14:59 +0200 (CEST) Received: from mail-wm0-f50.google.com (mail-wm0-f50.google.com [74.125.82.50]) by dpdk.org (Postfix) with ESMTP id C99232BD9 for ; Fri, 22 Apr 2016 23:14:58 +0200 (CEST) Received: by mail-wm0-f50.google.com with SMTP id u206so44427505wme.1 for ; Fri, 22 Apr 2016 14:14:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:organization:user-agent :in-reply-to:references:mime-version:content-transfer-encoding; bh=VSZKBWPp/xo3FEdjlVoTIOFBP5vtOawGxw2OaUyATyY=; b=yZ/DbrngRlYdqIFpBC4DPjFlryQQo1S2WQ/EcA/Cbi3OchSG63wTtd78DH15PUW7Yk kliDtwilAn7LBKoXlHOdi6z/tJAW77duU9cnXz2+cGm2cPIhrNR/hwsQQ+PC4Kj6/evH 9TPA62TVXPnlojRMNC1nk8tI+/pH4lGdZpr8K7EbF9+hPtHwq1rQ6BzQ2uJh1HuU/rma p/JLu6W7gUQBxGuvmMzPiqgNVPrZO0E/KVrempIC8jugJIO7N4onz/3NECDxbHRV/8eg qUSNATXNa26nHZ7J6Y1Zl0drrxiL1u59oG8VAztG/lRPdo9Vaurjw/DN2s6RcADVshXd r6ag== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:date:message-id:organization :user-agent:in-reply-to:references:mime-version :content-transfer-encoding; bh=VSZKBWPp/xo3FEdjlVoTIOFBP5vtOawGxw2OaUyATyY=; b=RzUcRq0/kV/S0SWZYTfcBhAUySOTeAwztnTRREeYbJ3M9JgrsWL5hXs6E8EFd4KOVc RzTHT5xqO69oO7URzvXVfG4n1WuLrmTJD/WzaNOxSd7BjHXvMej1QZeRFXFi05pJofQZ 2aQmSF4yB9n0MHI2lr9cTxP0qoHWkBOw9ZcpHTa0Z7rIY/rOyQQGrRpnQTPYZyFFdP9a ZfJB86Fh3aDCFMHu9MBuS/1T+XbXkmNBO6mLHQvDV85/+XnRYJATMHuSK5/0C/kLtdCA pMIuO+r4FqOGgzRDQBSCsA5tXWboKPWtA1UAgBNCH6ZU+L5MMzfFOl9HjQ77dUa1e1sY mjmQ== X-Gm-Message-State: AOPr4FWLYDOqBAMtxgKw13pqpJMSG0p6bxiyG2lTYOsWFKb5dXS+UtAt/XF7mkA7luPdJOC/ X-Received: by 10.28.187.5 with SMTP id l5mr5548165wmf.17.1461359698589; Fri, 22 Apr 2016 14:14:58 -0700 (PDT) Received: from xps13.localnet (APoitiers-658-1-63-44.w86-221.abo.wanadoo.fr. [86.221.58.44]) by smtp.gmail.com with ESMTPSA id o73sm5134040wme.16.2016.04.22.14.14.57 (version=TLSv1/SSLv3 cipher=OTHER); Fri, 22 Apr 2016 14:14:57 -0700 (PDT) From: Thomas Monjalon To: Yuanhan Liu Cc: dev@dpdk.org Date: Fri, 22 Apr 2016 23:14:35 +0200 Message-ID: <4685487.eCM9t2Cvuq@xps13> Organization: 6WIND User-Agent: KMail/4.14.10 (Linux/4.1.6-1-ARCH; KDE/4.14.11; x86_64; ; ) In-Reply-To: <20160422195110.GE7603@yliu-dev.sh.intel.com> References: <1461332640-24273-1-git-send-email-thomas.monjalon@6wind.com> <1461332640-24273-5-git-send-email-thomas.monjalon@6wind.com> <20160422195110.GE7603@yliu-dev.sh.intel.com> MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH 4/4] eal: add assert macro for debug X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2016-04-22 12:51, Yuanhan Liu: > On Fri, Apr 22, 2016 at 03:44:00PM +0200, Thomas Monjalon wrote: > > --- a/drivers/net/vmxnet3/vmxnet3_rxtx.c > > +++ b/drivers/net/vmxnet3/vmxnet3_rxtx.c > > @@ -662,8 +662,8 @@ vmxnet3_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t nb_pkts) > > > > PMD_RX_LOG(DEBUG, "rxd idx: %d ring idx: %d.", idx, ring_idx); > > > > - VMXNET3_ASSERT(rcd->len <= rxd->len); > > - VMXNET3_ASSERT(rbi->m); > > + RTE_ASSERT(rcd->len <= rxd->len); > > + RTE_ASSERT(rbi->m); > > Seems this piece of code introduces a build error: > > drivers/net/vmxnet3/vmxnet3_rxtx.c: In function ‘vmxnet3_recv_pkts’: > drivers/net/vmxnet3/vmxnet3_rxtx.c:635:18: error: variable ‘rxd’ set but not used > Vmxnet3_RxDesc *rxd; > ^ Indeed it can be seen with gcc when debug is disabled. It can be fixed this way: --- a/drivers/net/vmxnet3/vmxnet3_rxtx.c +++ b/drivers/net/vmxnet3/vmxnet3_rxtx.c @@ -658,6 +658,7 @@ vmxnet3_recv_pkts(void *rx_queue, struct rte_mbuf **rx_pkts, uint16_t nb_pkts) idx = rcd->rxdIdx; ring_idx = (uint8_t)((rcd->rqID == rxq->qid1) ? 0 : 1); rxd = (Vmxnet3_RxDesc *)rxq->cmd_ring[ring_idx].base + idx; + RTE_SET_USED(rxd); /* used only for assert when enabled */ rbi = rxq->cmd_ring[ring_idx].buf_info + idx; PMD_RX_LOG(DEBUG, "rxd idx: %d ring idx: %d.", idx, ring_idx);