03:14:53 Smithx10: unfortuantely, the way that aggrs work with mac for performance reasons, it's not a simple fix 04:46:54 maybe 1 day... link aggrs will support lldp :P 06:58:55 hello. is bhyve in illumos-gate now? i see the manpage is on illumos.org but i'm not sure branch those come from... 07:36:04 yes, for a long time 09:25:00 FYI: I updated Gerrit to 3.9.1 -- let me know if you see anything weird. 16:50:34 is the RTI queue being processed? 20:10:55 Trying to statically compile a go binary, GOOS=illumos CGO_ENABLED=1 go build -ldflags "-linkmode 'external' -extldflags '-static'" -v . results in ld: fatal: library -lsocket: not found and a few others 20:28:27 maybe the '-static' bit? 20:28:46 AFAIK, you can't statically link any of the system libraries 20:31:57 :( Was trying to write a small tool to take out interfaces from aggregations and get the LLDP info and put them back.... was gonna grab the lldp info with https://github.com/google/gopacket 20:32:40 all the CNs dont have libpcap, was hoping to just make it static* 20:43:09 we don't provide static versions of any system library 20:43:41 The linux instructions won't work, you can't create a static binary 20:43:44 you should be able to statically link libpcap, but dynamically link libsocket 20:43:52 but I can't tell you how to do that through the go soup unfortunately 20:44:00 But you might try something like: 20:44:02 richlowe that should probably work 20:44:15 go build -ldflags "-linkmode 'external' -extldflags '-Wl,-Bstatic -lpcap -Wl,-Bdynamic'" 20:45:03 which should only pick up libpcap statically 20:46:43 ptribble: you are the man :) 20:50:13 ping @andyf re fenix illumos#15998 ? 20:50:14 FEATURE 15998: Increase the rate and efficiency of vm_pageout (Closed) 20:50:14 ↳ https://www.illumos.org/issues/15998 | https://code.illumos.org/c/illumos-gate/+/1272 20:50:45 In the SmartOS version, there are several cv_signal() calls in usr/src/uts/common/vm/vm_page.c that got converted to cv_broadcast. 20:51:07 (And they got hidden by the redefined-for-upstream WAKE_PAGEOUT_SCANNER() macro.) 20:52:09 I'm curious, should those (as well as one in usr/src/uts/i86pc/vm/vm_machdep.c:3549) get upgraded to cv_broadcast? 20:56:58 Presumably they were changed in SmartOS because of the multiple scanning hands? 20:59:58 But isn't 15998 introducing those same multiple scanning hands? 21:00:25 Hmm, maybe I confused those bits then. Sorry. 21:00:54 It feels like either missing review feedback ("you don't need to cv_broadcast() there!") **OR** missing code changes. 21:01:20 "missing review feedback" ==> something internal that should've been documented on gerrit. 21:11:22 @ptribble actually hmmm ld.so.1: foo: fatal: libpcap.so.0: open failed: No such file or directory it made the binary but when I went to run it on the other box still got libpcap not found 21:25:36 On a related note to 15998, should fenix illumos#13567 be closed now? 21:25:37 FEATURE 15998: Increase the rate and efficiency of vm_pageout (Closed) 21:25:37 ↳ https://www.illumos.org/issues/15998 | https://code.illumos.org/c/illumos-gate/+/1272 21:25:56 wrong guess, fenix. how about 13567 ? 21:26:35 or https://www.illumos.org/issues/13567 ? 21:26:36 → BUG 13567: maxpgio is too low for modern systems (New) 21:27:55 Smithx10: why not just install libpcap on that machine? (possibly in a non-standard location, with LD_LIBRARY_PATH to find it if need be) 21:29:15 Smithx10: you can get the exact gcc command line it's issuing with 21:29:35 go build -ldflags "-linkmode 'external' -extldflags '-lssp -Wl,-Bstatic -lpcap -Wl,-Bdynamic' -v" 21:30:08 https://gist.github.com/Smithx10/54a58e5604ed34e1409bfe55895472e9 21:30:32 which might show if it's also pulling in libpcap a second time 21:30:38 Ah. Let me look 21:30:50 [illumos-gate] 16147 OpenZFS ZTS: normalize on use of `sync_pool` and `sync_all_pools` -- Allan Jude 21:32:24 Yep, there's the extra "-L" "/opt/local/lib" "-lpcap" so you need to make sure that it doesn't put that first "-lpcap" into the link line 21:34:14 @sommerfeld we could, I suppose. @jclulow, since he filed the bug, is likely the authority of this. 21:34:15 @ptribble i wonder if its getting that from https://github.com/google/gopacket/blob/master/pcap/pcap_unix.go#L26 21:37:40 Smithx10: I guess it will put those LDFLAGS first, then the extra flags you specify with extldflags, so I can imagine removing -lpcap from there ought to work 22:07:36 danmcd: on the cv_broadcast vs cv_signal thing, it may be more of a tuning issue than one of correctness. When memory is low, do you want to poke one scanner or all of them? (as an aside, it appears that the underling primitives will do FIFO wakeups among threads of the same priority so it should round-robin amongst all the scanner threads). Probably better to be consistent about it, though. 22:18:22 richlowe, sommerfeld: I'm not sure about closing 13567, to be honest. I am not sure if we've addressed the uses of maxpgio outside of the pageout rate stuff? 22:18:33 Or the async_list_size thing 22:19:40 I've put a note in the ticket there 22:36:56 [illumos-gate] 16148 zfs-tests: enospc_002_pos is failing -- Brian Behlendorf 23:18:23 jclulow: thanks for updating the bug. 23:26:15 You're welcome! 23:49:23 @jbk so it looks like I am able to at least read the frames from ToR on the interfaces in the aggregations