When running in a Solaris zone on T4/T5 what happens is that the architecture is reported as sun4v, so
is_niagara(features) is true, but family flag is for some reason stripped off.

I propose to remove the assert.

See vm_version_sparc.cpp, VM_Version::determine_features() .

The assert doesn't affect the product, but affects testing in zones, hence P4.