2015-11-10 10:22, Hunt, David:
> On 09/11/2015 17:46, Jan Viktorin wrote:
>  > Here is the log. You an see, that many tests fail just because
>  > of the missing hugetlb support. This is strange as I modified the
>  > autotest_runner.py to inject "--no-huge --no-pci" options when it
>  > detects armv7 architecture.
> 
> I think publishing these test results to the mailing list confuses 
> matters somewhat. The tests are not tuned for armv7, and there are many 
> false negatives, giving the impression that the results are worse than 
> they actually should be.
> 
> The tests need to be analysed to see if they respect the --no-huge and 
> --no-pci flags correctly, I suspect some tests are being run even though 
> these flags are enabled.
> 
> I believe enough of the tests pass to allow the initial version of the 
> patch set to be accepted. We can look at the failing cases later, and 
> improve the test suite as time goes on.

I agree.
The unit tests are know to require some refactoring.
I suggest to continue ARM integration while checking how to fix the
unit tests later. Maybe that Declan has some ideas or plans about the tests.

Reply via email to