Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752940AbbG3FkW (ORCPT ); Thu, 30 Jul 2015 01:40:22 -0400 Received: from cassarossa.samfundet.no ([193.35.52.29]:57462 "EHLO cassarossa.samfundet.no" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751177AbbG3FkV (ORCPT ); Thu, 30 Jul 2015 01:40:21 -0400 Date: Thu, 30 Jul 2015 07:40:07 +0200 From: Hans-Christian Egtvedt To: Peter Zijlstra Cc: mingo@kernel.org, linux-kernel@vger.kernel.org, will.deacon@arm.com, linux@arm.linux.org.uk, ralf@linux-mips.org, michael@amarulasolutions.com, cmetcalf@ezchip.com Subject: Re: [RFC][PATCH 3/7] avr32: Remove finish_arch_switch() Message-ID: <20150730054007.GA25446@samfundet.no> References: <20150729190848.678796654@infradead.org> <20150729191659.847539750@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150729191659.847539750@infradead.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1568 Lines: 49 Around Wed 29 Jul 2015 21:08:51 +0200 or thereabout, Peter Zijlstra wrote: > Fold the tracing hook into switch_to() in order to remove > finish_arch_switch(). AFAICT this should work, I do not have a nexus debugger to verify behavior. > Cc: Hans-Christian Egtvedt > Signed-off-by: Peter Zijlstra (Intel) Acked-by: Hans-Christian Egtvedt > --- > arch/avr32/include/asm/switch_to.h | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > --- a/arch/avr32/include/asm/switch_to.h > +++ b/arch/avr32/include/asm/switch_to.h > @@ -15,11 +15,13 @@ > */ > #ifdef CONFIG_OWNERSHIP_TRACE > #include > -#define finish_arch_switch(prev) \ > +#define ocd_switch(prev, next) \ > do { \ > ocd_write(PID, prev->pid); \ > - ocd_write(PID, current->pid); \ > + ocd_write(PID, next->pid); \ > } while(0) > +#else > +#define ocd_switch(prev, next) > #endif > > /* > @@ -38,6 +40,7 @@ extern struct task_struct *__switch_to(s > struct cpu_context *); > #define switch_to(prev, next, last) \ > do { \ > + ocd_switch(prev, next); \ > last = __switch_to(prev, &prev->thread.cpu_context + 1, \ > &next->thread.cpu_context); \ > } while (0) -- mvh Hans-Christian Egtvedt -- 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/