Re: [Qemu-devel] [PATCH] spapr: add missing break in h_get_cpu_characteristics()
On Thu, Feb 01, 2018 at 08:47:41PM +0100, Greg Kurz wrote: > Detected by Coverity (CID 1385702). This fixes the recently added hypercall > to let guests properly apply Spectre and Meltdown workarounds. > > Fixes: c59704b25473 "target/ppc/spapr: Add H-Call H_GET_CPU_CHARACTERISTICS" > Signed-off-by: Greg KurzApplied to ppc-for-2.12, thanks. > --- > hw/ppc/spapr_hcall.c |1 + > 1 file changed, 1 insertion(+) > > diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c > index 4d0e6eb0cf1d..596f58378a40 100644 > --- a/hw/ppc/spapr_hcall.c > +++ b/hw/ppc/spapr_hcall.c > @@ -1697,6 +1697,7 @@ static target_ulong > h_get_cpu_characteristics(PowerPCCPU *cpu, > switch (safe_indirect_branch) { > case SPAPR_CAP_FIXED: > characteristics |= H_CPU_CHAR_BCCTRL_SERIALISED; > +break; > default: /* broken */ > assert(safe_indirect_branch == SPAPR_CAP_BROKEN); > break; > -- David Gibson| I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson signature.asc Description: PGP signature
Re: [Qemu-devel] [PATCH] spapr: add missing break in h_get_cpu_characteristics()
On Thu, 2018-02-01 at 20:47 +0100, Greg Kurz wrote: > Detected by Coverity (CID 1385702). This fixes the recently added > hypercall > to let guests properly apply Spectre and Meltdown workarounds. > > Fixes: c59704b25473 "target/ppc/spapr: Add H-Call > H_GET_CPU_CHARACTERISTICS" > Signed-off-by: Greg KurzReviewed-by: Suraj Jitindar Singh > --- > hw/ppc/spapr_hcall.c |1 + > 1 file changed, 1 insertion(+) > > diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c > index 4d0e6eb0cf1d..596f58378a40 100644 > --- a/hw/ppc/spapr_hcall.c > +++ b/hw/ppc/spapr_hcall.c > @@ -1697,6 +1697,7 @@ static target_ulong > h_get_cpu_characteristics(PowerPCCPU *cpu, > switch (safe_indirect_branch) { > case SPAPR_CAP_FIXED: > characteristics |= H_CPU_CHAR_BCCTRL_SERIALISED; > +break; > default: /* broken */ > assert(safe_indirect_branch == SPAPR_CAP_BROKEN); > break; >
[Qemu-devel] [PATCH] spapr: add missing break in h_get_cpu_characteristics()
Detected by Coverity (CID 1385702). This fixes the recently added hypercall to let guests properly apply Spectre and Meltdown workarounds. Fixes: c59704b25473 "target/ppc/spapr: Add H-Call H_GET_CPU_CHARACTERISTICS" Signed-off-by: Greg Kurz--- hw/ppc/spapr_hcall.c |1 + 1 file changed, 1 insertion(+) diff --git a/hw/ppc/spapr_hcall.c b/hw/ppc/spapr_hcall.c index 4d0e6eb0cf1d..596f58378a40 100644 --- a/hw/ppc/spapr_hcall.c +++ b/hw/ppc/spapr_hcall.c @@ -1697,6 +1697,7 @@ static target_ulong h_get_cpu_characteristics(PowerPCCPU *cpu, switch (safe_indirect_branch) { case SPAPR_CAP_FIXED: characteristics |= H_CPU_CHAR_BCCTRL_SERIALISED; +break; default: /* broken */ assert(safe_indirect_branch == SPAPR_CAP_BROKEN); break;