diff options
| author | Rusty Russell <[email protected]> | 2015-06-17 06:16:52 +0930 |
|---|---|---|
| committer | Rusty Russell <[email protected]> | 2015-06-23 15:27:37 +0930 |
| commit | 74c3dea355245c17ee407a3ce3ea34f55b40f2eb (patch) | |
| tree | ca68b882f14d82c132395e165c303782ed32e220 | |
| parent | b6c09b512dc0c5f62a4cc40184f5de749e90fdac (diff) | |
params: suppress unused variable error, warn once just in case code changes.
It shouldn't fail due to OOM (it's boot time), and already warns if we
get two identical names. But you never know what the future holds, and
WARN_ON_ONCE() keeps gcc happy with minimal code.
Reported-by: Louis Langholtz <[email protected]>
Acked-by: Tejun Heo <[email protected]>
Signed-off-by: Rusty Russell <[email protected]>
| -rw-r--r-- | kernel/params.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/kernel/params.c b/kernel/params.c index 7edf31f2ce96..e906874da5fc 100644 --- a/kernel/params.c +++ b/kernel/params.c @@ -884,6 +884,7 @@ static void __init version_sysfs_builtin(void) mk = locate_module_kobject(vattr->module_name); if (mk) { err = sysfs_create_file(&mk->kobj, &vattr->mattr.attr); + WARN_ON_ONCE(err); kobject_uevent(&mk->kobj, KOBJ_ADD); kobject_put(&mk->kobj); } |