On 11/09/2015 02:02 PM, Pavel Hrdina wrote: > Kernel for ppc64 automatically enables serial console, there is no need > to report any warning. > > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1247434 > > Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx> > --- > > Pushed as trivial. > > virt-install | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/virt-install b/virt-install > index d509256..3656835 100755 > --- a/virt-install > +++ b/virt-install > @@ -520,6 +520,8 @@ def _show_nographics_warnings(options, guest): > console_type = serial_arm_arg > if guest.get_devices("console")[0].target_type in ["virtio", "xen"]: > console_type = hvc_arg > + if guest.os.is_ppc64(): > + return > > if console_type in (options.extra_args or ""): > return > I don't think this actually works as you expect, this check is done _after_ the warning mentioned in the bug report. I think it should be moved near the top of the function, where the similar arm machvirt check is done - Cole _______________________________________________ virt-tools-list mailing list virt-tools-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/virt-tools-list
