If THIS_MODULE is not set, the module would be removed while debugfs is
being used.
It eventually makes kernel panic.

Fixes: 8ca151b568b6 ("iwlwifi: add the MVM driver")
Signed-off-by: Taehee Yoo <ap420...@gmail.com>
---
 drivers/net/wireless/intel/iwlwifi/mvm/debugfs.h | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/debugfs.h 
b/drivers/net/wireless/intel/iwlwifi/mvm/debugfs.h
index a83d252c0602..5bf4f7801b83 100644
--- a/drivers/net/wireless/intel/iwlwifi/mvm/debugfs.h
+++ b/drivers/net/wireless/intel/iwlwifi/mvm/debugfs.h
@@ -63,6 +63,7 @@ static const struct file_operations iwl_dbgfs_##name##_ops = 
{                \
        .read = iwl_dbgfs_##name##_read,                                \
        .open = simple_open,                                            \
        .llseek = generic_file_llseek,                                  \
+       .owner = THIS_MODULE,                                           \
 }
 
 #define MVM_DEBUGFS_WRITE_WRAPPER(name, buflen, argtype)               \
@@ -87,6 +88,7 @@ static const struct file_operations iwl_dbgfs_##name##_ops = 
{                \
        .read = iwl_dbgfs_##name##_read,                                \
        .open = simple_open,                                            \
        .llseek = generic_file_llseek,                                  \
+       .owner = THIS_MODULE,                                           \
 };
 
 #define _MVM_DEBUGFS_WRITE_FILE_OPS(name, buflen, argtype)             \
@@ -95,4 +97,5 @@ static const struct file_operations iwl_dbgfs_##name##_ops = 
{                \
        .write = _iwl_dbgfs_##name##_write,                             \
        .open = simple_open,                                            \
        .llseek = generic_file_llseek,                                  \
+       .owner = THIS_MODULE,                                           \
 };
-- 
2.17.1

Reply via email to