Re: [PATCH] uprobes/core: handle breakpoint and signal stepexception.

From: Srikar Dronamraju
Date: Fri Feb 24 2012 - 00:32:24 EST


>
> > +static void handler_chain(struct uprobe *uprobe, struct pt_regs *regs)
> > +{
> > + struct uprobe_consumer *consumer;
> > +
> > + if (!(uprobe->flags & UPROBES_RUN_HANDLER))
> > + return;
> > +
> > + down_read(&uprobe->consumer_rwsem);
> > + consumer = uprobe->consumers;
> The line above is not needed, looks like a leftover. Perhaps Ingo can
> cut it before commit.

Right Anton.

Ingo,

Please let me know if you want me to send an updated patch with line removed.



>
> > + for (consumer = uprobe->consumers; consumer; consumer = consumer->next) {
> > + if (!consumer->filter || consumer->filter(consumer, current))
> > + consumer->handler(consumer, regs);
> > + }
> > + up_read(&uprobe->consumer_rwsem);
> > +}
> > +
> > /* Returns the previous consumer */

--
Thanks and Regards
Srikar

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/