Hoa Nguyen has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/39415 )

Change subject: dev,dev-arm: Add units to stats in /src/dev
......................................................................

dev,dev-arm: Add units to stats in /src/dev

Change-Id: I9d2449ef173f2f467717a9d233aef8d9a2f43f26
Signed-off-by: Hoa Nguyen <hoangu...@ucdavis.edu>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/39415
Tested-by: kokoro <noreply+kok...@google.com>
Reviewed-by: Daniel Carvalho <oda...@yahoo.com.br>
Reviewed-by: Giacomo Travaglini <giacomo.travagl...@arm.com>
Maintainer: Gabe Black <gabe.bl...@gmail.com>
Maintainer: Giacomo Travaglini <giacomo.travagl...@arm.com>
---
M src/dev/arm/flash_device.cc
M src/dev/arm/hdlcd.cc
M src/dev/arm/smmu_v3.cc
M src/dev/arm/smmu_v3_caches.cc
M src/dev/arm/ufs_device.cc
M src/dev/net/etherdevice.cc
M src/dev/net/sinic.cc
M src/dev/pci/copy_engine.cc
M src/dev/storage/ide_disk.cc
9 files changed, 174 insertions(+), 93 deletions(-)

Approvals:
  Giacomo Travaglini: Looks good to me, approved; Looks good to me, approved
  Daniel Carvalho: Looks good to me, approved
  Gabe Black: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/dev/arm/flash_device.cc b/src/dev/arm/flash_device.cc
index 533b3ed..a45ade0 100644
--- a/src/dev/arm/flash_device.cc
+++ b/src/dev/arm/flash_device.cc
@@ -458,12 +458,14 @@
 FlashDevice::
 FlashDeviceStats::FlashDeviceStats(Stats::Group *parent)
     : Stats::Group(parent, "FlashDevice"),
- ADD_STAT(totalGCActivations, "Number of Garbage collector activations"),
-    ADD_STAT(writeAccess, "Histogram of write addresses"),
-    ADD_STAT(readAccess, "Histogram of read addresses"),
-    ADD_STAT(fileSystemAccess, "Histogram of file system accesses"),
-    ADD_STAT(writeLatency, "Histogram of write latency"),
-    ADD_STAT(readLatency, "Histogram of read latency")
+    ADD_STAT(totalGCActivations, UNIT_COUNT,
+             "Number of Garbage collector activations"),
+    ADD_STAT(writeAccess, UNIT_COUNT, "Histogram of write addresses"),
+    ADD_STAT(readAccess, UNIT_COUNT, "Histogram of read addresses"),
+    ADD_STAT(fileSystemAccess, UNIT_COUNT,
+             "Histogram of file system accesses"),
+    ADD_STAT(writeLatency, UNIT_TICK, "Histogram of write latency"),
+    ADD_STAT(readLatency, UNIT_TICK, "Histogram of read latency")
 {
     using namespace Stats;

diff --git a/src/dev/arm/hdlcd.cc b/src/dev/arm/hdlcd.cc
index c6ff6db..85230be 100644
--- a/src/dev/arm/hdlcd.cc
+++ b/src/dev/arm/hdlcd.cc
@@ -81,7 +81,7 @@
 HDLcd::
 HDLcdStats::HDLcdStats(Stats::Group *parent)
     : Stats::Group(parent, "HDLcd"),
-      ADD_STAT(underruns, "Number of buffer underruns")
+      ADD_STAT(underruns, UNIT_COUNT, "Number of buffer underruns")
 {
     using namespace Stats;

diff --git a/src/dev/arm/smmu_v3.cc b/src/dev/arm/smmu_v3.cc
index 3076f5e..95712ed 100644
--- a/src/dev/arm/smmu_v3.cc
+++ b/src/dev/arm/smmu_v3.cc
@@ -746,12 +746,12 @@

 SMMUv3::SMMUv3Stats::SMMUv3Stats(Stats::Group *parent)
     : Stats::Group(parent),
-      ADD_STAT(steL1Fetches, "STE L1 fetches"),
-      ADD_STAT(steFetches, "STE fetches"),
-      ADD_STAT(cdL1Fetches, "CD L1 fetches"),
-      ADD_STAT(cdFetches, "CD fetches"),
-      ADD_STAT(translationTimeDist, "Time to translate address"),
-      ADD_STAT(ptwTimeDist, "Time to walk page tables")
+      ADD_STAT(steL1Fetches, UNIT_COUNT, "STE L1 fetches"),
+      ADD_STAT(steFetches, UNIT_COUNT, "STE fetches"),
+      ADD_STAT(cdL1Fetches, UNIT_COUNT, "CD L1 fetches"),
+      ADD_STAT(cdFetches, UNIT_COUNT, "CD fetches"),
+ ADD_STAT(translationTimeDist, UNIT_TICK, "Time to translate address"),
+      ADD_STAT(ptwTimeDist, UNIT_TICK, "Time to walk page tables")
 {
     using namespace Stats;

diff --git a/src/dev/arm/smmu_v3_caches.cc b/src/dev/arm/smmu_v3_caches.cc
index b8f284d..8bea3ea 100644
--- a/src/dev/arm/smmu_v3_caches.cc
+++ b/src/dev/arm/smmu_v3_caches.cc
@@ -84,14 +84,21 @@
 SMMUv3BaseCache::
 SMMUv3BaseCacheStats::SMMUv3BaseCacheStats(Stats::Group *parent)
     : Stats::Group(parent),
-      ADD_STAT(averageLookups, "Average number lookups per second"),
-      ADD_STAT(totalLookups, "Total number of lookups"),
-      ADD_STAT(averageMisses, "Average number misses per second"),
-      ADD_STAT(totalMisses, "Total number of misses"),
-      ADD_STAT(averageUpdates, "Average number updates per second"),
-      ADD_STAT(totalUpdates, "Total number of updates"),
-      ADD_STAT(averageHitRate, "Average hit rate"),
-      ADD_STAT(insertions, "Number of insertions (not replacements)")
+      ADD_STAT(averageLookups,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Second),
+               "Average number lookups per second"),
+      ADD_STAT(totalLookups, UNIT_COUNT, "Total number of lookups"),
+      ADD_STAT(averageMisses,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Second),
+               "Average number misses per second"),
+      ADD_STAT(totalMisses, UNIT_COUNT, "Total number of misses"),
+      ADD_STAT(averageUpdates,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Second),
+               "Average number updates per second"),
+      ADD_STAT(totalUpdates, UNIT_COUNT, "Total number of updates"),
+      ADD_STAT(averageHitRate, UNIT_RATIO, "Average hit rate"),
+      ADD_STAT(insertions, UNIT_COUNT,
+               "Number of insertions (not replacements)")
 {
     using namespace Stats;

diff --git a/src/dev/arm/ufs_device.cc b/src/dev/arm/ufs_device.cc
index e04cb39..f7b6d82 100644
--- a/src/dev/arm/ufs_device.cc
+++ b/src/dev/arm/ufs_device.cc
@@ -756,35 +756,56 @@
 UFSHostDevice::
 UFSHostDeviceStats::UFSHostDeviceStats(UFSHostDevice *parent)
     : Stats::Group(parent, "UFSDiskHost"),
-      ADD_STAT(currentSCSIQueue,
+      ADD_STAT(currentSCSIQueue, UNIT_COUNT,
                "Most up to date length of the command queue"),
-      ADD_STAT(currentReadSSDQueue,
+      ADD_STAT(currentReadSSDQueue, UNIT_COUNT,
                "Most up to date length of the read SSD queue"),
-      ADD_STAT(currentWriteSSDQueue,
+      ADD_STAT(currentWriteSSDQueue, UNIT_COUNT,
                "Most up to date length of the write SSD queue"),
       /** Amount of data read/written */
-      ADD_STAT(totalReadSSD, "Number of bytes read from SSD"),
-      ADD_STAT(totalWrittenSSD, "Number of bytes written to SSD"),
- ADD_STAT(totalReadDiskTransactions,"Number of transactions from disk"), - ADD_STAT(totalWriteDiskTransactions, "Number of transactions to disk"), - ADD_STAT(totalReadUFSTransactions, "Number of transactions from device"), - ADD_STAT(totalWriteUFSTransactions, "Number of transactions to device"),
+      ADD_STAT(totalReadSSD, UNIT_BYTE,
+               "Number of bytes read from SSD"),
+      ADD_STAT(totalWrittenSSD, UNIT_BYTE,
+               "Number of bytes written to SSD"),
+      ADD_STAT(totalReadDiskTransactions, UNIT_COUNT,
+               "Number of transactions from disk"),
+      ADD_STAT(totalWriteDiskTransactions, UNIT_COUNT,
+               "Number of transactions to disk"),
+      ADD_STAT(totalReadUFSTransactions, UNIT_COUNT,
+               "Number of transactions from device"),
+      ADD_STAT(totalWriteUFSTransactions, UNIT_COUNT,
+               "Number of transactions to device"),
       /** Average bandwidth for reads and writes */
-      ADD_STAT(averageReadSSDBW, "Average read bandwidth (bytes/s)",
+      ADD_STAT(averageReadSSDBW,
+               UNIT_RATE(Stats::Units::Byte, Stats::Units::Second),
+               "Average read bandwidth (bytes/s)",
                totalReadSSD / simSeconds),
-      ADD_STAT(averageWriteSSDBW, "Average write bandwidth (bytes/s)",
+      ADD_STAT(averageWriteSSDBW,
+               UNIT_RATE(Stats::Units::Byte, Stats::Units::Second),
+               "Average write bandwidth (bytes/s)",
                totalWrittenSSD / simSeconds),
-      ADD_STAT(averageSCSIQueue, "Average command queue length"),
-      ADD_STAT(averageReadSSDQueue, "Average read queue length"),
-      ADD_STAT(averageWriteSSDQueue, "Average write queue length"),
+      ADD_STAT(averageSCSIQueue,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Tick),
+               "Average command queue length"),
+      ADD_STAT(averageReadSSDQueue,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Tick),
+               "Average read queue length"),
+      ADD_STAT(averageWriteSSDQueue,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Tick),
+               "Average write queue length"),
       /** Number of doorbells rung*/
-      ADD_STAT(curDoorbell, "Most up to date number of doorbells used",
+      ADD_STAT(curDoorbell, UNIT_COUNT,
+               "Most up to date number of doorbells used",
                parent->activeDoorbells),
-      ADD_STAT(maxDoorbell, "Maximum number of doorbells utilized"),
-      ADD_STAT(averageDoorbell, "Average number of Doorbells used"),
+      ADD_STAT(maxDoorbell, UNIT_COUNT,
+               "Maximum number of doorbells utilized"),
+      ADD_STAT(averageDoorbell,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Tick),
+               "Average number of Doorbells used"),
       /** Latency*/
-      ADD_STAT(transactionLatency, "Histogram of transaction times"),
-      ADD_STAT(idleTimes, "Histogram of idle times")
+      ADD_STAT(transactionLatency, UNIT_TICK,
+               "Histogram of transaction times"),
+      ADD_STAT(idleTimes, UNIT_TICK, "Histogram of idle times")
 {
     using namespace Stats;

diff --git a/src/dev/net/etherdevice.cc b/src/dev/net/etherdevice.cc
index e279a9c..7c817f4 100644
--- a/src/dev/net/etherdevice.cc
+++ b/src/dev/net/etherdevice.cc
@@ -32,78 +32,121 @@

 EtherDevice::EtherDeviceStats::EtherDeviceStats(Stats::Group *parent)
     : Stats::Group(parent, "EtherDevice"),
-      ADD_STAT(postedInterrupts, "Number of posts to CPU"),
-      ADD_STAT(txBytes, "Bytes Transmitted"),
-      ADD_STAT(rxBytes, "Bytes Received"),
-      ADD_STAT(txPackets, "Number of Packets Transmitted"),
-      ADD_STAT(rxPackets, "Number of Packets Received"),
-      ADD_STAT(txBandwidth, "Transmit Bandwidth (bits/s)",
+      ADD_STAT(postedInterrupts, UNIT_COUNT, "Number of posts to CPU"),
+      ADD_STAT(txBytes, UNIT_BYTE, "Bytes Transmitted"),
+      ADD_STAT(rxBytes, UNIT_BYTE, "Bytes Received"),
+      ADD_STAT(txPackets, UNIT_COUNT, "Number of Packets Transmitted"),
+      ADD_STAT(rxPackets, UNIT_COUNT, "Number of Packets Received"),
+ ADD_STAT(txBandwidth, UNIT_RATE(Stats::Units::Bit, Stats::Units::Second),
+               "Transmit Bandwidth (bits/s)",
                txBytes * Stats::constant(8) / simSeconds),
-      ADD_STAT(rxBandwidth, "Receive Bandwidth (bits/s)",
+ ADD_STAT(rxBandwidth, UNIT_RATE(Stats::Units::Bit, Stats::Units::Second),
+               "Receive Bandwidth (bits/s)",
                rxBytes * Stats::constant(8) / simSeconds),
-      ADD_STAT(txIpChecksums, "Number of tx IP Checksums done by device"),
-      ADD_STAT(rxIpChecksums, "Number of rx IP Checksums done by device"),
- ADD_STAT(txTcpChecksums, "Number of tx TCP Checksums done by device"), - ADD_STAT(rxTcpChecksums, "Number of rx TCP Checksums done by device"), - ADD_STAT(txUdpChecksums, "Number of tx UDP Checksums done by device"), - ADD_STAT(rxUdpChecksums, "Number of rx UDP Checksums done by device"), - ADD_STAT(descDmaReads, "Number of descriptors the device read w/ DMA"), - ADD_STAT(descDmaWrites, "Number of descriptors the device wrote w/ DMA"),
-      ADD_STAT(descDmaRdBytes, "Number of descriptor bytes read w/ DMA"),
-      ADD_STAT(descDmaWrBytes, "Number of descriptor bytes write w/ DMA"),
-      ADD_STAT(totBandwidth, "Total Bandwidth (bits/s)",
+      ADD_STAT(txIpChecksums, UNIT_COUNT,
+               "Number of tx IP Checksums done by device"),
+      ADD_STAT(rxIpChecksums, UNIT_COUNT,
+               "Number of rx IP Checksums done by device"),
+      ADD_STAT(txTcpChecksums, UNIT_COUNT,
+               "Number of tx TCP Checksums done by device"),
+      ADD_STAT(rxTcpChecksums, UNIT_COUNT,
+               "Number of rx TCP Checksums done by device"),
+      ADD_STAT(txUdpChecksums, UNIT_COUNT,
+               "Number of tx UDP Checksums done by device"),
+      ADD_STAT(rxUdpChecksums, UNIT_COUNT,
+               "Number of rx UDP Checksums done by device"),
+      ADD_STAT(descDmaReads, UNIT_COUNT,
+               "Number of descriptors the device read w/ DMA"),
+      ADD_STAT(descDmaWrites, UNIT_COUNT,
+               "Number of descriptors the device wrote w/ DMA"),
+      ADD_STAT(descDmaRdBytes, UNIT_COUNT,
+               "Number of descriptor bytes read w/ DMA"),
+      ADD_STAT(descDmaWrBytes, UNIT_COUNT,
+               "Number of descriptor bytes write w/ DMA"),
+      ADD_STAT(totBandwidth,
+               UNIT_RATE(Stats::Units::Bit, Stats::Units::Second),
+               "Total Bandwidth (bits/s)",
                txBandwidth + rxBandwidth),
-      ADD_STAT(totPackets, "Total Packets", txPackets + rxPackets),
-      ADD_STAT(totBytes, "Total Bytes", txBytes + rxBytes),
-      ADD_STAT(totPacketRate, "Total Tranmission Rate (packets/s)",
+ ADD_STAT(totPackets, UNIT_COUNT, "Total Packets", txPackets + rxPackets),
+      ADD_STAT(totBytes, UNIT_BYTE, "Total Bytes", txBytes + rxBytes),
+      ADD_STAT(totPacketRate,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Second),
+               "Total Packet Tranmission Rate",
                totPackets / simSeconds),
-      ADD_STAT(txPacketRate, "Packet Tranmission Rate (packets/s)",
+      ADD_STAT(txPacketRate,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Second),
+               "Packet Tranmission Rate",
                txPackets / simSeconds),
-      ADD_STAT(rxPacketRate, "Packet Reception Rate (packets/s)",
+      ADD_STAT(rxPacketRate,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Second),
+               "Packet Reception Rate",
                rxPackets / simSeconds),
-      ADD_STAT(postedSwi, "Number of software interrupts posted to CPU"),
-      ADD_STAT(totalSwi, "Total number of Swi written to ISR"),
+      ADD_STAT(postedSwi, UNIT_COUNT,
+               "Number of software interrupts posted to CPU"),
+      ADD_STAT(totalSwi, UNIT_COUNT, "Total number of Swi written to ISR"),
       ADD_STAT(coalescedSwi,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Count),
                "Average number of Swi's coalesced into each post",
                totalSwi / postedInterrupts),
-      ADD_STAT(postedRxIdle,"Number of rxIdle interrupts posted to CPU"),
-      ADD_STAT(totalRxIdle, "Total number of RxIdle written to ISR"),
+      ADD_STAT(postedRxIdle, UNIT_COUNT,
+               "Number of rxIdle interrupts posted to CPU"),
+      ADD_STAT(totalRxIdle, UNIT_COUNT,
+               "Total number of RxIdle written to ISR"),
       ADD_STAT(coalescedRxIdle,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Count),
                "Average number of RxIdle's coalesced into each post",
                totalRxIdle / postedInterrupts),
-      ADD_STAT(postedRxOk, "Number of RxOk interrupts posted to CPU"),
-      ADD_STAT(totalRxOk, "Total number of RxOk written to ISR"),
+      ADD_STAT(postedRxOk, UNIT_COUNT,
+               "Number of RxOk interrupts posted to CPU"),
+ ADD_STAT(totalRxOk, UNIT_COUNT, "Total number of RxOk written to ISR"),
       ADD_STAT(coalescedRxOk,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Count),
                "Average number of RxOk's coalesced into each post",
                totalRxOk / postedInterrupts),
-      ADD_STAT(postedRxDesc, "Number of RxDesc interrupts posted to CPU"),
-      ADD_STAT(totalRxDesc, "Total number of RxDesc written to ISR"),
+      ADD_STAT(postedRxDesc, UNIT_COUNT,
+               "Number of RxDesc interrupts posted to CPU"),
+      ADD_STAT(totalRxDesc, UNIT_COUNT,
+               "Total number of RxDesc written to ISR"),
       ADD_STAT(coalescedRxDesc,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Count),
                "Average number of RxDesc's coalesced into each post",
                totalRxDesc / postedInterrupts),
-      ADD_STAT(postedTxOk, "Number of TxOk interrupts posted to CPU"),
-      ADD_STAT(totalTxOk, "Total number of TxOk written to ISR"),
+      ADD_STAT(postedTxOk, UNIT_COUNT,
+               "Number of TxOk interrupts posted to CPU"),
+      ADD_STAT(totalTxOk, UNIT_COUNT,
+               "Total number of TxOk written to ISR"),
       ADD_STAT(coalescedTxOk,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Count),
                "Average number of TxOk's coalesced into each post",
                totalTxOk / postedInterrupts),
-      ADD_STAT(postedTxIdle, "Number of TxIdle interrupts posted to CPU"),
-      ADD_STAT(totalTxIdle, "Total number of TxIdle written to ISR"),
+      ADD_STAT(postedTxIdle, UNIT_COUNT,
+               "Number of TxIdle interrupts posted to CPU"),
+      ADD_STAT(totalTxIdle, UNIT_COUNT,
+               "Total number of TxIdle written to ISR"),
       ADD_STAT(coalescedTxIdle,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Count),
                "Average number of TxIdle's coalesced into each post",
                totalTxIdle / postedInterrupts),
-      ADD_STAT(postedTxDesc, "Number of TxDesc interrupts posted to CPU"),
-      ADD_STAT(totalTxDesc, "Total number of TxDesc written to ISR"),
+      ADD_STAT(postedTxDesc, UNIT_COUNT,
+               "Number of TxDesc interrupts posted to CPU"),
+      ADD_STAT(totalTxDesc, UNIT_COUNT,
+               "Total number of TxDesc written to ISR"),
       ADD_STAT(coalescedTxDesc,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Count),
                "Average number of TxDesc's coalesced into each post",
                totalTxDesc / postedInterrupts),
-      ADD_STAT(postedRxOrn, "Number of RxOrn posted to CPU"),
-      ADD_STAT(totalRxOrn, "Total number of RxOrn written to ISR"),
+      ADD_STAT(postedRxOrn, UNIT_COUNT,
+               "Number of RxOrn posted to CPU"),
+      ADD_STAT(totalRxOrn, UNIT_COUNT,
+               "Total number of RxOrn written to ISR"),
       ADD_STAT(coalescedRxOrn,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Count),
                "Average number of RxOrn's coalesced into each post",
                totalRxOrn / postedInterrupts),
       ADD_STAT(coalescedTotal,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Count),
                "Average number of interrupts coalesced into each post"),
-      ADD_STAT(droppedPackets, "Number of packets dropped")
+      ADD_STAT(droppedPackets, UNIT_COUNT, "Number of packets dropped")
 {

     postedInterrupts
diff --git a/src/dev/net/sinic.cc b/src/dev/net/sinic.cc
index f270487..0607d26 100644
--- a/src/dev/net/sinic.cc
+++ b/src/dev/net/sinic.cc
@@ -98,11 +98,14 @@

 Device::DeviceStats::DeviceStats(Stats::Group *parent)
     : Stats::Group(parent, "SinicDevice"),
-      ADD_STAT(totalVnicDistance, "Total vnic distance"),
-      ADD_STAT(numVnicDistance, "Number of vnic distance measurements"),
-      ADD_STAT(maxVnicDistance, "Maximum vnic distance"),
-      ADD_STAT(avgVnicDistance, "Average vnic distance",
-               totalVnicDistance / numVnicDistance),
+      ADD_STAT(totalVnicDistance, UNIT_COUNT,
+               "Total vnic distance"),
+      ADD_STAT(numVnicDistance, UNIT_COUNT,
+               "Number of vnic distance measurements"),
+      ADD_STAT(maxVnicDistance, UNIT_COUNT, "Maximum vnic distance"),
+      ADD_STAT(avgVnicDistance,
+               UNIT_RATE(Stats::Units::Count, Stats::Units::Count),
+ "Average vnic distance", totalVnicDistance / numVnicDistance),
       _maxVnicDistance(0)
 {
 }
diff --git a/src/dev/pci/copy_engine.cc b/src/dev/pci/copy_engine.cc
index 02d3d86..48326cf 100644
--- a/src/dev/pci/copy_engine.cc
+++ b/src/dev/pci/copy_engine.cc
@@ -430,8 +430,10 @@
 CopyEngineStats::CopyEngineStats(Stats::Group *parent,
                                  const uint8_t &channel_count)
     : Stats::Group(parent, "CopyEngine"),
-      ADD_STAT(bytesCopied, "Number of bytes copied by each engine"),
- ADD_STAT(copiesProcessed, "Number of copies processed by each engine")
+      ADD_STAT(bytesCopied, UNIT_BYTE,
+               "Number of bytes copied by each engine"),
+      ADD_STAT(copiesProcessed, UNIT_COUNT,
+               "Number of copies processed by each engine")
 {
     bytesCopied
         .init(channel_count)
diff --git a/src/dev/storage/ide_disk.cc b/src/dev/storage/ide_disk.cc
index 808c705..63742b4 100644
--- a/src/dev/storage/ide_disk.cc
+++ b/src/dev/storage/ide_disk.cc
@@ -390,14 +390,17 @@
 IdeDisk::
 IdeDiskStats::IdeDiskStats(Stats::Group *parent)
     : Stats::Group(parent, "IdeDisk"),
-      ADD_STAT(dmaReadFullPages,
+      ADD_STAT(dmaReadFullPages, UNIT_COUNT,
                "Number of full page size DMA reads (not PRD)."),
-      ADD_STAT(dmaReadBytes,
+      ADD_STAT(dmaReadBytes, UNIT_BYTE,
                "Number of bytes transfered via DMA reads (not PRD)."),
-      ADD_STAT(dmaReadTxs, "Number of DMA read transactions (not PRD)."),
-      ADD_STAT(dmaWriteFullPages, "Number of full page size DMA writes."),
- ADD_STAT(dmaWriteBytes, "Number of bytes transfered via DMA writes."),
-      ADD_STAT(dmaWriteTxs, "Number of DMA write transactions.")
+      ADD_STAT(dmaReadTxs, UNIT_COUNT,
+               "Number of DMA read transactions (not PRD)."),
+      ADD_STAT(dmaWriteFullPages, UNIT_COUNT,
+               "Number of full page size DMA writes."),
+      ADD_STAT(dmaWriteBytes, UNIT_BYTE,
+               "Number of bytes transfered via DMA writes."),
+ ADD_STAT(dmaWriteTxs, UNIT_COUNT, "Number of DMA write transactions.")
 {
 }




The change was submitted with unreviewed changes in the following files:

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/39415
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: I9d2449ef173f2f467717a9d233aef8d9a2f43f26
Gerrit-Change-Number: 39415
Gerrit-PatchSet: 21
Gerrit-Owner: Hoa Nguyen <hoangu...@ucdavis.edu>
Gerrit-Reviewer: Andreas Sandberg <andreas.sandb...@arm.com>
Gerrit-Reviewer: Daniel Carvalho <oda...@yahoo.com.br>
Gerrit-Reviewer: Gabe Black <gabe.bl...@gmail.com>
Gerrit-Reviewer: Giacomo Travaglini <giacomo.travagl...@arm.com>
Gerrit-Reviewer: Hoa Nguyen <hoangu...@ucdavis.edu>
Gerrit-Reviewer: kokoro <noreply+kok...@google.com>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to