On Wed, 8 Jun 2022 02:49:47 +0300
Dmitry Kozlyuk <dkozl...@nvidia.com> wrote:

>  
> -def mount_huge(pagesize, mountpoint):
> +def mount_huge(pagesize, mountpoint, owner):
>      '''Mount the huge TLB file system'''
>      if mountpoint in get_mountpoints():
>          print(mountpoint, "already mounted")
> @@ -180,6 +180,8 @@ def mount_huge(pagesize, mountpoint):
>          cmd += ' -o pagesize={}'.format(pagesize * 1024)
>      cmd += ' nodev ' + mountpoint
>      os.system(cmd)
> +    if owner:
> +        os.system('chown {} {}'.format(owner, mountpoint))
>  

The hugetlb filesystem type already supports set owner.
Please use that instead of an additional chown command.

https://www.kernel.org/doc/Documentation/vm/hugetlbpage.txt



If the user applications are going to request huge pages using mmap system
call, then it is required that system administrator mount a file system of
type hugetlbfs:

  mount -t hugetlbfs \
        -o uid=<value>,gid=<value>,mode=<value>,pagesize=<value>,size=<value>,\
        min_size=<value>,nr_inodes=<value> none /mnt/huge

Reply via email to