[LU-336] Function module_loaded in test-framework.sh returns false positives

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.1.0
• Affects Version/s: Lustre 2.1.0
• Environment: RHEL6 based
• Severity: 3
• Assignee: Jian Yu
• Reporter: Prakash Surya (Inactive)

Description

The function module_loaded can easily produce false positives based on it's lenient way of grep'ing the output of lsmod.

Tightening up the regex can improve upon this.

For example, if checking to see if the 'ost' module is loaded.. It will report true if another module is already loaded and has 'ost' anywhere in its name (i.e. vhost_net).

Function's match criteria could mort strict to avoid this kind of problem:

module_loaded() {
        lsmod | grep ^$1 # Or perhaps even '^\b$1\b'
}

I will submit a patch to gerrit soon.