From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr1-f66.google.com (mail-wr1-f66.google.com [209.85.221.66]) by dpdk.org (Postfix) with ESMTP id 42FE537A2 for ; Fri, 27 Jul 2018 11:23:25 +0200 (CEST) Received: by mail-wr1-f66.google.com with SMTP id r16-v6so4343072wrt.11 for ; Fri, 27 Jul 2018 02:23:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=message-id:subject:from:to:cc:date:in-reply-to:references :content-transfer-encoding:mime-version; bh=yU+14NDrZMJU9W2dS0NiVMr4wLEDkcejnApNRcYHPn8=; b=P1wJ4JVrnq0WTKZYjuQcZjiyfgTituYP8k/UsJcY/Xg9d8uBZW6Z+eovNSchf03i+H xHoPgHccWBym8CGvSzvwFoGDvTTElrj2S0iZIcUDQLZNTy1N+412Gq1V5UGIy8MgA8Q4 oYgBrolHn9z1pcVkwQZe0BuNFwtTNzYrj4Za3Zvk1CCXdlWINphJ1jC0hVVuR+ABzjaR 8R2eQQj+n5J5jI00pw2EHnCfwjBc4FsVWVM5QJ+W4HJCynyv7yfQ7P5YkTml1vdV03aX an4Yb4sLrSo6aTL14DUa4qrS0gnJX7fNvYoPnO2rwHyzEdJ4y7zOnXA+mxA4ls1covtD vneg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to :references:content-transfer-encoding:mime-version; bh=yU+14NDrZMJU9W2dS0NiVMr4wLEDkcejnApNRcYHPn8=; b=Ea+1ca9eHFEBnpSFKK6hL2/mQp0jUSoq4gdAm5pYPFhi3Kvplw+BEkjlTTCMLz9agr Ti/uvqHvZhVjraAgOys9A+Pv9w+bQF3tmU7GuU4/y/ahQmGwcCfHSG2N+21mT08D1Dc/ UJT1lgyQWXRjDFDF00B1m1AW1Qfhi/+hdyBYIglitqfEuAB6ZMZoE8mDp65QPsJ4X90k PKxBLk6LsOtIoalf/8U1VLKc7OrDKkiWHY0oLfWP4e4d77RzC5SCrRAHoApgI+8wS52o g4wxPcAzzc2XOWFfJ/tW6SE/vCgbPJke2GVIJN9tUjV0JscowuZRPG2kAeEYpk8PQad9 EPYw== X-Gm-Message-State: AOUpUlHGDZI4hN+UXc4CLBbXisk4PCgWStf0XVoW1eBSycBfFsmy3oSP g2aJksDvUwfciyiEeqnIEvk= X-Google-Smtp-Source: AAOMgpcs4AoRJ1ZWXY20iRBlzo3W2YsNeEyXAE2AcUZlucTtTMbJIUmZv8okgZTzMGoqO6CmULADFQ== X-Received: by 2002:a5d:4cc5:: with SMTP id c5-v6mr4259401wrt.210.1532683404872; Fri, 27 Jul 2018 02:23:24 -0700 (PDT) Received: from localhost ([2a00:23c5:be94:4600:7b12:f8a0:fd52:f87d]) by smtp.gmail.com with ESMTPSA id x62-v6sm1936717wmg.1.2018.07.27.02.23.23 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 27 Jul 2018 02:23:23 -0700 (PDT) Message-ID: <1532683402.20916.33.camel@gmail.com> From: Luca Boccassi To: Ilya Maximets , Yongseok Koh , dpdk stable Cc: Maxime Coquelin , Peng He , Tiwei Bie Date: Fri, 27 Jul 2018 10:23:22 +0100 In-Reply-To: <20180727074215eucas1p19e45258ec15a10881b3006b70723c3f2~FKeqOKwSp3270732707eucas1p1b@eucas1p1.samsung.com> References: <20180727024008.39497-1-yskoh@mellanox.com> <20180727074215eucas1p19e45258ec15a10881b3006b70723c3f2~FKeqOKwSp3270732707eucas1p1b@eucas1p1.samsung.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Mailer: Evolution 3.22.6-1+deb9u1 Mime-Version: 1.0 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 09:23:25 -0000 On Fri, 2018-07-27 at 10:42 +0300, Ilya Maximets wrote: > [keeping in CC only e11411b52 related persons] > [CC: Luca Boccassi] >=20 > Hi. > I beleive that I was added to this thread as a reviewer of > "e11411b52=C2=A0=C2=A0Maxime Coquelin=C2=A0=C2=A0vhost: fix missing incre= ment of log > cache count". >=20 > 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 ? >=20 > P.S. I don't feel that I should backport e11411b52 myself, but I can > do that > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0if author/maintainers are too busy. Please,= let me know in that > case. > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0Regarding 16.11, to backport e11411b52 you = only need to change > the > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0destination file,because related code place= d in > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0lib/librte_vhost/virtio_net.c instead of > lib/librte_vhost/vhost.h. >=20 > Best regards, Ilya Maximets. Hi Ilya, Thanks for the hint w.r.t. 16.11, I have now applied and pushed the patch to dpdk-stable/16.11. > On 27.07.2018 05:40, Yongseok Koh wrote: > > Hi commit authors (and maintainers), > >=20 > > 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? > >=20 > > FYI, branch 17.11 is located at tree: > > =C2=A0=C2=A0=C2=A0git://dpdk.org/dpdk-stable > >=20 > > 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: > > =C2=A0=C2=A0=C2=A0=C2=A0[ backported from upstream commit xxx ] > >=20 > > Example: http://dpdk.org/browse/dpdk-stable/commit/?h=3D16.07&id=3Dc483 > > 1394c7d1944d8ec27d52c22997f20d19718e > >=20 > > Also please mention the target LTS in the subject line, as we have > > more than one > > at the same time, for example: > >=20 > > =C2=A0=C2=A0=C2=A0=C2=A0[PATCH 17.11] foo/bar: fix baz > >=20 > > With git send-email, this can be achieved by appending the > > parameter: > >=20 > > =C2=A0=C2=A0=C2=A0=C2=A0--subject-prefix=3D'17.11' > >=20 > > 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". > >=20 > > Thanks. > >=20 > > Yongseok > >=20 > > --- > > 422515b9a=C2=A0=C2=A0Adrien Mazarguil app/testpmd: fix crash when attac= hing a > > device > > 66f2ac0f8=C2=A0=C2=A0Adrien Mazarguil net/mlx4: fix minor resource leak > > during init > > 0cc1fcdeb=C2=A0=C2=A0Ajit Khaparde=C2=A0=C2=A0=C2=A0=C2=A0net/bnxt: che= ck filter type before > > clearing it > > 6a9360963=C2=A0=C2=A0Ajit Khaparde=C2=A0=C2=A0=C2=A0=C2=A0net/bnxt: fix= clear port stats > > 3c2e87c10=C2=A0=C2=A0Ajit Khaparde=C2=A0=C2=A0=C2=A0=C2=A0net/bnxt: fix= close operation > > 90362583e=C2=A0=C2=A0Ajit Khaparde=C2=A0=C2=A0=C2=A0=C2=A0net/bnxt: fix= incorrect IO address > > handling in Tx > > 812bb2087=C2=A0=C2=A0Andrew Rybchenko net/sfc: handle unknown L3 packet= class > > in EF10 event parser > > 2a4ed7233=C2=A0=C2=A0Damjan Marion=C2=A0=C2=A0=C2=A0=C2=A0net/i40e: do = not reset device info data > > 64cb90f88=C2=A0=C2=A0Fiona Trahe=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0cry= pto/qat: fix checks for 3GPP algo > > bit params > > 2a7e3d546=C2=A0=C2=A0Hyong Youb Kim=C2=A0=C2=A0=C2=A0net/enic: do not o= verwrite admin Tx > > queue limit > > e11411b52=C2=A0=C2=A0Maxime Coquelin=C2=A0=C2=A0vhost: fix missing incr= ement of log > > cache count > > 79d098921=C2=A0=C2=A0Moti Haimovsky=C2=A0=C2=A0=C2=A0net/mlx5: fix buil= d with old kernels > > c44fbc7cc=C2=A0=C2=A0Nelio Laranjeiro net/mlx5: clean-up developer logs > > 179c7e893=C2=A0=C2=A0Pavan Nikhilesh=C2=A0=C2=A0mempool/octeontx: fix p= ool to aura > > mapping > > 196e9a486=C2=A0=C2=A0Qi Zhang=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0eal: fix hotplug add and remove > > 9f32c7e7e=C2=A0=C2=A0Rafal Kozik=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0net= /ena: change memory type > > 46916aa17=C2=A0=C2=A0Rafal Kozik=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0net= /ena: check pointer before memset > > 41e59028d=C2=A0=C2=A0Rafal Kozik=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0net= /ena: set link speed as none > > 3abe17195=C2=A0=C2=A0Rahul Lakkireddy net/cxgbe: fix Rx channel map and= queue > > type > > 1fc3afdf7=C2=A0=C2=A0Rasesh Mody=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0doc= : update qede management firmware > > guide > > c6dd1eb8a=C2=A0=C2=A0Rasesh Mody=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0net= /qede: fix default extended VLAN > > offload config > > a16aef523=C2=A0=C2=A0Rasesh Mody=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0net= /qede: fix for devargs > > e46821e9f=C2=A0=C2=A0Shahaf Shuler=C2=A0=C2=A0=C2=A0=C2=A0net/mlx5: sep= arate generic tunnel TSO > > from the standard one > > 496f15c2c=C2=A0=C2=A0Shahed Shaikh=C2=A0=C2=A0=C2=A0=C2=A0net/qede: fix= incorrect link status > > update > > c7641841a=C2=A0=C2=A0Shahed Shaikh=C2=A0=C2=A0=C2=A0=C2=A0net/qede: fix= unicast MAC address > > handling in VF > > 82dbef2c8=C2=A0=C2=A0Somnath Kotur=C2=A0=C2=A0=C2=A0=C2=A0net/bnxt: rev= ert reset of L2 filter id > > 0989161b2=C2=A0=C2=A0Tiwei Bie=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0vhost: release locks on RARP packet > > failure > > b2a4a69f7=C2=A0=C2=A0Xiaoxin Peng=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0net/bnxt= : fix Tx with multiple mbuf > >=20 > >=20 --=20 Kind regards, Luca Boccassi