Looks fine to me.

In D30793#692367, @imp wrote:

That is, I'd rather see ZFS64

include GENERIC64
options ZFS

etc

Address @alfredo's feedback

Is this still needed? We've had ifuncs for a year now, with no problems.

Address @bdragon's feedback.

switch to new openzfs made this redundant.

Seems to work fine on my G4.

In D29810#669221, @markj wrote:

Can you provide more info about the problem? Does it arise because of the roundup of namesz and descsz?

This is superseded by D29136.

No need for this anymore. We have clang, and GCC has abandoned powerpcspe (last supported version is gcc8, used as ports gcc).

Committed as 6260bfb

Address @mw's feedback.

Closed by b6d8f3b5

On a dual socket POWER9:

@luporl we want a LE 64-bit number here.

Since mail/thunderbird now builds and works on powerpc64, is this OBE? Or is more needed?

Looks fine to me

Maybe after it's in, someone can try to figure out why enabling superpages causes a 2% performance penalty in buildworld.

Maybe we should stop playing whack-a-mole and just use an overkill bounce page that can hold the full name field (2048 bytes?)

Is there any more work needed on this?

I like the refactoring!

Seems fine to me. It's consistent with all other archs.

Looks good, glad you got the speed back!

This is mostly a series of experiments to see what can reduce the locking contention on the 'page pv' lock (PV_LOCK()/PV_PAGE_LOCK()) since those seem to be pretty heavily contended, as measured with lockstat(1).