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 56E4842F53; Wed, 26 Jul 2023 18:31:33 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id F029940E25; Wed, 26 Jul 2023 18:31:32 +0200 (CEST) Received: from mail-pf1-f181.google.com (mail-pf1-f181.google.com [209.85.210.181]) by mails.dpdk.org (Postfix) with ESMTP id CAF8440A79 for ; Wed, 26 Jul 2023 18:31:30 +0200 (CEST) Received: by mail-pf1-f181.google.com with SMTP id d2e1a72fcca58-686b91c2744so41803b3a.0 for ; Wed, 26 Jul 2023 09:31:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=networkplumber-org.20221208.gappssmtp.com; s=20221208; t=1690389089; x=1690993889; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:to:from:date:from:to:cc:subject:date:message-id :reply-to; bh=0UTPHLhZhdl0FVLZ01gLULyKuS8fz2pRUKo61BuEx/U=; b=PpkiyKKT1NcK1DZkHJx7lAq+JORLgDXq6ZWKgzDK/X/257sGeeH+8ZdN6NJQFayr5F gvbpbi7Rzh6Yuy3nlqFokIhM+JLpxvjmDHrH9GR3GKX/iHyqK79IrTL21k9VyAsQxJy9 zHVY3cPxeKC5n4WveNz790IXxBAddqJG8PW0hqKEx1LRw1g9QRVoKt3xS6w0zm3GGTkt JpmYxTRG/Sqmk8RFyeFj68bu0JIZmvGJxeXfONhob3uDC7PaDfCtxPrnoOsRJR94AEmN 0c8fG7FKlQYKmSX81OCJZIAqvyGd8HE3E6l6S+1iVUVifuCMrdxM07yylZazH4RCprMY 9tXA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1690389089; x=1690993889; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:to:from:date:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=0UTPHLhZhdl0FVLZ01gLULyKuS8fz2pRUKo61BuEx/U=; b=HVzSU7tYnkcnLL1f+bzRQ+cE0v53xsH2NkJDCyPxbspNclaIEd6G2fbWS48Znhqw5+ mSZIwN9vTfi010wE8ZCQ1icJsbHxmpWDjmpzNpYL6IQhe/70ViIVnXOEirqDUwVOp+V9 1f8WZ+tdeJ4Cn17Gn4BTn3ZeD4B+idB8Zro5L1t8RoVCV/FyzowFdbh5YOvYhUFjQfP6 95HFjm99u8aSDkCa5AKIBBibNE4Xv8FhwE1/kiupRfJ5Diy00CTFoy2NE3ReSvc3IQDB lNMcCaXo2Eb2SZqVqysYDxyD3Ywob65IGzLI8SJlDyljxWNthK1WwHWiZOtRjQzz6Pjj G94A== X-Gm-Message-State: ABy/qLa/srwLXFcPJQ1cSW/cBb2g0HR7lxD2+hT2wfF1soIlou0iCaTa D/Q05BXlwHeYUrYpRI3jSm51oOxuzG38AnRx2WG53Q== X-Google-Smtp-Source: APBJJlHB551NJiwoQZcIlO06/yQziVwiB91/9afSBOSFt5oPMYO79izWbTsfaU2ePFTbAiRinsmOCA== X-Received: by 2002:a05:6300:8089:b0:134:40f0:5d04 with SMTP id ap9-20020a056300808900b0013440f05d04mr2365269pzc.13.1690389089529; Wed, 26 Jul 2023 09:31:29 -0700 (PDT) Received: from hermes.local (204-195-127-207.wavecable.com. [204.195.127.207]) by smtp.gmail.com with ESMTPSA id d5-20020aa78685000000b00686edbc3d90sm241396pfo.127.2023.07.26.09.31.29 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 26 Jul 2023 09:31:29 -0700 (PDT) Date: Wed, 26 Jul 2023 09:31:27 -0700 From: Stephen Hemminger To: dev@dpdk.org Subject: Re: [PATCH v3] tap: fix build of TAP BPF program Message-ID: <20230726093127.553ddb65@hermes.local> In-Reply-To: <20230720232549.63619-1-stephen@networkplumber.org> References: <20230716212544.5625-1-stephen@networkplumber.org> <20230720232549.63619-1-stephen@networkplumber.org> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII 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 Thu, 20 Jul 2023 16:25:49 -0700 Stephen Hemminger wrote: > Move the BPF program related code into a subdirectory. > And add a Makefile for building it. > > The code was depending on old versions of headers from iproute2. > Include those headers here so that build works. > > The standalone build was also broken because by > commit ef5baf3486e0 ("replace packed attributes") > which introduced __rte_packed into this code. > > Add a python program to extract the resulting BPF into > a format that can be consumed by the TAP driver. > > Update the documentation. > > Signed-off-by: Stephen Hemminger Some backstory notes: 0. No longer work for Microsoft, so don't have Azure/Failsafe/TAP setup. Therefore compile tested only. 1. Went down this rabbit hole while trying to clear some of the DPDK patch backlog, and ran into this one fix for RSS on fragmented packets https://patchwork.dpdk.org/project/dpdk/patch/20220420112400.2271-1-madhuker.mythri@oracle.com/ 2. This version of patch only makes the BPF program build. It does not change the BPF program used currently by TAP device. 3. Doing build will change the bpf program since clang is probably optimizing more now. Bottom line: Merging the patch won't fix the RSS bug, it won't change the current TAP device behavior (same BPF). But it will allow for easier later fix in next release.