From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id A2D65A04AC; Tue, 1 Sep 2020 14:49:11 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 84A9F1C1E9; Tue, 1 Sep 2020 14:49:11 +0200 (CEST) Received: from mail-wr1-f65.google.com (mail-wr1-f65.google.com [209.85.221.65]) by dpdk.org (Postfix) with ESMTP id 675AC1C1D3; Tue, 1 Sep 2020 14:49:09 +0200 (CEST) Received: by mail-wr1-f65.google.com with SMTP id e16so1404742wrm.2; Tue, 01 Sep 2020 05:49:09 -0700 (PDT) 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:user-agent:mime-version; bh=PBWaVLxAZhbV7DqmLkvliW0jsm09M8XdzitgjLeG7Nc=; b=QvK5CWYgooiRFRepzml1fhnh8m+IhkOuRKOf17EvOykpIkF68QfVTH2wpHKEMGLe2I p1QSo1kVM+AHrpirJ0ZZ8KSYdZYtu8iWrwVaoZfHvaz7Gpz0ACiMs5WPoLYudrx0302J e9VsjhOxHZgmugwB30sOGimFZkkvLxImI/TT2HFTrVIvPwv5u7ewRlGYTRFKQniLKB12 mn2IVbBJSKTwVhax63gtExUyc0ln2T08fTKnrUjPLt2EaKV2TzHQo4gN6YrflH9YAJ3/ dK0eMzBE0uJaZzjBS3wA3vbXbm3hOJ2vfQmRno0GhsEjtRolvflhXqp1oxfC2zGgjgBG 6npg== X-Gm-Message-State: AOAM533dNba1CNkW0wDxlGvzOmiZ3O3nJPLCejQhiGN/qKYwmuKX+uxG FJyk/Lm+ohRt5jOUeihCOos= X-Google-Smtp-Source: ABdhPJzRB03YNHfn+S7ywqzoAMC6i64li81Sm3i2CqMpuzhirALJHMD8wFEfuX0HVg4Uhu4BFVoD6A== X-Received: by 2002:adf:fcd2:: with SMTP id f18mr1753039wrs.78.1598964549145; Tue, 01 Sep 2020 05:49:09 -0700 (PDT) Received: from localhost ([2a01:4b00:f419:6f00:7a8e:ed70:5c52:ea3]) by smtp.gmail.com with ESMTPSA id v9sm2085219wru.37.2020.09.01.05.49.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 01 Sep 2020 05:49:08 -0700 (PDT) Message-ID: <98c92e1040e24eaee4a6659e81bfd5f940a1926c.camel@debian.org> From: Luca Boccassi To: Christian Ehrhardt , Ian Stokes , "Richardson, Bruce" Cc: stable@dpdk.org, dev Date: Tue, 01 Sep 2020 13:49:07 +0100 In-Reply-To: References: <20200818181222.8462-1-bluca@debian.org> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.30.5-1.1 MIME-Version: 1.0 Subject: Re: [dpdk-dev] 19.11.4 patches review and test X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, 2020-09-01 at 14:32 +0200, Christian Ehrhardt wrote: > On Tue, Sep 1, 2020 at 10:30 AM Luca Boccassi wrote: > > On Tue, 2020-08-18 at 19:12 +0100, Luca Boccassi wrote: > > > Hi all, > > >=20 > > > Here is a list of patches targeted for stable release 19.11.4. > > >=20 > > > The planned date for the final release is August 31st. > > >=20 > > > Please help with testing and validation of your use cases and report > > > any issues/results with reply-all to this mail. For the final release > > > the fixes and reported validations will be added to the release notes= . > > >=20 > > > A release candidate tarball can be found at: > > >=20 > > > https://dpdk.org/browse/dpdk-stable/tag/?id=3Dv19.11.4-rc1 > > >=20 > > > These patches are located at branch 19.11 of dpdk-stable repo: > > > https://dpdk.org/browse/dpdk-stable/ > > >=20 > > > Thanks. > > >=20 > > > Luca Boccassi > >=20 > > Microsoft's regression tests are still running, delaying until Thursday > > the 3rd. Apologies for any inconvenience. >=20 > Due to report on OVS failing to build I happened to find that 19.11.4 > has massively changed linking. > =3D> https://paste.ubuntu.com/p/znCRR4gpjP/ >=20 > This was meant to be helpful for sure and I assume is around: > 48f7fd27f6 build/pkg-config: prevent overlinking > 2d1535d592 build/pkg-config: improve static linking flags > 9fb13a12c1 build/pkg-config: output drivers first for static build > 59b108d824 build/pkg-config: move pkg-config file creation > aea915e944 devtools: test static linkage with pkg-config >=20 > But overlinking has effectively become underlinking now > https://launchpadlibrarian.net/495845224/buildlog_ubuntu-groovy-amd64.ope= nvswitch_2.13.1-0ubuntu2~ppa1_BUILDING.txt.gz >=20 > /usr/bin/ld: /usr/lib/gcc/x86_64-linux-gnu/10/../../../x86_64-linux-gnu/l= ibrte_pmd_ring.a(net_ring_rte_eth_ring.c.o): > in function `rte_eth_from_rings': > (.text+0x91c): undefined reference to `rte_vdev_init' > /usr/bin/ld: /usr/lib/gcc/x86_64-linux-gnu/10/../../../x86_64-linux-gnu/l= ibrte_pmd_ring.a(net_ring_rte_eth_ring.c.o): > in function `vdrvinitfn_pmd_ring_drv': > (.text.startup+0x28): undefined reference to `rte_vdev_register' > collect2: error: ld returned 1 exit status >=20 > Also as you can see in the pastebin above, CFlags and Libs massively > shrunk and likely too much so. >=20 > Given that this should be a stable release I'd ask to back out those > changes so that a test build creates the same .pc file as before - > would that make sense? Paging Bruce and Ian - IIRC OVS explicitly asked for that series to be backported and took a dependency on it, so removing it might cause breakages for them? Ian? Bruce, have you see the error reported by Christian before? --=20 Kind regards, Luca Boccassi