Mirror of the lttng-dev mailing list
 help / color / mirror / Atom feed
From: mathieu.desnoyers@efficios.com (Mathieu Desnoyers)
Subject: [ltt-dev] [PATCH 3/3] Remove unused sync_core() definitions
Date: Sun, 13 Jun 2010 17:30:29 -0400	[thread overview]
Message-ID: <20100613213029.GH5427@Krystal> (raw)
In-Reply-To: <1276286430-21147-3-git-send-email-paulmck@linux.vnet.ibm.com>

* Paul E. McKenney (paulmck at linux.vnet.ibm.com) wrote:
> Signed-off-by: Paul E. McKenney <paulmck at linux.vnet.ibm.com>

Merged and pushed, thanks !

Mathieu

> ---
>  urcu/arch_generic.h |    4 ----
>  urcu/arch_ppc.h     |    5 -----
>  urcu/arch_x86.h     |   14 --------------
>  3 files changed, 0 insertions(+), 23 deletions(-)
> 
> diff --git a/urcu/arch_generic.h b/urcu/arch_generic.h
> index e9b1b82..0484618 100644
> --- a/urcu/arch_generic.h
> +++ b/urcu/arch_generic.h
> @@ -121,10 +121,6 @@ extern "C" {
>  #define cpu_relax()	 barrier()
>  #endif
>  
> -#ifndef sync_core
> -#define sync_core()	 mb()
> -#endif
> -
>  #ifdef __cplusplus
>  }
>  #endif
> diff --git a/urcu/arch_ppc.h b/urcu/arch_ppc.h
> index eeeb753..185cb22 100644
> --- a/urcu/arch_ppc.h
> +++ b/urcu/arch_ppc.h
> @@ -34,11 +34,6 @@ extern "C" {
>  
>  #define mb()    asm volatile("sync":::"memory")
>  
> -/*
> - * Serialize core instruction execution. Also acts as a compiler barrier.
> - */
> -#define sync_core()	asm volatile("isync" : : : "memory")
> -
>  #define mftbl()						\
>  	({ 						\
>  		unsigned long rval;			\
> diff --git a/urcu/arch_x86.h b/urcu/arch_x86.h
> index 64cc026..e3171a2 100644
> --- a/urcu/arch_x86.h
> +++ b/urcu/arch_x86.h
> @@ -47,20 +47,6 @@ extern "C" {
>  
>  #define cpu_relax()	asm volatile("rep; nop" : : : "memory");
>  
> -/*
> - * Serialize core instruction execution. Also acts as a compiler barrier.
> - * On PIC ebx cannot be clobbered
> - */
> -#ifdef __PIC__
> -#define sync_core()							  \
> -	asm volatile("push %%ebx; cpuid; pop %%ebx"			  \
> -		     : : : "memory", "eax", "ecx", "edx");
> -#endif
> -#ifndef __PIC__
> -#define sync_core()							  \
> -	asm volatile("cpuid" : : : "memory", "eax", "ebx", "ecx", "edx");
> -#endif
> -
>  #define rdtscll(val)							  \
>  	do {						  		  \
>  	     unsigned int __a, __d;					  \
> -- 
> 1.7.0.6
> 

-- 
Mathieu Desnoyers
Operating System Efficiency R&D Consultant
EfficiOS Inc.
http://www.efficios.com




      parent reply	other threads:[~2010-06-13 21:30 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20100611195937.GA18891@linux.vnet.ibm.com>
     [not found] ` <1276286430-21147-1-git-send-email-paulmck@linux.vnet.ibm.com>
2010-06-13 21:28   ` [ltt-dev] [PATCH 1/3] Fix the "unknown" case Mathieu Desnoyers
2010-06-14  0:41     ` Paul E. McKenney
2010-06-14 16:51       ` Paolo Bonzini
2010-06-14 17:05         ` Paul E. McKenney
2010-06-14 16:52       ` Paolo Bonzini
     [not found] ` <1276286430-21147-3-git-send-email-paulmck@linux.vnet.ibm.com>
2010-06-13 21:30   ` Mathieu Desnoyers [this message]

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=20100613213029.GH5427@Krystal \
    --to=mathieu.desnoyers@efficios.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