This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] Improved linker-debugger interface


Hi Mark,

On Monday, May 07 2012, Mark Kettenis wrote:

> I'm pretty annoyed by the whole SystemTap thing.  You presented this
> as being something pretty generic.  But it turns out this is not only
> Linux-specfic, but pretty much a completely RedHat-specific thing it
> seems.  And I think I've figured out why: SystemTap relies on utrace,
> which is not present in the official Linux kernel source tree.  And as
> far as I can see it will remain that way in the near future.  So
> unless you are running RedHat Linux, you'll not only need to build a
> patched glibc, but you also need to build a patched kernel to be able
> to use these new SystemTap probes.  Not many people will do that!

SystemTap is not a Red Hat-specific thing, and the support that I have
added on GDB does not rely on utrace at all.  It seems we are talking
about two different things here (userspace probes vs. kernel probes).

GDB currently supports static probes in userspace.  Those happen to be
implemented by SystemTap using <sys/sdt.h>, and (again) does not depend
on utrace, but rather on GCC + ELF support (that's why Jan asked me to
make the compilation of `stap-probe.c' conditional to the ELF support on
the target).

FWIW, Debian also ships a version of <sys/sdt.h>:

  http://packages.debian.org/squeeze/all/systemtap-sdt-dev/filelist

Some distros do not ship it, but they can perfectly do so if their
community wants.

And the upstream GCC already have static probes in it, along with many
more projects.

So in my view, this is not Red Hat specific, nor utrace-dependent.

> Having the basic SystemTap support in GDB is fine.  But depending on
> it to fix issues with core GDB functionality like the ability to debug
> shared libraries is a different matter.  It means that people using
> RedHat Linux, almost certainly including any RedHat engineers
> contributing here will no longer test the codepaths that don't rely on
> SystemTap.  And people on other Linux variants will never test the
> codepaths that rely on SystemTap.  That'll inevitably lead to more
> breakage.
>
>> I do agree that we should make another attempt to get the probes
>> upstream; I just don't think success at that should be a precondition
>> for this patch.
>
> If you ask me, having utrace in the official Linux kernel should be a
> precondition for this patch as well.

I hope I made it clear that, as a non-utrace-dependent feature, I think
this precondition is not valid anymore.

>> FWIW, we already have support for the glibc longjmp probes in the tree
>> now.
>
> In the glibc tree?  Or in the GDB tree?  If these probes are not
> present in the official glibc tree, the support for those particular
> probes should not be in the official GDB tree either.

The support is present in the GDB tree.

-- 
Sergio


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]