-
jbk
jeez... > 24 hours for opengrok to index illumos-gate, freebsd, and linux
-
antranigv
jbk any data on how long took each?
-
gitomat
[illumos-gate] 16258 telnet: parameter defaults to 'int' -- Toomas Soome <tsoome⊙mc>
-
sommerfeld
possibly dumb question, possibly asked at the wrong time: why don't we install ctfconvert and ctfmerge to /usr/bin ?
-
rmustacc
Mostly because I'm a bum.
-
sommerfeld
(discovered yesterday that the python support in pstack requires python to be built with CTF to get argument counts, so I started opening that can of worms...)
-
sjorge
pretty sure python is a can of snakes, not worms
-
tsoome_
-
sommerfeld
ah, next can of worms: oi python is built with feedback-directed optimization which splits code for function "foo" into "foo" and "foo.cold" (and the latter doesn't get ctf argument info...)
-
rmustacc
sommerfeld: Are you taking advantage of the ctfconvert at the end of the link?
-
sommerfeld
uh, probably not?
-
rmustacc
Oh.
-
rmustacc
Let me say it differently, are you trying to use ctfmerge?
-
sommerfeld
(I tweaked the python makefiles to insert ctfconvert / ctfmerge like the way we build illumos-gate)
-
rmustacc
Ah, so we added the ability to just do a single ctfconvert of the resulting binary.
-
andyf
In OmniOS we do a batch ctfconvert at the end, on DESTDIR
-
andyf
(don't know if that approach helps you)
-
rmustacc
You don't need to do the old merge path.
-
rmustacc
So mentioning that in case those symbols exist for some reason and have dwarf but aren't getting filled in.
-
sommerfeld
had no idea that had any chance of working!
-
rmustacc
Oh, sorry.
-
rmustacc
I redid a bunch of this a while back.
-
rmustacc
The goal is you shouldn't need ctfmerge except for exceptions.
-
rmustacc
But I never went back and changed all the makefiles in gate.
-
sommerfeld
so build with -g -gdwarf-2 -gstrict-dwarf -msave-args, then at the very end ctfconvert all the delivered executables and shared libraries?
-
rmustacc
Or -gdwarf-4 should work.
-
rmustacc
But yeah.
-
sommerfeld
gate does gdwarf-2 still
-
rmustacc
However, if they don't have dwarf for the .cold variants not a ton I can do immediately.
-
rmustacc
I know.
-
rmustacc
Again, I'm a bum.
-
rmustacc
The tools and test suite generally work with gcc/clang and dwarf4.
-
rmustacc
But it shouldn't make much of a difference what you're doing at your end.
-
sommerfeld
trying to get the pstack python support working again. multiple complications (not least of which is that they renamed some functions and juggled argument order within python that will require changes in both pstack and the python debug helper)
-
rmustacc
Yeah, I was following.
-
rmustacc
I just meant -gdwarf-2/dwarf-4 may or may not make a ton of difference is all.
-
sommerfeld
okay, got something that looks like a python-annotated stack trace
-
sommerfeld
(python 3.9, built without the .cold stuff, with a hacked up pstack and libpython3.9_db
-
sommerfeld
but I think I can fix it "correctly" w/o too much effort.
-
sommerfeld
-
sommerfeld
so (1) need CTF for python; (2) need a patch to libpython39_db.c to look for the correct libpython symbol, and (3) need pstack to pass argument 1 rather than argument 0 because that's where python puts its python frame now.
-
sommerfeld
(3) is an interface change so I think I need to rename the helper function to signal which argument to pass..
-
andyf
That's really nice!
-
sommerfeld
-
fenix
→
BUG 16272: python support in pstack should work again (New)