From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Stancek Date: Mon, 25 Feb 2019 02:37:50 -0500 (EST) Subject: [LTP] [PATCH v2] syscalls/mprotect: align exec_func to 64 bytes In-Reply-To: References: Message-ID: <1508347197.2635651.1551080270164.JavaMail.zimbra@redhat.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ltp@lists.linux.it ----- Original Message ----- > On Tue, Feb 12, 2019 at 6:15 AM Jan Stancek 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 > > > > Reviewed-by: Li Wang Pushed. Regards, Jan > > > > +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. > https://travis-ci.com/wangli5665/ltp/builds/101429714 > > Beside that, I also manually tried that with old gcc(gcc version 4.1.2) and > verify that works fine. > > -- > Regards, > Li Wang >