From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.1 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, MENTIONS_GIT_HOSTING,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 70430C433DF for ; Mon, 15 Jun 2020 09:45:13 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 407C7206D7 for ; Mon, 15 Jun 2020 09:45:13 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (1024-bit key) header.d=redhat.com header.i=@redhat.com header.b="JO1aFHUO" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 407C7206D7 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Received: from localhost ([::1]:55750 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jklg0-0008Ri-IU for qemu-devel@archiver.kernel.org; Mon, 15 Jun 2020 05:45:12 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:40348) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jklf9-0007cJ-9u for qemu-devel@nongnu.org; Mon, 15 Jun 2020 05:44:19 -0400 Received: from us-smtp-delivery-1.mimecast.com ([205.139.110.120]:52278 helo=us-smtp-1.mimecast.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1jklf6-0006K3-Q3 for qemu-devel@nongnu.org; Mon, 15 Jun 2020 05:44:19 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1592214255; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:openpgp:openpgp; bh=72rKnDWlp5+yiOfk+vKoPcjcrSgCF10ukOWzQh+dO8U=; b=JO1aFHUOCOA7PmrtaZfOCA7zcXfaXv2UrXYjcukyqFC6J0pNr/5LV0mMew/4e81fktU4fN M6k56PExSrHCMarJ5zK7mDmiinEXMZCrbRvtmHP1wsvegOWzPWEn7P5Zb0rj7JKlP1A+v3 LdWbObX1d6Tjhhz7ZKy8W4IqjHGewtE= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-202-HNK2pMvFNyu7zF_7Zay7ew-1; Mon, 15 Jun 2020 05:44:14 -0400 X-MC-Unique: HNK2pMvFNyu7zF_7Zay7ew-1 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id B236118585B9; Mon, 15 Jun 2020 09:44:12 +0000 (UTC) Received: from thuth.remote.csb (ovpn-112-111.ams2.redhat.com [10.36.112.111]) by smtp.corp.redhat.com (Postfix) with ESMTPS id A11565D9CC; Mon, 15 Jun 2020 09:44:10 +0000 (UTC) Subject: Re: [PATCH v3] configure: Let SLOF be initialized by ./scripts/git-submodule.sh To: =?UTF-8?Q?Daniel_P=2e_Berrang=c3=a9?= , =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= References: <20200615074919.12552-1-f4bug@amsat.org> <20200615091925.GB269638@redhat.com> From: Thomas Huth Openpgp: preference=signencrypt Message-ID: Date: Mon, 15 Jun 2020 11:44:08 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: <20200615091925.GB269638@redhat.com> Content-Language: en-US X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Received-SPF: pass client-ip=205.139.110.120; envelope-from=thuth@redhat.com; helo=us-smtp-1.mimecast.com X-detected-operating-system: by eggs.gnu.org: First seen = 2020/06/14 22:37:26 X-ACL-Warn: Detected OS = Linux 2.2.x-3.x [generic] [fuzzy] X-Spam_score_int: -30 X-Spam_score: -3.1 X-Spam_bar: --- X-Spam_report: (-3.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-1, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001 autolearn=_AUTOLEARN X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Fam Zheng , =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= , Cornelia Huck , Mark Cave-Ayland , qemu-devel@nongnu.org, qemu-s390x@nongnu.org, =?UTF-8?Q?Alex_Benn=c3=a9e?= Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On 15/06/2020 11.19, Daniel P. Berrangé wrote: > On Mon, Jun 15, 2020 at 09:49:19AM +0200, Philippe Mathieu-Daudé wrote: >> The git-submodule.sh script is called by make and initialize the >> submodules listed in the GIT_SUBMODULES variable generated by >> ./configure. >> >> SLOF is required for building the s390-ccw firmware on s390x, since >> it is using the libnet code from SLOF for network booting. >> >> Add it to the GIT_SUBMODULES when buildint the s390-ccw firmware, >> to fix: >> >> $ ( cd ${SRC_DIR} ; git submodule update --init roms/SLOF ) >> Submodule 'roms/SLOF' (https://git.qemu.org/git/SLOF.git) registered for path 'roms/SLOF' >> Cloning into '/home/travis/build/user/qemu/roms/SLOF'... >> fatal: unable to access 'https://git.qemu.org/git/SLOF.git/': Could not resolve host: git.qemu.org >> fatal: clone of 'https://git.qemu.org/git/SLOF.git' into submodule path '/home/travis/build/user/qemu/roms/SLOF' failed >> Failed to clone 'roms/SLOF'. Retry scheduled >> Cloning into '/home/travis/build/user/qemu/roms/SLOF'... >> fatal: unable to access 'https://git.qemu.org/git/SLOF.git/': Could not resolve host: git.qemu.org >> fatal: clone of 'https://git.qemu.org/git/SLOF.git' into submodule path '/home/travis/build/user/qemu/roms/SLOF' failed >> Failed to clone 'roms/SLOF' a second time, aborting >> The command "( cd ${SRC_DIR} ; git submodule update --init roms/SLOF )" exited with 1. > > The "git-submodule.sh" script just invokes "git submodule". So if > you are getting DNS failures from "git submodule", using git-submodule.sh > instead is not going to fix the problem. > >> >> Reported-by: Mark Cave-Ayland >> Suggested-by: Thomas Huth >> Signed-off-by: Philippe Mathieu-Daudé >> --- >> configure | 5 +++++ >> .travis.yml | 1 - >> 2 files changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/configure b/configure >> index bb7fd12612..927e4a3d06 100755 >> --- a/configure >> +++ b/configure >> @@ -6533,6 +6533,11 @@ if test "$cpu" = "s390x" ; then >> write_c_skeleton >> if compile_prog "-march=z900" ""; then >> roms="$roms s390-ccw" >> + # SLOF is required for building the s390-ccw firmware on s390x, >> + # since it is using the libnet code from SLOF for network booting. >> + if test -e "${source_path}/.git" ; then >> + git_submodules="${git_submodules} roms/SLOF" >> + fi >> fi >> fi > > This whole bit of configure looks a bit dubious. > > For all the other firmware images we ship as pre-built blobs, we never > try to re-build them even if the host compiler supports it. All the other firmwares have separate projects and repositories, so it's normal that we don't try to rebuild them from the QEMU repo. But the s390-ccw firmware is part of the QEMU repository (for some good reasons), so of course we should build it during the QEMU build process if possible. And no, please don't suggest to move it to a separate repository instead, IIRC we've had this discussion in the past already, and it is easier if it stays in the current place. > So I don't think we need to make SLOF special. We are not talking about SLOF here. We are talking about the s390-ccw firmware which is part of the QEMU repository (see pc-bios/s390-ccw). It just requires the SLOF submodule for building its network booting part, since it re-uses the network stack from SLOF. Thomas