netdev-dpdk: Fix possible memory leak configuring VF MAC address. #453
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| name: Build and Test | |
| on: [push, pull_request] | |
| env: | |
| python_default: 3.12 | |
| jobs: | |
| build-linux: | |
| env: | |
| dependencies: | | |
| automake libtool gcc bc libjemalloc2 libjemalloc-dev \ | |
| libssl-dev llvm-dev libelf-dev libnuma-dev libpcap-dev \ | |
| ninja-build selinux-policy-dev | |
| AFXDP: ${{ matrix.afxdp }} | |
| ASAN: ${{ matrix.asan }} | |
| UBSAN: ${{ matrix.ubsan }} | |
| CC: ${{ matrix.compiler }} | |
| DPDK: ${{ matrix.dpdk }} | |
| DPDK_SHARED: ${{ matrix.dpdk_shared }} | |
| KERNEL: ${{ matrix.kernel }} | |
| LIBS: ${{ matrix.libs }} | |
| M32: ${{ matrix.m32 }} | |
| OPTS: ${{ matrix.opts }} | |
| STD: ${{ matrix.std }} | |
| TESTSUITE: ${{ matrix.testsuite }} | |
| name: linux ${{ join(matrix.*, ' ') }} | |
| runs-on: ubuntu-22.04 | |
| timeout-minutes: 30 | |
| strategy: | |
| fail-fast: false | |
| matrix: | |
| include: | |
| - compiler: gcc | |
| opts: --disable-ssl | |
| - compiler: clang | |
| opts: --disable-ssl | |
| - compiler: gcc | |
| std: c99 | |
| - compiler: clang | |
| std: c99 | |
| - compiler: gcc | |
| testsuite: test | |
| - compiler: clang | |
| testsuite: test | |
| asan: asan | |
| - compiler: clang | |
| testsuite: test | |
| ubsan: ubsan | |
| - compiler: gcc | |
| testsuite: test | |
| opts: --enable-shared | |
| - compiler: clang | |
| testsuite: test | |
| opts: --enable-shared | |
| - compiler: gcc | |
| testsuite: test | |
| dpdk: dpdk | |
| - compiler: clang | |
| testsuite: test | |
| dpdk: dpdk | |
| - compiler: gcc | |
| testsuite: test | |
| libs: -ljemalloc | |
| - compiler: clang | |
| testsuite: test | |
| libs: -ljemalloc | |
| - compiler: gcc | |
| afxdp: afxdp | |
| kernel: 5.4 | |
| - compiler: clang | |
| afxdp: afxdp | |
| kernel: 5.4 | |
| - compiler: gcc | |
| dpdk: dpdk | |
| opts: --enable-shared | |
| - compiler: clang | |
| dpdk: dpdk | |
| opts: --enable-shared | |
| - compiler: gcc | |
| dpdk_shared: dpdk-shared | |
| - compiler: clang | |
| dpdk_shared: dpdk-shared | |
| - compiler: gcc | |
| dpdk_shared: dpdk-shared | |
| opts: --enable-shared | |
| - compiler: clang | |
| dpdk_shared: dpdk-shared | |
| opts: --enable-shared | |
| - compiler: gcc | |
| m32: m32 | |
| opts: --disable-ssl | |
| steps: | |
| - name: checkout | |
| uses: actions/checkout@v4 | |
| - name: update PATH | |
| run: | | |
| echo "$HOME/bin" >> $GITHUB_PATH | |
| echo "$HOME/.local/bin" >> $GITHUB_PATH | |
| - name: set up python | |
| uses: actions/setup-python@v5 | |
| with: | |
| python-version: ${{ env.python_default }} | |
| - name: create ci signature file for the dpdk cache key | |
| if: matrix.dpdk != '' || matrix.dpdk_shared != '' | |
| # This will collect most of DPDK related lines, so hash will be different | |
| # if something changed in a way we're building DPDK including DPDK_VER. | |
| # This also allows us to use cache from any branch as long as version | |
| # and a way we're building DPDK stays the same. | |
| run: | | |
| grep -irE 'RTE_|DPDK|meson|ninja' -r .ci/ > dpdk-ci-signature | |
| cat dpdk-ci-signature | |
| - name: cache | |
| if: matrix.dpdk != '' || matrix.dpdk_shared != '' | |
| uses: actions/cache@v4 | |
| env: | |
| matrix_key: ${{ matrix.dpdk }}${{ matrix.dpdk_shared }} | |
| ci_key: ${{ hashFiles('dpdk-ci-signature') }} | |
| with: | |
| path: dpdk-dir | |
| key: ${{ env.matrix_key }}-${{ env.ci_key }} | |
| - name: update APT cache | |
| run: sudo apt update || true | |
| - name: install common dependencies | |
| run: sudo apt install -y ${{ env.dependencies }} | |
| - name: install libunbound libunwind | |
| if: matrix.m32 == '' | |
| run: sudo apt install -y libunbound-dev libunwind-dev | |
| - name: Reduce ASLR entropy | |
| if: matrix.asan != '' || matrix.ubsan != '' | |
| # Asan in llvm 14 provided in ubuntu-22.04 is incompatible with | |
| # high-entropy ASLR configured in much newer kernels that GitHub | |
| # runners are using leading to random crashes: | |
| # https://github.com/actions/runner-images/issues/9491 | |
| run: sudo sysctl -w vm.mmap_rnd_bits=28 | |
| - name: prepare | |
| run: ./.ci/linux-prepare.sh | |
| - name: build | |
| run: ./.ci/linux-build.sh | |
| - name: copy logs on failure | |
| if: failure() || cancelled() | |
| run: | | |
| # upload-artifact throws exceptions if it tries to upload socket | |
| # files and we could have some socket files in testsuite.dir. | |
| # Also, upload-artifact doesn't work well enough with wildcards. | |
| # So, we're just archiving everything here to avoid any issues. | |
| mkdir logs | |
| cp config.log ./logs/ | |
| cp -r ./*/_build/sub/tests/testsuite.* ./logs/ || true | |
| tar -czvf logs.tgz logs/ | |
| - name: upload logs on failure | |
| if: failure() || cancelled() | |
| uses: actions/upload-artifact@v4 | |
| with: | |
| name: logs-linux-${{ join(matrix.*, '-') }} | |
| path: logs.tgz | |
| build-osx: | |
| env: | |
| CC: clang | |
| OPTS: --disable-ssl | |
| name: osx clang --disable-ssl | |
| runs-on: macos-latest | |
| timeout-minutes: 30 | |
| strategy: | |
| fail-fast: false | |
| steps: | |
| - name: checkout | |
| uses: actions/checkout@v4 | |
| - name: update PATH | |
| run: | | |
| echo "$HOME/bin" >> $GITHUB_PATH | |
| echo "$HOME/.local/bin" >> $GITHUB_PATH | |
| - name: set up python | |
| uses: actions/setup-python@v5 | |
| with: | |
| python-version: ${{ env.python_default }} | |
| - name: install dependencies | |
| run: brew install automake libtool | |
| - name: prepare | |
| run: ./.ci/osx-prepare.sh | |
| - name: build | |
| run: ./.ci/osx-build.sh | |
| - name: upload logs on failure | |
| if: failure() | |
| uses: actions/upload-artifact@v4 | |
| with: | |
| name: logs-osx-clang---disable-ssl | |
| path: config.log | |
| build-linux-deb: | |
| env: | |
| deb_dependencies: | | |
| linux-headers-$(uname -r) build-essential fakeroot devscripts equivs | |
| DEB_PACKAGE: yes | |
| DPDK: ${{ matrix.dpdk }} | |
| name: linux deb ${{ matrix.dpdk }} dpdk | |
| runs-on: ubuntu-22.04 | |
| timeout-minutes: 30 | |
| strategy: | |
| fail-fast: false | |
| matrix: | |
| include: | |
| - dpdk: no | |
| - dpdk: shared | |
| steps: | |
| - name: checkout | |
| uses: actions/checkout@v4 | |
| - name: update PATH | |
| run: | | |
| echo "$HOME/bin" >> $GITHUB_PATH | |
| echo "$HOME/.local/bin" >> $GITHUB_PATH | |
| - name: update APT cache | |
| run: sudo apt update || true | |
| - name: install dependencies for debian packages | |
| run: sudo apt install -y ${{ env.deb_dependencies }} | |
| - name: install dpdk-dev | |
| if: matrix.dpdk != 'no' | |
| run: sudo apt install -y libdpdk-dev | |
| - name: prepare | |
| run: ./.ci/linux-prepare.sh | |
| - name: build | |
| run: ./.ci/linux-build.sh | |
| - name: upload deb packages | |
| uses: actions/upload-artifact@v4 | |
| with: | |
| name: deb-packages-${{ matrix.dpdk }}-dpdk | |
| path: '/home/runner/work/ovs/*.deb' |