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 879D54407C; Mon, 20 May 2024 23:43:13 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1C25E402F2; Mon, 20 May 2024 23:43:13 +0200 (CEST) Received: from mail-yw1-f169.google.com (mail-yw1-f169.google.com [209.85.128.169]) by mails.dpdk.org (Postfix) with ESMTP id 6C21B402E9 for ; Mon, 20 May 2024 23:43:11 +0200 (CEST) Received: by mail-yw1-f169.google.com with SMTP id 00721157ae682-620314aa1d7so31821837b3.2 for ; Mon, 20 May 2024 14:43:11 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1716241390; x=1716846190; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=rUSJitaYCJz0gPQhTXsTfTsbplxMi1YXc/Mc6AP9B9I=; b=e5dG39a/wQBZ+7Z49SE8E/Z+z5eSTD7TeE0K9PzuIosHVb0zd0/fhZ08F5P/3GFNwX 35gHdY6xAd0+LT95/Npai46I5S6Li+ZCjj28Uo43T9YhpCvmPJYy/zNhdjPD1VJyz0f+ pl6+gS2qY92z+cD6PjUZ2UNeYmj48DYoDDoDHAUKG39u7W3+ISKVB+yWXG6ogHSX4wLW BUvPQSNuuQbyTWD0U2o3jolq+iluvZcNBpEjI/WI3lwS70v/mLvQWl0RKoOzMazY9udp HwGOZQIUHbRDJZOBW2r7hFkySen8yWHQLm8nvDLlLkJoBeDqCnCeSDzEhD1pr1QGCDCX Y47g== X-Forwarded-Encrypted: i=1; AJvYcCW3O3Zn1HKSliQUhQ3oY7dOcgm4H2F67w7Lw5IuOX6gE808+vhuTrE5G8r5muqgeCdtN8rGAaJRs4SmEAA= X-Gm-Message-State: AOJu0YyRD4DWG06kgm02RBTvzuVKiwGyqPXlpcXPUdP6zjGMwb2tR8AH uEx2swoDVhqQsckyRuGovZDrrr3nSucv7JSg7thJdwhdOEPYI33j51fO4ruj X-Google-Smtp-Source: AGHT+IEuik8Z9Lyl18XYjguoyITjLSbR0zuhY5CAElDoEpbp0SaP1Hzh9RY4k5q1cKSmo/4WmHXNzw== X-Received: by 2002:a81:ae68:0:b0:61b:e871:9115 with SMTP id 00721157ae682-622affa8a1bmr306443867b3.17.1716241390484; Mon, 20 May 2024 14:43:10 -0700 (PDT) Received: from mail-yb1-f179.google.com (mail-yb1-f179.google.com. [209.85.219.179]) by smtp.gmail.com with ESMTPSA id 00721157ae682-6209e379299sm49530597b3.108.2024.05.20.14.43.09 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 20 May 2024 14:43:09 -0700 (PDT) Received: by mail-yb1-f179.google.com with SMTP id 3f1490d57ef6-de5ea7edb90so2798593276.1 for ; Mon, 20 May 2024 14:43:09 -0700 (PDT) X-Forwarded-Encrypted: i=1; AJvYcCVP75loxvQ5fMEIR28uFw/3Jd9xfSk7WqWKwhIDruzjC1J+XxA8zS8+HueTpUp0FgGp47v5tw/+tP3XfTw= X-Received: by 2002:a25:8382:0:b0:deb:3b7c:5268 with SMTP id 3f1490d57ef6-dee4f2e8616mr30815465276.29.1716241389358; Mon, 20 May 2024 14:43:09 -0700 (PDT) MIME-Version: 1.0 References: <20240130034925.44869-1-stephen@networkplumber.org> <20240502213618.11391-1-stephen@networkplumber.org> <20240502213618.11391-8-stephen@networkplumber.org> <6bb00014-b2ee-4fc9-bdfd-7583e172d3e2@amd.com> <20240520111804.0ee1fff2@hermes.local> In-Reply-To: <20240520111804.0ee1fff2@hermes.local> From: Luca Boccassi Date: Mon, 20 May 2024 22:42:57 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v12 07/12] net/tap: use libbpf to load new BPF program To: Stephen Hemminger Cc: Ferruh Yigit , Christian Ehrhardt , Patrick Robb , dpdklab , Aaron Conole , dev@dpdk.org 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 Mon, 20 May 2024 at 19:43, Stephen Hemminger wrote: > > On Mon, 20 May 2024 18:49:19 +0100 > Ferruh Yigit wrote: > > > On 5/2/2024 10:31 PM, Stephen Hemminger wrote: > > > There were multiple issues in the RSS queue support in the TAP > > > driver. This required extensive rework of the BPF support. > > > > > > Change the BPF loading to use bpftool to > > > create a skeleton header file, and load with libbpf. > > > The BPF is always compiled from source so less chance that > > > source and instructions diverge. Also resolves issue where > > > libbpf and source get out of sync. The program > > > is only loaded once, so if multiple rules are created > > > only one BPF program is loaded in kernel. > > > > > > The new BPF program only needs a single action. > > > No need for action and re-classification step. > > > > > > It also fixes the missing bits from the original. > > > - supports setting RSS key per flow > > > - level of hash can be L3 or L3/L4. > > > > > > Bugzilla ID: 1329 > > > > > > Signed-off-by: Stephen Hemminger > > > > > > > > > The libbpf version in my Ubuntu box, installed with package manager, is > > 'libbpf.so.0.5.0', so it doesn't satisfy the requirement and bpf support > > is not compiled for me. > > > > > > @Christian, 'libbpf.so.0.5.0'seems old, it is from 2021, do you know is > > there a reason Ubuntu stick to this version? And can we expect an update > > soon? > > > > > > @Patric, I assume test environment also doesn't have 'libbpf', version: > > '>= 1.0' which we need to test this feature. > > Is it possible to update test environment to justify this dependency? > > > > I think we need to verify at least build (with and without dependency > > met) for the set. > > The BPF API changed a lot, and it is not really possible to support > both. It can be done, but it is a _lot_ of work and requires a lot of shims, so for something optional it's not really worth it. Given libbpf 1.0 also broke ABI, Ubuntu 22.04 and older cannot really get a new version as it's incompatible, so this pmd will simply be skipped there. I think it's fine. 24.04 has a new one.