Re: [Kgdb-bugreport] [PATCH 2/4] perf, hw_breakpoint: add lockless reservation forhw_breaks
Status: Beta
Brought to you by:
jwessel
From: Jason W. <jas...@wi...> - 2010-01-27 22:30:15
|
Frederic Weisbecker wrote: > On Tue, Jan 26, 2010 at 01:25:19PM -0600, Jason Wessel wrote: > >> @@ -250,11 +326,16 @@ int reserve_bp_slot(struct perf_event *b >> >> mutex_lock(&nr_bp_mutex); >> >> + ret = dbg_hw_breakpoint_alloc(bp->cpu); >> + if (ret) >> + goto end; >> + >> > > > > This is totally breaking all the constraints that try to > make the reservation cpu-wide/task-wide aware. > Agreed. I was unaware of all the various pieces that the toggle_bp handled. A revised series is on the way based on the off-list discussion. Thanks, Jason. |