Re: Ping: [PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

2015-12-09 Thread Leon Romanovsky
On Wed, Dec 09, 2015 at 10:11:55AM +0100, Hans Westgaard Ry wrote: > Sorry but I didn't see that "something" gobbled part of the mail. > The correct lines are (as in the original): > > src/perftest_resources.c | 22 -- > 1 file changed, 16 insertions(+), 6 deletions(-) > >

Re: Ping: [PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

2015-12-09 Thread Hans Westgaard Ry
Sorry but I didn't see that "something" gobbled part of the mail. The correct lines are (as in the original): src/perftest_resources.c | 22 -- 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/src/perftest_resources.c b/src/perftest_resources.c index

Ping: [PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

2015-12-09 Thread Hans Westgaard Ry
Any comments on this patch ? On 11/18/2015 01:27 PM, Hans Westgaard Ry wrote: The test uses a signal-handler to periodically print a line of report. Snip libc info: 24.4.6 Signal Handling and Nonreentrant Functions Handler functions usually don't do very much. The best practice is to write

Re: Ping: [PATCH 1/1] Fix deadlock when running run_iter_bw_infinitely

2015-12-09 Thread Leon Romanovsky
On Wed, Dec 09, 2015 at 09:33:11AM +0100, Hans Westgaard Ry wrote: > Any comments on this patch ? > > > > On 11/18/2015 01:27 PM, Hans Westgaard Ry wrote: > >The test uses a signal-handler to periodically print a line of report. > >+if (trig_print_report_bw) { > >+