00:25:34 tsoome: did your mdb_printf fixes come with a PRINTFLIKE annotation somewhere? 05:35:48 yes 07:06:32 but apparently the mdb print support is not trivial and needs more than just filling up few tables, I havent had time to dig deep into it, but then again, even simple approach is rather revealing.... 13:44:52 it's missing more formatting? 16:26:00 mixed up, usual stuff - signed versus unsigned, long versus not long or long long etc. 16:29:17 like this one too: ../../../common/modules/genunix/avl.c:107:63: error: format '%d' expects argument of type 'int', but argument 3 has type 'size_t' {aka 'long unsigned int'} [-Werror=format=] 16:29:43 ahh.. so more callers are the issue 16:31:38 yep. 16:33:53 the same old story, compiler can help us, but the feature is either not implemented or warnings are disabled... 16:36:04 yeah.. i try to remember when messing with a module to try to clean up any silenced warnings while I'm in there unless it's too messy and really needs a dedicated change 20:17:25 printflike isn't silenced in the traditional sense 20:17:48 the problem is that we use `-fno-builtin` which means the compiler (rightfully, but annoyingly) can't assume what `printf` etc are 20:18:00 so we lack it in places we didn't annotate (annoying), and in places we shouldn't have to annotate 20:18:11 but we sort of asked for it to be a pain 20:22:34 no, with printf the problem is that we need compiler support (like there is for cmn_err()), and it is something we havent implemented yet. thats it. 20:26:13 tsoome: actual printf(3c)? no that's not it 20:26:45 mdb_printf() yes, that's what I was asking for :) 20:27:07 (because otherwise it won't know %H, the best format character ever) 20:27:53 with printf(3c) we have unrecognized "%ws", and some other minor details. 20:28:19 and yes, mdb printf is much more "fun". 20:29:11 but %ws prevents us to annotate printf family for userland. at least as much as I have played with it. 20:30:55 ofc the good question is, should we actually have that %ws. 21:28:41 tsoome: probably if you follow that thought too far the answer is related to the standards check symbols we have 21:29:07 so that we support %ws _unless_ ... 21:29:36 (and anything else that isn't in the standard, when the runtime standard bits are set) 21:30:00 and the compiler would agree with that via -std=... 21:30:01 and 21:33:48 so a very early drafty take on the TCP small queues thing I was talking about yesterday: https://code.illumos.org/c/illumos-gate/+/4594 21:33:49 → CODE REVIEW 4594: 17909 tcp could benefit from a small-queues transmit throttle (NEW) | https://www.illumos.org/issues/17909 21:37:01 sommerfeld: i see you and danmcd have similar taste in magic constants :) 21:38:36 Yeah, it's my fault. Plus 0x90125xxx needs an xxx I can't think would have a good answer off the top of my head. 21:38:53 (*partially* my fault to be 100% accurate... I can't speak for sommerfeld) 21:39:23 jbk: indeed 21:49:22 am I missing a reference or cute trick in the numbers? 21:54:33 There are a pair of initializers in his changeset. And the hint is "album names". 21:55:25 Also cf. ipseckey(8). Also, for even more blatant ones, cf. ike.preshared(5) 21:55:41 oh, rush 21:55:44 and probably van halen 21:56:06 ^^^ give that man a prize! 21:56:13 sorry, I'm old but younger, and I always assume there's math I don't know :) 21:57:38 was hoping for comments on the content rather than the magic numbers but I'll take what I can get... 21:57:43 No sorry needed! 21:58:00 (and those magic numbers will go away when I fix up how the buffer is allocated..) 21:58:05 sommerfeld: Not enough review brainpower for that today. All of that is $DAYJOB related for the next several days. 21:58:39 sommerfeld: unfortunately, the meat of my look at it was judging how much I would need to do in remedial study to have opinions 21:59:09 so I think having attracted rpz to it is the best I can probably achieve 21:59:11 The fact that driver reclaim tweaks are required has me concerned about the solution space and what assumptions this is going to create and assume. 22:00:47 It's not clera from reading the ticket why the proposed solution is the right one and how we expect it to change and impact things. Particulalry when the way traffic to zones works means you have no assumptions about quick freeing of data. 22:01:03 I think there's enough there to buy the problem. 22:01:45 And to be clear, it all might be, but just saying for the few paragraphs for a fundamental design change and the throwaway driver problem leaves me with more questions than answers. 22:03:00 mblk_t lifetime isn't tied inherently to the protocol lifetime, paticularly when going across mac loopback. 22:04:01 yeah, much to think about here but unfortunately I have to run right now... 22:08:32 [illumos-gate] 17900 svc/configd: format issue while printing pid_t -- Toomas Soome