Re: [PATCH 03/10] tests/avocado/intel_iommu.py: increase timeout

2023-12-14 Thread Eric Auger
On 12/14/23 10:41, Alex Bennée wrote: > Eric Auger writes: > >> Hi Cleber, >> >> On 12/13/23 21:08, Cleber Rosa wrote: >>> Alex Bennée writes: >>> Cleber Rosa writes: > Based on many runs, the average run time for these 4 tests is around > 250 seconds, with 320 seconds being

Re: [PATCH 03/10] tests/avocado/intel_iommu.py: increase timeout

2023-12-14 Thread Alex Bennée
Eric Auger writes: > Hi Cleber, > > On 12/13/23 21:08, Cleber Rosa wrote: >> Alex Bennée writes: >> >>> Cleber Rosa writes: >>> Based on many runs, the average run time for these 4 tests is around 250 seconds, with 320 seconds being the ceiling. In any way, the default 120 secon

Re: [PATCH 03/10] tests/avocado/intel_iommu.py: increase timeout

2023-12-14 Thread Philippe Mathieu-Daudé
On 14/12/23 08:24, Eric Auger wrote: Hi Cleber, On 12/13/23 21:08, Cleber Rosa wrote: Alex Bennée writes: Cleber Rosa writes: Based on many runs, the average run time for these 4 tests is around 250 seconds, with 320 seconds being the ceiling. In any way, the default 120 seconds timeout

Re: [PATCH 03/10] tests/avocado/intel_iommu.py: increase timeout

2023-12-13 Thread Eric Auger
Hi Cleber, On 12/13/23 21:08, Cleber Rosa wrote: > Alex Bennée writes: > >> Cleber Rosa writes: >> >>> Based on many runs, the average run time for these 4 tests is around >>> 250 seconds, with 320 seconds being the ceiling. In any way, the >>> default 120 seconds timeout is inappropriate in my

Re: [PATCH 03/10] tests/avocado/intel_iommu.py: increase timeout

2023-12-13 Thread Cleber Rosa
Alex Bennée writes: > Cleber Rosa writes: > >> Based on many runs, the average run time for these 4 tests is around >> 250 seconds, with 320 seconds being the ceiling. In any way, the >> default 120 seconds timeout is inappropriate in my experience. > > I would rather see these tests updated to

Re: [PATCH 03/10] tests/avocado/intel_iommu.py: increase timeout

2023-12-12 Thread Alex Bennée
Akihiko Odaki writes: > On 2023/12/12 2:01, Alex Bennée wrote: >> Cleber Rosa writes: >> >>> Based on many runs, the average run time for these 4 tests is around >>> 250 seconds, with 320 seconds being the ceiling. In any way, the >>> default 120 seconds timeout is inappropriate in my experien

Re: [PATCH 03/10] tests/avocado/intel_iommu.py: increase timeout

2023-12-12 Thread Akihiko Odaki
On 2023/12/12 2:01, Alex Bennée wrote: Cleber Rosa writes: Based on many runs, the average run time for these 4 tests is around 250 seconds, with 320 seconds being the ceiling. In any way, the default 120 seconds timeout is inappropriate in my experience. I would rather see these tests upda

Re: [PATCH 03/10] tests/avocado/intel_iommu.py: increase timeout

2023-12-11 Thread Alex Bennée
Cleber Rosa writes: > Based on many runs, the average run time for these 4 tests is around > 250 seconds, with 320 seconds being the ceiling. In any way, the > default 120 seconds timeout is inappropriate in my experience. I would rather see these tests updated to fix: - Don't use such an old

[PATCH 03/10] tests/avocado/intel_iommu.py: increase timeout

2023-12-08 Thread Cleber Rosa
Based on many runs, the average run time for these 4 tests is around 250 seconds, with 320 seconds being the ceiling. In any way, the default 120 seconds timeout is inappropriate in my experience. Let's increase the timeout so these tests get a chance to completion. Signed-off-by: Cleber Rosa -