search

dentproject / dentOS

dentOS SwitchDev based NOS
Other
206 stars 59 forks source link

[tx4810] Update the MFT PCI drivers to latest version 4.16.3. #63

Closed chenglin-tsai closed 3 years ago

chenglin-tsai commented 3 years ago

Since the kernel 5.8 used by tx4810 is removed [1], the tx4810 was configured to work on kernel 5.10. But the MFT PCI drivers on tx4810 are failed to create device node of "/dev/mst/mt52100_pciconf0" with kernel 5.10. The error logs like below:

ioctl (PCICONF_INIT): Inappropriate ioctl for device
ioctl (PCI_INIT): Inappropriate ioctl for device
cat: /dev/mst/mt52100_pci_cr0: No such file or directory
ioctl (PCICONF_INIT): Inappropriate ioctl for device
ioctl (PCI_INIT): Inappropriate ioctl for device
cat: /dev/mst/mt52100_pci_cr0: No such file or directory
ioctl (PCICONF_INIT): Inappropriate ioctl for device
ioctl (PCI_INIT): Inappropriate ioctl for device
cat: /dev/mst/mt52100_pci_cr0: No such file or directory

To fix this, we have to update MFT PCI drivers to its latest version from Mellanox official website [2] and the source codes can be found in downlaod MFT package [3].

[1]: https://github.com/dentproject/dentOS/pull/59 [2]: https://www.mellanox.com/products/adapter-software/firmware-tools [3]: https://github.com/dentproject/dentOS/pull/63#issuecomment-815822791

Test log: dentOS_tx4810_test_onlp_with_mft_pci_driver_v4.16.3_20210408.log

Signed-off-by: Chenglin Tsai chenglin.tsai@deltaww.com

paulmenzel commented 3 years ago

The referenced Mellanox site only offers DEB/RPM packages, or am I missing something? Can you please add the commands you used to extract the (included) driver files to the commit message?

taraschornyiplv commented 3 years ago

@paulmenzel this is a list of commands that can be used to get file from MFT package

tar xf mft-4.16.3-12-arm64-deb.tgz ls mft-4.16.3-12-arm64-deb/SDEBS/kernel-mft-dkms_4.16.3-12_all.deb cd mft-4.16.3-12-arm64-deb/SDEBS/ ar xv kernel-mft-dkms_4.16.3-12_all.deb tar xf data.tar.xz ls usr/src/kernel-mft-dkms-4.16.3/driver_common.c

paulmenzel commented 3 years ago

Yes, thank you, that is what I thought. I would always add these things to the commit message, so it’s crystal clear, where the code came from.

chenglin-tsai commented 3 years ago

@taraschornyiplv, I appreciate your detailed explanations.