-
rmustacc
The reality is that most folks are coming to ask for features from these because some app wants it, so that's always the lens to remember. I don't think anyone is going to be trying to pass a conformance test.
-
warden
Hi to all, I'm moving my first steps with IlumOS distributions, and I'm banging my head around to find how to import a root CA certificate into OmniOS CA bundle... anyone here can kindly point me to the right direction? :)
-
Agnar
warden: I could tell you how to do it on OpenIndiana since I wrote the responsible script ;)
-
Agnar
you could check for a SMF service called ca-certificates (svcs ca-certificates) and it it exists, put you new root ca in PEM format into /etc/certs/CA and run svcadm refresh ca-certificates
-
warden
Agnar: thanks, I already searched for it, but no service like this seems to exist in OmniOS :(
-
Agnar
they should adopt it ;)
-
Agnar
sorry...
-
Agnar
have you checked their website?
-
warden
You are right!:) I guess the bundled CA list in OmniOS is under /etc/ssl , but I didn't find any documentation about that
-
Agnar
or just find /etc -name \*.pem
-
Agnar
and the bundle of course, ca-certificates.crt
-
warden
putting my CA pem file into /etc/ssl/certs didn't seem to work
-
warden
well, I'll go on with a trial-and-error process... :)
-
Agnar
your tools might read the bundle file, so you need to append it there also
-
warden
actually my wish is to make integrated wget and curl binaries to work without the "-k" workaround
-
warden
Agnar: I'll go on trying to mangle further into /etc/ssl contents. Thanks for your suggestion.
-
andyf
warden what version of OmniOS are you running? I can help once I am back at a proper keyboard but I am surprised you need to use "-k" if it's a public web server. Perhaps omnios is missing some certificate updates and we should get that fixed.
-
warden
andyf: thank, but the bundled CA list is ok as for me, I'm only trying to add my private root CA into it
-
andyf
We should definitely do something to make that easier! I think you need to put the certificate into `/etc/ssl/certs` with a name ending in `.pem`, and then run `c_rehash -v /etc/ssl/certs`
-
Agnar
andyf: why not adopt OIs service, it simulates the behaviour of solaris 11.3...
-
Agnar
-
Agnar
-
warden
andyf: thank you so much, I launched the 'c_rehash' and it worked!
-
otis
when you put certificate into /etc/ssl/cert, you most probablt also need c_rehash or such
-
andyf
Agnar - thanks, looks useful!
-
Agnar
I'm following
illumos.org/docs/contributing/gerrit, but I fail on the step to scp the hook script: scp olafbohlen⊙cio:hooks/commit-msg .git/hooks/ # subsystem request failed on channel 0
-
andyf
Try adding `-O` to scp
-
Agnar
ah right
-
Agnar
switched to sftp
-
Agnar
thanks for returning a favor :)
-
Agnar
yay, that worked. the gerrit doc is phantastic btw,
-
Agnar
very very clear instructions, a great example
-
Agnar
so, if someone wants to review a 6lines code removal in emlxs_solaris.c:
code.illumos.org/c/illumos-gate/+/3415 ;)
-
fenix
→ CODE REVIEW 3415: 16459 want emlxs to support Oracle branded LP adapters (NEW) |
illumos.org/issues/16459
-
Agnar
so I see it correctly that I now git commit --amend, add reviewers to the commit message and then prepare the RTI?
-
andyf
That sounds right. Some people also push again to gerrit and some others even set the integration ready bit, but it just generates more emails to people who are copied on it.
-
Agnar
okies.
-
Agnar
I'll re-run the built tonight, my previous built unfortunately contained mofifications somewhere else in the tree which I forgot to branch correctly, so I'll rebuilt to get a clean mail_msg
-
andyf
At some point in the future we can hopefully integrate through gerrit directly in which case the IA flag becomes useful.
-
Agnar
ok, good to know
-
Agnar
it's phantastic how pain can motivate you to learn about code contribution ;)
-
andyf
It's always great when somebody goes through the process for the first time too. It helps make sure the documentation is up to snuff.
-
Agnar
andyf: I went through gerrit the first time, other contributions are already 7y ago :/
-
Agnar
too much paid work to do
-
tsoome_
I use --amend mostly, and sometimes --author and --date is useful too;)
-
Agnar
tsoome_: I actually looked at one of your commits for a template ;)
-
tsoome_
ouch:)
-
gitomat
[illumos-gate] 16461 Introduce sequence to clear Branch History Buffer (BHB) -- Dan McDonald <danmcd⊙mi>
-
vetal
Hey, How can I use TSO/LRO in illumos? "dladm show-linkprop" doesn't show so much.
-
vetal
Is it supported in illumos?
-
Agnar
if the driver supports iit, it should show up on dladm show-linprop -p lro <linkname>
-
Agnar
at least for e1000g and ixgbe it seems not to be supported
-
vetal
So it looks like i40e doesn't support it at all?
-
andyf
The man pages list the supported features fwiw -
illumos.org/man/4D/i40e
-
andyf
I don't know how much is plumbed up to dladm today.
-
Agnar
seems i40e has LSO but not TSP/LRO
-
Agnar
also, I have seen a lot of issues in the past with TSO on other operating systems btw...
-
andyf
and LSO is on by default
-
jbk
i'm not sure it's a dladm property and might just be a driver.conf property
-
jbk
but yeah, i believe it's enabled by default
-
jbk
I also don't believe the i40e hardware itself supports LRO
-
jbk
(though in general any hardware that does support LRO is going to have it disabled by default)
-
Agnar
ok, git pbchk fails for me with: b"fatal: ambiguous argument 'origin/master..': unknown revision or path not in the working tree.\nUse '--' to separate paths from revisions, like this:\n'git <command> [<revision>...] -- [<file>...]'\n"
-
Agnar
I guess I'm missing arguments, however
illumos.org/docs/contributing doesn't specify anything precisly
-
andyf
If pbchk can't find the parent commit, you can specify what you're comparing against as the last argument.
-
andyf
So if you just have one commit, something like `git pbchk -p HEAD~` works
-
Agnar
ah, it needs the previous commit
-
andyf
but usually you point it at the master branch from illumos-gate - in your case it's not finding origin/master which is usually where I'd expect it to be.
-
Agnar
and what does it mean with: only complete workspaces may be pbchk'd
-
andyf
Looking at the code, you have an extra parameter at the end of the line that it doesn't like
-
andyf
which it's interpreting as [path...] and you can't specify paths with pbchk
-
Agnar
hmm, where exactly?
-
andyf
What was your command?
-
Agnar
git pbchk master
-
andyf
You missed the `-p`
-
andyf
git pbchk -p master
-
Agnar
oh, me stupid
-
andyf
so it thought master was a filename
-
Agnar
now it works :)
-
andyf
How bad was that file? :)
-
Agnar
just missing copyright, but I will not put my copyright in it for deleting 6 lines ;)
-
andyf
Ah nice. Sometimes you make a small change like that and pbchk shows you things from everywhere else in the file (I think fixing most of them is optional but encouraged).
-
Agnar
ah, yeah.
-
Agnar
no, nothing - I send my mail to advocates
-
Agnar
have
-
vetal
andyf: "dladm show-linkprop i40e5|grep lso" doesn't show anything. Or it is not supposed to be set on/off?
-
jbk
LSO is not managed via dladm
-
rzezeski
Yea, LSO is managed through the driver conf file (i40e.conf). And it's enabled by default unless you set tx_lso_enable to 0.
-
danmcd
(That should probably be a private dladm(8) attribute using the private _<name> convention.)
-
danmcd
(Unless it's something that can only be enabled/disabled at attach time.)
-
vetal
Good. The rest question is how to check whether it works or not ;)
-
rzezeski
vetal: One way to check is to use dtrace to see if `i40e_lso_chain()` is being called.
-
jbk
i _think_ it could probably be made a private dladm property (which means it doesn't show up unless you explicitly ask for it)
-
rzezeski
and remember that LSO doesn't really kick in if the application isn't trying to send large buffers
-
jbk
it looks like really just needs to be set/unset before m_start() is called
-
rzezeski
For the dladm prop you'll have to think about quiescence and how to deal with outstanding LSO mblks coming from userland. Though we do have generic LSO emulation code now, so that could help.
-
jbk
(it looks like mac will call m_getcapab to get LSO parameters after the instance is started, so you'd not want to toggle after it's started)
-
andyf
Agnar - the RTI message looks good to me but your mail_msg shows some build errors - possibly something around perl.
-
rzezeski
One thing rmustacc and I have talked about in years past is just saying _everything_ supports stuff like checksum offload and LSO. Then, if the driver doesn't actually support those features, you rectify it with emulation code just before handing the mblk off to the driver. If that were our world than I imagine we could just expose a standard dladm option in the case that we do want to turn those features off.
-
Agnar
andyf: yes, a lot of perl. let me correct the perl patch and rebuild
-
Agnar
I forgot to set the correct perl path
-
Agnar
:/
-
andyf
I think it's safe to say that we've all done it!
-
andyf
Either perl or python anyway..
-
jbk
perl and python version problems are like 97% of all my build failures
-
Agnar
it must be the p
-
Agnar
perl...python...php...penguins... ;)
-
gitomat
[illumos-gate] 16460 dmu_zfetch_stream_fini leaks refcount -- Andy Fiddaman <illumos⊙fn>
-
jbk
is there a way to see the various tcp socket options of a socket belonging to an arbitrary process (at least without a lot of mdb spelunking)?
-
rmustacc
pfiles?
-
jbk
ahh.. actually that does show what i wanted (once I fould find the actual fd amongst the noise :P)
-
jbk
trying to diagnose glacial zfs send perf, so trying to look at each piece along the way
-
Agnar
oh wonderful, took 20s to remove 6 lines from a c file, now finding obscure perl bugs during build :)
-
jbk
make sure the version in your env file matches what's on the system
-
jbk
they don't always get updated when distros bump perl versions
-
jbk
that's usually what bites me, then i get upset for myself for having wasted 1-4 hours
-
jbk
err at myself
-
jbk
which should probably motivate me to update my 'theperlthe' branch :)
-
vetal
rzezeski: It looks like it should be improved somehow. For the instance Solaris has ability to modify LRO value via 'dladm set-linkprop'.
-
vetal
rzezeski: As I checked i40e_lso_chain() is called for output traffic. Is it possible to verify LRO is used for received traffic ?
-
jbk
AFAIK i40e doesn't support LRO.. as in the actual hardware itself doesn't support it
-
jbk
just send
-
jbk
which is kinda surprising consider how much kitchen sink stuff it does have :)
-
gitomat
[illumos-gate] 16464 Typo in i40e(4D) -- Andy Fiddaman <illumos⊙fn>
-
gitomat
[illumos-gate] 16463 zfs_ioc_recv leaks nvlist -- Andy Fiddaman <illumos⊙fn>
-
rmustacc
I've thought about writing up an ipd around better dynamic control of those properties and the resulting GLD APIs we'd want.
-
rzezeski
vetal: As jbk mentioned, the 700-series parts do not support hardware LRO.
-
rmustacc
And I think we want to use the software LRO stuff we prototyped anyways as that simplifies a lot of the driver chicanery involved.
-
rmustacc
As then you have internal timers that it's using to decide when to merge and having to figure out how to get up to 64 KiB buffers consumed.
-
rzezeski
Yes, it seems better if we just had a generic LRO feature in mac that all drivers can take advantage of.