[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 4/6] cfi: Initial support for cfi-icall in QEMU
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH v2 4/6] cfi: Initial support for cfi-icall in QEMU |
Date: |
Mon, 26 Oct 2020 10:52:41 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.3.1 |
On 23/10/20 22:06, Daniele Buono wrote:
> +
> +#ifdef CONFIG_CFI
> +/* If CFI is enabled, use an attribute to disable cfi-icall on the following
> + * function */
> +#define __disable_cfi__ __attribute__((no_sanitize("cfi-icall")))
> +#else
> +/* If CFI is not enabled, use an empty define to not change the behavior */
> +#define __disable_cfi__
> +#endif
> +
> diff --git a/plugins/core.c b/plugins/core.c
__disable_cfi__ is a reserved identifier, since it starts with two
underscores. Please name it QEMU_DISABLE_CFI and put it in
include/qemu/compiler.h.
Thanks,
Paolo
- Re: [PATCH v2 2/6] configure: avoid new clang 11+ warnings, (continued)
[PATCH v2 3/6] configure: add option to enable LTO, Daniele Buono, 2020/10/23
[PATCH v2 4/6] cfi: Initial support for cfi-icall in QEMU, Daniele Buono, 2020/10/23
[PATCH v2 5/6] check-block: enable iotests with cfi-icall, Daniele Buono, 2020/10/23
[PATCH v2 6/6] configure: add support for Control-Flow Integrity, Daniele Buono, 2020/10/23
Re: [PATCH v2 0/6] Add support for Control-Flow Integrity, Eric Blake, 2020/10/23