[LTP] [PATCH v2] syscalls/mprotect: align exec_func to 64 bytes

Li Wang liwang@redhat.com
Tue Feb 19 14:29:04 CET 2019

On Tue, Feb 12, 2019 at 6:15 AM Jan Stancek <jstancek@redhat.com> wrote:

> exec_func() is dummy/empty function. Try to align it so we don't
> need to worry about copying 2 pages. But also check that compiler
> aligned it and there's sufficient space between start of func_exec
> and end of page.
> This patch also removes copy_sz, which is now replaced with page_sz.
> Signed-off-by: Jan Stancek <jstancek@redhat.com>

Reviewed-by: Li Wang <liwang@redhat.com>

> +mprotect04: CFLAGS += -falign-functions=64

One of my concern was this option maybe not supported by kind of compiler
and probably broken in the compiling phase. So I just submitted these
two(another is Daniel's) patches to Travis CI, the result looks good and no
compiler error found there.

Beside that, I also manually tried that with old gcc(gcc version 4.1.2) and
verify that works fine.

Li Wang
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.linux.it/pipermail/ltp/attachments/20190219/73dc6da9/attachment.html>

More information about the ltp mailing list