This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][Kasan][PING] Allow to override Asan shadow offset from command line
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Yury Gribov <y dot gribov at samsung dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Konstantin Serebryany <konstantin dot s dot serebryany at gmail dot com>, Dmitry Vyukov <dvyukov at google dot com>, Andrey Ryabinin <a dot ryabinin at samsung dot com>
- Date: Thu, 18 Sep 2014 13:01:20 +0200
- Subject: Re: [PATCH][Kasan][PING] Allow to override Asan shadow offset from command line
- Authentication-results: sourceware.org; auth=none
- References: <540DBD37 dot 9060702 at samsung dot com> <5416B566 dot 2000601 at samsung dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, Sep 15, 2014 at 01:46:14PM +0400, Yury Gribov wrote:
> On 09/08/2014 06:29 PM, Yury Gribov wrote:
> >Kasan developers has asked for an option to override offset of Asan
> >shadow memory region. This should simplify experimenting with memory
> >layouts on 64-bit architectures.
> >
> >I've bootstrapped and regtested this on x64.
> >
> >Ok to commit?
I don't like it at all. For the kernel-address perhaps it might make sense
as a param, but for userland, as it is an ABI changing option, I'm afraid
people would start to create objects/shared libraries/binaries with
ABI incompatible values.
So, if you need it for kernel, use a param that can be eventually dropped,
and limit it to kernel-address mode only.
Jakub