diff mbox series

[ovs-dev,v9,2/2] rhel: Enable USDT scripts by default in Fedora builds.

Message ID 20240220214753.2799436-3-aconole@redhat.com
State Changes Requested
Headers show
Series debugging: Add a revalidator probe, and monitor script | expand

Checks

Context Check Description
ovsrobot/apply-robot success apply and check: success
ovsrobot/github-robot-_Build_and_Test success github build: passed
ovsrobot/intel-ovs-compilation success test: success

Commit Message

Aaron Conole Feb. 20, 2024, 9:47 p.m. UTC
All supported versions of Fedora do package libbpf, so it
makes sense to enable USDT support.

Acked-by: Simon Horman <horms@ovn.org>
Signed-off-by: Aaron Conole <aconole@redhat.com>
---
 rhel/openvswitch-fedora.spec.in | 8 ++++++++
 1 file changed, 8 insertions(+)

Comments

Eelco Chaudron March 1, 2024, 2:06 p.m. UTC | #1
On 20 Feb 2024, at 22:47, Aaron Conole wrote:

> All supported versions of Fedora do package libbpf, so it
> makes sense to enable USDT support.
>
> Acked-by: Simon Horman <horms@ovn.org>
> Signed-off-by: Aaron Conole <aconole@redhat.com>

This patch looks good to me.

Acked-by: Eelco Chaudron <echaudro@redhat.com>
diff mbox series

Patch

diff --git a/rhel/openvswitch-fedora.spec.in b/rhel/openvswitch-fedora.spec.in
index 5d24ebcda8..94b6d7431c 100644
--- a/rhel/openvswitch-fedora.spec.in
+++ b/rhel/openvswitch-fedora.spec.in
@@ -28,6 +28,8 @@ 
 %bcond_with dpdk
 # To disable AF_XDP support, specify '--without afxdp' when building
 %bcond_without afxdp
+# To control the USDT support
+%bcond_without usdt
 
 # If there is a need to automatically enable the package after installation,
 # specify the "--with autoenable"
@@ -77,6 +79,9 @@  Provides: %{name}-dpdk = %{version}-%{release}
 %if %{with afxdp}
 BuildRequires: libxdp-devel libbpf-devel numactl-devel
 %endif
+%if %{with usdt}
+BuildRequires: libbpf-devel systemtap-sdt-devel
+%endif
 BuildRequires: unbound unbound-devel
 
 Requires: openssl hostname iproute module-init-tools unbound
@@ -173,6 +178,9 @@  This package provides IPsec tunneling support for OVS tunnels.
         --enable-afxdp \
 %else
         --disable-afxdp \
+%endif
+%if %{with usdt}
+        --enable-usdt-probes \
 %endif
         --enable-ssl \
         --disable-static \