public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: chrubis@suse.cz
To: Stanislav Kholmanskikh <stanislav.kholmanskikh@oracle.com>
Cc: vasily.isaenko@oracle.com, ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH 2/2] msg_common.h: corrected __sparc64__ definition
Date: Thu, 15 May 2014 14:30:59 +0200	[thread overview]
Message-ID: <20140515123059.GA28445@rei.Home> (raw)
In-Reply-To: <1399962023-23105-2-git-send-email-stanislav.kholmanskikh@oracle.com>

Hi!
> __sparc64__ is not defined by cpp. We should use
> '__sparc__' and '__arch64__' instead.
> 
> Author: Jose E. Marchesi <jose.marchesi@oracle.com>
> 
> Signed-off-by: Stanislav Kholmanskikh <stanislav.kholmanskikh@oracle.com>
> ---
>  testcases/kernel/syscalls/utils/msg_common.h |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/testcases/kernel/syscalls/utils/msg_common.h b/testcases/kernel/syscalls/utils/msg_common.h
> index 0471291..2ecd27e 100644
> --- a/testcases/kernel/syscalls/utils/msg_common.h
> +++ b/testcases/kernel/syscalls/utils/msg_common.h
> @@ -26,7 +26,8 @@
>  #ifndef MSG_CMSG_COMPAT
>  
>  #if defined(__powerpc64__) || defined(__mips64) || defined(__x86_64__) || \
> -	defined(__sparc64__) || defined(__ia64__) || defined(__s390x__)
> +	defined(__ia64__) || defined(__s390x__) || \
> +	defined(__sparc__) && defined(__arch64__)
>  #define MSG_CMSG_COMPAT 0x80000000
>  #else
>  #define MSG_CMSG_COMPAT 0

These two patches are OK, acked.

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
"Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE
Instantly run your Selenium tests across 300+ browser/OS combos.
Get unparalleled scalability from the best Selenium testing platform available
Simple to use. Nothing to install. Get started now for free."
http://p.sf.net/sfu/SauceLabs
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  reply	other threads:[~2014-05-15 12:31 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-08  9:06 [LTP] SPARC fixes Stanislav Kholmanskikh
2014-05-08  9:06 ` [LTP] [PATCH 1/4] linux syscall numbers: fixed the checkings for SPARC Stanislav Kholmanskikh
2014-05-08  9:06 ` [LTP] [PATCH 2/4] linux syscall numbers: removed unimplemented syscalls for sparc64 Stanislav Kholmanskikh
2014-05-08  9:06 ` [LTP] [PATCH 3/4] syscalls: corrected the rest of __sparc64__ defines Stanislav Kholmanskikh
2014-05-12 12:59   ` chrubis
2014-05-13  6:20     ` [LTP] [PATCH 1/2] sendmsg, recvmsg: moved MSG_CMSG_COMPAT definition to a header Stanislav Kholmanskikh
2014-05-13  6:20     ` [LTP] [PATCH 2/2] msg_common.h: corrected __sparc64__ definition Stanislav Kholmanskikh
2014-05-15 12:30       ` chrubis [this message]
2014-05-08  9:06 ` [LTP] [PATCH 4/4] rt_sigaction*, rt_sigprocmask01, rt_sigsuspend01: SPARC support Stanislav Kholmanskikh
2014-05-12 13:08   ` chrubis
     [not found]     ` <5371C09F.7050205@oracle.com>
2014-05-13 11:39       ` chrubis
2014-05-20 12:24         ` [LTP] [RFC PATCH] A draft of the rt_sigaction syscall wrapper Stanislav Kholmanskikh
2014-05-20 14:42           ` chrubis
2014-05-21 10:24             ` [LTP] [PATCH 1/3] rt_sigaction, rt_sigprocmask, rt_sigsuspend: cleanup Stanislav Kholmanskikh
2014-05-21 10:24             ` [LTP] [PATCH 2/3] rt_sigaction, rt_sigprocmask01, rt_sigsuspend: use rt_sigaction wrapper Stanislav Kholmanskikh
2014-06-04 15:53               ` chrubis
2014-05-21 10:24             ` [LTP] [PATCH 3/3] ltp_rt_sigaction: added SPARC support Stanislav Kholmanskikh
2014-05-21 10:31               ` Stanislav Kholmanskikh
2014-06-04 15:49               ` chrubis
2014-06-17  9:08             ` [LTP] [PATCH V2 1/3] rt_sigaction, rt_sigprocmask, rt_sigsuspend: cleanup Stanislav Kholmanskikh
2014-06-17  9:08             ` [LTP] [PATCH V2 2/3] rt_sigaction, rt_sigprocmask01, rt_sigsuspend: use rt_sigaction wrapper Stanislav Kholmanskikh
2014-06-17  9:08             ` [LTP] [PATCH V2 3/3] ltp_rt_sigaction.h: added SPARC support Stanislav Kholmanskikh
2014-06-23 15:21               ` chrubis
2014-08-11 15:04               ` chrubis
     [not found]                 ` <53E9C7B8.1040504@oracle.com>
2014-08-12  9:04                   ` chrubis
     [not found]                     ` <53E9F644.5000101@oracle.com>
2014-08-12 11:26                       ` chrubis
2014-08-12 11:59                         ` [LTP] [PATCH] rt_sigaction.h: define HAVE_SA_RESTORER for most platforms Stanislav Kholmanskikh
2014-08-12 12:24                           ` chrubis

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140515123059.GA28445@rei.Home \
    --to=chrubis@suse.cz \
    --cc=ltp-list@lists.sourceforge.net \
    --cc=stanislav.kholmanskikh@oracle.com \
    --cc=vasily.isaenko@oracle.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox