From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id 26C975320 for ; Wed, 21 Jun 2017 10:33:02 +0200 (CEST) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id BCDBD2096C; Wed, 21 Jun 2017 04:33:01 -0400 (EDT) Received: from frontend1 ([10.202.2.160]) by compute1.internal (MEProxy); Wed, 21 Jun 2017 04:33:01 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc:x-sasl-enc; s=mesmtp; bh=RxYls3ptTFShr7H r0HCRBdY2N/MPW6XzqZ5BtaKN5KM=; b=VZ2YHKjUY8gyCLj3Sq56Ha1APPMTftY ZI9RrMQu3ZHrEbM6B/wdRqHXb4BLxOkbC2iOgNhRh3m5t6zQ8qP5p0EKJMFc91wL rHfUXeKgOtijrkmZWpYQU0IFKmJBNEU/RFY3xHYajgdCHo6+pI8VBJ0Wrtak4rOM Sju02+va0+EI= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-sender:x-me-sender:x-sasl-enc:x-sasl-enc; s= fm1; bh=RxYls3ptTFShr7Hr0HCRBdY2N/MPW6XzqZ5BtaKN5KM=; b=Ra9fRirr 3jFWDlwHFxTkTEoIg0P5FZk8H7UlNYGL0+SYadZeY+Yvb6ljyQ5qIpuuTQ2gYK68 BmSmC46tu+VKCyF7XQlLDumAmBmC7tBsok37qrInwLglZPRLM73sflvDD+7m8VRf Bxpyi25+0S7UcHHaoHJCnTQn826AwkgLYIVz5CxXYUP89q5oBEA5lWdgvECFWrBc /gRmYeg5k0WJlTHiRN5EarwMcDguu38xMQ1dfBKQiPOhnAZnSkzVDjnLk9zMFff3 JYq22HxIWiTuDvQkidek4CC+p1HXbY8ctmjgswbTN65Tb8QZr+P4j2go5U3g1LlL q4DlFfoPXCku1Q== X-ME-Sender: X-Sasl-enc: stnxWNaF+IucYPN6UNRj9DFraB179ySjxvbNhPuzHA5t 1498033981 Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id 5E1737E6B1; Wed, 21 Jun 2017 04:33:01 -0400 (EDT) From: Thomas Monjalon To: "Wei, FangfangX" Cc: ci@dpdk.org, "Richardson, Bruce" , "Xu, Qian Q" , "O'Driscoll, Tim" , Eugene Voronov Date: Wed, 21 Jun 2017 10:33:00 +0200 Message-ID: <1814490.xy7qWLUraa@xps> In-Reply-To: <59AF69C657FD0841A61C55336867B5B0721805ED@IRSMSX104.ger.corp.intel.com> References: <26FA93C7ED1EAA44AB77D62FBE1D27BA722C837C@IRSMSX108.ger.corp.intel.com> <067B569323FEB248B5CB480E1954F4346F4174FD@SHSMSX101.ccr.corp.intel.com> <59AF69C657FD0841A61C55336867B5B0721805ED@IRSMSX104.ger.corp.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-ci] script to determine target repo (was DPDK Lab) X-BeenThere: ci@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK CI discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 21 Jun 2017 08:33:02 -0000 I agree with Bruce. Sorry for not having written the scripts yet. Someone else in Mellanox should do it in July. In the meantime, do not hesitate to share your code if it speed up things. Thanks a lot 21/06/2017 10:20, Richardson, Bruce: > Hi Fangfang, > > My comments on the questions you asked: > > 1. For determining the repo tree to target, I don't believe that we can ever come up with a 100% accurate rule, as the tree to which a set is to be applied can be difficult to determine, so it may be done on the basis of on-list discussion. A 90% accurate rule it what we may have to accept. However, since applying a patchset to a tree should not be a time-consuming operation, I suggest any script produce a list of possible trees in priority order to try. If not net, then try main, etc. etc. > > 2. Using the order from patch titles is correct. > > /Bruce > > > -----Original Message----- > > From: Wei, FangfangX > > Sent: Wednesday, June 21, 2017 8:46 AM > > To: Thomas Monjalon ; Xu, Qian Q > > > > Cc: ci@dpdk.org; Richardson, Bruce ; > > O'Driscoll, Tim > > Subject: RE: [dpdk-ci] script to determine target repo (was DPDK Lab) > > > > Hi Thomas, > > > > What's the status about the script which determine target repo for each > > patch set? > > Currently I used the rule which you told me before: "the subject prefix on > > the patches should provide the hint. If the majority of the patches start > > with "net/" then it's for next-net, and similarly for crypto. Patches with > > virtio or vhost go to that tree, etc". > > But I'm not sure about the 100% correct repo. > > Looking forward to your script. > > > > Another thing, Qian told me that you want to know the apply sequence about > > many patches in one patch set. > > I don't apply patches with the order of patchwork id, but the names' order > > about them. > > Such as, patchset 25081-25085, the order of patchwork id is not the > > correct patches' order. > > http://dpdk.org/patch/25081, subject: [dpdk-dev,v2,1/5] net/qede: > > refactoring vport handling code http://dpdk.org/patch/25083, subject: > > [dpdk-dev,v2,2/5] net/qede: refactoring multi-queue implementation > > http://dpdk.org/patch/25084, subject: [dpdk-dev,v2,3/5] net/qede/base: > > upgrade the FW to 8.20.0.0 http://dpdk.org/patch/25082, subject: [dpdk- > > dev,v2,4/5] net/qede: fix VXLAN tunnel Tx offload flag setting > > http://dpdk.org/patch/25085, subject: [dpdk-dev,v2,5/5] net/qede: refactor > > Tx routine After download them, the patches' names are as below: > > http://dpdk.org/patch/25081: dpdk-dev-v2-1-5-net-qede-refactoring-vport- > > handling-code.patch > > http://dpdk.org/patch/25083: dpdk-dev-v2-2-5-net-qede-refactoring-multi- > > queue-implementation.patch > > http://dpdk.org/patch/25084: dpdk-dev-v2-3-5-net-qede-base-upgrade-the-FW- > > to-8.20.0.0.patch > > http://dpdk.org/patch/25082: dpdk-dev-v2-4-5-net-qede-fix-VXLAN-tunnel-Tx- > > offload-flag-setting.patch > > http://dpdk.org/patch/25085: dpdk-dev-v2-5-5-net-qede-refactor-Tx- > > routine.patch > > > > So applying these patches based on the order of their names, it will be > > correct. > > > > > > Best Regards > > Fangfang Wei > > > > > > -----Original Message----- > > From: Thomas Monjalon [mailto:thomas@monjalon.net] > > Sent: Tuesday, April 25, 2017 2:51 PM > > To: Xu, Qian Q > > Cc: ci@dpdk.org; Richardson, Bruce ; Wei, > > FangfangX ; O'Driscoll, Tim > > ; Liu, Yuanhan > > Subject: Re: [dpdk-ci] script to determine target repo (was DPDK Lab) > > > > 25/04/2017 08:23, Xu, Qian Q: > > > Thanks Thomas and Bruce for your comments. Currently we are trying to > > > start the per patch set performance test investigation and need to know > > the Target repo for each patch set. > > > The goal is to get 100% correct repo to go for each patch set. It's > > important for the credit of the performance result. > > > @Thomas, could you provide any script to support the auto-filter after > > the 17.05 release? Currently, we don't have the ability to make it. > > > > OK, it is on my todo list. > > > > Other task: I want to check the series support in the patchwork git.