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 4D444A04FF for ; Mon, 4 Apr 2022 12:33:18 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 313F74068C; Mon, 4 Apr 2022 12:33:18 +0200 (CEST) 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 E35E042821 for ; Mon, 4 Apr 2022 12:33:16 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1649068396; 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=AEXauAOnAdurl9tdoOLaWttmHHs3HWkqjk2NImbZ5tM=; b=io8ZFetM4+bRp1flAqtBhnrCVN7zySGXrkQHNjAOW/TzqHck+58zU+GSeJMfTR4XNplKE2 8h565z4GMVrH8GGN3mjwRRVxVtSBuHQlD0WZeozqEn+/RNwgG/fctY05UZLg8xrCd04lS0 fKjiUyDo1puUXzxK17wJeMagcdd8txg= Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-441-JuCgCl3oM_KLnC_Mf2gNWw-1; Mon, 04 Apr 2022 06:33:15 -0400 X-MC-Unique: JuCgCl3oM_KLnC_Mf2gNWw-1 Received: by mail-wr1-f69.google.com with SMTP id v3-20020adf8b43000000b00205e463b017so1588177wra.10 for ; Mon, 04 Apr 2022 03:33:15 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent :content-language:to:cc:references:from:subject:in-reply-to :content-transfer-encoding; bh=AEXauAOnAdurl9tdoOLaWttmHHs3HWkqjk2NImbZ5tM=; b=yYgbgT7MEoEZ7E6qKNPy3LxxDnSf2rSBmbpJT575d4CNMKiTp7K06FtVnQLTsAllcL 1/SEzBEgzLJIJzXQ3AV4m8qwRK1DfAK5Rb38mXqSbOMC8tLQwnXVeHB/TQG+GLPArGdR 7yb6+MkuXUz8RUIL4I7ImpomjkuXe06VWP/DvwITbmaltsQtM1Eu+WHkOyS0hwIphhig U4nzAmDANA6DWj0EaKoONe6i8LG7yKLx/KX0s7jihLI6/i9zjynCy7Om4NUmfQplTMT0 edX5TqcoWE5XF5PJU7zeDrBJNzBepIxBDRK/TcM8p5bsIdWHPjvTUXAIAByw+90hrBsp EKtA== X-Gm-Message-State: AOAM530DCjg9FhQ//3zlQ+5nYAQ3lBVytRF3QuhvCKxzRt8D/wbb+H7I rEBUBOlrHH9E0p4uR0aaLiZwRl2rsxBQ2IJEEeJi3/ELkKOs39vcsuXVxcX1jv8eqp8MGZF+tXk +4WfiVvU= X-Received: by 2002:a5d:598c:0:b0:205:a7ab:fe52 with SMTP id n12-20020a5d598c000000b00205a7abfe52mr16279142wri.411.1649068390725; Mon, 04 Apr 2022 03:33:10 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwNni1xe+2OzrQ/6LN9PaOou5/GWqh9V8+9jcTmy+GkNq1J8SD0X4oafcI9HzoPDHNMVUMvgw== X-Received: by 2002:a5d:598c:0:b0:205:a7ab:fe52 with SMTP id n12-20020a5d598c000000b00205a7abfe52mr16279128wri.411.1649068390461; Mon, 04 Apr 2022 03:33:10 -0700 (PDT) Received: from [192.168.0.36] ([78.19.110.230]) by smtp.gmail.com with ESMTPSA id b1-20020adfd1c1000000b002058537af75sm9405221wrd.104.2022.04.04.03.33.09 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 04 Apr 2022 03:33:09 -0700 (PDT) Message-ID: <95d923da-0216-9a66-2713-17f0a6442a0c@redhat.com> Date: Mon, 4 Apr 2022 11:33:09 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.7.0 To: =?UTF-8?Q?Morten_Br=c3=b8rup?= , Ben Magistro , dev@dpdk.org Cc: ben.magistro@trinitycyber.com, stefan.baranoff@trinitycyber.com, stable@dpdk.org, Luca Boccassi , Christian Ehrhardt , "Xueming(Steven) Li" References: <20220404005634.2657-1-koncept1@gmail.com> <98CBD80474FA8B44BF855DF32C47DC35D86F9B@smartserver.smartshare.dk> From: Kevin Traynor Subject: Re: [PATCH] mbuf: expose outer vlan in mbuf dump In-Reply-To: <98CBD80474FA8B44BF855DF32C47DC35D86F9B@smartserver.smartshare.dk> Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=ktraynor@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: 8bit X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org On 04/04/2022 07:15, Morten Brørup wrote: >> From: Ben Magistro [mailto:koncept1@gmail.com] >> Sent: Monday, 4 April 2022 02.57 >> >> Enable printing of the outer vlan if flags indicate it is present. >> >> Signed-off-by: Ben Magistro >> >> --- >> >> While troubleshooting some QinQ offloads with various Intel i40e >> firmware[1], it was >> helpful to expose the outer vlan in the dump mbuf calls. This should >> be straightforward >> to backport and happy to do the work if accepted. I understand that >> this may not be a >> widely supported capability at this time, so we are okay if this is not >> accepted and >> we just maintain a local patch. > > Features are usually not backported, only bug fixes. > > However, since this patch proved helpful finding a bug, and it is very simple, it could be considered by the LTS maintainers. > Suggest to tag the patch with 'Cc: stable@dpdk.org' to indicate it is requested for stable branches. It will be caught by stable maintainer filters when it is time for backports and can be discussed further then. (Note, it would need small rebase for older than 21.11 LTS to account for the namespaces that were added) Kevin. >> >> --- >> lib/mbuf/rte_mbuf.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/lib/mbuf/rte_mbuf.c b/lib/mbuf/rte_mbuf.c >> index 87592faccb..a2307cebe6 100644 >> --- a/lib/mbuf/rte_mbuf.c >> +++ b/lib/mbuf/rte_mbuf.c >> @@ -674,6 +674,9 @@ rte_pktmbuf_dump(FILE *f, const struct rte_mbuf *m, >> unsigned dump_len) >> fprintf(f, " pkt_len=%u, ol_flags=%#"PRIx64", nb_segs=%u, >> port=%u", >> m->pkt_len, m->ol_flags, m->nb_segs, m->port); >> >> + if (m->ol_flags & (RTE_MBUF_F_RX_QINQ | RTE_MBUF_F_TX_QINQ)) >> + fprintf(f, ", vlan_tci_outer=%u", m->vlan_tci_outer); >> + >> if (m->ol_flags & (RTE_MBUF_F_RX_VLAN | RTE_MBUF_F_TX_VLAN)) >> fprintf(f, ", vlan_tci=%u", m->vlan_tci); >> >> -- >> 2.27.0 >> > > Reviewed-by: Morten Brørup >