-
neirac
who has recently worked in the smartos ipxe?
github.com/TritonDataCenter/ipxe currently I'm trying to boot using netboot but I'm stuck at
github.com/TritonDataCenter/ipxe/bl…rc/arch/x86/image/multiboot2.c#L813 but I don't have a clue on how to continue
-
rmustacc
When you say you're stuck can you clarify what you mean?
-
neirac
rmustacc yes, I mean that the last thing I see on the debug build is that "MULTIBOOT2 exit_boot_services" is printed then the call to get_efi_mmap never returns and then after a while the process exists to the efi shell again
-
rmustacc
So, exiting boot services is a way of telling firmware it's done and the primary ownership of the system is the OS.
-
neirac
rmustacc oh, yes, but it never actually exits boot services it seems stuck, let me get the last debug messages I have
-
rmustacc
So is it stuck getting the memory map or in the call to exit boot services?
-
neirac
rmustacc
ctrl.vi/i/u1epRh9Ip, yes I added debug messages once is stuck in the memory map call
-
neirac
rmustacc I'm sorry, yes I see that is stuck in the GetMemoryMap call, according to debug messages I added sometime ago when boot_services calls get_efi_mmap
-
tsoome_
neirac GetMemoryMap should either return memory map or tell the buffer is not large enough and returns the expected buffer size. Now the issue might be the expected size is not exact as memoty map can grow during the GetMemoryMap call itself... so, we should round the buffer size up. So, if the system is stuck in GetMemoryMap, it hints that we did call it with some buffer space, and the GetMemoryMap may be running
-
tsoome_
ast buffer end and thus corrupting whatever memory there is and somehow this will eventually trigger exit from ipxe and we end up in UEFI shell.
-
tsoome_
neirac to work around this issue, we call GetMemoryMap initially with NULL buffer to get suggested buffer size, and we allocate buffer with some number of pages larger than hinted.
-
sjorge
OK so ran out of space again, but my zvol is 100G but illumos only sees 10G -_- thats unfortunately
-
sjorge
Well, 3rd time is the charm, I hope
-
sjorge
So this is the only fine I need to copy to the new be right, assuming everything is build from the same commit as what I'm currently running? ./proto/root_i386-nd/usr/kernel/drv/amd64/viona
-
neirac
tsoome thanks I'll try to check that but I could not find the GetMemoryMap implementation in the ipxe code
-
sjorge
Here goes nothing, lets reboot
-
sjorge
Ok, I'm booted into the BE and nothing exploded yet
-
sjorge
Now to figure out how the properly format the tunable name
-
sjorge
OK with a vq of 128 I hit the issue almost imediatly after booting the VM
-
sjorge
Good good
-
sjorge
OMG It works, well at least viona_tx_copy_needed now returns 1 instead of 0 after setting the tunable and rebooting
-
sjorge
And somehow i am not left holding multiple pieces
-
sjorge
OK, the change is super super tiny
-
sjorge
But still
-
sjorge
Also TIL fbt probes only show up in dtrace -l after the first viona instances is created
-
sjorge
Now the big test, does pushing large amounts of data over it with forced copying prevent it from locking up or not
-
sjorge
I may have squeeze the vqsize down to 8 *whistles*
-
sjorge
Looks to be holding up, there is a performance impact, which is to be expected I guess
-
sjorge
NFS now tops out around 437 MB/s
-
sjorge
Which is still very very good IMHO, before I was seeing closer to 7xx MB/s range
-
sjorge
At first glance nfsstat also has less random super high latancy spikes, which is unexpected but interesting
-
sjorge
jclulow: do I file a sub-ticket for adding the force copy tunable to viona ? Seems mine tiny patch works, or do I use 13463 ?
-
sjorge
I think adding notes on what I did and how I tested to 13463 would just clutter that one up, given this doesn't fix the actual problem and is just a bandaid.
-
nbjoerg
any suggestions for debugging my other network problem? :)
-
nbjoerg
also, is there a way to change the queue size via vmadm now?
-
nbjoerg
or how to play with it?
-
sjorge
Not sure you can via vmadm
-
andyf
Has anyone had much luck running the zfs testsuite recently?
-
andyf
I'm getting lots of errors that look like it's assuming bash (local: not found) and path problems - digest: not found / uuidgen: not found
-
andyf
I only got 91% success, which is a lot lower than I'm used to
-
jbk
there seems like there's a couple of new failures..
-
jbk
for the zfs stats i ran the test suite against an unmodified illumos-gate (since it had been a while), then a build w/ the change applied..
-
jbk
and most of the failures were ones that have been on my list (I've filed tickets for all of those)
-
jbk
but it did seem like there was 1-2 new ones (even after re-running)
-
danmcd
@andyf --> I now run them under the umbrella of `smartos-test` every release week.
-
danmcd
Last run: Out of 1292 lines stating PASS/FAIL/SKIP, I get 1070 PASS, 75 FAIL, and 147 SKIP. *which* ZFS test(s) fails does vary from release to release, suggestiong transient failures.
-
danmcd
The %-ages do seem high, however. I've NOT taken deep-dives yet into why.
-
jbk
apparently you're supposed to be able to set ZFS_ABORT to force zpool commands to create a core when they exit, but I think that either is broken or the test isn't properly detecting things -- that seems to be at least a few of the failures
-
jbk
others seem like some test don't always properly clean up, and some later test (not always the immediate one afterwards) then fails because of that (but is fine when run on their own)
-
jbk
unfortunately, such things are not a priority at $JOB, so it's hard to find time to dig into them
-
andyf
jbk - I also had hanging tests, so it's likely I'm doing something daft, although I've run these frequently in the past
-
andyf
I think it's the first time I've used two NVMe and one vioblk disks for the test temporary ones though
-
andyf
I shall try going back to vioblk for all!
-
gitomat
[illumos-gate] 15194 libdladm: tautological-pointer-compare -- Toomas Soome <tsoome⊙mc>
-
neirac
tsoome is it bad when ipxe is loading the boot_archive saying that the image has no multiboot2 header?
-
danmcd
neirac: You're using the Triton iPXE booter?
-
neirac
danmcd I'm using netboot.smartos.org/smartos.ipxe
-
danmcd
That's a config file.
-
neirac
dancmd I mean I compiled the one that's on the triton repo to load it
-
danmcd
Which ipxe *binary* are you using... ahh, you typed the answer I wanted.
-
danmcd
So you compiled it yourself.
-
danmcd
(which shouldn't be a problem FTR).
-
neirac
danmcd that's right I added debug to see where it's stuck, but my last test exit_bootservices returned EFI_SUCCESS so after that the vm get's back to the efi shell
-
neirac
danmcd on the ipxe shell I'm just typing dhcp then chain
netboot.smartos.org/smartos.ipxe. then choose text instead of a tty
-
neirac
danmcd I was wrong get_efi_mmap returne ok, I'll activate more debug on boot_services
-
jclulow
sjorge: Not a sub ticket but just a new ticket, and you can add the original bug as a related issue
-
sjorge
OK then i did it correctly
-
ptribble
So who is off to FOSDEM?
-
sjorge
do we have a booth?
-
tsoome_
neirac boot_archive will not have MB header, kernel does. and MB bootloaders expect kernel first..
-
jbk
so really dumb question.. shouldn't 'chmod 2750 dir' result in '-r-xr-s--' ? it's stripping the setgid bit when i try it so i feel like i've forgotten something i should know
-
jbk
-
jbk
chmod g+s does work however
-
danmcd
Odd indeed.
-
danmcd
Yeah. 2 in that first octal digit (even or-ed with other bits) seems to be a nop.
-
danmcd
`chmod 6750 kebe` gives me:
-
danmcd
drwsr-x--- 2 root root 512 Dec 20 21:11 kebe
-
danmcd
Ahh.
-
danmcd
-
danmcd
@jbk ^^^
-
danmcd
Now WHY that's there is a mystery to me, and I'm sure would make a good history story.
-
danmcd
The behavior IS documented in the chmod(1) man page too.
-
jbk
ok.. so i'm not crazy or forgetting something silly..
-
danmcd
You're not crazy. You HAVE piqued my curiousity as to WHY this behavior is there. I don't see it on MacOS, e.g.
-
danmcd
My gut says SysV-ism, but :shrug:
-
jbk
yeah.. we have someone that has an app that for whatever reason wants 2750 on an NFS share, and chmod wasn't working
-
danmcd
Linux (ubuntu16 in an LX zone) chmod also behaves like MacOS.
-
danmcd
There's gotta be some OG SysV thing I'm missing.
-
jbk
oh... there is a note at the very bottom of chmod(1)... that contradicts the stuff earlier about it
-
jbk
well kinda...
-
jbk
it could maybe make the distinction of 'when present' vs 'when you try to set it'
-
danmcd
Eeesh, example 5 is broken.
-
alanc
jbk/danmcd: if I recall correctly, this is a holdover from SysV using that bit for mandatory locking
-
danmcd
Thank you!
-
alanc
though I may be remembering wrong, since the comment Don Cragun left when closing
illumos.org/opensolaris/bugdb/bug.html#!4067561 says:
-
fenix
→ OpenSolaris issue 4067561: chmod 2xxx fails on directories (Closed)
-
alanc
Doing what is being suggested here would not make the system more
-
alanc
user friendly. It would create chaos as regular users accidentally
-
alanc
change the way group IDs are set when subdirectories are created.
-
alanc
The current method of only changing the GID inheritance mode when
-
alanc
done by symbolic mode changes was done as a result of a user interface
-
alanc
design study. It may seem strange to users who are not used to
-
alanc
systems that allow the user to specify whether subdirectories are to be
-
alanc
created with the group ID of the creating process or the group ID of
-
alanc
the containing directory. Switching the behavior to allow absolute
-
alanc
mode changes (as well as symbolic mode changes) to set and clear the
-
alanc
set-GID bit on directories should only be done if using the set-GID
-
alanc
bit on a directory to determine which group-ID assignment to use on
-
alanc
subdirectories is removed from the system at the same time. Since
-
alanc
there has been no suggestion that this capability should be removed
-
alanc
from Solaris systems, I am closing this bug again as Not A Defect.
-
richlowe
well, that certainly sounds like don
-
alanc
...but since Linux & BSD allowed it, we ignored Don's advice and changed it anyway in 12.0^H^H^H^H^H11.4
-
alanc
as part of our adding GNU-compatible options to /usr/bin/chown
-
richlowe
yeah, I don't follow any individual part of don's reasoning (fitting...), so I would do it too
-
richlowe
"it's so confusing we don't let you do it when you use chmod in the greybeard octal way nobody does by accident, only the easy way"
-
danmcd
I'll happily code-review or RTI-approve such a fix, richlowe
-
alanc
though I've also just noticed our man page still claims "Absolute changes do not work for the set-group-ID bit of a directory.
-
alanc
You must use g+s or g-s."
-
danmcd
That was the text I saw in ours.
-
copec
I always just thought I was missing something since I "grew up" on Linux
-
nbjoerg
I don't understand that argument
-
nbjoerg
but then I spend half my life on BSD
-
danmcd
SysV was always odd. ALWAYS.
-
nbjoerg
and that was even before it become LarryOS :)