Re: [PULL 2/3] tests/avocado: add timeout to the aspeed tests

2022-08-16 Thread Peter Maydell
On Tue, 16 Aug 2022 at 14:34, Alex Bennée wrote: > Peter Maydell writes: > > Is there some place (in the superclass??) that we can put a > > default timeout that applies to *all* avocado tests, so we > > don't have the risk of forgetting it in a particular test? > > It's a bit muddy. Most tests a

Re: [PULL 2/3] tests/avocado: add timeout to the aspeed tests

2022-08-16 Thread Alex Bennée
Peter Maydell writes: > On Tue, 16 Aug 2022 at 13:26, Alex Bennée wrote: >> >> On some systems the test can hang. At least defining a timeout stops >> it from hanging forever. > > Aha. Yeah, I've seen this test hang forever sometimes. > > Is there some place (in the superclass??) that we can p

Re: [PULL 2/3] tests/avocado: add timeout to the aspeed tests

2022-08-16 Thread Peter Maydell
On Tue, 16 Aug 2022 at 13:26, Alex Bennée wrote: > > On some systems the test can hang. At least defining a timeout stops > it from hanging forever. Aha. Yeah, I've seen this test hang forever sometimes. Is there some place (in the superclass??) that we can put a default timeout that applies to

[PULL 2/3] tests/avocado: add timeout to the aspeed tests

2022-08-16 Thread Alex Bennée
On some systems the test can hang. At least defining a timeout stops it from hanging forever. Signed-off-by: Alex Bennée Reviewed-by: Philippe Mathieu-Daudé Message-Id: <20220811151413.3350684-7-alex.ben...@linaro.org> diff --git a/tests/avocado/machine_aspeed.py b/tests/avocado/machine_aspeed.