On Tue, Oct 06, 2015 at 09:49:29AM +0200, Arnd Bergmann wrote: > On Tuesday 06 October 2015 11:05:43 Manjeet Pawar wrote: > > MINSIGSTKSZ and SIGSTKSZ for ARM64 are not correctly set in latest kernel. > > This patch fixes this issue. > > > > This issue is reported in LTP (testcase: sigaltstack02.c). > > Testcase failed when sigaltstack() called with stack size "MINSIGSTKSZ - 1" > > Since in Glibc-2.22, MINSIGSTKSZ is set to 5120 but in kernel > > it is set to 2048 so testcase gets failed. > > > > Testcase Output: > > sigaltstack02 1 TPASS : stgaltstack() fails, Invalid Flag value,errno:22 > > sigaltstack02 2 TFAIL : sigaltstack() returned 0, expected -1,errno:12 > > > > Reported Issue in Glibc Bugzilla: > > https://sourceware.org/bugzilla/show_bug.cgi?id=16850 > > > > Bugfix in Glibc-2.22: > > https://sourceware.org/git/gitweb.cgi?p=glibc.git;a=blob;f=sysdeps/unix/ > > sysv/linux/aarch64/bits/sigstack.h;h=8f2fb76e3e81734ef8a9cf9ae40daf4705 > > f31c35;hb=b763f6ae859ecea70a5dacb8ad45c71d5f667e2e > > > > Signed-off-by: Akhilesh Kumar <akhiles...@samsung.com> > > Signed-off-by: Manjeet Pawar <manjee...@samsung.com> > > Signed-off-by: Rohit Thapliyal <r.thapli...@samsung.com> > > This looks correct now. A few more points though: > > * My first thought would have been to do this by first defining the > two symbols before the #include, and then adding an #ifdef in > the generic file. Both approaches work though, any other opinions > on this?
That's what I was thinking as well. Maybe with a single #ifndef MINSIGSTKSZ to cover both macros. > * Do we need to backport this to stable? I think it does. -- Catalin -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/