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 A91E6430E0; Wed, 23 Aug 2023 09:33:57 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7F52D410F1; Wed, 23 Aug 2023 09:33:57 +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 96BE040ED6 for ; Wed, 23 Aug 2023 09:33:56 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1692776036; 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=sFgjEjt9USWsD4/DNrltD6EucOu9JcXJa/v1htsmzBs=; b=f5pYSONQG3XKPckBNZKsFCLp05tVlRYjUXcI0ZcLmar7xEEfyNKW2NOM+KSP9VEU8N5wr4 YT3nWlUG2Wuxx6SFp7uNDbGF0h6F3PVSOEaHAbwilmlmseylkmcKjgVLHCVmR6O1Z+i+dG 8gs1Rqitd6jZvArA9jyCESgGDHtGpFY= Received: from mail-lf1-f70.google.com (mail-lf1-f70.google.com [209.85.167.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-387-XuzJRLwkOjy7WPDxPUlfCA-1; Wed, 23 Aug 2023 03:33:54 -0400 X-MC-Unique: XuzJRLwkOjy7WPDxPUlfCA-1 Received: by mail-lf1-f70.google.com with SMTP id 2adb3069b0e04-4fe157ebea0so5438101e87.1 for ; Wed, 23 Aug 2023 00:33:53 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692776033; x=1693380833; h=content-transfer-encoding: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=sFgjEjt9USWsD4/DNrltD6EucOu9JcXJa/v1htsmzBs=; b=dBwlMrvs6GYrMHQswWdoh1nXKXjDdgOUojG9QeyK3jO++JiKVIKP+By5Zwd3W8IfGZ EO//Dg5s/8HOZo4z1UM0B1TwHUuAmm1XQ10YKrAoRD+y1kSzQ+AwxLNJcsBPW37Q5XNN DIlAXupgO9ztm6k7MwkSuiIipPErJSZ5tVQpK2N1gFS1Q8wker71qusT6l0quk9nFAdV 9p2IrY0q4ULP8S38x/2bt37y+gYgv+cEZ+4cV+WhzY/SZ2XQgoCyqYbRKZCNlPfEVIbg oRhipEl88fLor+ylwgcR7QL3PdLjDJ6/Iy8zk0bc4fn2ruNwC87mV19CJT/HksaUbLLv 4UDg== X-Gm-Message-State: AOJu0YwjUg9e+UGRLdMA1EoAmCV4x0yvWYpMcN5wYsg2rDMpU/QT3fLs OoI3muKQ0u+DRi1jerd4v4gwzOpsC45jaYZ+Z+n/XXe6kCpxalek59DPMCDlw8q+4z/jblvtY9r X3GAEAG+OGR9npPOFcM8= X-Received: by 2002:a05:6512:200d:b0:4fb:9d61:db4d with SMTP id a13-20020a056512200d00b004fb9d61db4dmr6733494lfb.18.1692776032753; Wed, 23 Aug 2023 00:33:52 -0700 (PDT) X-Google-Smtp-Source: AGHT+IHZ7Y5205eKCYrS0NMwom9mObRKgve9SKwpzp6M2/6jWTdYvsXB/xWJLBJfTgKR4Csk2eP682qaG0eM3gB+ApE= X-Received: by 2002:a05:6512:200d:b0:4fb:9d61:db4d with SMTP id a13-20020a056512200d00b004fb9d61db4dmr6733486lfb.18.1692776032417; Wed, 23 Aug 2023 00:33:52 -0700 (PDT) MIME-Version: 1.0 References: <20230721115125.55137-1-bruce.richardson@intel.com> <20230816153439.551501-1-bruce.richardson@intel.com> In-Reply-To: <20230816153439.551501-1-bruce.richardson@intel.com> From: David Marchand Date: Wed, 23 Aug 2023 09:33:40 +0200 Message-ID: Subject: Re: [PATCH v6 00/11] expand list of optional libraries To: Bruce Richardson Cc: dev@dpdk.org, probb@iol.unh.edu, mb@smartsharesystems.com, Aaron Conole , Thomas Monjalon X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable 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 Hello Bruce, On Wed, Aug 16, 2023 at 5:35=E2=80=AFPM Bruce Richardson wrote: > > This patchset contains changes to DPDK to make the build more configurabl= e > overall. The changes can be grouped into two areas: > > * Firstly, there are changes to make the build of the unit tests more > flexible and maintainable. These 7 patches switch the unit tests from > having separate hard-coded lists of files and tests, with lots of > conditional logic to add/remove items from the list, to instead being > a single list of files and dependencies per-file. The actual lists of > test-suites are built up dynamically from the list of files whose > dependencies are met and therefore will be part of the build. > This rework enables... > * The second set of patches which expands the list of libraries which > can be disabled at build-time, while still giving a successful build > of DPDK. Overall, after this set is applied, the list of optional > libraries in DPDK is approx doubled in size. [15 additions giving > 31 optional libs] > > NOTES: > * as stated in the RFC, the test binary moves in the build as a result > of this set - from app/test/dpdk-test to app/dpdk-test. > * running the suites of tests no longer works after patch 4 until patch > 5-7 is applied. If this is an issue, the patches can be merged on > apply. However, the individual tests are still runnable through the > test binary - it's only through "meson test" that there are issues, so > I'm keeping the patches separate since each is relatively complex in it= s > own right. Compilation is fine between those patches (which is the minimum). Seeing the size of the changes, I agree it is more readable to keep them separate. And I think we can live with unit tests being "broken" (well, absent) between those patches. > * A number of files in the test directory were not being built, and seem > to have never been built with meson. This requires separate work, outsi= de > the scope of the patchset to fix. > * Similarly, some unit tests were never added to test suites. These can > now be found by using grep to find the old REGISTER_TEST_COMMAND macro. > Again, adding orphan tests to suites is outside the scope of this set. As discussed in the thread for patch 5 (https://inbox.dpdk.org/dev/CAJvnSUBcsA3=3D9CZvnU+g5CJ261X=3DtJMivcvygO-D= =3Djcu+akEGg@mail.gmail.com/), merging this series renders ARM container unit tests unavailable at UNH. This is not perfect, but I prefer we make progress on the build framework changes the sooner possible. This is a temporary state until either the possibility to skip test is added (and UNH makes use of it instead of patching app/test/meson.build) or the fixes I proposed for ethdev cleanup are merged (and UNH removes the patching of meson). Hopefully, we will solve this remaining issue in the next weeks. Series applied, thanks for this nice cleanup and enhancement Bruce. As a followup on the optional libraries topic, next, I will merge the enable_libs option: https://patchwork.dpdk.org/project/dpdk/list/?series=3D29044 --=20 David Marchand