From 83a668ad2564d1231f59f130c81cab6e7e31c364 Mon Sep 17 00:00:00 2001 From: Andreas Hansson Date: Fri, 7 Aug 2015 04:55:38 -0400 Subject: mem: Remove extraneous acquire/release flags and attributes This patch removes the extraneous flags and attributes from the request and packet, and simply leaves the new commands. The change introduced when adding acquire/release breaks all compatibility with existing traces, and there is really no need for any new flags and attributes. The commands should be sufficient. This patch fixes packet tracing (urgent), and also removes the unnecessary complexity. --- src/mem/packet.cc | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'src/mem/packet.cc') diff --git a/src/mem/packet.cc b/src/mem/packet.cc index bda5ff24f..b1e6dcd63 100644 --- a/src/mem/packet.cc +++ b/src/mem/packet.cc @@ -166,13 +166,13 @@ MemCmd::commandInfo[] = /* IntResp -- for interrupts */ { SET2(IsWrite, IsResponse), InvalidCmd, "MessageResp" }, /* ReleaseReq -- for release synchronization */ - { SET3(IsRelease, IsRequest, NeedsResponse), ReleaseResp, "ReleaseReq" }, + { SET2(IsRequest, NeedsResponse), ReleaseResp, "ReleaseReq" }, /* ReleaseResp -- for release synchronization */ - { SET2(IsRelease, IsResponse), InvalidCmd, "ReleaseResp" }, + { SET1(IsResponse), InvalidCmd, "ReleaseResp" }, /* AcquireReq -- for release synchronization */ - { SET3(IsAcquire, IsRequest, NeedsResponse), AcquireResp, "AcquireReq" }, + { SET2(IsRequest, NeedsResponse), AcquireResp, "AcquireReq" }, /* AcquireResp -- for release synchronization */ - { SET3(IsAcquire, IsResponse, NeedsResponse), InvalidCmd, "AcquireResp" }, + { SET2(IsResponse, NeedsResponse), InvalidCmd, "AcquireResp" }, /* InvalidDestError -- packet dest field invalid */ { SET2(IsResponse, IsError), InvalidCmd, "InvalidDestError" }, /* BadAddressError -- memory address invalid */ -- cgit v1.2.3