From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergey Bronnikov Subject: Re: [kvm-unit-tests PATCH] Use /bin/env in shebang to make scripts more portable Date: Fri, 17 Mar 2017 14:11:18 +0300 Message-ID: <20170317111118.GA67603@Sergeys-MacBook-Pro-2.local> References: <20170315102550.GA44201@Sergeys-MacBook-Pro-2.local> <29e0ed97-d18a-acf8-2483-5c53db9ca053@redhat.com> <20170317051122.GA61717@Sergeys-MacBook-Pro-2.local> <9bcf5946-dd02-c312-0b00-3dcb09568fda@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Cc: kvm@vger.kernel.org To: Thomas Huth Return-path: Received: from [87.240.31.184] ([87.240.31.184]:25204 "EHLO gw.bronevichok.ru" rhost-flags-FAIL-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1751022AbdCQLMd (ORCPT ); Fri, 17 Mar 2017 07:12:33 -0400 Content-Disposition: inline In-Reply-To: <9bcf5946-dd02-c312-0b00-3dcb09568fda@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: Hi, Thomas On 09:04 Fri 17 Mar , Thomas Huth wrote: > On 17.03.2017 06:11, Sergey Bronnikov wrote: > > On 08:38 Thu 16 Mar , Thomas Huth wrote: > >> On 15.03.2017 11:25, Sergey Bronnikov wrote: > >>> Some operating systems installs bash executable file to other directory > >>> than /bin. So it is better to use env utility to find bash. > >> > >> Which operating systems do you have in mind here? > > > > OpenBSD > > Ok, then I'd say this patch simply does not make sense. We're talking > about kvm-unit-tests here - and KVM is pretty much Linux-only as far as > I know. according to description on the project page: "Unit tests provide KVM and virtual hardware functional testing by targeting the features through minimal implementations of their use per the hardware specification." http://www.linux-kvm.org/page/KVM-unit-tests So testsuite is applicable to any virtual machine with hardware emulation. OpenBSD has QEMU in ports and it's own hypervisor vmd(8) (http://man.openbsd.org/vmd). It would be perfect to use testsuite for regression testing of these hypervisors on OpenBSD too. Moreover my patch don't broke anything on Linux. Why not to commit it and make tests more portable? $ qemu-system-x86_64 -nodefaults -device isa-debug-exit,iobase=0xf4,iosize=0x4 -vnc none -serial stdio -kernel x86/msr.flat enabling apic PASS: IA32_SYSENTER_CS PASS: MSR_IA32_SYSENTER_ESP PASS: IA32_SYSENTER_EIP PASS: MSR_IA32_MISC_ENABLE PASS: MSR_IA32_CR_PAT PASS: MSR_FS_BASE PASS: MSR_GS_BASE PASS: MSR_KERNEL_GS_BASE PASS: MSR_EFER PASS: MSR_LSTAR PASS: MSR_CSTAR PASS: MSR_SYSCALL_MASK PASS: MSR_*STAR eager loading SUMMARY: 13 tests $ uname -a OpenBSD openbsd.example.com 6.0 GENERIC.MP#2319 amd64 $ > Every Linux distro that I've seen so far provides /bin/bash, so > I don't think we've got to change this. > > Thomas