...
> > +
> > + virHostMsgCheck(hvname, "%s", _("for secure guest
support"));
> > + if (ARCH_IS_S390(arch)) {
>
> We don't need ^this check here, facility 158 won't be available on x86.
Since it is very true that the facility 158 won't be available on x86 you do
not want to get the message "Hardware or firmware does not provide support
for IBM Secure Execution" on x86.
On the other hand you can be on s390 and not have the facility available and
want to know see the message.
Oops, I missed the last nested else block, please ignore what I said then.
--
Erik Skultety