stats: Create HDF5 stat files relative to simout
authorAndreas Sandberg <andreas.sandberg@arm.com>
Fri, 30 Aug 2019 12:59:59 +0000 (13:59 +0100)
committerAndreas Sandberg <andreas.sandberg@arm.com>
Mon, 2 Sep 2019 12:20:12 +0000 (12:20 +0000)
When using the HDF5 stats backend, we currently pass the filename from
the command line straight to the HDF5 library. This behaviour is
different from the expected behaviour when using plain text stat files
where relative paths are resolved relative to simout. This change adds
support for resolving hdf5 paths relative to simout.

Change-Id: I09cb0b7468735e697125eda7f04c5a85725ca8d0
Signed-off-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/20508
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Jason Lowe-Power <jason@lowepower.com>

src/base/stats/hdf5.cc

index 1e11d56278cf5f97355dcfd07b11fda37ce53281..0345c8c62d362911a0dd956161d6ca5ebed2e483 100644 (file)
@@ -319,7 +319,7 @@ initHDF5(const std::string &filename, unsigned chunking,
          bool desc, bool formulas)
 {
     return  std::unique_ptr<Output>(
-        new Hdf5(filename, chunking, desc, formulas));
+        new Hdf5(simout.resolve(filename), chunking, desc, formulas));
 }
 
 }; // namespace Stats