00:47:47 ty everyone, I archived my illumos box and have the files on the vm host 00:48:05 i think it was rmustacc who helped 00:48:34 I have a machine being donated purportedly 00:49:02 and I know there's a net build host, perhaps I should just ask about using that? 10:13:16 Yep build host and distribution packaging are there to help with that :) 19:35:14 *sigh* one of those days when you gcc -o file.c by mistake :) 19:43:41 this is why all my outputs get named foo :) 20:24:56 also somewhat surprising 20:25:00 (again one of those days) 20:25:12 if you build a shared library with gcc, but omit a source 20:25:22 it still generates a (mostly) empty .so file 20:25:35 (well surprising to me since i've not deliberately tried to do this) 20:29:42 you're not omitting source, the CRT etc objects will still be implicitly there 20:29:46 and it will just link them together. 20:30:08 not the most user friendly of viewpoints, but not off the wall 21:25:42 the power to create empty libraries 21:46:51 pmooney: An update on fenix` illumos 14892 21:46:59 fenix illumos 14892 21:47:00 BUG 14892: pollhead lifetime too short in signalfd (In Progress) 21:47:01 ↳ https://www.illumos.org/issues/14892 21:47:20 1.) Got it to build cleanly on SmartOS (brand hooks in signalfd only change). 21:47:51 2.) Made changes to be pbchk clean and added things to help there as well (e.g. usr/src/pkg and adding the mentioned-but-not-there "common.h"). 21:48:35 3.) I have a test program that *might* help induce 14892, but I haven't been able to do it yet. I'm not sure if the program is not going to cause the problem, or if the program needs to get bombarded with more signals, or if I need to apply chill() in the right kernel paths. 21:49:22 Building illumos-gate plus the ~10 changes (your three plus 6 tiny incremental ones and one refactor that makes it easy for SmartOS and OmniOS) to make sure it's a clean build. 21:49:32 pbchk's only complaints on gate are 2022 vs. 2023 ones. :) 21:49:40 Re: 2 - Yeah, that's work I hadn't focused on until the tests were fleshed out 21:49:56 (which is obviously where my effort stalled) 21:49:58 Understood. 21:50:17 I modified one of your tests to do a lot more, but I'm not clear if it's enough to induce the panic. 21:50:25 Lemme put it up on kebe.com for download... 21:50:47 I think andyf (and/or hadfl, I can't recall) was able to get the problem reproduced w/ the postgres test suite 21:51:24 that was, to date, one of the more reliable repros 21:51:35 And Andy tried again this week and COULD NOT DO IT. (It's as if the bug knows we're hunting it now.) 21:51:48 Anyway: https://kebe.com/~danmcd/webrevs/14892/14892-repro.c 21:51:55 that may be a DEBUG vs non-DEBUG thing 21:52:09 I know I had some trouble getting it to repro as well 21:52:10 it *works*, but it does not, using simple externals, cause a panic yet on a non-debug stock SmartOS. 21:52:27 Good point about DEBUG, however. I've an OmniOS VM setup for just this sort of thing. 21:52:29 postgres may also have tweaked some of the bits 21:52:50 I'll be putting other things into the 14892/ directory on kebe.com as they arise (e.g. SmartOS PI with the fix in it). 21:53:16 I'm going to let the build churn, and come back to it. I wanted to make sure I communicated somewhere about this where you and others can see it. 21:53:19 as far as testing goes, my bigger concern was around ensuring that I hadn't broken signalfd with the change 21:53:32 rather than nailing down a reliable reproducer for 14892 itself 21:53:51 not to say I'd turn one down if it presented itself 21:53:53 Well so far, so good, in that I seem to see them in my test, which is a proper superset of your second one. 21:54:11 I'll be back in 20 or so. 21:54:25 yeah, I just wasn't all that confident yet in the test coverage 21:54:29 lots of edges on signalfd 21:54:36 signals are bastards. 21:54:41 they sure are 21:55:03 signalfd is certainly one less-hairy way of handling them 21:55:18 but it's a bit spicy 22:10:15 It is spicy and they are bastards. 22:27:13 And it built. Now https://kebe.com/~danmcd/webrevs/14892/ has "gate" and "smartos" directories with patch sets to download and look. 22:27:31 (The "smartos" ones Should Just Drop into OmniOS, FWIW.)