Sorry for lack of responses, I did see your requests but got lost in work stuff.

Few little style nits but apart from that much prefer this approach.

Is there anything happening with this as head is still 1.7.12-k which is very far behind the latest intel download which is 1.9.7?

Closed by https://svnweb.freebsd.org/changeset/base/330950

Its only a problem for devices that report they support TRIM so it wouldn't impact HDD's but would effect SSD's or MD devices.

Sorry guys this dropped off my radar, only just spotted your updates when reviewing other changes.

Added howto param name to prototypes.

Correct size argument as zio_trim is offset + size not start + end.

In D14347#301632, @truckman wrote:

I don't remember seeing any finalizer related crashes. The failures were mostly malloc-related and looked like they could be caused by arena corruption. I can try to dig up the logs later. I don't recall seeing any build failures on my FX CPU, but lang/go would almost always fail to build on Ryzen.

In D14347#300971, @truckman wrote:

As noted in my email comment, this patch appears to have resolved a number of randomish-looking ports build failures on my Ryzen machine, in particular lang/go and anything related to lang/guile.

Nice improvements, I learned some interesting details while following these improvements :-)

LGTM

Few nits

LGTM

LGM, much cleaner than it was.

Wow that's a lot of changes.

Convert timestamp to ticks to ensure that comparisons are correct for
hz != 1000.

Lawrence is now happy with this in is current form, so just a prod to see if I get any comments for or against from transport members / gnn?

Extract common logic to tcp_autorcvbuf as suggested by lstewart.

Some style nits but otherwise LGM

Thanks for the feedback Lawrence, based on that I've updated to use just the SRTT check, added the fastpath version and generally cleaned up.

Corrected dtrace autoresize mbuf parameter definition.

Updated dtrace:

dtrace
#!/usr/sbin/dtrace -s

Eliminated timestamp specific code path resulting in a simplified yet still effective single SRTT path.

LGTM

Massive amounts of changes in this so impossible to see if everything is good, however a number of style related bits highlighted, some of which are regressions.

book -> bool

In D9829#203014, @jhb wrote:

I agree with shortening the line a bit so it fits in 80 cols. If you are worried about duplicating "ARC:" you could perhaps just use a whitespace prefix so you end up with something like:

Mem: 754M Active, 836M Inact, 75M Laundry, 3761M Wired, 10G Free
ARC: 2476M Total, 914M MFU, 1302M MRU, 1696K Anon, 140M Header, 118M Other
     1899M Compressed, 6065M Uncompressed, 2.45:1 Ratio, 318M Overhead
Swap: 3072M Total, 3072M Free

Some may find it useful if it was bit shorter, currently its 82 chars in the example.