On 28/01/2018 22:28, David Gibson wrote: > + switch (safe_indirect_branch) { > + case SPAPR_CAP_FIXED: > + characteristics |= H_CPU_CHAR_BCCTRL_SERIALISED;
Missing "break;" here. Paolo > + default: /* broken */ > + assert(safe_indirect_branch == SPAPR_CAP_BROKEN); > + break;