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


Groups > linux.kernel > #1195442 > unrolled thread

[RFC][PATCH 1/7] arm: Remove finish_arch_switch

Started byPeter Zijlstra <peterz@infradead.org>
First post2015-07-29 21:30 +0200
Last post2015-08-04 10:40 +0200
Articles 4 — 4 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  [RFC][PATCH 1/7] arm: Remove finish_arch_switch Peter Zijlstra <peterz@infradead.org> - 2015-07-29 21:30 +0200
    Re: [RFC][PATCH 1/7] arm: Remove finish_arch_switch Ingo Molnar <mingo@kernel.org> - 2015-08-04 09:50 +0200
      Re: [RFC][PATCH 1/7] arm: Remove finish_arch_switch Russell King - ARM Linux <linux@arm.linux.org.uk> - 2015-08-04 10:30 +0200
        Re: [RFC][PATCH 1/7] arm: Remove finish_arch_switch Will Deacon <will.deacon@arm.com> - 2015-08-04 10:40 +0200

#1195442 — [RFC][PATCH 1/7] arm: Remove finish_arch_switch

FromPeter Zijlstra <peterz@infradead.org>
Date2015-07-29 21:30 +0200
Subject[RFC][PATCH 1/7] arm: Remove finish_arch_switch
Message-ID<pRFd7-AH-5@gated-at.bofh.it>
From: Will Deacon <will.deacon@arm.com>

Fold finish_arch_switch() into switch_to().

Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
---
 arch/arm/include/asm/switch_to.h |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

--- a/arch/arm/include/asm/switch_to.h
+++ b/arch/arm/include/asm/switch_to.h
@@ -10,7 +10,9 @@
  * CPU.
  */
 #if defined(CONFIG_PREEMPT) && defined(CONFIG_SMP) && defined(CONFIG_CPU_V7)
-#define finish_arch_switch(prev)	dsb(ish)
+#define __complete_pending_tlbi()	dsb(ish)
+#else
+#define __complete_pending_tlbi()
 #endif
 
 /*
@@ -22,6 +24,7 @@ extern struct task_struct *__switch_to(s
 
 #define switch_to(prev,next,last)					\
 do {									\
+	__complete_pending_tlbi();					\
 	last = __switch_to(prev,task_thread_info(prev), task_thread_info(next));	\
 } while (0)
 


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1199488

FromIngo Molnar <mingo@kernel.org>
Date2015-08-04 09:50 +0200
Message-ID<pTF90-41P-21@gated-at.bofh.it>
In reply to#1195442
* Peter Zijlstra <peterz@infradead.org> wrote:

> From: Will Deacon <will.deacon@arm.com>
> 
> Fold finish_arch_switch() into switch_to().
> 
> Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>

Will, can I add your:

 Signed-off-by: Will Deacon <will.deacon@arm.com>

?

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1199519

FromRussell King - ARM Linux <linux@arm.linux.org.uk>
Date2015-08-04 10:30 +0200
Message-ID<pTFLH-51d-9@gated-at.bofh.it>
In reply to#1199488
On Tue, Aug 04, 2015 at 09:39:43AM +0200, Ingo Molnar wrote:
> 
> * Peter Zijlstra <peterz@infradead.org> wrote:
> 
> > From: Will Deacon <will.deacon@arm.com>
> > 
> > Fold finish_arch_switch() into switch_to().
> > 
> > Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
> 
> Will, can I add your:
> 
>  Signed-off-by: Will Deacon <will.deacon@arm.com>

I've already merged this patch as Will had submitted it to me for merging.
Peter's S-o-b should be an acked-by though in that case...

-- 
FTTC broadband for 0.8mile line: currently at 10.5Mbps down 400kbps up
according to speedtest.net.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1199528

FromWill Deacon <will.deacon@arm.com>
Date2015-08-04 10:40 +0200
Message-ID<pTFVo-5cP-7@gated-at.bofh.it>
In reply to#1199519
On Tue, Aug 04, 2015 at 09:26:22AM +0100, Russell King - ARM Linux wrote:
> On Tue, Aug 04, 2015 at 09:39:43AM +0200, Ingo Molnar wrote:
> > 
> > * Peter Zijlstra <peterz@infradead.org> wrote:
> > 
> > > From: Will Deacon <will.deacon@arm.com>
> > > 
> > > Fold finish_arch_switch() into switch_to().
> > > 
> > > Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
> > 
> > Will, can I add your:
> > 
> >  Signed-off-by: Will Deacon <will.deacon@arm.com>
> 
> I've already merged this patch as Will had submitted it to me for merging.

Yup. Peter said he would pick up what's left after the merge window, so
arch maintainers can pick up the patches now for 4.3 (I think Ralf had
picked up the mips part already).

> Peter's S-o-b should be an acked-by though in that case...

I guess, although he turned my diff into a patch (i.e. wrote the commit
message) and posted it, so I just left it there. Not fussed either way.

Will
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web