Re: [PATCH] efi/libstub: Disable some warnings for x86{,_64}

2018-11-06 Thread Sedat Dilek
On Mon, Nov 5, 2018 at 1:52 PM Ard Biesheuvel wrote: > > On 24 October 2018 at 09:22, Sedat Dilek wrote: > > On Tue, Oct 23, 2018 at 7:53 PM Nathan Chancellor > > wrote: > >> > >> On Fri, Oct 12, 2018 at 06:03:49PM -0700, Nathan Chancellor wrote: > >>

Re: [PATCH] efi/libstub: Disable some warnings for x86{,_64}

2018-10-24 Thread Sedat Dilek
isable-warning, > > address-of-packed-member) \ > > + $(call cc-disable-warning, gnu) > > > > # arm64 uses the full KBUILD_CFLAGS so it's necessary to explicitly > > # disable the stackleak plugin > > -- > > 2.19.1 > > > > + Sedat for review/testing. Thanks Nathan. Reviewed-by: Sedat Dilek Tested-by: Sedat Dilek [ Reported-by: Sedat Dilek ] [ See duplicate "arch/x86/include/asm/processor.h: Warning -Waddress-of-packed-member " <https://github.com/ClangBuiltLinux/linux/issues/234> ] Regards, - Sedat -

Re: [PATCH v5 2/3] x86/asm: add _ASM_ARG* constants for argument registers to

2018-06-14 Thread Sedat Dilek
for arguments; make them >> > available so we don't have to #ifdef in the actual code. >> > >> > Native size and specified size (q, l, w, b) versions are provided. >> > >> > Suggested-by: Sedat Dilek >> > Signed-off-by: H. Peter Anvin >&g

Re: [PATCH v5 0/3] extern inline native_save_fl for paravirt

2018-06-14 Thread Sedat Dilek
On Wed, Jun 13, 2018 at 11:05 PM, Nick Desaulniers wrote: > paravirt depends on a custom calling convention (callee saved), but > expects this from a static inline function that it then forces to be > outlined. This is problematic because different compilers or flags can > then add a stack guard

Re: [PATCH v2 1/2] compiler-gcc.h: add gnu_inline to all inline declarations

2018-06-08 Thread Sedat Dilek
On Wed, Jun 6, 2018 at 6:39 PM, wrote: ... >>Beyond this, a general question: Can someone explain why all these >>inline defines are in compiler-gcc.h (as there exists compiler.h and >>compiler-clang.h)? >> >>Thanks. >> >>Regards, >>- Sedat - > > Because gcc itself also supports both GNU89-style

Re: [PATCH v4 1/3] compiler-gcc.h: add gnu_inline to all inline declarations

2018-06-08 Thread Sedat Dilek
On Fri, Jun 8, 2018 at 1:28 PM, Andrea Parri wrote: > On Fri, Jun 08, 2018 at 12:04:36PM +0200, Sedat Dilek wrote: >> On Fri, Jun 8, 2018 at 9:59 AM, Arnd Bergmann wrote: >> > On Thu, Jun 7, 2018 at 10:49 PM, Nick Desaulniers >> > wrote: >> >> Fun

Re: [PATCH v2 1/2] compiler-gcc.h: add gnu_inline to all inline declarations

2018-06-06 Thread Sedat Dilek
Hi, when discovering 'gnu_inline', I found ... $ git grep -w __FORTIFY_INLINE include/linux/string.h:#define __FORTIFY_INLINE extern __always_inline __attribute__((gnu_inline)) include/linux/string.h:__FORTIFY_INLINE char *strncpy(char *p, const char *q, __kernel_size_t size)

Re: [PATCH v2 2/2] x86: paravirt: make native_save_fl extern inline

2018-06-05 Thread Sedat Dilek
On Tue, Jun 5, 2018 at 7:31 PM, H. Peter Anvin wrote: > On 06/05/18 10:28, H. Peter Anvin wrote: >> On 06/05/18 10:05, Nick Desaulniers wrote: >>> + >>> +/* >>> + * void native_restore_fl(unsigned long flags) >>> + * %rdi: flags >>> + */ >>> +ENTRY(native_restore_fl) >>> +push %_ASM_DI >>> +

Re: linux-next: Tree for Apr 26

2013-04-29 Thread Sedat Dilek
On Mon, Apr 29, 2013 at 8:58 AM, Geert Uytterhoeven ge...@linux-m68k.org wrote: On Mon, Apr 29, 2013 at 1:36 AM, Stephen Rothwell s...@canb.auug.org.au wrote: P.S. Geert, please trim your replies. Sorry, I noticed only after sending. (First time I wanted to reply without quoting anything