From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cyril Hrubis Date: Fri, 9 Mar 2018 13:04:14 +0100 Subject: [LTP] [PATCH] mallocstress: extend test time on arm In-Reply-To: <2012939043.6317537.1520592935484.JavaMail.zimbra@redhat.com> References: <981e2a8523db8616a4ed442635a493928376489d.1520589632.git.jstancek@redhat.com> <20180309104146.GA28698@rei> <2012939043.6317537.1520592935484.JavaMail.zimbra@redhat.com> Message-ID: <20180309120414.GD24656@rei> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ltp@lists.linux.it Hi! > Just wanted to go with minimal impact on other arches (since they worked > OK so far as I'm aware). I don't have strong opinion either way, > I can make the timeout unconditional. Given that the test seems to take 4m on successful run 5m timeout seems to be a bit too strict to me. We are not primarily benchmarking the kernel, so these timeouts are better to be set to be double of the expecte runtime. Also I like to avoid #ifdefs whenever possible. -- Cyril Hrubis chrubis@suse.cz