From: Alexander Potapenko <glider@google.com>
To: Andrey Konovalov <adech.fo@gmail.com>,
Christoph Lameter <cl@linux.com>,
Dmitriy Vyukov <dvyukov@google.com>,
Andrew Morton <akpm@linux-foundation.org>,
Andrey Ryabinin <ryabinin.a.a@gmail.com>,
Steven Rostedt <rostedt@goodmis.org>,
Joonsoo Kim <iamjoonsoo.kim@lge.com>,
JoonSoo Kim <js1304@gmail.com>,
Kostya Serebryany <kcc@google.com>
Cc: kasan-dev <kasan-dev@googlegroups.com>,
LKML <linux-kernel@vger.kernel.org>,
Linux Memory Management List <linux-mm@kvack.org>
Subject: Re: [PATCH v4 0/7] SLAB support for KASAN
Date: Wed, 9 Mar 2016 12:12:29 +0100 [thread overview]
Message-ID: <CAG_fn=WAs6a1jbX1A3RBJM8-n3OhNJYZz4xsrjY3xX7rrTkxfg@mail.gmail.com> (raw)
In-Reply-To: <cover.1457519440.git.glider@google.com>
Sorry, this cover letter had incorrect version, I've resent it with
the corrected one.
On Wed, Mar 9, 2016 at 12:05 PM, Alexander Potapenko <glider@google.com> wrote:
> This patch set implements SLAB support for KASAN
>
> Unlike SLUB, SLAB doesn't store allocation/deallocation stacks for heap
> objects, therefore we reimplement this feature in mm/kasan/stackdepot.c.
> The intention is to ultimately switch SLUB to use this implementation as
> well, which will save a lot of memory (right now SLUB bloats each object
> by 256 bytes to store the allocation/deallocation stacks).
>
> Also neither SLUB nor SLAB delay the reuse of freed memory chunks, which
> is necessary for better detection of use-after-free errors. We introduce
> memory quarantine (mm/kasan/quarantine.c), which allows delayed reuse of
> deallocated memory.
>
> Alexander Potapenko (7):
> kasan: Modify kmalloc_large_oob_right(), add
> kmalloc_pagealloc_oob_right()
> mm, kasan: SLAB support
> mm, kasan: Added GFP flags to KASAN API
> arch, ftrace: For KASAN put hard/soft IRQ entries into separate
> sections
> mm, kasan: Stackdepot implementation. Enable stackdepot for SLAB
> kasan: Test fix: Warn if the UAF could not be detected in kmalloc_uaf2
> mm: kasan: Initial memory quarantine implementation
> ---
> v2: - merged two patches that touched kmalloc_large_oob_right
> - moved stackdepot implementation to lib/
> - moved IRQ definitions to include/linux/interrupt.h
>
> v3: - minor description changes
> - store deallocation info in the "mm, kasan: SLAB support" patch
>
> v4: - fix kbuild error reports
>
> v5: - SLAB allocator, stackdepot: adopted suggestions by Andrey Ryabinin
> - IRQ: fixed kbuild warnings
> ---
>
> Documentation/kasan.txt | 5 +-
> arch/arm/include/asm/exception.h | 2 +-
> arch/arm/kernel/vmlinux.lds.S | 1 +
> arch/arm64/include/asm/exception.h | 2 +-
> arch/arm64/kernel/vmlinux.lds.S | 1 +
> arch/blackfin/kernel/vmlinux.lds.S | 1 +
> arch/c6x/kernel/vmlinux.lds.S | 1 +
> arch/metag/kernel/vmlinux.lds.S | 1 +
> arch/microblaze/kernel/vmlinux.lds.S | 1 +
> arch/mips/kernel/vmlinux.lds.S | 1 +
> arch/nios2/kernel/vmlinux.lds.S | 1 +
> arch/openrisc/kernel/vmlinux.lds.S | 1 +
> arch/parisc/kernel/vmlinux.lds.S | 1 +
> arch/powerpc/kernel/vmlinux.lds.S | 1 +
> arch/s390/kernel/vmlinux.lds.S | 1 +
> arch/sh/kernel/vmlinux.lds.S | 1 +
> arch/sparc/kernel/vmlinux.lds.S | 1 +
> arch/tile/kernel/vmlinux.lds.S | 1 +
> arch/x86/kernel/Makefile | 1 +
> arch/x86/kernel/vmlinux.lds.S | 1 +
> include/asm-generic/vmlinux.lds.h | 12 +-
> include/linux/ftrace.h | 11 --
> include/linux/interrupt.h | 20 +++
> include/linux/kasan.h | 63 ++++++--
> include/linux/slab.h | 10 +-
> include/linux/slab_def.h | 14 ++
> include/linux/slub_def.h | 11 ++
> include/linux/stackdepot.h | 32 ++++
> kernel/softirq.c | 2 +-
> kernel/trace/trace_functions_graph.c | 1 +
> lib/Kconfig | 3 +
> lib/Kconfig.kasan | 5 +-
> lib/Makefile | 3 +
> lib/stackdepot.c | 275 +++++++++++++++++++++++++++++++
> lib/test_kasan.c | 59 ++++++-
> mm/Makefile | 1 +
> mm/kasan/Makefile | 4 +
> mm/kasan/kasan.c | 221 +++++++++++++++++++++++--
> mm/kasan/kasan.h | 45 ++++++
> mm/kasan/quarantine.c | 306 +++++++++++++++++++++++++++++++++++
> mm/kasan/report.c | 64 ++++++--
> mm/mempool.c | 23 +--
> mm/page_alloc.c | 2 +-
> mm/slab.c | 53 +++++-
> mm/slab.h | 4 +-
> mm/slab_common.c | 8 +-
> mm/slub.c | 19 +--
> 47 files changed, 1205 insertions(+), 92 deletions(-)
> create mode 100644 include/linux/stackdepot.h
> create mode 100644 lib/stackdepot.c
> create mode 100644 mm/kasan/quarantine.c
>
> --
> 2.7.0.rc3.207.g0ac5344
>
--
Alexander Potapenko
Software Engineer
Google Germany GmbH
Erika-Mann-Straße, 33
80636 München
Geschäftsführer: Matthew Scott Sucherman, Paul Terence Manicle
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
next prev parent reply other threads:[~2016-03-09 11:12 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-09 11:05 Alexander Potapenko
2016-03-09 11:05 ` [PATCH v5 1/7] kasan: Modify kmalloc_large_oob_right(), add kmalloc_pagealloc_oob_right() Alexander Potapenko
2016-03-09 11:05 ` [PATCH v5 2/7] mm, kasan: SLAB support Alexander Potapenko
2016-03-11 11:47 ` Andrey Ryabinin
2016-03-11 13:05 ` Alexander Potapenko
2016-03-11 13:57 ` Andrey Ryabinin
2016-03-09 11:05 ` [PATCH v5 3/7] mm, kasan: Added GFP flags to KASAN API Alexander Potapenko
2016-03-09 11:05 ` [PATCH v5 4/7] arch, ftrace: For KASAN put hard/soft IRQ entries into separate sections Alexander Potapenko
2016-03-09 11:05 ` [PATCH v5 5/7] mm, kasan: Stackdepot implementation. Enable stackdepot for SLAB Alexander Potapenko
2016-03-09 20:09 ` Andrew Morton
2016-03-10 14:00 ` Alexander Potapenko
2016-03-09 11:05 ` [PATCH v5 6/7] kasan: Test fix: Warn if the UAF could not be detected in kmalloc_uaf2 Alexander Potapenko
2016-03-09 11:05 ` [PATCH v5 7/7] mm: kasan: Initial memory quarantine implementation Alexander Potapenko
2016-03-09 20:21 ` Andrew Morton
2016-03-10 13:50 ` Alexander Potapenko
2016-03-10 20:14 ` Andrew Morton
2016-03-11 10:05 ` Alexander Potapenko
2016-03-11 17:12 ` Alexander Potapenko
2016-03-09 11:12 ` Alexander Potapenko [this message]
2016-03-09 20:23 ` [PATCH v5 0/7] SLAB support for KASAN Andrew Morton
2016-03-10 17:01 ` Andrey Ryabinin
-- strict thread matches above, loose matches on Subject: below --
2016-02-26 16:48 [PATCH v4 " Alexander Potapenko
2016-02-26 22:28 ` Andrew Morton
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAG_fn=WAs6a1jbX1A3RBJM8-n3OhNJYZz4xsrjY3xX7rrTkxfg@mail.gmail.com' \
--to=glider@google.com \
--cc=adech.fo@gmail.com \
--cc=akpm@linux-foundation.org \
--cc=cl@linux.com \
--cc=dvyukov@google.com \
--cc=iamjoonsoo.kim@lge.com \
--cc=js1304@gmail.com \
--cc=kasan-dev@googlegroups.com \
--cc=kcc@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=rostedt@goodmis.org \
--cc=ryabinin.a.a@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox