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=-15.1 required=3.0 tests=BAYES_00,DKIM_INVALID, DKIM_SIGNED,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, 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 0CF0BC433E0 for ; Thu, 21 Jan 2021 18:33:10 +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 9C71323A40 for ; Thu, 21 Jan 2021 18:33:09 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9C71323A40 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]:45142 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1l2elY-0007XB-HM for qemu-devel@archiver.kernel.org; Thu, 21 Jan 2021 13:33:08 -0500 Received: from eggs.gnu.org ([2001:470:142:3::10]:45580) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1l2ehf-00046S-6v for qemu-devel@nongnu.org; Thu, 21 Jan 2021 13:29:07 -0500 Received: from us-smtp-delivery-124.mimecast.com ([63.128.21.124]:59520) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_CBC_SHA1:256) (Exim 4.90_1) (envelope-from ) id 1l2ehZ-0005n5-8R for qemu-devel@nongnu.org; Thu, 21 Jan 2021 13:29:06 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1611253739; 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; bh=sd8mGbbt087SLu4CCuRW27oFTksKiXBeUd3gsuKAEaY=; b=BlcBQ9lgUQQitxZGAQUkSGKs5qp+djQgm2U8bp0SmxbWSyh3MAJx+N+CkPsVA+B+1pVu9f 0kFQiYBN9C3uHYkm9anbcS0eBummuBdak/nVpVJzgRXip7bEBAmRfEYNEHkAlupgnjLxAl m39++9NZI+ysA16AOXp1A3Bcws81pOw= 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-478-q_swCdOHOIufpTOsWd4T-Q-1; Thu, 21 Jan 2021 13:28:55 -0500 X-MC-Unique: q_swCdOHOIufpTOsWd4T-Q-1 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 1E103425CB; Thu, 21 Jan 2021 18:28:54 +0000 (UTC) Received: from thuth.remote.csb (ovpn-112-82.ams2.redhat.com [10.36.112.82]) by smtp.corp.redhat.com (Postfix) with ESMTP id A32656267F; Thu, 21 Jan 2021 18:28:48 +0000 (UTC) Subject: Re: [RFC PATCH 2/2] gitlab-ci: Add a job building TCI with Clang To: =?UTF-8?Q?Daniel_P=2e_Berrang=c3=a9?= , Wainer dos Santos Moschetta References: <20210110162739.858087-1-f4bug@amsat.org> <20210110162739.858087-3-f4bug@amsat.org> <78a9718b-dec0-cc31-7ada-e815d9022e65@redhat.com> <20210121181303.GR3125227@redhat.com> From: Thomas Huth Message-ID: <89b62df4-e077-fb82-75af-e180a89fba27@redhat.com> Date: Thu, 21 Jan 2021 19:28:47 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.6.0 MIME-Version: 1.0 In-Reply-To: <20210121181303.GR3125227@redhat.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=thuth@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Received-SPF: pass client-ip=63.128.21.124; envelope-from=thuth@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.168, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, NICE_REPLY_A=-0.001, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no 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: Stefan Weil , Richard Henderson , =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= , Wataru Ashihara , qemu-devel@nongnu.org, =?UTF-8?Q?Alex_Benn=c3=a9e?= , =?UTF-8?Q?Philippe_Mathieu-Daud=c3=a9?= Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On 21/01/2021 19.13, Daniel P. Berrangé wrote: > On Thu, Jan 21, 2021 at 03:05:43PM -0300, Wainer dos Santos Moschetta wrote: >> Hi, >> >> On 1/21/21 7:08 AM, Thomas Huth wrote: >>> On 10/01/2021 17.27, Philippe Mathieu-Daudé wrote: >>>> Split the current GCC build-tci job in 2, and use Clang >>>> compiler in the new job. >>>> >>>> Signed-off-by: Philippe Mathieu-Daudé >>>> --- >>>> RFC in case someone have better idea to optimize can respin this patch. >>>> >>>>   .gitlab-ci.yml | 22 ++++++++++++++++++++-- >>>>   1 file changed, 20 insertions(+), 2 deletions(-) >>> >>> I'm not quite sure whether we should go down this road ... if we wanted >>> to have full test coverage for clang, we'd need to duplicate *all* jobs >>> to run them once with gcc and once with clang. And that would be just >>> overkill. >> >> I agree with Thomas. >> >>> >>> >>> I think we already catch most clang-related problems with the clang jobs >>> that we already have in our CI, so problems like the ones that you've >>> tried to address here should be very, very rare. So I'd rather vote for >>> not splitting the job here. >> >> We got only one clang job on GitLab CI... >> >>   build-clang: >>     <<: *native_build_job_definition >>     variables: >>       IMAGE: fedora >>       CONFIGURE_ARGS: --cc=clang --cxx=clang++ >>       TARGETS: alpha-softmmu arm-softmmu m68k-softmmu mips64-softmmu >>         ppc-softmmu s390x-softmmu arm-linux-user >>       MAKE_CHECK_ARGS: check >> >> ... and others on Travis: >> >>   "Clang (user)" >> >>   "Clang (main-softmmu)" >> >>   "Clang (other-softmmu)" > > I guess these three overlap partially with the build-clang job. > >>   "[s390x] Clang (disable-tcg)" > > Don't forget the Cirrus CI jobs for freebsd and macOS will > be using CLang too. Right... we should work towards getting cirrus-run into the QEMU-CI, too, to finally have these in the gitlab-ci dashboard, too. >> >> So I've some questions: >> >>  * Can we move those first three Travis jobs to Gitlab CI? (I can work on >> that) > > Yeah, if we move those three travis jobs they can replace the existing > build-clang job. We don't neccesssarily need to keep them as three > separate jobs - that split was just due to the Travis time limits. > If a different split works better on GitLab we can do that. Well, if we really want to increase the amount clang jobs, one of them should likely use TCI, as Phillippe suggested. >>  * Do you think they cover the most common problems with clang? > > Should do I think, especially in addition to the Cirrus CI jobs. I concur. Thomas