From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH] ixgbe: Fix compilation issue in vpmd Date: Tue, 21 Oct 2014 10:36:22 +0200 Message-ID: <1652314.f9VDNlESRS@xps13> References: <1413874756-18669-1-git-send-email-changchun.ouyang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: "Ouyang, Changchun" , "De Lara Guarch, Pablo" Return-path: In-Reply-To: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" 2014-10-21 08:28, Ouyang, Changchun: > From: De Lara Guarch, Pablo > > From: dev [mailto:dev-bounces-VfR2kkLFssw@public.gmane.org] On Behalf Of Thomas Monjalon > > > 2014-10-21 14:59, Ouyang Changchun: > > > > Fix the compilation issue in vector PMD when macro RTE_MBUF_REFCNT > > > > is disabled. > > > > > > > > Signed-off-by: Changchun Ouyang > > > > > > Acked-by: Thomas Monjalon > > > > > > Applied > > > > I was checking this patch right now, and I come across a second compilation > > issue, because rte_mbuf_refcnt_update and rte_pktmbuf_attach are not > > declared, and Bond PMD and IP fragmentation libraries use those functions. > > > > I guess that it is late to NACK this :P, but we require a second patch to fix > > completely this issue. > > As it fixes the compilation issue in vpmd, so no reason to NACK it, :-) Exact > In my config, both BOND and IP fragment is disabled. So I don't come across your issues. > Yes, agree with you, we need another patch to fix compilation issue in other both places. Yes, I'm aware of these limitations. Please, first explain why mbuf refcnt is needed for these features. Then we have 2 options: remove the dependency or add more ifdefs. Thanks -- Thomas