linux-kernel - Re: [PATCH 00/15] kasan: x86: arm64: risc-v: KASAN tag-based mode for x86

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CA+fCnZfTvFzX32ZU=Xa0qsNACM4Y1vA1xQDtJkhhk1fYH1QxRA@mail.gmail.com>
Date: Thu, 6 Feb 2025 19:10:50 +0100
From: Andrey Konovalov <andreyknvl@...il.com>
To: Maciej Wieczor-Retman <maciej.wieczor-retman@...el.com>
Cc: luto@...nel.org, xin@...or.com, kirill.shutemov@...ux.intel.com, 
	palmer@...belt.com, tj@...nel.org, brgerst@...il.com, ardb@...nel.org, 
	dave.hansen@...ux.intel.com, jgross@...e.com, will@...nel.org, 
	akpm@...ux-foundation.org, arnd@...db.de, corbet@....net, dvyukov@...gle.com, 
	richard.weiyang@...il.com, ytcoode@...il.com, tglx@...utronix.de, 
	hpa@...or.com, seanjc@...gle.com, paul.walmsley@...ive.com, 
	aou@...s.berkeley.edu, justinstitt@...gle.com, jason.andryuk@....com, 
	glider@...gle.com, ubizjak@...il.com, jannh@...gle.com, bhe@...hat.com, 
	vincenzo.frascino@....com, rafael.j.wysocki@...el.com, 
	ndesaulniers@...gle.com, mingo@...hat.com, catalin.marinas@....com, 
	junichi.nomura@....com, nathan@...nel.org, ryabinin.a.a@...il.com, 
	dennis@...nel.org, bp@...en8.de, kevinloughlin@...gle.com, morbo@...gle.com, 
	dan.j.williams@...el.com, julian.stecklina@...erus-technology.de, 
	peterz@...radead.org, cl@...ux.com, kees@...nel.org, 
	kasan-dev@...glegroups.com, x86@...nel.org, 
	linux-arm-kernel@...ts.infradead.org, linux-riscv@...ts.infradead.org, 
	linux-kernel@...r.kernel.org, linux-mm@...ck.org, llvm@...ts.linux.dev, 
	linux-doc@...r.kernel.org
Subject: Re: [PATCH 00/15] kasan: x86: arm64: risc-v: KASAN tag-based mode for x86

On Thu, Feb 6, 2025 at 11:41 AM Maciej Wieczor-Retman
<maciej.wieczor-retman@...el.com> wrote:
>
> >I started reviewing the patches, but this is somewhat complicated, as
> >the dense mode changes are squashed together with the generic ones for
> >x86 support. Could you please split this series into 2? Or at least
> >reorder the patches so that everything needed for basic x86 support
> >comes first and can be reviewed and tested separately.
>
> I'll try reordering first and see if it looks nice. Since the dense mode would
> make some parts arch specific I think it's better to have the two parts in one
> series for easier reference. But if it turns out more convoluted I'll just split
> it as you suggested.

Yes, please do. I also think if you split the series, we can land the
basic x86 support fairly quickly, or at least I can do the review and
give the ack from the KASAN side. For the dense mode part, I'd like to
also hear the opinion of other KASAN developers wrt the overall
design.

> >Also feel free to drop the dependency on that risc-v series, as it
> >doesn't get updated very often. But up to you.
>
> Okay, I was mostly interested in the patch that redefines KASAN_SHADOW_END as
> KASAN_SHADOW_OFFSET and then gets shadow addresses by using a signed offset. But
> I suppose I can just take that patch and prepend my series with that? (after
> applying your comments from that series)

Sounds good to me!