Commit 8070b2b1 authored by David S. Miller's avatar David S. Miller

[IOAT]: Do not dereference THIS_MODULE directly to set unsafe.

Use the __unsafe() macro instead.

Noticed by Miles Lane.
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 52383678
...@@ -824,10 +824,9 @@ static int __init ioat_init_module(void) ...@@ -824,10 +824,9 @@ static int __init ioat_init_module(void)
{ {
/* it's currently unsafe to unload this module */ /* it's currently unsafe to unload this module */
/* if forced, worst case is that rmmod hangs */ /* if forced, worst case is that rmmod hangs */
if (THIS_MODULE != NULL) __unsafe(THIS_MODULE);
THIS_MODULE->unsafe = 1;
return pci_module_init(&ioat_pci_drv); pci_module_init(&ioat_pci_drv);
} }
module_init(ioat_init_module); module_init(ioat_init_module);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment