Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1428679

Re: Build failures in -next due to 'locking/atomic, arch/arc: Implement atomic_fetch_{add,sub,and,andnot,or,xor}()'

From Peter Zijlstra <peterz@infradead.org>
Newsgroups linux.kernel
Subject Re: Build failures in -next due to 'locking/atomic, arch/arc: Implement atomic_fetch_{add,sub,and,andnot,or,xor}()'
Date 2016-06-22 12:50 +0200
Message-ID <rMNTj-2fc-5@gated-at.bofh.it> (permalink)
References <rL369-7I5-25@gated-at.bofh.it> <rL3fP-7LV-19@gated-at.bofh.it> <rL3fP-7LV-17@gated-at.bofh.it> <rMNqi-25n-31@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Wed, Jun 22, 2016 at 03:34:54PM +0530, Vineet Gupta wrote:
> 
> Peter the fixlet below doesn't show up in linux-next yet. Could you please do that
> soon. Our internal build lords are yelling at me :-)
> 
> To keep things bisectable, it would be desirable to squash this with the orig commit !

The fix seems to be in tip/locking/arch-atomic, albeit not squashed.

Ingo?

if you're going to rebase that branch, I have one more fix for
tilepro, so let me know.

> > ---
> >  arch/arc/include/asm/atomic.h | 8 --------
> >  1 file changed, 8 deletions(-)
> > 
> > diff --git a/arch/arc/include/asm/atomic.h b/arch/arc/include/asm/atomic.h
> > index bd9c51cb2bfd..4e3c1b6b0806 100644
> > --- a/arch/arc/include/asm/atomic.h
> > +++ b/arch/arc/include/asm/atomic.h
> > @@ -71,7 +71,6 @@ static inline int atomic_##op##_return(int i, atomic_t *v)		\
> >  static inline int atomic_fetch_##op(int i, atomic_t *v)			\
> >  {									\
> >  	unsigned int val, orig;						\
> > -	SCOND_FAIL_RETRY_VAR_DEF                                        \
> >  									\
> >  	/*								\
> >  	 * Explicit full memory barrier needed before/after as		\
> > @@ -84,11 +83,8 @@ static inline int atomic_fetch_##op(int i, atomic_t *v)			\
> >  	"	" #asm_op " %[val], %[orig], %[i]	\n"		\
> >  	"	scond   %[val], [%[ctr]]		\n"		\
> >  	"						\n"		\
> > -	SCOND_FAIL_RETRY_ASM						\
> > -									\
> >  	: [val]	"=&r"	(val),						\
> >  	  [orig] "=&r" (orig)						\
> > -	  SCOND_FAIL_RETRY_VARS						\
> >  	: [ctr]	"r"	(&v->counter),					\
> >  	  [i]	"ir"	(i)						\
> >  	: "cc");							\
> > @@ -199,10 +195,6 @@ ATOMIC_OPS(andnot, &= ~, bic)
> >  ATOMIC_OPS(or, |=, or)
> >  ATOMIC_OPS(xor, ^=, xor)
> >  
> > -#undef SCOND_FAIL_RETRY_VAR_DEF
> > -#undef SCOND_FAIL_RETRY_ASM
> > -#undef SCOND_FAIL_RETRY_VARS
> > -
> >  #else /* CONFIG_ARC_PLAT_EZNPS */
> >  
> >  static inline int atomic_read(const atomic_t *v)

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Build failures in -next due to 'locking/atomic, arch/arc: Implement  atomic_fetch_{add,sub,and,andnot,or,xor}()' Guenter Roeck <linux@roeck-us.net> - 2016-06-17 16:40 +0200
  Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Peter Zijlstra <peterz@infradead.org> - 2016-06-17 16:50 +0200
    Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Guenter Roeck <linux@roeck-us.net> - 2016-06-17 17:00 +0200
      Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2016-06-17 17:10 +0200
        Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Peter Zijlstra <peterz@infradead.org> - 2016-06-17 17:30 +0200
          Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2016-06-17 17:40 +0200
    Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Ingo Molnar <mingo@kernel.org> - 2016-06-17 18:10 +0200
      Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Peter Zijlstra <peterz@infradead.org> - 2016-06-17 18:20 +0200
      Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Vineet Gupta <vgupta@synopsys.com> - 2016-06-17 22:30 +0200
        Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Guenter Roeck <linux@roeck-us.net> - 2016-06-17 23:30 +0200
        Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Peter Zijlstra <peterz@infradead.org> - 2016-06-18 09:10 +0200
          cross compilers [was build failure of sorts] Peter Zijlstra <peterz@infradead.org> - 2016-06-18 12:20 +0200
            Re: cross compilers [was build failure of sorts] Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2016-06-21 08:00 +0200
              Re: cross compilers [was build failure of sorts] Peter Zijlstra <peterz@infradead.org> - 2016-06-21 09:50 +0200
                Re: cross compilers [was build failure of sorts] Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2016-06-21 11:30 +0200
                Re: cross compilers [was build failure of sorts] Peter Zijlstra <peterz@infradead.org> - 2016-06-21 11:50 +0200
                Re: cross compilers [was build failure of sorts] Peter Zijlstra <peterz@infradead.org> - 2016-06-21 12:10 +0200
                Re: cross compilers [was build failure of sorts] Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2016-06-21 12:20 +0200
    Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Vineet Gupta <Vineet.Gupta1@synopsys.com> - 2016-06-22 12:20 +0200
      Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Peter Zijlstra <peterz@infradead.org> - 2016-06-22 12:50 +0200
  Re: Build failures in -next due to 'locking/atomic, arch/arc:  Implement atomic_fetch_{add,sub,and,andnot,or,xor}()' Peter Zijlstra <peterz@infradead.org> - 2016-06-17 16:50 +0200

csiph-web