From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id D5BA2C6FA82 for ; Tue, 27 Sep 2022 13:13:35 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4BF3D8E00D3; Tue, 27 Sep 2022 09:13:35 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 46EF78E00C1; Tue, 27 Sep 2022 09:13:35 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 30F858E00D3; Tue, 27 Sep 2022 09:13:35 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0014.hostedemail.com [216.40.44.14]) by kanga.kvack.org (Postfix) with ESMTP id 21FE18E00C1 for ; Tue, 27 Sep 2022 09:13:35 -0400 (EDT) Received: from smtpin24.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay02.hostedemail.com (Postfix) with ESMTP id DAC77120C37 for ; Tue, 27 Sep 2022 13:13:34 +0000 (UTC) X-FDA: 79957907148.24.5716B23 Received: from mail-ej1-f45.google.com (mail-ej1-f45.google.com [209.85.218.45]) by imf15.hostedemail.com (Postfix) with ESMTP id 83DFAA000F for ; Tue, 27 Sep 2022 13:13:34 +0000 (UTC) Received: by mail-ej1-f45.google.com with SMTP id bj12so20552118ejb.13 for ; Tue, 27 Sep 2022 06:13:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=user-agent:in-reply-to:content-disposition:mime-version:references :message-id:subject:cc:to:from:date:from:to:cc:subject:date; bh=xKQUJ8crDSiafngxlHWxH3oZauT5Eq9qaoxq/VTtdws=; b=QCzn9+LBEWtv03A3zMGdq2xANA/WgN/QhSTWl3M74FSY91wfGosM7O1ACH+OdpvawE u7oFFk3kwcfdA7yhe+NsGeCkvb2djwO5FLIjxmCKMH7rdlM0zQJjvqGQz/IIIJtV0aC5 TECmKd4yjnBB9BPuma+AmqfHl99lj8jX3hsHL+szmPhbZs8Ovxj32ggBzL1Cao6b3K58 gPk9skKwAo+hOw+jatMChu1pXB+6TmFRgBXsjzEojgs23mKEFvr6OpDXsPZTaoJka3d2 bbNUvBeEamzTowLKXCBGcGnIhU4zfjF8LSCrTdb1vDImANmZsa6lTXSgO8AkYOUhft/C lc6Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=user-agent:in-reply-to:content-disposition:mime-version:references :message-id:subject:cc:to:from:date:x-gm-message-state:from:to:cc :subject:date; bh=xKQUJ8crDSiafngxlHWxH3oZauT5Eq9qaoxq/VTtdws=; b=SptMgwLgbpL2T20OUH4WlUwT+JhxMw2saCu2ixl84qWE92oEaKvSo3PElz1Pxno5EV EvCTCSvUp4h6pLl40EgW888+6C/+c7Rj5fpZL8Yj3VqEgdXzoK9hI4q7cOgT18g6LQN8 ucg2KQYiFOi68RprcqUL52RzQBj7uWOY/IgVvjLt2qstI+zw5Pr3BcwqawuakMcFrF9f tmJqPt0Kc51H92/pqKTWExhO46+UsjgONLvyDjbn7iKdTTnfR6+mr3OMErB8YFxqjJOA Da5o3ZkxZilEzO6a4GKCCmwMiVsBAzBb4hv4AoM+5kC+GSRBLMRD4WZPaqKkMe4wX428 7X2Q== X-Gm-Message-State: ACrzQf2t3Hlw1AKfD2WWgmQIDFO5lNeIwXDoq1E0fWt8EPDDEKgTBegZ kWWczZNH1MuMRF4ZLEELBcMSdQ== X-Google-Smtp-Source: AMsMyM6IazSuWmATHKaG9FWJD3wvDPiaZfgItkify6a5DbB7PexUdLedG0xN1DdrVlMBYfTd77o9QQ== X-Received: by 2002:a17:906:8473:b0:77b:efa8:50e4 with SMTP id hx19-20020a170906847300b0077befa850e4mr22495363ejc.250.1664284412975; Tue, 27 Sep 2022 06:13:32 -0700 (PDT) Received: from elver.google.com ([2a00:79e0:9c:201:693c:15a1:a531:bb4e]) by smtp.gmail.com with ESMTPSA id u24-20020a056402065800b004571907240asm1265720edx.36.2022.09.27.06.13.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 27 Sep 2022 06:13:32 -0700 (PDT) Date: Tue, 27 Sep 2022 15:13:25 +0200 From: Marco Elver To: andrey.konovalov@linux.dev Cc: Alexander Potapenko , Andrey Konovalov , Dmitry Vyukov , Andrey Ryabinin , kasan-dev@googlegroups.com, Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Andrey Konovalov Subject: Re: [PATCH mm 1/3] kasan: switch kunit tests to console tracepoints Message-ID: References: <653d43e9a6d9aad2ae148a941dab048cb8e765a8.1664044241.git.andreyknvl@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <653d43e9a6d9aad2ae148a941dab048cb8e765a8.1664044241.git.andreyknvl@google.com> User-Agent: Mutt/2.2.7 (2022-08-07) ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1664284414; a=rsa-sha256; cv=none; b=kJuaiG5x4vRJ0md80e7AQQE5K40efqY5Tm9TDTN5//kJWLeyf2AFhtxp3QpbRMjYGadXf1 C1Nklx6GvDlWt0jRm+c0RdfTvyqTaXfb/RYsK/LgsUK9wBiGwRwS9KVReUiFL7PsrshGM2 Zc+qR0JxwahTTQLPyORt9aV+M9V2JX0= ARC-Authentication-Results: i=1; imf15.hostedemail.com; dkim=pass header.d=google.com header.s=20210112 header.b=QCzn9+LB; dmarc=pass (policy=reject) header.from=google.com; spf=pass (imf15.hostedemail.com: domain of elver@google.com designates 209.85.218.45 as permitted sender) smtp.mailfrom=elver@google.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1664284414; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=xKQUJ8crDSiafngxlHWxH3oZauT5Eq9qaoxq/VTtdws=; b=R2CxxcqtURuDxHiaBmjG+v0PWcirNX05Adh4A0UOMAlYO4T4Bv70CNLgoNu8WNbZFviOwK 0MlC/ZeHR0XVjYl9XEMim+le6U2BKHFEwKvJdy286ShhTGb20ttya8IKHkDH7DuZI1LB4B gC1rhnuzar86ypnXtaijPjHqRkwA/e0= X-Stat-Signature: fur4rmsgrg3uycnx7bjhhus1rcucxacu X-Rspam-User: X-Rspamd-Server: rspam04 Authentication-Results: imf15.hostedemail.com; dkim=pass header.d=google.com header.s=20210112 header.b=QCzn9+LB; dmarc=pass (policy=reject) header.from=google.com; spf=pass (imf15.hostedemail.com: domain of elver@google.com designates 209.85.218.45 as permitted sender) smtp.mailfrom=elver@google.com X-Rspamd-Queue-Id: 83DFAA000F X-HE-Tag: 1664284414-686222 X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Sat, Sep 24, 2022 at 08:31PM +0200, andrey.konovalov@linux.dev wrote: > From: Andrey Konovalov > > Switch KUnit-compatible KASAN tests from using per-task KUnit resources > to console tracepoints. > > This allows for two things: > > 1. Migrating tests that trigger a KASAN report in the context of a task > other than current to KUnit framework. > This is implemented in the patches that follow. > > 2. Parsing and matching the contents of KASAN reports. > This is not yet implemented. > > Signed-off-by: Andrey Konovalov > --- > lib/Kconfig.kasan | 2 +- > mm/kasan/kasan_test.c | 85 +++++++++++++++++++++++++++++++------------ > mm/kasan/report.c | 31 ---------------- > 3 files changed, 63 insertions(+), 55 deletions(-) > > diff --git a/lib/Kconfig.kasan b/lib/Kconfig.kasan > index ca09b1cf8ee9..ba5b27962c34 100644 > --- a/lib/Kconfig.kasan > +++ b/lib/Kconfig.kasan > @@ -181,7 +181,7 @@ config KASAN_VMALLOC > > config KASAN_KUNIT_TEST > tristate "KUnit-compatible tests of KASAN bug detection capabilities" if !KUNIT_ALL_TESTS > - depends on KASAN && KUNIT > + depends on KASAN && KUNIT && TRACEPOINTS > default KUNIT_ALL_TESTS > help > A KUnit-based KASAN test suite. Triggers different kinds of > diff --git a/mm/kasan/kasan_test.c b/mm/kasan/kasan_test.c > index f25692def781..3a2886f85e69 100644 > --- a/mm/kasan/kasan_test.c > +++ b/mm/kasan/kasan_test.c > @@ -5,8 +5,12 @@ > * Author: Andrey Ryabinin > */ > > +#define pr_fmt(fmt) "kasan_test: " fmt > + > +#include > #include > #include > +#include > #include > #include > #include > @@ -14,21 +18,28 @@ > #include > #include > #include > +#include > #include > #include > +#include > #include > -#include > #include > -#include > +#include > > #include > > -#include > - > #include "kasan.h" > > #define OOB_TAG_OFF (IS_ENABLED(CONFIG_KASAN_GENERIC) ? 0 : KASAN_GRANULE_SIZE) > > +static bool multishot; > + > +/* Fields set based on lines observed in the console. */ > +static struct { > + bool report_found; > + bool async_fault; > +} test_status; > + > /* > * Some tests use these global variables to store return values from function > * calls that could otherwise be eliminated by the compiler as dead code. > @@ -36,35 +47,61 @@ > void *kasan_ptr_result; > int kasan_int_result; > > -static struct kunit_resource resource; > -static struct kunit_kasan_status test_status; Also remove this struct from kasan.h?