diff options
author | 2020-04-17 02:58:37 +0000 | |
---|---|---|
committer | 2020-04-27 11:10:10 -0700 | |
commit | 52785b6ae8eded7ac99d65c92d989b702e5b4376 (patch) | |
tree | a9934deb29022a7cdb18f3aca9eff7adcb00b80a /kernel/kcsan | |
parent | Improve KCSAN documentation a bit (diff) | |
download | linux-52785b6ae8eded7ac99d65c92d989b702e5b4376.tar.gz linux-52785b6ae8eded7ac99d65c92d989b702e5b4376.tar.bz2 linux-52785b6ae8eded7ac99d65c92d989b702e5b4376.zip |
kcsan: Use GFP_ATOMIC under spin lock
A spin lock is held in insert_report_filterlist(), so the krealloc()
should use GFP_ATOMIC. This commit therefore makes this change.
Reviewed-by: Marco Elver <elver@google.com>
Signed-off-by: Wei Yongjun <weiyongjun1@huawei.com>
Signed-off-by: Paul E. McKenney <paulmck@kernel.org>
Diffstat (limited to 'kernel/kcsan')
-rw-r--r-- | kernel/kcsan/debugfs.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kernel/kcsan/debugfs.c b/kernel/kcsan/debugfs.c index 1a08664a7fab..023e49c58d55 100644 --- a/kernel/kcsan/debugfs.c +++ b/kernel/kcsan/debugfs.c @@ -230,7 +230,7 @@ static ssize_t insert_report_filterlist(const char *func) /* initial allocation */ report_filterlist.addrs = kmalloc_array(report_filterlist.size, - sizeof(unsigned long), GFP_KERNEL); + sizeof(unsigned long), GFP_ATOMIC); if (report_filterlist.addrs == NULL) { ret = -ENOMEM; goto out; @@ -240,7 +240,7 @@ static ssize_t insert_report_filterlist(const char *func) size_t new_size = report_filterlist.size * 2; unsigned long *new_addrs = krealloc(report_filterlist.addrs, - new_size * sizeof(unsigned long), GFP_KERNEL); + new_size * sizeof(unsigned long), GFP_ATOMIC); if (new_addrs == NULL) { /* leave filterlist itself untouched */ |