00:03:49 <gemelen> richlowe: I see there is a pgp key at https://illumos.org/docs/security/, I should use it to sign my message, right?
00:10:03 <gemelen> all the logs are publicly available on the internet, though - https://log.omnios.org/illumos/2026-09-26
01:14:49 <gemelen> hope I sent it in a correct way
07:07:33 <wiedi> gemelen: thanks for trying that! :)
15:01:29 <gemelen> wiedi: np. it was an interesting exercise too
17:23:00 <siemato> hi, i kind of need assistance. i've patched the I226 driver and MAC in the illumos kernel for my own appliances for parallel receive paths, but i've never committed anything to illumos before. how should i approach trying to upstream it?
17:30:27 <rmustacc> https://www.illumos.org/docs/contributing/ has the basic docs here.
17:30:58 <rmustacc> But basically, I would focus on getting the change up on gerrit (ttps://www.illumos.org/docs/contributing/gerrit/) and then we'll review. Feel free to tag me in it as I had done initial work on the driver.
17:31:37 <rmustacc> I wouldn't expect that adding multiple receive rings would require changing the mac kernel module though.
17:32:07 <siemato> so what's the plan. should i break it down into multiple smaller PRs?
17:32:32 <rmustacc> I would just expect one change that has the totality of the work so we can review and integrate it coherently.
17:32:34 <siemato> i could see the MAC change land fairly isolated, if i want to get something in in preparation without breaking too much
17:32:40 <rmustacc> Why is MAC changing?
17:33:18 <rmustacc> It may be reasonable, I just wouldn't expect it for adding RSS support to igc.
17:38:02 <siemato> okay how do i describe that. i hit a bug that seems to affect multiple drivers (found some chelsio one on the bug tracker) as well as my x553 and i226, essentially, a network card's receive ring in polling mode with interrupts turned off after its queue empties
17:39:25 <rmustacc> Ah, a bug in MAC versus new features to implement RSS.
17:39:52 <siemato> yeah, but i'm not pushing a driver update, if it triggers a bug somewhere
17:39:53 <rmustacc> Okay, I would probably structure that separately then.
17:40:14 <rmustacc> And do one change with the mac bits and one with the igc changes on top of it.
17:40:26 <rmustacc> I took your original message as having to add new functionality to mac.
17:40:28 <siemato> that's related to setting maxbw as well, which i assume is used
17:41:16 <rmustacc> If there are specific bugs there, then yeah, breaking that separately is reasonable.
17:41:27 <rmustacc> Sorry, kind of hard to give specific advice without the context.
17:42:11 <siemato> yeah, i guess i'll just try to post it, and see where it goes. 
17:42:39 <rmustacc> If helpful you can push a branch somewhere, send it to me, and I can make a more specific suggestion if that's helpful.
17:43:34 <rmustacc> We appreciate you putting in the work, so least we can do to help in that regard.
17:43:36 <siemato> let me pull it apart and prepare
23:49:07 <siemato> @rmustacc https://github.com/siemato/illumos-gate/tree/mac-igc this is the fork. feel free to have a look at it, suggestions are welcome