Re: [PATCH] virt-install: Do not warn about consoles on s390x

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Mar 14, 2019 at 05:08:02PM +0100, Christian Borntraeger wrote:
> 
> 
> On 14.03.2019 16:00, Boris Fiuczynski wrote:
> > On 3/14/19 9:50 AM, Christian Borntraeger wrote:
> >> s390x guests usually detect the sclp or virtio console. The current
> >> warning might actually break setups. Let us disable the warning.
> >>
> >> Reported-by: Andre Wild <wild1@xxxxxxxxxxxxx>
> >> Signed-off-by: Christian Borntraeger <borntraeger@xxxxxxxxxx>
> >> ---
> >>   virt-install | 6 +++---
> >>   1 file changed, 3 insertions(+), 3 deletions(-)
> >>
> >> diff --git a/virt-install b/virt-install
> >> index d7af107b..291b4a0a 100755
> >> --- a/virt-install
> >> +++ b/virt-install
> >> @@ -402,9 +402,9 @@ def _show_nographics_warnings(options, guest, installer):
> >>           console_type = serial_arm_arg
> >>       if guest.devices.console[0].target_type in ["virtio", "xen"]:
> >>           console_type = hvc_arg
> >> -    if guest.os.is_ppc64() or guest.os.is_arm_machvirt():
> >> -        # Later arm/ppc kernels figure out console= automatically, so don't
> >> -        # warn about it.
> >> +    if guest.os.is_ppc64() or guest.os.is_arm_machvirt() or guest.os.is_s390x:
> > s/guest.os.is_s390x/guest.os.is_s390x()/
> 
> Yes, indeed. Shall I resend or is anyone willing to fix this up while applying?

Fixed and pushed.

Thanks,
Pavel

Attachment: signature.asc
Description: PGP signature

_______________________________________________
virt-tools-list mailing list
virt-tools-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/virt-tools-list

[Index of Archives]     [Linux Virtualization]     [KVM Development]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]     [Video 4 Linux]

  Powered by Linux