search

OpenVisualCloud / Media-Transport-Library

A real-time media transport(DPDK, AF_XDP, RDMA) stack for both raw and compressed video based on COTS hardware.
BSD 3-Clause "New" or "Revised" License
163 stars 53 forks source link

mtl-ice 1.12.7 does not compile under kernel 6.6.16 #748

Closed sammirata closed 6 months ago

sammirata commented 7 months ago

We recently had to upgrade our OS from 6.4.x to the latest stable kernel (6.6.16). Unfortunately, the mtl-ice v1.12.7 no longer compiles. Here is the error:

/mtl-ice-1.12.7/src/ice_txrx.h:363:29: error: field, xdp_rxq has incomplete type 363 | struct xdp_rxq_info xdp_rxq;

Any help or patches would be appreciated.

ricmli commented 7 months ago

Have you correctly installed the kernel headers for 6.6.16?

sammirata commented 7 months ago

Yes, I just double checked. The headers come from the exact same linux version and are the only ones installed in the build system (buildroot). This same compilation script was working before I upgraded the headers and kernel.

ricmli commented 7 months ago

It seems that the ice driver is not compatible with the newer kernel. If you only use NIC's PF, you do not have to build the ice driver, just use the in-tree kernel driver. If you need to use VF, another option may be to use old kernel and patched ice driver on host, then use newer kernel inside a VM and passsthrough the VFs to it.

frankdjx commented 7 months ago

I discovered that there is a new version, 1.13.7, available at https://www.intel.com/content/www/us/en/download/19630/intel-network-adapter-driver-for-e810-series-devices-under-linux.html. I'm wondering if the original ice 1.13.7 can be compiled on the 6.6.16 kernel? If it can, then we will need to update the IMTL patches to 1.13.7. However, if it cannot, this is a problem for the upstream ice driver.

sammirata commented 7 months ago

yes, the original 1.13.7 does compile under 6.6.16. Can you port your patches to it?

sammirata commented 7 months ago

I created a pull request with the ported patches in case that helps.

frankdjx commented 6 months ago

Close this one since it fixed already. Thanks.