Commit 4b428979 authored by Dariusz Stojaczyk's avatar Dariusz Stojaczyk Committed by Jim Harris
Browse files

setup.sh: fix chown 



Patch da7e3bb8 [1] disabled chown for vfio
groups and hugepage mountpoints. Chown code
was never being executed.

[1] da7e3bb8 ("setup.sh: replace username param with TARGET_USER env
variable")

Change-Id: Ie5c9ac42fca5b60c72ced501d4065842a5bef9cd
Signed-off-by: default avatarDariusz Stojaczyk <dariuszx.stojaczyk@intel.com>
Reviewed-on: https://review.gerrithub.io/395925


Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-by: default avatarBen Walker <benjamin.walker@intel.com>
parent 68b8237c

scripts/setup.sh

+2 −2
Original line number Diff line number Diff line
@@ -84,7 +84,7 @@ function linux_bind_driver() { 


	iommu_group=$(basename $(readlink -f /sys/bus/pci/devices/$bdf/iommu_group))

	if [ -e "/dev/vfio/$iommu_group" ]; then

		if [ -z "$TARGET_USER" ]; then

		if [ -n "$TARGET_USER" ]; then

			chown "$TARGET_USER" "/dev/vfio/$iommu_group"

		fi

	fi
@@ -237,7 +237,7 @@ function configure_linux { 
	fi



	if [ "$driver_name" = "vfio-pci" ]; then

		if [ -z "$TARGET_USER" ]; then

		if [ -n "$TARGET_USER" ]; then

			chown "$TARGET_USER" "$hugetlbfs_mount"

			chmod g+w "$hugetlbfs_mount"

		fi