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 2D58941E9F; Wed, 15 Mar 2023 12:47:15 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 06DBE40A7A; Wed, 15 Mar 2023 12:47:15 +0100 (CET) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by mails.dpdk.org (Postfix) with ESMTP id 5DF3F40141 for ; Wed, 15 Mar 2023 12:47:13 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1678880832; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=MhQZ0d2pNfkMaD0EedTKCI9/islabuN8XhF1pGHCHSw=; b=BFo0KB3b9o12vbNT/L3tsqMzvVlfEqZ0Pc5VYAzKvTjfnsp0F1fC424iLpilhjeyRIpA/I 98RVQTCH8nFhARcnnvcXA05l+mqakTRS41d3c84etopkXezakqHCK8PPgVfBdMvMLVpTpN PqTeBSQ2dvjmoEd3OTm3O9MJRIpwPUE= Received: from mail-wm1-f69.google.com (mail-wm1-f69.google.com [209.85.128.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-657-Sql_jxDXPUycbXVNQjUtiA-1; Wed, 15 Mar 2023 07:47:11 -0400 X-MC-Unique: Sql_jxDXPUycbXVNQjUtiA-1 Received: by mail-wm1-f69.google.com with SMTP id ay12-20020a05600c1e0c00b003ed201dcf71so3817218wmb.4 for ; Wed, 15 Mar 2023 04:47:11 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1678880830; h=content-transfer-encoding:in-reply-to:subject:references:cc:to:from :content-language:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=MhQZ0d2pNfkMaD0EedTKCI9/islabuN8XhF1pGHCHSw=; b=Cw8zJfpocZnXx1QjVJoZ6oFJDP4g/+3E8CjHV83/+a+4/vv6XsnhQYjrNl6kZt3hL/ lH0gF46vaMV55FjuN0yUwzMLriVXp5P75R2LRXM3ffVRCArhkID1ol3IScF74h9vYM5L GHR3NgmQWNEfzFGGySoUTbPOkj2icj7A9h/qbiug6X/3b2wpvcFOPShedAUtlnquVjSS Ow77GgXIEzPgVpWoFh4zD7km88a3yB5LNPNQk4MI/MjzkOWJ146m1AShB7EEDFOJhSQx ihEzZF5wRes4XB5L//0RhXk5dhq0bVUNXFYSJxhEx+3Z3b3QtpnOTI/fB2GFmIlw10vn ApsA== X-Gm-Message-State: AO0yUKWgeje62VipUXX89l5WC6K8+cwbamVDdIsOwYufewsabntkV97e RKYiO2dgYQkmNPslYf6NqTazi0/nTFAs/Fi1kODIzv9Pw2V5GoL8OaSHI9gk3yALG9rd+kDwbwU 92JE= X-Received: by 2002:adf:ef0e:0:b0:2cf:ee6b:36aa with SMTP id e14-20020adfef0e000000b002cfee6b36aamr1630872wro.64.1678880830663; Wed, 15 Mar 2023 04:47:10 -0700 (PDT) X-Google-Smtp-Source: AK7set9+fZsK8/q0jpr9u8CJ+X502ZfhgC+DdUtGxSTjRerzyY4q7UZ1OC9fz/8eevVVfbyY/y/6dQ== X-Received: by 2002:adf:ef0e:0:b0:2cf:ee6b:36aa with SMTP id e14-20020adfef0e000000b002cfee6b36aamr1630857wro.64.1678880830367; Wed, 15 Mar 2023 04:47:10 -0700 (PDT) Received: from [192.168.0.36] ([78.19.103.115]) by smtp.gmail.com with ESMTPSA id j8-20020a5d5648000000b002c559843748sm4367442wrw.10.2023.03.15.04.47.08 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 15 Mar 2023 04:47:09 -0700 (PDT) Message-ID: Date: Wed, 15 Mar 2023 11:47:08 +0000 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.8.0 From: Kevin Traynor To: Andrew Rybchenko , Ciara Loftus , dev@dpdk.org, "Zhang, Qi Z" Cc: thomas@monjalon.net, ferruh.yigit@xilinx.com, David Marchand References: <20220624102354.1516606-1-ciara.loftus@intel.com> <0fcfe78a-72c6-788d-867e-0ff554ba5004@oktetlabs.ru> <29f03d86-bc81-c39f-76b1-bb51b9999cfb@redhat.com> Subject: Re: [PATCH] net/af_xdp: make compatible with libbpf v0.8.0 In-Reply-To: <29f03d86-bc81-c39f-76b1-bb51b9999cfb@redhat.com> X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit 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 21/12/2022 09:28, Kevin Traynor wrote: > On 21/12/2022 06:09, Andrew Rybchenko wrote: >> Hi Kevin, >> >> On 12/20/22 17:05, Kevin Traynor wrote: >>> On 24/06/2022 11:23, Ciara Loftus wrote: >>>> libbpf v0.8.0 deprecates the bpf_get_link_xdp_id and bpf_set_link_xdp_fd >>>> functions. Use meson to detect if libbpf >= v0.7.0 is linked and if >>>> so, use >>>> the recommended replacement functions bpf_xdp_query_id, bpf_xdp_attach >>>> and bpf_xdp_detach which are available to use since libbpf v0.7.0. >>>> >>>> Also prevent linking with libbpf versions > v0.8.0. >>>> >>>> Signed-off-by: Ciara Loftus >>> >>> Hi Andrew/Qi (assuming Ciara is still out of office), >>> >>> I am seeing a similar issue [1] on 21.11 branch with Fedora 37 >>> (libbpf-0.8.0-2.fc37.x86_64 and libxdp-1.2.6-1.fc37.x86_64). >>> >>> This patch alone won't apply as there are other dependencies. Looking at >>> the commits in main branch, it seems like I could take all these [2] to >>> resolve the issue. With these cherry-picked the build warnings on Fedora >>> 37 are removed. >>> >>> It's a bit late to take these for DPDK 21.11.3 as I intend to release >>> later today/tomorrow, so it can be resolved for DPDK 21.11.4. >>> >>> Do the commits below look ok for backport? Main branch might be able to >>> demand user uses new libbpf/libxdp versions etc, but with stable we >>> never want to break the users existing setup when they upgrade from >>> 2X.11.n to 2X.11.n+1. N.B. ^^^^ >>> >>> Let me know what you think? >> >> IMO these patches are to to be backported to stable branch. > > Thanks Andrew. > >> However, af_xdp maintainers opinion is more important here. >> > > Qi, what do you think? > Hi Qi/Ciara, this issue is still present approaching 21.11.4. What is your opinion on backporting these patches? Please especially note the paragraph above wrt users not being required to upgrade libbpf. thanks, Kevin. >>> >>> thanks, >>> Kevin. >>> >>> [1] https://paste.centos.org/view/e4eec764 >>> [2] >>> 1eb1846b1a net/af_xdp: make compatible with libbpf 0.8.0 >>> 5ff3dbe6ce net/af_xdp: add log on XDP program removal failures >>> 0ed0bc3834 net/af_xdp: avoid version-based check for program load >>> e024c7e838 net/af_xdp: avoid version-based check for shared UMEM >>> f76dc44ded net/af_xdp: make clear which libxdp version is required >>> 50b855fc47 net/af_xdp: move XDP library presence flag setting >>> >> >