search

ARM-software / synchronization-benchmarks

Collection of synchronization micro-benchmarks and traces from infrastructure applications
Other
37 stars 36 forks source link

Add half-hour test timeout with sh(_timeout=1800) #39

Closed zoybai closed 6 years ago

zoybai commented 6 years ago

To prevent system live-lock with faulty configuration, we need to add timeout as a safety measure. Currently our longest test used about 23 minutes for swap_mutex with 200+ hyper-threads system, therefore we chose 30 minutes as the default sh timeout value. We can easily increase this value if future system has more cores and slower.

This parameter doesn't change successful test case's running time. Because this value should not be changed very often, therefore we used an internal value instead of a yaml config parameter.

geoffreyblake commented 6 years ago

Can one of the admins verify this patch?