From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2 7/7]app/testpmd: add commands and config functions for i40e flow director support Date: Thu, 28 Aug 2014 13:00:33 +0200 Message-ID: <32773310.iMGaa2PpNO@xps13> References: <1409105634-29980-1-git-send-email-jingjing.wu@intel.com> <6406194.6E43HquPHH@xps13> <9BB6961774997848B5B42BEC655768F8ADC10D@SHSMSX104.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: "Wu, Jingjing" Return-path: In-Reply-To: <9BB6961774997848B5B42BEC655768F8ADC10D-0J0gbvR4kTg/UvCtAeCM4rfspsVTdybXVpNB7YpNyf8@public.gmane.org> 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-08-28 09:01, Wu, Jingjing: > I have another question: > If we use the way 'rx_classification_filter_ctl' works, the specific > structures defined in rte_i40e.h will be visible in user's application, > such as testpmd. > I know I shouldn't make commands linked with i40e like what I did before. > But will the i40e specific structures become visible be acceptable? I think testpmd should be limited to generic API. So it wouldn't be acceptable to be dependent of i40e files. But having some specific i40e tests in examples or app/test is OK. -- Thomas