Mellanox bullseye merge (#1)

* Make neccesary changed to mellanox platform code to build on Debian 11 * Revert use of backported kernel to build mft and elect to only build kernel module under bullseye
saiarcot895 · Nov 9, 2021 · 04dcd6f · 04dcd6f
1 parent 19e3e29
commit 04dcd6f
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 13 deletions.
diff --git a/platform/mellanox/mft.mk b/platform/mellanox/mft.mk
@@ -26,11 +26,11 @@ $(MFT)_SRC_PATH = $(PLATFORM_PATH)/mft
 SONIC_MAKE_DEBS += $(MFT)
 
 ifeq ($(BLDENV), bullseye)
-$(MFT)_DEPENDS = $(LINUX_HEADERS) $(LINUX_HEADERS_COMMON)
-endif
+$(MFT)_DEPENDS += $(LINUX_HEADERS) $(LINUX_HEADERS_COMMON)
 
 KERNEL_MFT = kernel-mft-dkms-modules-$(KVERSION)_$(MFT_VERSION)_amd64.deb
 $(eval $(call add_derived_package,$(MFT),$(KERNEL_MFT)))
+endif
 
 MFT_OEM = mft-oem_$(MFT_VERSION)-$(MFT_REVISION)_amd64.deb
 $(eval $(call add_derived_package,$(MFT),$(MFT_OEM)))
diff --git a/platform/mellanox/mft/Makefile b/platform/mellanox/mft/Makefile
@@ -35,6 +35,7 @@ $(addprefix $(DEST)/, $(MAIN_TARGET)): $(DEST)/% :
  wget -O $(MFT_TGZ) http://www.mellanox.com/downloads/MFT/$(MFT_TGZ)
  tar xzf $(MFT_TGZ)
 
+ifeq ($(BLDENV), bullseye)
  pushd $(MFT_NAME)/SDEBS
 
  # put a lock here because dpkg does not allow installing packages in parallel
@@ -46,7 +47,7 @@ $(addprefix $(DEST)/, $(MAIN_TARGET)): $(DEST)/% :
 
  popd
 
- sudo dkms build kernel-mft-dkms/$(MFT_VERSION) -k $(KVERSION) -a amd64 --kernelsourcedir /usr/src/linux-headers-$(KVERSION)
+ sudo dkms build kernel-mft-dkms/$(MFT_VERSION) -k $(KVERSION) -a amd64
  sudo dkms mkbmdeb kernel-mft-dkms/$(MFT_VERSION) -k $(KVERSION) -a amd64
 
  # w/a: remove dependencies
@@ -62,6 +63,7 @@ $(addprefix $(DEST)/, $(MAIN_TARGET)): $(DEST)/% :
  popd
 
  rm -rf $(DKMS_TMP)
+endif
 
  # fix timestamp because we do not actually build tools, only kernel
  touch $(MFT_NAME)/DEBS/*.deb

diff --git a/platform/mellanox/rules.mk b/platform/mellanox/rules.mk
@@ -14,12 +14,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 #
-
-# Set KVERSION correctly depending on build environment
-ifeq ($(BLDENV), buster)
-KVERSION = 5.10.0-0.bpo.9-amd64
-endif
-
 include $(PLATFORM_PATH)/sdk.mk
 include $(PLATFORM_PATH)/fw.mk
 include $(PLATFORM_PATH)/mft.mk

diff --git a/sonic-slave-buster/Dockerfile.j2 b/sonic-slave-buster/Dockerfile.j2
@@ -153,10 +153,6 @@ RUN apt-get update && apt-get install -y \
 # For mellanox sai build
  libtool-bin \
  libxml2-dev \
-# For mellanox MFT build
- linux-image-5.10.0-0.bpo.9-amd64 \
- linux-headers-5.10.0-0.bpo.9-amd64 \
- linux-headers-5.10.0-0.bpo.9-common \
 # For BFN sdk build
  libusb-1.0-0-dev \
  libcurl3-nss-dev \