diff options
| author |   Jia-Ju Bai <baijiaju1990@gmail.com> | 2018-07-27 17:28:25 +0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2018-07-27 13:45:15 -0700 |
| commit | 04b9ce48ef19e09d8c65eb506b7982e99db212d7 (patch) | |
| tree | d82b4bd78cda0757069c571d3af16a80ab02839e /net/tipc | |
| parent | net: usb: sr9700: Replace mdelay() with msleep() in sr9700_bind() (diff) | |
| download | linux-dev-04b9ce48ef19e09d8c65eb506b7982e99db212d7.tar.xz linux-dev-04b9ce48ef19e09d8c65eb506b7982e99db212d7.zip | |
net: tipc: name_table: Replace GFP_ATOMIC with GFP_KERNEL in tipc_nametbl_init()
tipc_nametbl_init() is never called in atomic context.
It calls kzalloc() with GFP_ATOMIC, which is not necessary.
GFP_ATOMIC can be replaced with GFP_KERNEL.
This is found by a static analysis tool named DCNS written by myself.
Signed-off-by: Jia-Ju Bai <baijiaju1990@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/tipc')
| -rw-r--r-- | net/tipc/name_table.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/tipc/name_table.c b/net/tipc/name_table.c index bebe88cae07b..88f027b502f6 100644 --- a/net/tipc/name_table.c +++ b/net/tipc/name_table.c @@ -735,7 +735,7 @@ int tipc_nametbl_init(struct net *net) struct name_table *nt; int i; - nt = kzalloc(sizeof(*nt), GFP_ATOMIC); + nt = kzalloc(sizeof(*nt), GFP_KERNEL); if (!nt) return -ENOMEM; |