dev: Remove the return type from DmaPort::dmaAction.
authorGabe Black <gabe.black@gmail.com>
Thu, 24 Dec 2020 11:17:29 +0000 (03:17 -0800)
committerGabe Black <gabe.black@gmail.com>
Wed, 6 Jan 2021 06:33:50 +0000 (06:33 +0000)
commit131ae4a2eb42724f320da113532c47c580502fc0
tree4d4ceba082336b26800312b73dbf73c63efa0d58
parente5c8f03b21a906c3413d0236f9154690d48741a4
dev: Remove the return type from DmaPort::dmaAction.

This function had a comment claiming that returning an arbitrary request
from the call was necessary for page table walker statistics, but
looking at the actual code, the return type was never used. Also
returning whatever the last request happens to be seems arbitrary, and a
bad boundary for modularization. The page table walker should not depend
on the internal implementation of the DMA port.

Change-Id: I00281fbaf6aeb85b15baf54f3d4a23ca1ac72b8b
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/38716
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Maintainer: Andreas Sandberg <andreas.sandberg@arm.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/dev/dma_device.cc
src/dev/dma_device.hh