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 8A1F2A0556 for ; Mon, 17 Oct 2022 09:30:44 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7625940143; Mon, 17 Oct 2022 09:30:44 +0200 (CEST) Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by mails.dpdk.org (Postfix) with ESMTP id 5A01E400D7 for ; Mon, 17 Oct 2022 09:30:42 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1665991841; 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: in-reply-to:in-reply-to:references:references; bh=Mv8mble9sup0HRS4j0LTNL5otiwQ/XU+fKNVd3jlQNY=; b=iQuLRYGyS4DtyLKcAfhkHxshgHhf7VPf8Sbwb6iJ9Txe7QHrMoCY0fp2nAyTTXzVkzz1Ay 81x1ExEBt5Havk0+gmm1B7R/vJTXHlPW2Zruv4YcROAO1c95Mqhqxl6J1/Pxb6QRzhmN67 OXXe20vHkC2kTPJqtPYJoo0cOf23y58= Received: from mail-pf1-f200.google.com (mail-pf1-f200.google.com [209.85.210.200]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_128_GCM_SHA256) id us-mta-564-tM7aJBXbNMW0HrnXDqfk5A-1; Mon, 17 Oct 2022 03:30:38 -0400 X-MC-Unique: tM7aJBXbNMW0HrnXDqfk5A-1 Received: by mail-pf1-f200.google.com with SMTP id o14-20020a056a00214e00b0056238ef46ebso5641644pfk.2 for ; Mon, 17 Oct 2022 00:30:38 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=Mv8mble9sup0HRS4j0LTNL5otiwQ/XU+fKNVd3jlQNY=; b=If5kzoWXtTV6po4xX5MOx4QOBwcy5x/lVEcoB0+IrEvT2wUnI+1eT/Wkpbhy5S3Gfe TAjdh+4GrUQ06PO+3t2nR6Wva99D//rF/ZVY/VFiDZl+2OURNGQVloZx3yvBeXDZYnza rge6oY7V8+0BmouL6YVjjvoXhdE8HPPHNDuAOVEgS0j22gGtzCC7e2eHvW/SbG17qLwa KIpqb8rK1OF5F/6yFNZlkMNdBMZ5yGLAfNS5ZaB8DJan8QdZ+I5fENFfQaglkPhzzuxl MxipWSzsvVYy2AWTCInZztDoO4AZgU5HIs964xKsNKnr/Ps9auqqby0Fh14ZkgcTUCWH VRwg== X-Gm-Message-State: ACrzQf0DcisE/zg6pdggY7jIkX/ddzZUxGkd5VcHrgxcha2xGPhfGz27 cCGiQumnreuWUR30ymN8dh/LeA35uCyDPWwrA+cCcFXcEnPl759LIMMnLmm2POlSc+Vn5lV2TfU WZzJG04vj8gSEws+Yxs6GLUo= X-Received: by 2002:a17:903:2307:b0:181:e618:b4c5 with SMTP id d7-20020a170903230700b00181e618b4c5mr10432723plh.172.1665991837783; Mon, 17 Oct 2022 00:30:37 -0700 (PDT) X-Google-Smtp-Source: AMsMyM61mxrf9KR+zHeUolQhCMFMz4YSPDMCtuqfyBxjOMQEKxdpfA0ETfDk5sWOEtqPxKwShp/9c/92I4lxstsc5hk= X-Received: by 2002:a17:903:2307:b0:181:e618:b4c5 with SMTP id d7-20020a170903230700b00181e618b4c5mr10432706plh.172.1665991837512; Mon, 17 Oct 2022 00:30:37 -0700 (PDT) MIME-Version: 1.0 References: <20221014203710.6172-1-olivier.matz@6wind.com> In-Reply-To: <20221014203710.6172-1-olivier.matz@6wind.com> From: David Marchand Date: Mon, 17 Oct 2022 09:30:26 +0200 Message-ID: Subject: Re: [PATCH 1/2] event/sw: fix missing flow ID init in selftest To: Olivier Matz Cc: dev@dpdk.org, Harry van Haaren , Bruce Richardson , David Hunt , Anatoly Burakov , =?UTF-8?Q?Morten_Br=C3=B8rup?= , Andrew Rybchenko , stable@dpdk.org X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" 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 Fri, Oct 14, 2022 at 10:38 PM Olivier Matz wrote: > > The issue is seen by unit tests: > > > root@dpdk-VF-dut247:~/dpdk# MALLOC_PERTURB_=204 \ > > DPDK_TEST=eventdev_selftest_sw \ > > /root/dpdk/x86_64-native-linuxapp-gcc/app/test/dpdk-test -c 0xff > > (...) > > *** Running XStats ID Reset test... > > 12: 1761: qid_0_port_2_pinned_flows value , expected 1 got 7 > > 1778: qid_0_port_2_pinned_flows value incorrect, expected 1 got 7 > > ERROR - XStats ID Reset test FAILED. > > SW Eventdev Selftest Failed. > > Test Failed > > The flow id is not set in the event, which results in an undefined > flow, whose value depends on what was previously in stack. Having > different flows for the packets makes the test to fail, since only one > flow is expected. > > This only happens in -O3, where the same stack area is shared by the > event object and the address of the mbuf allocated in rte_gen_arp(). Good catch. That's scary... > > Fix this by properly initializing the flow id. > > Bugzilla ID: 1101 > Fixes: e21df4b062b5 ("test/eventdev: add SW xstats tests") > Cc: stable@dpdk.org > > Signed-off-by: Olivier Matz Reviewed-by: David Marchand -- David Marchand