Re: [Xen-devel] [PATCH 3/9] passthrough/amd: Missing 'break'

[Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx>]

On 09/12/2013 07:15 AM, Tim Deegan wrote:
> Coverity CID 1055502
>
> Signed-off-by: Tim Deegan <tim@xxxxxxx>
> Cc: Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx>
> Cc: Jacob Shin <jacob.shin@xxxxxxx>
> ---
>   xen/drivers/passthrough/amd/iommu_guest.c |    1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/xen/drivers/passthrough/amd/iommu_guest.c 
> b/xen/drivers/passthrough/amd/iommu_guest.c
> index 85f2361..952600a 100644
> --- a/xen/drivers/passthrough/amd/iommu_guest.c
> +++ b/xen/drivers/passthrough/amd/iommu_guest.c
> @@ -728,6 +728,7 @@ static void guest_iommu_mmio_write64(struct guest_iommu 
> *iommu,
>           break;
>       case IOMMU_EVENT_LOG_BASE_LOW_OFFSET:
>           u64_to_reg(&iommu->event_log.reg_base, val);
> +        break;
>       case IOMMU_PPR_LOG_BASE_LOW_OFFSET:
>           u64_to_reg(&iommu->ppr_log.reg_base, val);
>           break;
Acked-by: Suravee Suthikulpanit <suravee.suthikulpanit@xxxxxxx>


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel