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 B4F9143076; Tue, 15 Aug 2023 21:05:21 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 128CE43251; Tue, 15 Aug 2023 21:05:21 +0200 (CEST) Received: from mail-oo1-f45.google.com (mail-oo1-f45.google.com [209.85.161.45]) by mails.dpdk.org (Postfix) with ESMTP id 9BF2F4114B for ; Tue, 15 Aug 2023 21:05:19 +0200 (CEST) Received: by mail-oo1-f45.google.com with SMTP id 006d021491bc7-56cc3453e31so4167348eaf.1 for ; Tue, 15 Aug 2023 12:05:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=iol.unh.edu; s=unh-iol; t=1692126319; x=1692731119; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=L+zo40wGVPxuILwQutKho1swAMX2ZJfNM+HLmPvdwU0=; b=aidE4Pk9TNJTD+1djxpf/eJmxK+WHrfXj/WPJY4OD+OAa5VOL++IEv143E70mdh6ra cK5SUvD/iJnh0wVmfNvoIQAv2dQWb0cszRSHQ1adtEi40Jsdla3aiTLFZLtYl/bTFA6d 4aNLav4u0id1MS1Cp33hMStRpkj0CY3CD/0L8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692126319; x=1692731119; 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=L+zo40wGVPxuILwQutKho1swAMX2ZJfNM+HLmPvdwU0=; b=i3Nyxk5tTOjuo3rkP+J5XY/NiuoS7fh/7c3yzRkZTD4pgMbP/Zv0ILovuYTeP5OU0j 0ge54I45uvKju+wsD5l3CTRS4cPAhjz5L72RbRVcGJO/R5bVCVKQgznVkLWiWPSfnkQ4 SBzl3dqab5aJyo6kosAopophDX4lw8s6ln15emTKebh91d6vPOUM035FJ5PjmdHHKw+4 as30vUL7njqw56JjMYXhx2iEw7DLeh4ghy/z8ob3enXw8AXEsSN0ZgKPmqkGCaLhGsLf OZgrls8JPkv9jP+jMFjpHSuhHtAtqAtYC/YoxZHBKUjh7GfEwa6aPOAUVtauUSJ+Pgxq gCQQ== X-Gm-Message-State: AOJu0Yy/vzlQd9yYtAtqqrKEOVjxUdRUGOft7HY+T3b2WqRTcWCEjIux eXsjBjiFgAMIdWWQ/t9sYdx+wle8VWhMcy+FNusftQ== X-Google-Smtp-Source: AGHT+IF9PIW+IIOgYIV6uSfEiza11cgmJFocfkm5oZZZUQY/yD6hRKlTAUtprGCDc16SgZM69MmvAQCKh1NbH0JmaDI= X-Received: by 2002:a4a:2502:0:b0:56e:4bb5:3095 with SMTP id g2-20020a4a2502000000b0056e4bb53095mr2446388ooa.5.1692126318748; Tue, 15 Aug 2023 12:05:18 -0700 (PDT) MIME-Version: 1.0 References: <20230721115125.55137-1-bruce.richardson@intel.com> <20230815151053.996469-1-bruce.richardson@intel.com> <20230815151053.996469-5-bruce.richardson@intel.com> In-Reply-To: <20230815151053.996469-5-bruce.richardson@intel.com> From: Patrick Robb Date: Tue, 15 Aug 2023 15:05:08 -0400 Message-ID: Subject: Re: [PATCH v5 04/10] app/test: build using per-file dependency matrix To: Bruce Richardson Cc: dev@dpdk.org, ci@dpdk.org, =?UTF-8?Q?Morten_Br=C3=B8rup?= Content-Type: multipart/alternative; boundary="00000000000024fde90602fadc11" 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 --00000000000024fde90602fadc11 Content-Type: text/plain; charset="UTF-8" Adam from our team just raised something important about this patch and UNH CI which I missed during the RFC discussion. Presently, eal_flags_file_prefix_autotest fails on arm tx2 systems (arm people are aware of this, they think it might be a memory leak from mlx5 driver), so at their request we are disabling this unit test (on arm only) for them for the time being. This failure was discovered when we were initially standing up arm unit testing on tx2 servers earlier this year - previously there was no coverage for this. We do this filtering by going through the /app/test/meson.build file and resetting the eal_flags_file_prefix_autotest line to an empty string. This process is broken by your patch series. Again, I'm sorry I didn't catch this concern when discussing it during the RFC. This is why you are not getting unit test results for arm64, we can't run unit tests given the changes in this patch. Your refactor likely means that going forward, we will no longer be able to tailor the fast test suite (or any suite) per vendor request. That might actually be a good thing. In any case, if this is merged now, it is going to put the tree in a state where our CI doesn't run any unit testing on ARM. I don't know how close this patch series is to possibly hitting mainline, but, if possible, can that be delayed, pending us figuring out how we will respond to this situation? --00000000000024fde90602fadc11 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Adam from our team just raised something = important about this patch and UNH CI which I missed during the RFC discuss= ion. Presently,=C2=A0eal_flags_file_prefix_autotest fails on arm tx2 system= s (arm people are aware of this, they think it might be a memory leak from = mlx5 driver), so at their request we are disabling this unit test (on arm o= nly) for them for the time being. This failure was discovered when we were = initially standing up arm unit testing on tx2 servers earlier this year - p= reviously there was no coverage for this. We do this filtering by going=C2= =A0through the /app/test/meson.build file and resetting the=C2=A0eal_flags_= file_prefix_autotest line to an empty string. This process is broken by you= r patch series. Again, I'm sorry I didn't catch this concern when d= iscussing it during the RFC. This is why you are not getting unit test resu= lts for arm64, we can't run unit tests given the changes in this patch.= =C2=A0

Your refactor likely means that= going forward, we will no longer be able to tailor the fast test suite (or= any suite) per vendor request. That might actually be a good thing. In any= case, if this is merged now, it is going to put the tree in a state where = our CI doesn't run any unit testing on ARM. I don't know how close = this patch series is to possibly hitting mainline, but, if possible, can th= at be delayed, pending us figuring out how we will respond to this situatio= n?=C2=A0


--00000000000024fde90602fadc11--