From fc28ded7a9217c85eb3c68871e2c9527bffc6f67 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Petr=20P=C3=ADsa=C5=99?= Date: Wed, 11 May 2011 15:30:51 +0200 Subject: [PATCH] Fix maximal RSS report --- ...7-ru_maxrss-is-in-kilobytes-on-Linux.patch | 32 +++++++++++++++++++ time.spec | 7 +++- 2 files changed, 38 insertions(+), 1 deletion(-) create mode 100644 time-1.7-ru_maxrss-is-in-kilobytes-on-Linux.patch diff --git a/time-1.7-ru_maxrss-is-in-kilobytes-on-Linux.patch b/time-1.7-ru_maxrss-is-in-kilobytes-on-Linux.patch new file mode 100644 index 0000000..1f68f9b --- /dev/null +++ b/time-1.7-ru_maxrss-is-in-kilobytes-on-Linux.patch @@ -0,0 +1,32 @@ +From ad24a929bdcc15abae14a64ea21b821bcd8cb030 Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Petr=20P=C3=ADsa=C5=99?= +Date: Wed, 11 May 2011 15:19:11 +0200 +Subject: [PATCH] ru_maxrss is in kilobytes on Linux + +Since 2.6.32 Linux returns ru_maxrss in kilobytes. Not in pages. + +See http://lists.gnu.org/archive/html/bug-gnu-utils/2008-12/msg00047.html +for discussion. +--- + time.c | 4 ++++ + 1 files changed, 4 insertions(+), 0 deletions(-) + +diff --git a/time.c b/time.c +index d15fee4..43aec0b 100644 +--- a/time.c ++++ b/time.c +@@ -395,7 +395,11 @@ summarize (fp, fmt, command, resp) + ptok ((UL) resp->ru.ru_ixrss) / MSEC_TO_TICKS (v)); + break; + case 'M': /* Maximum resident set size. */ ++#ifdef __linux__ ++ fprintf (fp, "%ld", resp->ru.ru_maxrss); ++#else + fprintf (fp, "%lu", ptok ((UL) resp->ru.ru_maxrss)); ++#endif + break; + case 'O': /* Outputs. */ + fprintf (fp, "%ld", resp->ru.ru_oublock); +-- +1.7.4.4 + diff --git a/time.spec b/time.spec index ac0716a..133738f 100644 --- a/time.spec +++ b/time.spec @@ -1,13 +1,14 @@ Summary: A GNU utility for monitoring a program's use of system resources Name: time Version: 1.7 -Release: 38%{?dist} +Release: 39%{?dist} License: GPLv2+ Group: Applications/System Url: http://www.gnu.org/software/time/ Source: ftp://prep.ai.mit.edu/pub/gnu/%{name}/%{name}-%{version}.tar.gz Patch0: time-1.7-destdir.patch Patch1: time-1.7-verbose.patch +Patch2: time-1.7-ru_maxrss-is-in-kilobytes-on-Linux.patch BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) Requires(post): /sbin/install-info Requires(preun): /sbin/install-info @@ -21,6 +22,7 @@ the results. %setup -q %patch0 -p1 %patch1 -p1 +%patch2 -p1 -b .ru_maxrss %build echo "ac_cv_func_wait3=\${ac_cv_func_wait3='yes'}" >> config.cache @@ -51,6 +53,9 @@ fi %{_infodir}/time.info* %changelog +* Wed May 11 2011 Petr Pisar - 1.7-39 +- Fix maximal RSS report (bug #702826) + * Wed Feb 09 2011 Fedora Release Engineering - 1.7-38 - Rebuilt for https://fedoraproject.org/wiki/Fedora_15_Mass_Rebuild