[LU-7086] RHEL made /sbin a symlink to /usr/sbin/

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.8.0
• Severity: 3
• Assignee: Emoly Liu
• Reporter: Oleg Drokin

Description

We have a code in test-framework.sh that tries to avoid double-mounting /sbin/mount.lustre when running from a built tree:

       local mount_lustre=$LUSTRE/utils/mount.lustre
        if [ -f $mount_lustre ]; then
                local sbin_mount=/sbin/mount.lustre
                if grep -qw "$sbin_mount" /proc/mounts; then
                        cmp -s $mount_lustre $sbin_mount || umount $sbin_mount
                fi

In rhel7 and other new distros with the /sbin -> /usr/sbin symlink that never matches, /proc/mounts output is like this:

192.168.10.1:/exports/centos7-nfsroot/home/green/git/lustre-release/lustre/utils/mount.lustre /usr/sbin/mount.lustre nfs4 rw,relatime,vers=4.0,rsize=1048576,wsize=1048576,namlen=255,hard,proto=tcp,port=0,timeo=600,retrans=2,sec=sys,clientaddr=192.168.10.232,local_lock=none,addr=192.168.10.1 0 0

This is sounds mostly harmless, after all whats acouple of more bind mounts, right? Except for an unknown reason (I am still looking), rhel7 kernel is doubling size of mnt_cache slab on every such bind mount, so 7 iterations in the cache takes 120 megabytes, next iteration brings it to 240 and so on. It does add up quickly and some other slabs also grow (though not as drastically), so we quickly run out of memory (I tried and rhel6 kernel does not have this problem).

Comments

[Joseph Gmitter (Inactive)]

Landed for 2.8

[Gerrit Updater]

Oleg Drokin ([e-mail]) merged in patch http://review.whamcloud.com/16606/
Subject: LU-7086 tests: resolve /sbin symlink in test-framework.sh
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 8f9ab85d3c79f06c9db6b7b44368857a4714d455

[Gerrit Updater]

Emoly Liu ([e-mail]) uploaded a new patch: http://review.whamcloud.com/16606
Subject: LU-7086 tests: resolve /sbin symlink in test-framework.sh
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 55d693fbe3ce9633c992761d2887174716b64706

[Peter Jones]

Emoly

Could you please look into this issue?

Thanks

Peter