s390x: sigp: Force Set Architecture to return Invalid Parameter
According to the Principles of Operation, the SIGP Set Architecture
order will return Incorrect State if some CPUs are not stopped, but
only if the CZAM facility is not present. If it is, the order will
return Invalid Parameter because the architecture mode cannot be
changed.
Since CZAM always exists when S390_FEAT_ZARCH exists, which in turn
exists for every defined CPU model, we can simplify this code.
Fixes: 075e52b816
("s390x/cpumodel: we are always in zarchitecture mode")
Signed-off-by: Eric Farman <farman@linux.ibm.com>
Reviewed-by: Christian Borntraeger <borntraeger@de.ibm.com>
Reviewed-by: Janosch Frank <frankja@linux.ibm.com>
Message-Id: <20211008203811.1980478-2-farman@linux.ibm.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: David Hildenbrand <david@redhat.com>
Signed-off-by: Thomas Huth <thuth@redhat.com>
This commit is contained in:
parent
f142010102
commit
998eb7448c
@ -428,26 +428,10 @@ static int handle_sigp_single_dst(S390CPU *cpu, S390CPU *dst_cpu, uint8_t order,
|
|||||||
static int sigp_set_architecture(S390CPU *cpu, uint32_t param,
|
static int sigp_set_architecture(S390CPU *cpu, uint32_t param,
|
||||||
uint64_t *status_reg)
|
uint64_t *status_reg)
|
||||||
{
|
{
|
||||||
CPUState *cur_cs;
|
|
||||||
S390CPU *cur_cpu;
|
|
||||||
bool all_stopped = true;
|
|
||||||
|
|
||||||
CPU_FOREACH(cur_cs) {
|
|
||||||
cur_cpu = S390_CPU(cur_cs);
|
|
||||||
|
|
||||||
if (cur_cpu == cpu) {
|
|
||||||
continue;
|
|
||||||
}
|
|
||||||
if (s390_cpu_get_state(cur_cpu) != S390_CPU_STATE_STOPPED) {
|
|
||||||
all_stopped = false;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
*status_reg &= 0xffffffff00000000ULL;
|
*status_reg &= 0xffffffff00000000ULL;
|
||||||
|
|
||||||
/* Reject set arch order, with czam we're always in z/Arch mode. */
|
/* Reject set arch order, with czam we're always in z/Arch mode. */
|
||||||
*status_reg |= (all_stopped ? SIGP_STAT_INVALID_PARAMETER :
|
*status_reg |= SIGP_STAT_INVALID_PARAMETER;
|
||||||
SIGP_STAT_INCORRECT_STATE);
|
|
||||||
return SIGP_CC_STATUS_STORED;
|
return SIGP_CC_STATUS_STORED;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user