01:42:00 <gitomat> [illumos-gate] 18166 Create header-tests package -- Gordon Ross <gordon.w.ross⊙gc>
01:42:00 <gitomat> [illumos-gate] 18111 Want C++ header tests -- Gordon Ross <gordon.w.ross⊙gc>
01:42:00 <gitomat> [illumos-gate] 18192 header-tests programs could be improved -- Gordon Ross <gordon.w.ross⊙gc>
14:00:37 <gitomat> [illumos-gate] 18219 mdb: cmihdl dcmd actually does use slashes for -c -- Toomas Soome <tsoome⊙mc>
18:05:39 <jbk> i don't suppose anyone has a system with SATA port multipliers by any chance?
18:13:09 <sommerfeld> they have a reputation of not working well so I think people tend to avoid them.
18:15:49 <jbk> certainly on illumos, I think I've found some potential causes :)
18:16:31 <jbk> so like, I can point at the spec and fix the code, but aside from 'yeah non-PM is fine with it'.. kinda stuck in making sure a PM would be ok with it
18:22:40 <sommerfeld> sort of a self-fulfilling prophecy.   Though it looks like amazon has something that might implement that spec for cheap: https://www.amazon.com/Splitter-Adapter-Converter-Multiplier-Selector/dp/B0GR993TJG
18:23:29 <sommerfeld> or https://www.amazon.com/Cablecc-Splitter-Converter-Multiplier-Selector/dp/B0BYNFR2FC
18:24:06 <sommerfeld> so $25ish gets you a 1-5 expansion if you have the right upstream sata controller..
18:25:19 <jbk> https://github.com/illumos/illumos-gate/blob/master/usr/src/uts/common/io/sata/adapters/ahci/ahci.c#L9603 heh
18:50:47 <jbk> (it also calls ahci_software_reset() w/ addr uninitialized, but AFAICT, ahci_software_reset() never actually uses the parameter)
21:08:22 <jbk> somewhat related, for a > 1ms wait in an _attach routine, would delay(9F) be preferable over drv_usecwait()?
21:09:07 <jbk> given how long that is, i'm thinking 'probably', but also not sure because we're attaching if there might be other considerations where 'no' might be better despite the length
21:09:20 <danmcd> I think the preferred approach is drv_usecwait()?
21:09:48 <richlowe> assuming you didn't disable mtc, we multi-thread all of that pretty widely
21:09:56 <richlowe> so if you're worrying about your thread sleeping, that probably doesn't matter much
21:41:38 <jbk> hrm.. ok..
22:10:41 <sommerfeld> richlowe: I hope my fix for https://www.illumos.org/issues/18185 won't cause hearburn for the arm port..
22:10:42 <fenix> → BUG 18185: retire run_arch_tests.ksh in libc-tests (In Progress) | https://code.illumos.org/c/illumos-gate/+/4859
22:17:50 <richlowe> sommerfeld: probably makes things simpler, actually.
22:18:05 <richlowe> though what's in there right now is aimed at being easy to merge, not nice
22:52:42 <rmustacc> jbk: Generally I'd use delay there.
22:52:50 <rmustacc> Unless you're in a special case.
22:53:14 <rmustacc> drv_usecwait is a spin loop, so unless you need to spin,  you generally shouldn't for a ms.
23:12:37 <sommerfeld> gwr's change to split the header tests out of libc-test nicely sped up the (reduced-size) suite..
23:13:03 <richlowe> I'm excited to re-sort my list to run the libc tests sooner