History

selftests: kmod: use variable NAME in kmod_test_0001()

Patch series "kmod/umh: a few fixes". Tiezhu Yang had sent out a patch set with a slew of kmod selftest fixes, and one patch which modified kmod to return 254 when a module was not found. This opened up pandora's box about why that was being used for and low and behold its because when UMH_WAIT_PROC is used we call a kernel_wait4() call but have never unwrapped the error code. The commit log for that fix details the rationale for the approach taken. I'd appreciate some review on that, in particular nfs folks as it seems a case was never really hit before. This patch (of 5): Use the variable NAME instead of "\000" directly in kmod_test_0001().

Tiezhu Yang committed 6 years ago

Commit aaa3e7f1 parent a089e3f1 file · +2 −2Browse files

Unified · Split

tools/testing/selftests/kmod/kmod.sh+2−2
@@ -343,7 +343,7 @@ kmod_test_0001_driver() · 342 unmodified lines
343343
344344 kmod_defaults_driver
345345 config_num_threads 1
346− printf '\000' >"$DIR"/config_test_driver
346+ printf $NAME >"$DIR"/config_test_driver
347347 config_trigger ${FUNCNAME[0]}
348348 config_expect_result ${FUNCNAME[0]} MODULE_NOT_FOUND
349349 }
@@ -354,7 +354,7 @@ kmod_test_0001_fs() · 4 unmodified lines
354354
355355 kmod_defaults_fs
356356 config_num_threads 1
357− printf '\000' >"$DIR"/config_test_fs
357+ printf $NAME >"$DIR"/config_test_fs
358358 config_trigger ${FUNCNAME[0]}
359359 config_expect_result ${FUNCNAME[0]} -EINVAL
360360 }

Keyboard shortcuts

General

Show keyboard shortcuts
?
Focus the current search
/

Issues and pull requests

Open filters
f

Issues, pull requests and Inbox

Select next item
j↓
Select previous item
k↑
Open selected item
Enter
Dismiss filters or clear selection
Esc

Repository code

Go to file
t