Skip to content

Commit e6ff30a

Browse files
author
Matthias Koefferlein
committed
Modifying definition of Timer#memory_size on Linux to become resident size.
1 parent db728dc commit e6ff30a

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

src/gsi/gsi/gsiDeclTl.cc

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,11 @@ Class<tl::Timer> decl_Timer ("tl", "Timer",
147147
gsi::method ("memory_size", &tl::Timer::memory_size,
148148
"@brief Gets the current memory usage of the process in Bytes\n"
149149
"\n"
150-
"This method has been introduced in version 0.27."
150+
"The returned value is the resident memory size on Linux and MacOS and "
151+
"the working set size on Windows.\n"
152+
"\n"
153+
"This method has been introduced in version 0.27. The value has been changed "
154+
"to be resident size (instead of virtual size) on Linux in version 0.30."
151155
) +
152156
gsi::method ("user", &tl::Timer::sec_user,
153157
"@brief Returns the elapsed CPU time in user mode from start to stop in seconds\n"

src/tl/tl/tlTimer.cc

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -245,8 +245,8 @@ Timer::memory_size ()
245245
"%*d " // 0
246246
"%*d " // itrealvalue
247247
"%*u " // starttime
248-
"%lu " // vsize
249-
"%*d " // rss
248+
"%*u " // vsize
249+
"%lu " // rss
250250
"%*u " // rlim
251251
"%*u " // startcode
252252
"%*u " // endcode
@@ -270,7 +270,8 @@ Timer::memory_size ()
270270
memsize = 0;
271271
}
272272
}
273-
return size_t (memsize);
273+
274+
return size_t (memsize) * size_t (getpagesize ());
274275

275276
#else
276277
# error Unsupported platform

0 commit comments

Comments
 (0)