From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by dpdk.org (Postfix) with ESMTP id C9BD9532C for ; Fri, 27 Jul 2018 18:26:03 +0200 (CEST) Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.rdu2.redhat.com [10.11.54.6]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 0B59E87A74; Fri, 27 Jul 2018 16:26:03 +0000 (UTC) Received: from ktraynor.remote.csb (ovpn-117-208.ams2.redhat.com [10.36.117.208]) by smtp.corp.redhat.com (Postfix) with ESMTP id 6BCF52156701; Fri, 27 Jul 2018 16:25:59 +0000 (UTC) To: Ilya Maximets , Yongseok Koh , dpdk stable , Luca Boccassi Cc: Maxime Coquelin , Peng He , Tiwei Bie References: <20180727024008.39497-1-yskoh@mellanox.com> <20180727074215eucas1p19e45258ec15a10881b3006b70723c3f2~FKeqOKwSp3270732707eucas1p1b@eucas1p1.samsung.com> From: Kevin Traynor Organization: Red Hat Message-ID: <80ad5117-ceb1-7d59-0e12-96f4ab4ed90d@redhat.com> Date: Fri, 27 Jul 2018 17:25:59 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.0 MIME-Version: 1.0 In-Reply-To: <20180727074215eucas1p19e45258ec15a10881b3006b70723c3f2~FKeqOKwSp3270732707eucas1p1b@eucas1p1.samsung.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.78 on 10.11.54.6 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.1]); Fri, 27 Jul 2018 16:26:03 +0000 (UTC) X-Greylist: inspected by milter-greylist-4.5.16 (mx1.redhat.com [10.11.55.1]); Fri, 27 Jul 2018 16:26:03 +0000 (UTC) for IP:'10.11.54.6' DOMAIN:'int-mx06.intmail.prod.int.rdu2.redhat.com' HELO:'smtp.corp.redhat.com' FROM:'ktraynor@redhat.com' RCPT:'' Subject: Re: [dpdk-stable] please help backporting some patches to LTS release 17.11.4 X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 27 Jul 2018 16:26:04 -0000 On 07/27/2018 08:42 AM, Ilya Maximets wrote: > [keeping in CC only e11411b52 related persons] > [CC: Luca Boccassi] > > Hi. > I beleive that I was added to this thread as a reviewer of > "e11411b52 Maxime Coquelin vhost: fix missing increment of log cache count". > > Looking at the current dpdk-stable/17.11 I see that the patch could not be > applied because it fixes issue introduced by another patch > c16915b87109 ("vhost: improve dirty pages logging performance") that doesn't > exist in this branch. OTOH, this patch exists in stable 16.11. > So, I have a question: Is there any reason why c16915b87109 was backported > to 16.11 but not to 17.11 ? > Hi Ilya, Guessing it was because it was an optimization as opposed to a functional fix. See below commit wrt clarifying this - ref last para. Kevin. commit 56c7ddc1381a56612018b75a848169b917bc33fb Author: Kevin Traynor Date: Sat Jun 30 00:10:43 2018 +0100 doc: add default that all fixes should be backported Set the starting point that all commits on master branch with Fixes tag should be backported to relevant stable/LTS branches, and explain that the submitter may indicate it is not suitable for backport. Of course there will be exceptions that will crop up from time to time that need discussion, so also add a sentence for that. This is to ensure that there is consistency between what is backported to stable/LTS branches, remove some subjectivity as to what constitutes "a fix" and avoid possible conflicts for future backports. Signed-off-by: Kevin Traynor Acked-by: Luca Boccassi Acked-by: Aaron Conole Acked-by: Thomas Monjalon > P.S. I don't feel that I should backport e11411b52 myself, but I can do that > if author/maintainers are too busy. Please, let me know in that case. > Regarding 16.11, to backport e11411b52 you only need to change the > destination file,because related code placed in > lib/librte_vhost/virtio_net.c instead of lib/librte_vhost/vhost.h. > > Best regards, Ilya Maximets. > > On 27.07.2018 05:40, Yongseok Koh wrote: >> Hi commit authors (and maintainers), >> >> I didn't manage to apply following commits from upstream to stable branch >> 17.11: conflict happens. I'm wondering can the authors check the following >> list and backport those patches belong to you? >> >> FYI, branch 17.11 is located at tree: >> git://dpdk.org/dpdk-stable >> >> It'd be great if you could do that in one or two weeks. Also, please add a >> heading line like below before the commit log body: >> [ backported from upstream commit xxx ] >> >> Example: http://dpdk.org/browse/dpdk-stable/commit/?h=16.07&id=c4831394c7d1944d8ec27d52c22997f20d19718e >> >> Also please mention the target LTS in the subject line, as we have more than one >> at the same time, for example: >> >> [PATCH 17.11] foo/bar: fix baz >> >> With git send-email, this can be achieved by appending the parameter: >> >> --subject-prefix='17.11' >> >> Please let me know if you have any comments, say, need more time, or it's >> worthless to packport it. And please send it to "stable@dpdk.org", but not >> "dev@dpdk.org". >> >> Thanks. >> >> Yongseok >> >> --- >> 422515b9a Adrien Mazarguil app/testpmd: fix crash when attaching a device >> 66f2ac0f8 Adrien Mazarguil net/mlx4: fix minor resource leak during init >> 0cc1fcdeb Ajit Khaparde net/bnxt: check filter type before clearing it >> 6a9360963 Ajit Khaparde net/bnxt: fix clear port stats >> 3c2e87c10 Ajit Khaparde net/bnxt: fix close operation >> 90362583e Ajit Khaparde net/bnxt: fix incorrect IO address handling in Tx >> 812bb2087 Andrew Rybchenko net/sfc: handle unknown L3 packet class in EF10 event parser >> 2a4ed7233 Damjan Marion net/i40e: do not reset device info data >> 64cb90f88 Fiona Trahe crypto/qat: fix checks for 3GPP algo bit params >> 2a7e3d546 Hyong Youb Kim net/enic: do not overwrite admin Tx queue limit >> e11411b52 Maxime Coquelin vhost: fix missing increment of log cache count >> 79d098921 Moti Haimovsky net/mlx5: fix build with old kernels >> c44fbc7cc Nelio Laranjeiro net/mlx5: clean-up developer logs >> 179c7e893 Pavan Nikhilesh mempool/octeontx: fix pool to aura mapping >> 196e9a486 Qi Zhang eal: fix hotplug add and remove >> 9f32c7e7e Rafal Kozik net/ena: change memory type >> 46916aa17 Rafal Kozik net/ena: check pointer before memset >> 41e59028d Rafal Kozik net/ena: set link speed as none >> 3abe17195 Rahul Lakkireddy net/cxgbe: fix Rx channel map and queue type >> 1fc3afdf7 Rasesh Mody doc: update qede management firmware guide >> c6dd1eb8a Rasesh Mody net/qede: fix default extended VLAN offload config >> a16aef523 Rasesh Mody net/qede: fix for devargs >> e46821e9f Shahaf Shuler net/mlx5: separate generic tunnel TSO from the standard one >> 496f15c2c Shahed Shaikh net/qede: fix incorrect link status update >> c7641841a Shahed Shaikh net/qede: fix unicast MAC address handling in VF >> 82dbef2c8 Somnath Kotur net/bnxt: revert reset of L2 filter id >> 0989161b2 Tiwei Bie vhost: release locks on RARP packet failure >> b2a4a69f7 Xiaoxin Peng net/bnxt: fix Tx with multiple mbuf >> >>