[PATCH 1/1] kasan: remove unnecessary sync argument from start_report()

[Maninder Singh]

commit 7ce0ea19d50e ("kasan: switch kunit tests to console tracepoints")
removed use of sync variable, thus removing that extra argument also.

Signed-off-by: Maninder Singh <manin...@samsung.com>
---
mm/kasan/report.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/mm/kasan/report.c b/mm/kasan/report.c
index 62c01b4527eb..27efb78eb32d 100644
--- a/mm/kasan/report.c
+++ b/mm/kasan/report.c
@@ -203,7 +203,7 @@ static inline void fail_non_kasan_kunit_test(void) { }

static DEFINE_RAW_SPINLOCK(report_lock);

-static void start_report(unsigned long *flags, bool sync)
+static void start_report(unsigned long *flags)
{
fail_non_kasan_kunit_test();
/* Respect the /proc/sys/kernel/traceoff_on_warning interface. */
@@ -543,7 +543,7 @@ void kasan_report_invalid_free(void *ptr, unsigned long ip, enum kasan_report_ty
if (unlikely(!report_enabled()))
return;

- start_report(&flags, true);
+ start_report(&flags);

__memset(&info, 0, sizeof(info));
info.type = type;
@@ -581,7 +581,7 @@ bool kasan_report(const void *addr, size_t size, bool is_write,
goto out;
}

- start_report(&irq_flags, true);
+ start_report(&irq_flags);

__memset(&info, 0, sizeof(info));
info.type = KASAN_REPORT_ACCESS;
@@ -615,7 +615,7 @@ void kasan_report_async(void)
if (unlikely(!report_enabled()))
return;

- start_report(&flags, false);
+ start_report(&flags);
pr_err("BUG: KASAN: invalid-access\n");
pr_err("Asynchronous fault: no details available\n");
pr_err("\n");
--
2.34.1

[Andrey Ryabinin]

Maninder Singh <manin...@samsung.com> writes:

> commit 7ce0ea19d50e ("kasan: switch kunit tests to console tracepoints")
> removed use of sync variable, thus removing that extra argument also.
>
> Signed-off-by: Maninder Singh <manin...@samsung.com>

Acked-by: Andrey Ryabinin <ryabin...@gmail.com>

[Andrey Konovalov]

Reviewed-by: Andrey Konovalov <andre...@gmail.com>

Thanks!