28/11/2023 15:56, Luca Boccassi: > On Tue, 28 Nov 2023 at 14:40, <christian.ehrha...@canonical.com> wrote: > > > > From: Christian Ehrhardt <christian.ehrha...@canonical.com> > > > > On ppc64, without huge pages, pa would be the default but fails like: > > EAL: Cannot use IOVA as 'PA' since physical addresses are not available > > > > On a normal system setup we'd expect an admin to set up huge pages > > and for the unlikely case that they do not do that set iova=va > > themselve. But the testing infrastructure does not have this conscious > > admin that will do that, so fix test execution that by adjusting the > > tests arguments accordingly if the conditions are met. > > > > Signed-off-by: Christian Ehrhardt <christian.ehrha...@canonical.com> > > Acked-by: Luca Boccassi <bl...@debian.org>
Moved the comment inside the "if condition" context. Applied, thanks.