Fri, Apr 13
Thu, Apr 12
I think this should be fine. Can always be tweaked later.
Is this good to land? Thanks!
Wed, Mar 28
Seems ok to me.
Should I make any more changes for this to land? As an aside note: I've tried to make the interface as simple as possible to use with the args[0-2] being consistent.
Mar 18 2018
Adding Tycho to this.
Mar 11 2018
Jan 12 2018
Thanks. I'll commit this later tonight.
Address the comments by markj@ and add tests in safety/ for jailname and jid.
Looks ok to me, thanks. Did you end up writing test cases for safety/ as we discussed on IRC?
After you explained that zonename is for compatibility, making it easier to port scripts from Solaris, this seems like a good change. Thank you for you hard work. Cheers!
Update the diff to add more context. No actual code was changed.
I just noticed that the ifdef for solaris was removed, resulting in the declaration of zonename. Was this intentional, and if-so, why?
Jan 11 2018
Tracing on a per-jail basis using a predicate was already possible (example below):
Mar 27 2017
Feb 23 2017
By my opinion, kernel should call dtrace_trap_func () only in unresolvable kernel trap case, as alternative to panic.
Something like: oups, dtrace probe causes panic, recovery me.
Confirmed that calling dtrace_trap_func from abort_fatal() is correct (my mistake).
In my example a malformed script results in a memory access to 0x02. The resulting translation fault brings the system down. This should not happen.
Yes, due to missing FAULT_TRAN_L: case in dtrace_trap().
Can you be, please, little more specific for
"abort_handler - didn't call the dtrace_trap() function early enough to handle faults caused by DTrace " ?
Feb 21 2017
Feb 18 2017
Feb 3 2017
Feb 2 2017
Feb 1 2017
Jan 13 2017
Review comments addressed.
Jan 6 2017
This seems fine to me - aside from the licensing, my comments are all style nits.