From 1c4606bc4d2589152eb09a74c8fe62e3004097a7 Mon Sep 17 00:00:00 2001 From: Siddhesh Poyarekar Date: Mon, 19 Feb 2018 00:42:21 +0530 Subject: dev: Don't fall through into BRAR after RFDR case If the switch block inside the RFDR case selects the non-default case and breaks out, it will fall through into the BRAR case, which seems incorrect. Put in a break to ensure that it breaks out of the parent switch block as well. Change-Id: Ie4cedf66954b7e8f4b884ad9e3a653968bbfaef7 Signed-off-by: Siddhesh Poyarekar Reviewed-on: https://gem5-review.googlesource.com/8563 Reviewed-by: Gabe Black Maintainer: Gabe Black --- src/dev/net/ns_gige.cc | 1 + 1 file changed, 1 insertion(+) diff --git a/src/dev/net/ns_gige.cc b/src/dev/net/ns_gige.cc index dbed29f8b..e1289b4b1 100644 --- a/src/dev/net/ns_gige.cc +++ b/src/dev/net/ns_gige.cc @@ -742,6 +742,7 @@ NSGigE::write(PacketPtr pkt) panic("writing RFDR for something other than pattern matching " "or hashing! %#x\n", rfaddr); } + break; case BRAR: regs.brar = reg; -- cgit v1.2.3