From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v5 01/13] librte_ether: modify internal callback function Date: Thu, 06 Oct 2016 17:41:28 +0200 Message-ID: <4741428.MAME74GNI7@xps13> References: <1475592734-22355-1-git-send-email-bernard.iremonger@intel.com> <1884200.vdWHEKFaxn@xps13> <8CEF83825BEC744B83065625E567D7C21A08F8EF@IRSMSX108.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, "Shah, Rahul R" , "Lu, Wenzhuo" , "az5157@att.com" , "jerin.jacob@caviumnetworks.com" To: "Iremonger, Bernard" Return-path: Received: from mail-wm0-f51.google.com (mail-wm0-f51.google.com [74.125.82.51]) by dpdk.org (Postfix) with ESMTP id 710145934 for ; Thu, 6 Oct 2016 17:41:32 +0200 (CEST) Received: by mail-wm0-f51.google.com with SMTP id i130so12059216wmg.1 for ; Thu, 06 Oct 2016 08:41:32 -0700 (PDT) In-Reply-To: <8CEF83825BEC744B83065625E567D7C21A08F8EF@IRSMSX108.ger.corp.intel.com> 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-10-06 15:32, Iremonger, Bernard: > > > > 2016-10-06 12:26, Bernard Iremonger: > > > > > void > > > > > _rte_eth_dev_callback_process(struct rte_eth_dev *dev, > > > > > - enum rte_eth_event_type event) > > > > > + enum rte_eth_event_type event, void *param) > > > > > > > > You need to squash the patches updating the calls to this function. > > > > Otherwise, this patch does not compile. > > > > > > I will have to squash everything into one patch, separate patches will not > > compile. > > > > No you can keep a separate patch for the VF event in ixgbe. > > I have 4 patches at present > > librte_ether > net/ixgbe > drivers/net > app/test > > Would this be acceptable or do you just want everything squashed into librte_ether except for net/ixgbe? You can test the compilation of each patch with this command: git rebase -i origin/master -x scripts/test-build.sh or just: git rebase -i origin/master -x make I think you need to squash drivers/net (adding just NULL param) and app/test in ethdev patch.