[PATCH v6 3/3] KGDB: make kgdb_breakpoint() as noinline

Will Deacon will.deacon at arm.com
Wed Jan 15 11:48:42 EST 2014


On Wed, Jan 15, 2014 at 04:44:27PM +0000, Jason Wessel wrote:
> On 12/19/2013 05:50 AM, vijay.kilari at gmail.com wrote:
> > From: Vijaya Kumar K <Vijaya.Kumar at caviumnetworks.com>
> >
> > The function kgdb_breakpoint() sets up break point at
> > compile time by calling arch_kgdb_breakpoint();
> > Though this call is surrounded by wmb() barrier,
> > the compile can still re-order the break point,
> > because this scheduling barrier is not a code motion
> > barrier in gcc.
> >
> > Making kgdb_breakpoint() as noinline solves this problem
> > of code reording around break point instruction and also
> > avoids problem of being called as inline function from
> > other places
> >
> > More details about discussion on this can be found here
> > http://comments.gmane.org/gmane.linux.ports.arm.kernel/269732
> >
> > Signed-off-by: Vijaya Kumar K <Vijaya.Kumar at caviumnetworks.com>
> > Acked-by: Will Deacon <will.deacon at arm.com>
> 
> Acked-by: Jason Wessel <jason.wessel at windriver.com>

Thanks Jason! We'll take this series via the arm64 tree.

Will



More information about the linux-arm-kernel mailing list