Mirror of the gdb mailing list
 help / color / mirror / Atom feed
* Re: PSA: Formalizing the glibc<->gdb probe-based interface.
       [not found] <f73d8f0d-32ac-487d-8f90-fdac7b1be8e8@redhat.com>
@ 2019-02-13 17:13 ` Tom Tromey
  2019-02-13 18:47   ` Carlos O'Donell
  0 siblings, 1 reply; 2+ messages in thread
From: Tom Tromey @ 2019-02-13 17:13 UTC (permalink / raw)
  To: Carlos O'Donell; +Cc: GNU C Library, gdb

>>>>> "Carlos" == Carlos O'Donell <carlos@redhat.com> writes:

Carlos> We document *other* probes in manual/probes.texi, but we should
Carlos> add a section for stable guaranteed probes, and discuss a way
Carlos> to deprecate the probes should we ever need to do that.

Because probes are introspectable, it's actually better to remove them
than to change their semantics -- the tools can adapt more easily to
this.  Of course it's good to have a transition period, or at least some
sort of warning.

As I recall we had some thoughts about ABI compatibility when adding the
probes; namely that it is ok to add arguments to the end, but not ok to
remove arguments or to change the meaning of existing arguments.

HTH,
Tom


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: PSA: Formalizing the glibc<->gdb probe-based interface.
  2019-02-13 17:13 ` PSA: Formalizing the glibc<->gdb probe-based interface Tom Tromey
@ 2019-02-13 18:47   ` Carlos O'Donell
  0 siblings, 0 replies; 2+ messages in thread
From: Carlos O'Donell @ 2019-02-13 18:47 UTC (permalink / raw)
  To: Tom Tromey; +Cc: GNU C Library, gdb

On 2/13/19 12:13 PM, Tom Tromey wrote:
>>>>>> "Carlos" == Carlos O'Donell <carlos@redhat.com> writes:
> 
> Carlos> We document *other* probes in manual/probes.texi, but we should
> Carlos> add a section for stable guaranteed probes, and discuss a way
> Carlos> to deprecate the probes should we ever need to do that.
> 
> Because probes are introspectable, it's actually better to remove them
> than to change their semantics -- the tools can adapt more easily to
> this.  Of course it's good to have a transition period, or at least some
> sort of warning.
> 
> As I recall we had some thoughts about ABI compatibility when adding the
> probes; namely that it is ok to add arguments to the end, but not ok to
> remove arguments or to change the meaning of existing arguments.

Agreed.

In general I think the probe name should always change if:
* semantics change.
* # of arguments change.

Then you'll always be safe.

If you can't keep implementing the probe it is safest to remove it.

Talk to the users of the probe first and reach consensus :-)

-- 
Cheers,
Carlos.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-02-13 18:47 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <f73d8f0d-32ac-487d-8f90-fdac7b1be8e8@redhat.com>
2019-02-13 17:13 ` PSA: Formalizing the glibc<->gdb probe-based interface Tom Tromey
2019-02-13 18:47   ` Carlos O'Donell

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox