Move WANT_PGSQL up

Update order

I got the disjoint OPTIONS_DEFAULT ordering from Example 5.39 in https://www.freebsd.org/doc/en/books/porters-handbook/makefile-options.html

I remember we tried to analyze and improve this and found some unintended consequences between @hiren and @lstewart https://reviews.freebsd.org/D8225 so it got backed out. @lstewart do you remember the details for backing it out?

We ran this configuration for a very long time at LLNW

I worked with rrs to suss out some bugs on this use case when it was new at job-1. I recently tested it to 40gbps single flow on my own pedestrian hardware (1st gen Xeon e3, ixl on one side chelsio T5 on the other) and the CPU usage didn't stand out to me. Yes there will be more timer soft interrupts but there are actually a lot of branching and date movement optimizations in the RACK stack that also help this single flow case. I'm not necessarily saying don't improve the default stack, just that it would probably be a good thing for your customers if you had the totality of the RACK stack improvements in there.

Are you aware of the RACK stack in /usr/src/sys/netinet/tcp_stacks/? It probably does everything you want

You may be able to add something to /usr/src/tests/sys/

Fixes https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=215825

Will address in a version bump

I would prefer to see this bootstrapped and only for 12.0

Your PR patch looks good to me

This works for me and is the last qt4 application I had

It's the way it was, dropped them.

Add MOVED entry. This doesn't need an updating entry there are only a couple users and I am in communication with them.

@krion or @timur can I get approval for this to go into unfrozen head?

kev-ws-aurora% portlint -A
WARN: /usr/home/kev009/c/fbsd-rw/ports-head/emulators/linux-power9-functional-sim/pkg-descr: includes lines that exceed 80 characters.
0 fatal errors and 1 warning found.

Addressed the two inlines but the overall intent here is to not think about this and follow amd64 GENERIC. When they reduce due to devmatch, we replay the change.

Address erj review

Address review

Sorry the first part of my comment made no sense, ixl is the pf and will remain for 25/40g hw parts.

Can you rename the sys/dev/ixl dir to iavf as part of this? I'd really like to see this change done now in 12.0, it would be very awkward to try and MFC later.

Yes I have given remote access to my P9. We also now have P8 reference machines in the FreeBSD cluster.

Nice work, thanks for this fix!

This LGTM taking @hiren suggestion to drop the unused TCPTV_CPU_VAR define. At LLNW we ran w/o delack to work around most of the issues this addresses in a more elegant way.

Commandeer to close