From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 9AF29A0508; Thu, 14 Apr 2022 07:52:48 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 435FD40687; Thu, 14 Apr 2022 07:52:48 +0200 (CEST) Received: from smtp-relay-internal-0.canonical.com (smtp-relay-internal-0.canonical.com [185.125.188.122]) by mails.dpdk.org (Postfix) with ESMTP id C8F294003F for ; Thu, 14 Apr 2022 07:52:46 +0200 (CEST) Received: from mail-qk1-f197.google.com (mail-qk1-f197.google.com [209.85.222.197]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by smtp-relay-internal-0.canonical.com (Postfix) with ESMTPS id 4CFBE3F841 for ; Thu, 14 Apr 2022 05:52:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1649915565; bh=QjYOqqY0Ex+t3l/BKUDl+YcfUBzB+hTn4ieRkkeFhHM=; h=MIME-Version:References:In-Reply-To:From:Date:Message-ID:Subject: To:Cc:Content-Type; b=APdXdx0rnNdIcyM3FbCIwL3MRU7qIZDW+p8EjGMxVulEjqOnR9KniaBD8tekaKCic 0eOaoyEnjXXcaayu4Z2HId9scc+2aHmTe+4ZaTMeQ6Q7DA/S6OlCsQ/w89Yz82+20n PrNadaoV7B+TSg1pOuowyND2F/3a+S6MQCMkpBCpMS8Rc7DdgK+3lvEbpHJsZlVHz/ eJyTBPVAK2fAFrsYRD3gyeqDUV6/bDXBpZFKBTfQYys+D7hDPD1LiAIBqmkZ0EaI5j p4gxyGcZrx/PkA9Ut4td42Bi6fVvLi/BX3MUBzrkbtrEsZDb8GNUb4V4XeKZ8Knrp9 YiSaD5znafL0A== Received: by mail-qk1-f197.google.com with SMTP id de26-20020a05620a371a00b00699f7065b5cso2599355qkb.5 for ; Wed, 13 Apr 2022 22:52:45 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=QjYOqqY0Ex+t3l/BKUDl+YcfUBzB+hTn4ieRkkeFhHM=; b=Jb5freS924UWoXwTWxwnJIE9uDa+ZE/lQwI4y/gMSuI8gRUcKoo0blke15ROMyB+Ph hiFJZ6ybXaXvPcSgI+w4dHz9iEcJOy6ZnXbulaah7BBqOcZvysrGmyyQpr0/KzHxcdwk hXkGaX1WAgEGEqIcOLyGqbWj7xq0t7EpVhXS0fXDm1wl+MwsuCBTT1L0DkrYFEWcmEwb hC6JbJx3iRmVJbed2fcVjUuJX3MX0EnNXWtZw5tN1Fcii3nTMZJ5MI3o7DQS9ijxOd02 9w9x/bT+VTAfwQOCet0Fit4KRZAyFiSBXcJt0AuB8Cgp05+wSj/jbqMijAz27Hc4g0Hz WlKA== X-Gm-Message-State: AOAM533jBSJCxtUWh4xqiSX21wJqwfteGQVAbGOenJmR2e1Vx3Jsqzoy Sx8ZzoZ6XAvuvZPm63XURRVd3rwW3tjqMpOt3i6NIJ803ll7EjOXVHg/hQopjpKlYc3OsZzQZaW O+9IyU6q8P1gdZ73jdtCB1C49SR/YSYxvfCx7 X-Received: by 2002:a05:6214:2b07:b0:432:f7e6:e443 with SMTP id jx7-20020a0562142b0700b00432f7e6e443mr1918480qvb.125.1649915563881; Wed, 13 Apr 2022 22:52:43 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyjO2K22TCv3nH+7WwMibBsP1BuYOD17uqApu0jjxVX/FPvuUj0od3H2K2T9+McrSVb0MKMMUAwsgu1iQZVig4= X-Received: by 2002:a05:6214:2b07:b0:432:f7e6:e443 with SMTP id jx7-20020a0562142b0700b00432f7e6e443mr1918450qvb.125.1649915563575; Wed, 13 Apr 2022 22:52:43 -0700 (PDT) MIME-Version: 1.0 References: <20220401102216.642587-1-ktraynor@redhat.com> <42961b14-a277-8cfb-13c2-66b904015df2@redhat.com> In-Reply-To: <42961b14-a277-8cfb-13c2-66b904015df2@redhat.com> From: Christian Ehrhardt Date: Thu, 14 Apr 2022 07:52:17 +0200 Message-ID: Subject: Re: 21.11.1 patches review and test To: Kevin Traynor Cc: Thomas Monjalon , Ori Kam , stable@dpdk.org, dev@dpdk.org, Abhishek Marathe , Ali Alnubani , benjamin.walker@intel.com, David Christensen , Hemant Agrawal , Ian Stokes , Jerin Jacob , John McNamara , Ju-Hyoung Lee , Luca Boccassi , Pei Zhang , qian.q.xu@intel.com, Raslan Darawsheh , yuan.peng@intel.com, zhaoyan.chen@intel.com Content-Type: text/plain; charset="UTF-8" X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On Wed, Apr 13, 2022 at 12:06 PM Kevin Traynor wrote: > > Hi Christian/Thomas/Ori, > > On 11/04/2022 07:58, Christian Ehrhardt wrote: > > On Fri, Apr 1, 2022 at 12:22 PM Kevin Traynor wrote: > >> Hi all, > >> > >> Here is a list of patches targeted for stable release 21.11.1. > > Hi Kevin, > > this breaks on me at build time due to symbol changes. > > It is a wild mix of Base->Internal/Experimental, a few new symbols, > > and even just removed ones (in gpu which is experimental, but still > > would that need a minor soname bump?). > > They might be intentional, but it felt too much to me without at least > > discussing it. > > Could you have a look if you think that they are all intentional, save > > and correct for an LTS release? > > > > Thanks for the report. I've looked through these. Comments below. > > > dpkg-gensymbols: warning: some new symbols appeared in the symbols > > file: see diff output below > > dpkg-gensymbols: warning: debian/librte-common-cnxk22/DEBIAN/symbols > > doesn't match completely debian/librte-common-cnxk22.symbols > > --- debian/librte-common-cnxk22.symbols > > (librte-common-cnxk22_21.11.1~rc1-0ubuntu1~jammyppa2_amd64) > > +++ dpkg-gensymbolsUuRb8d 2022-04-11 06:46:22.276766813 +0000 > > @@ -197,6 +197,7 @@ > > roc_nix_ptp_clock_read@INTERNAL 21.08 > > roc_nix_ptp_info_cb_register@INTERNAL 21.08 > > roc_nix_ptp_info_cb_unregister@INTERNAL 21.08 > > + roc_nix_ptp_is_enable@INTERNAL 21.11.1~rc1-0ubuntu1~jammyppa2 > > roc_nix_ptp_rx_ena_dis@INTERNAL 21.08 > > roc_nix_ptp_sync_time_adjust@INTERNAL 21.08 > > roc_nix_ptp_tx_ena_dis@INTERNAL 21.08 > > > > This is a new internal from: > commit 28acfe550dcb12b0908df754a4307b8b4d1fe5b0 > Author: Harman Kalra > Date: Thu Mar 3 12:30:42 2022 +0530 > > common/cnxk: fix mbuf data offset for VF > > [ upstream commit 8f98e3ecc55e02234f8bec7213b0b6a69c086949 ] > > Looks ok to me. > > > dpkg-gensymbols: warning: some new symbols appeared in the symbols > > file: see diff output below > > dpkg-gensymbols: warning: debian/librte-ethdev22/DEBIAN/symbols > > doesn't match completely debian/librte-ethdev22.symbols > > --- debian/librte-ethdev22.symbols > > (librte-ethdev22_21.11.1~rc1-0ubuntu1~jammyppa2_amd64) > > +++ dpkg-gensymbolskEnokB 2022-04-11 06:46:25.252795157 +0000 > > @@ -37,6 +37,7 @@ > > rte_eth_dev_flow_ctrl_get@DPDK_22 21.11 > > rte_eth_dev_flow_ctrl_set@DPDK_22 21.11 > > rte_eth_dev_fw_version_get@DPDK_22 21.11 > > + rte_eth_dev_get_by_name@INTERNAL 21.11.1~rc1-0ubuntu1~jammyppa2 > > rte_eth_dev_get_dcb_info@DPDK_22 21.11 > > rte_eth_dev_get_eeprom@DPDK_22 21.11 > > rte_eth_dev_get_eeprom_length@DPDK_22 21.11 > > > > This is a new internal > Added by: > commit 721d0bbd1668d2a4b617a4a4de0b93dd60283d58 > Author: Kumara Parameshwaran > Date: Mon Jan 31 20:02:33 2022 +0530 > > ethdev: add internal function to device struct from name > > [ upstream commit 961fb4029b8c52c0e8230d34993c354d70e10e14 ] > > Used by: > commit ac180f4d2662503ecd18a2e94689a229104d3d61 > Author: Kumara Parameshwaran > Date: Mon Jan 31 20:02:34 2022 +0530 > > net/tap: fix to populate FDs in secondary process > > [ upstream commit c36ce7099c2187926cd62cff7ebd479823554929 ] > > Looks ok to me. > > > dpkg-gensymbols: warning: some new symbols appeared in the symbols > > file: see diff output below > > dpkg-gensymbols: error: some symbols or patterns disappeared in the > > symbols file: see diff output below > > dpkg-gensymbols: warning: debian/librte-regexdev22/DEBIAN/symbols > > doesn't match completely debian/librte-regexdev22.symbols > > --- debian/librte-regexdev22.symbols > > (librte-regexdev22_21.11.1~rc1-0ubuntu1~jammyppa2_amd64) > > +++ dpkg-gensymbolsPD0Ygo 2022-04-11 06:46:33.368872490 +0000 > > @@ -1,6 +1,8 @@ > > librte_regexdev.so.22 librte-regexdev22 #MINVER# > > EXPERIMENTAL@EXPERIMENTAL 20.11 > > - rte_regex_devices@Base 20.11 > > + INTERNAL@INTERNAL 21.11.1~rc1-0ubuntu1~jammyppa2 > > + rte_regex_devices@EXPERIMENTAL 21.11.1~rc1-0ubuntu1~jammyppa2 > > rte_regexdev_attr_get@EXPERIMENTAL 20.11 > > rte_regexdev_attr_set@EXPERIMENTAL 20.11 > > rte_regexdev_close@EXPERIMENTAL 20.11 > > @@ -8,12 +10,16 @@ > > rte_regexdev_count@EXPERIMENTAL 20.11 > > rte_regexdev_dump@EXPERIMENTAL 20.11 > > rte_regexdev_get_dev_id@EXPERIMENTAL 20.11 > > - rte_regexdev_get_device_by_name@Base 20.11 > > + rte_regexdev_get_device_by_name@INTERNAL 21.11.1~rc1-0ubuntu1~jammyppa2 > > rte_regexdev_info_get@EXPERIMENTAL 20.11 > > - rte_regexdev_is_valid_dev@Base 20.11 > > - rte_regexdev_logtype@Base 20.11 > > + rte_regexdev_is_valid_dev@EXPERIMENTAL 21.11.1~rc1-0ubuntu1~jammyppa2 > > + rte_regexdev_logtype@EXPERIMENTAL 21.11.1~rc1-0ubuntu1~jammyppa2 > > rte_regexdev_queue_pair_setup@EXPERIMENTAL 20.11 > > - rte_regexdev_register@Base 20.11 > > + rte_regexdev_register@INTERNAL 21.11.1~rc1-0ubuntu1~jammyppa2 > > rte_regexdev_rule_db_compile_activate@EXPERIMENTAL 20.11 > > rte_regexdev_rule_db_export@EXPERIMENTAL 20.11 > > rte_regexdev_rule_db_import@EXPERIMENTAL 20.11 > > @@ -21,7 +27,8 @@ > > rte_regexdev_selftest@EXPERIMENTAL 20.11 > > rte_regexdev_start@EXPERIMENTAL 20.11 > > rte_regexdev_stop@EXPERIMENTAL 20.11 > > - rte_regexdev_unregister@Base 20.11 > > + rte_regexdev_unregister@INTERNAL 21.11.1~rc1-0ubuntu1~jammyppa2 > > rte_regexdev_xstats_by_name_get@EXPERIMENTAL 20.11 > > rte_regexdev_xstats_get@EXPERIMENTAL 20.11 > > rte_regexdev_xstats_names_get@EXPERIMENTAL 20.11 > > > > +cc Ori, regex maintainer. > > Regex library is an experimental API so everything should have been > experimental or internal. This is fixing that issue. > > It is fixed with > commit 6e7f8939f23c2c8ed80602bc0d62990eebe52013 > Author: Thomas Monjalon > Date: Sun Mar 6 10:20:22 2022 +0100 > > regexdev: fix section attribute of symbols > > [ upstream commit 89e290eb8ca99af9f7cfc3292d93860f8e672708 ] > > and > > commit 026470bafaa02cba0d46ed7b7e835262399a009a > Author: Thomas Monjalon > Date: Sun Mar 6 10:20:23 2022 +0100 > > build: hide local symbols in shared libraries > > [ upstream commit b403498e14229ee903c8fff9baefcb72894062f3 ] > > > The fact that they are redesignated to correctly be > experimental/internal seems ok to me. > > > dpkg-gensymbols: error: some symbols or patterns disappeared in the > > symbols file: see diff output below > > dpkg-gensymbols: warning: debian/librte-gpudev22/DEBIAN/symbols > > doesn't match completely debian/librte-gpudev22.symbols > > --- debian/librte-gpudev22.symbols > > (librte-gpudev22_21.11.1~rc1-0ubuntu1~jammyppa2_amd64) > > +++ dpkg-gensymbols4qkXdt 2022-04-11 06:46:34.552883776 +0000 > > @@ -1,7 +1,7 @@ > > librte_gpudev.so.22 librte-gpudev22 #MINVER# > > EXPERIMENTAL@EXPERIMENTAL 21.11 > > INTERNAL@INTERNAL 21.11 > > - gpu_logtype@Base 21.11 > > rte_gpu_add_child@EXPERIMENTAL 21.11 > > rte_gpu_allocate@INTERNAL 21.11 > > rte_gpu_attach@INTERNAL 21.11 > > > > The missing wildcard meant this symbol escaped in 21.11. > > It is fixed by: > commit 026470bafaa02cba0d46ed7b7e835262399a009a > Author: Thomas Monjalon > Date: Sun Mar 6 10:20:23 2022 +0100 > > build: hide local symbols in shared libraries > > [ upstream commit b403498e14229ee903c8fff9baefcb72894062f3 ] > > In this case the symbol is not redesignated but removed, but it doesn't > look to have any use to a user, so I think it can be safe to remove. I'm 100% with all others, thanks for having a look. On this one I can easily follow the argument of the fix for the newest release. But for stable we can never really know if there are users. In theory for anything that shipped in a Distribution someone might have coded and linked something against it - we would not know. The meant to be "stable" update will then break them the hard way. In this case gladly the function wasn't anything that one would consider useful for use from outside, so I think it is ok. But still I wanted to make the point that in general a symbol: 1. once released might be used and we can not never be sure if no one uses them 2. even being EXPERIMENTAL, touching them too much in stable updates means not-stable. Should we at least try to minimize the impact to stable releases? For now I'm adapting my checkers and will continue testing ... > There are updates to the libabigail.ignore for regex and gpu_dev to > ignore ABI changes for these fixes. > > -- > > I'm ok with changes above for 21.11.1, what do others think? > > Kevin. > > > > > Full log: > > https://launchpadlibrarian.net/596047842/buildlog_ubuntu-jammy-amd64.dpdk_21.11.1~rc1-0ubuntu1~jammyppa2_BUILDING.txt.gz > > > -- Christian Ehrhardt Staff Engineer, Ubuntu Server Canonical Ltd