Re: [PATCH v2 1/1] Fix int1 recursion when no perf_bp_event is registeredy

2015-12-13 Thread Jeff Merkey
Is there anything else I need to do on this patch, is it acceptable, do you want a v3 that disables the errant breakpoints? Problem with that approach is if in fact there is a "lazy" breakpoint we might end up clearing it being set if it fires off too soon. So this is the first time I have submit

Re: [PATCH v2 1/1] Fix int1 recursion when no perf_bp_event is registeredy

2015-12-11 Thread Jeff Merkey
One thing I noticed in this section of code: /* Handle all the breakpoints that were triggered */ for (i = 0; i < HBP_NUM; ++i) { if (likely(!(dr6 & (DR_TRAP0 << i continue; ... snip ... if (bp->hw.info.t

Re: [PATCH v2 1/1] Fix int1 recursion when no perf_bp_event is registeredy

2015-12-11 Thread Jeff Merkey
I have completed testing of the following patch, added the fix and log a message when the error occurs. I also reviewed the code paths that touch this section of code. The lazy debug register behaviors pass state back and forth with this subsystem through the thread.debugred6 virtualized register