On Thu, Sep 05, 2013 at 03:24:25PM +0100, Christoph Lameter wrote: > On Thu, 5 Sep 2013, Will Deacon wrote: > > > > > > > if (el == DBG_ACTIVE_EL1 && > > > - local_dec_and_test(this_cpu_ptr(&kde_ref_count))) > > > + this_cpu_dec_return(kde_ref_count)) > > > disable &= ~DBG_MDSCR_KDE; > > > > Almost! I think we just need an '== 0' check on the result from the > > decrement, since local_dec_and_test is simply a way to check that we've > > decremented to zero, so this patch would otherwise invert the meaning. > > Ok can you take it from here and modify it? I have no build and test > environment set up for ARM. Sure. Does that include the original arm/arm64 patches from your v3 series as well as these two fixups? Will -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html