On 3/13/19 12:25 AM, David Gibson wrote: > On Tue, Mar 12, 2019 at 09:46:14PM +0100, Cédric Le Goater wrote: >> On 3/12/19 6:18 PM, Cleber Rosa wrote: >>> Because of the changes in da874d90a, the Fedora 29 kernel doesn't >>> boot with the modified version of the Power9 CPU. Let's set the >>> power8 CPU instead. >>> >>> Signed-off-by: Cleber Rosa <cr...@redhat.com> >>> CC: Cédric Le Goater <c...@kaod.org> >>> CC: David Gibson <da...@gibson.dropbear.id.au> >> >> Looks fine to me as POWER9 support still lacks a skiboot update. > > AFAICT this is about a pseries boot, not a powernv boot.
yes. I realize that now. I got confused by the other thread. POWER9 should work on pseries. I agree. C. >> >> Reviewed-by: Cédric Le Goater <c...@kaod.org> >> >> Thanks, >> >> C. >> >>> --- >>> tests/acceptance/boot_linux_console.py | 3 ++- >>> 1 file changed, 2 insertions(+), 1 deletion(-) >>> >>> diff --git a/tests/acceptance/boot_linux_console.py >>> b/tests/acceptance/boot_linux_console.py >>> index edc7722abe..50ced6549b 100644 >>> --- a/tests/acceptance/boot_linux_console.py >>> +++ b/tests/acceptance/boot_linux_console.py >>> @@ -149,7 +149,8 @@ class BootLinuxConsole(Test): >>> self.vm.set_console() >>> kernel_command_line = self.KERNEL_COMMON_COMMAND_LINE + >>> 'console=hvc0' >>> self.vm.add_args('-kernel', kernel_path, >>> - '-append', kernel_command_line) >>> + '-append', kernel_command_line, >>> + '-cpu', 'power8') >>> self.vm.launch() >>> console_pattern = 'Kernel command line: %s' % kernel_command_line >>> self.wait_for_console_pattern(console_pattern) >>> >> >