I wasn't saing you shoudl force yourself to do the test.. just whether it is theoretically possible,, anyhow I'm happy with the result. Don't let me stop you from committing it and I would say that the only case for keeping it in 13 is for the brief period while an upgrade is occurring. (for example I know people who have their interface connected into a bridge so that if the bridge wasn't configured correctly they'd be off the air.. It would basically be for compatibility up the 12 line rather than being needed in the 13 line.
Thu, Oct 10
That's why I added the cookies to the API.
Can you use both binaries? e.g. a script uses one and manually you use a newer one?
It should be backward compatible for 12 at least. Probably not needed for 13 but no harm in carrying it for one release.
Mon, Oct 7
Jun 25 2019
Jun 15 2019
Oct 16 2018
experimenting with the pastbin.. ignore..
May 17 2018
May 15 2018
only one thought really.
May 9 2018
It would be nice to add a mention of rc.resume and the new resume keyword to rc(8) as Julian suggested.
I would rather not duplicate same documentation in different places.
/etc/rc.resume is not used for auto-reboot nor general daemon startup described in rc(8).
Do you have any idea how to mention rc.resume in rc(8) correctly?
May 5 2018
should probably be documented somewhere.
mabe add a cross ref (see also) in the rc and rcorder man pages and add sections to whatever man pages mention how to resume (or add one)
May 2 2018
Apr 18 2018
happy but not able to test right now..
Apr 4 2018
Can we make a list of other things that we'd need to change?
ABIs for Start
Mar 7 2018
I'm curious why you don't setup a kqueue event structure to point to the file and then just poll it each time through. That would be no more expensive than the time() call, and you would have no lags. Would also save all the stat calls too, which would be a win.
Despite my grumpy responses (Get off my lawn!) I appreciate the comments.. At least you didn't say "What a stupid idea, who would want to do that?" Anyhow you've given me some things to think about which is the aim. right?
In D4154#295111, @kevin.bowling_kev009.com wrote:
This was committed over 2 years ago. So that sounds like a pretty local mod. I would recommend either carrying it in your local tree or otherwise upstreaming the estimator.
that's what we are doing (just adding the fields back locally..
The estimator is, unfortunately, a bit crude for public consumption, or I'd commit it.
I tried to read the introduction with a web browser (it's a .html) but it stuck it all together..
can you add some basic formatting?
Jan 25 2018
dammit we have a functional bandwidth estimator that used this field..
Jan 2 2018
is there a formatted version of this available somewhere so it can be read in final form?
Dec 5 2017
Nov 10 2017
Have applied this patch at $JOB now to stable/10
Oct 30 2017
what are the chances of this improvement making it into stable/10?
Oct 20 2017
really minor things only.. don't get discouraged..
Oct 5 2017
Sep 30 2017
Sep 15 2017
I've applied these changes to 10.3 and will give them a run..
Sep 14 2017
so there are bits of this missing if you are (like us) trying to make this work in stable/10 or stable/11 because Isilon have already aommitted parts of it..
like change 313475 for example..
I need to merge that into our 10 based code until I can use this..
Sep 12 2017
It seems that ZFS (which we don't use) has some ABI dependency on these constants via zfs_cmd, and this change breaks a CTASSERT on line 202 of zfs_ioctl.c. For some reason passing 3 MAXPATHLENs and 1 MAXNAMELEN via ioctl is too much. Someone would need to figure out how to fix ZFS to use this patch.
Aug 19 2017
Very glad to see this. at $JOB we create an appliance and we have ami, azure and vmware versions and I can predict that google-compute is on the Horizon (actually it's already prototyped).
so I'm sure I will get to know and love (?) this port .. :-)
Aug 16 2017
New, BSD license, code does not need to be in cddl. Most of the new D scripts and programs go into either share/dtrace or the DTrace Toolkit port. Since this is a script I'd put it into share/dtrace if you want it in the src tree.
so, like is this getting committed some time?
May 4 2017
DOn't see issues in this code but see other comment
Apr 6 2017
Who;s going to commit?
any doc change required?
Mar 16 2017
note that this was committed
Mar 10 2017
just an idea for sometime
we are going to run out of available tricks for this in ipfw at some stage.. Can we re-use () like used in table? or make the : part of the keep-state.. so that "keep_state: state1" or keep_state(state1) vs "keep_state :state1"
just my $0.02
will work on this this a bit more this weekend
Mar 7 2017
First off in the unix philosophy /we also be adding a relname in the spirit of
basename and dirname that calls this library so that any shell script or
Makefile, etc could easily add this functionality?
p.s. I'm still considering line 266/267 in ln.c which came with the original patch, but I have not totally understood yet..
Bapt has convinced me to add these functions to libutil instead of keeping them in libpathconv as they originally came.
The Pathconvert package has been incorporated over the years into many projects.
The simplest way to see how to use this is to imagine how difficult it can be sometimes to generate relative symlinks when installing packages or generated binaries when the source and destination are not fixed relative to each other but expressed as two absolute paths. This becomes trivial by adding the -r option and specifying the available absolute paths. I originally started checking this in as libpathconv, before Bapt got to m. I will remove that at the same time I add this.
Feb 9 2017
I think that "collision" is ok.
This is the source comment that should have been submitted with my 'request for change' action, had I pressed the buttons in a way that didn't dispose of it instead.
so what change did pkelsey request? I can't see any request to change anything.
who is this mysterious group reviewer "transport"? And where is their negative review? I only see gnn's
my only comment is that I ask you to just confirm that a listening socket can not collide with the socket we are allocating.
i don't think it does, but I may have missed something
As long as features are DISABLED and not removed (e.g. at $JOB we need MD2) then ok.
I think we should make two versions of this port: openssl and openssl-insecure, which has features like sslv2 and MD2 enabled for people who need that functionality. (e.g. for talking to old machines or handling old certificates etc.)
maybe with a different .so number. (just an idea).
Jan 27 2017
Jan 22 2017
as far as I can see it looks good. Same default value as before..
Considering how much work went into writing this and the fact that we had to get it approved by verizon (or whatever they were called at the time) its amazing that I remember almost nothing about how it works.
Jan 20 2017
we should probably skip ports that have a listenig socket on them.. do we?
Jan 19 2017
looks good to me. Hit this in 10.3
Jan 4 2017
this is obviously part of a bigger change, so I can't comment on that part but the part about the EXTRA_PATCH_TREE looks like it should work.
I don't see why one would have a compressed patch file but handling it shouldn't hurt. (except by hiding bugs through added complexity.)
Dec 27 2016
Dec 22 2016
Dec 2 2016
Nov 24 2016
I vaguely remember another superio driver for another model/maker super io chip
Nov 23 2016
Nov 15 2016
Nov 10 2016
looks reasonable, but am unsure of all the possible cases for *end given garbage input
Nov 3 2016
I'd still vote for using rcvif if a new field is out of the question.