mem: Bug fix in write merge logic

Information
Submitter:	Rizwana Begum
Repository:	gem5
Branch:	default
Bugs:
Depends On:
Reviewers
Groups:	Default
People:

Description

Changeset 10782:5fdf351e41e1
---------------------------
mem: Bug fix in write merge logic

This patch fixes the write merge logic. Previously, new write
requests were merged with the existing writes across burst
boundaries. This patch ensures that only writes falling within a
burst boundary are merged.

Testing Done

This patch changes stats. Number of writes merged are less due to this patch, which impacts memory and systems stats.

Ship It!

Thanks for the fix. I'm ok with this going in, but I have a simplification that I think we should consider instead. Let me post if and see what you think.

Andreas Hansson April 24, 2015, 9:33 a.m. (April 24, 2015, 9:33 a.m.)
```
http://reviews.gem5.org/r/2746/
Let me know what you think.
```
Andreas Hansson April 30, 2015, 12:36 a.m. (April 30, 2015, 12:36 a.m.)
```
Shall we discard this patch in favour of the other one?
```
Rizwana Begum April 30, 2015, 10:37 a.m. (April 30, 2015, 10:37 a.m.)
```
Yes.
```

You have a pending review.

Review Board 2.0.15

This change has been discarded.

Screenshots

Files

Status: Discarded