dpdk: fix build with new DPDK 23.11

EttusResearch / uhd

The USRP™ Hardware Driver Repository

http://uhd.ettus.com

Other

1k stars 666 forks source link

dpdk: fix build with new DPDK 23.11 #714

Closed cpaelzer closed 9 months ago

cpaelzer commented 11 months ago

Description

Since 23.03 rte_thread_setname was deprecated and in 23.11 it is gone. Adapt to the new name rte_thread_set_name. See the release notes for more.

Testing Done

Test build against DPDK 23.11 as it is prepared for Debian and Ubuntu => https://launchpad.net/~paelzer/+archive/ubuntu/dpdk-23.11-test-builds/+build/27035932 Build fails without the fix https://launchpad.net/~paelzer/+archive/ubuntu/dpdk-23.11-test-builds/+build/27035895

Checklist

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project. See CODING.md.
[n/a] I have updated the documentation accordingly.
[n/a] I have added tests to cover my changes, and all previous tests pass.
[n/a] I have checked all compat numbers if they need updating (FPGA compat, MPM compat, noc_shell, specific RFNoC block, ...)

github-actions[bot] commented 11 months ago

CLA Assistant Lite bot:
Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

_{You can retrigger this bot by commenting recheck in this Pull Request}

cpaelzer commented 11 months ago

If not strictly required I'd like to save the time to read all the details of the CLA and signing it and instead rely on this term from CONTRIBUTING.md:

Small changes (less than 10 lines of code), or documentation fixes, may be       
accepted without prior submission of a CLA. This decision is made by             
Ettus Research / National Instruments.

mbr0wn commented 11 months ago

If not strictly required I'd like to save the time to read all the details of the CLA and signing it and instead rely on this term from CONTRIBUTING.md:
Small changes (less than 10 lines of code), or documentation fixes, may be       
accepted without prior submission of a CLA. This decision is made by             
Ettus Research / National Instruments.

You're good -- thanks again. This patch will percolate our internal CI and review pipelines (potentially with minor modifications) before going live, so don't be surprised if there's a delay between this PR closing and the patch going live.