From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=YvWI+q/0; spf=pass (domain: linaro.org, ip: 209.85.221.66, mailfrom: leif.lindholm@linaro.org) Received: from mail-wr1-f66.google.com (mail-wr1-f66.google.com [209.85.221.66]) by groups.io with SMTP; Thu, 08 Aug 2019 04:30:50 -0700 Received: by mail-wr1-f66.google.com with SMTP id q12so4291445wrj.12 for ; Thu, 08 Aug 2019 04:30:49 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=d6E1yrk0NkVLqgc+VKQHz3NjHymmdTxMHLR0d9CxGPU=; b=YvWI+q/03gGjTPQPV9m4uhRXvP1/jGu1oCNW+Vj597BQipdlVAQJ09YfYVbwdyrfTm uAW9ygmUYE/N/bZjBLkyl5QUi1JMupEGXqPVRV4V+Gvn3UJbaYbACSrgrKj7k+gPbQPg gptvVlRRw2RhGZ+qbVgXDGlC2IJXU0fxSK7hiVvSqDDxRMkX+syrQDozF0G22fHn3eto cwTzqdOa8uyz2acRstI9T7kGdiR1/uS5lnVC7YgMQBOWymluu8Xf9iO3qsWr8AtIrZRG Z3b2XoVB0EdRQekp4fkLwX53351MUX3vypSaRwQnpEvScFE9mxVlpEdQz/7BUJM+HL3q SWaA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=d6E1yrk0NkVLqgc+VKQHz3NjHymmdTxMHLR0d9CxGPU=; b=GbsNr6yPZHbeqm2WXMqxCuxPjbtBMhB4R7ty8HPNAUJ7xfVf8NEcuHDGkJQ2Dv+f6I h07WB9kxPIevyLwMSEVz96WlbtjEn+e8FZSsWz1S0z2O4VqCUAFGSMstebEBt75ivqM8 pV52a4oUR0qou/yt79dp3Hj9J+hNQGZ0RE5pF5hlbEdkmfeHYavbmpN1NXQPETTSRgra /qWXNuQIYPor0jBAlBBxSHl3maliLW/BfO8F+CA8DbLjH/+/hR9lWv8vyWo6Fxdf4wDP gHmro9qzVKHa/FXo7JZOGhg6I4AoTZLAGCLTzVKylohlxhDzoihJGml8wheMGzTaAZIL gBLA== X-Gm-Message-State: APjAAAWvClIwOw4POc2nWY7fNZmSOUoeInxad+6HOXlEDR7i7mBUcwlO ZrbuNrBcQfRyson5MuUqOltY9w== X-Google-Smtp-Source: APXvYqzzw4E4PQf2K86tIZ80WZMY6grILZujeUyaX0IUdNiq8/xAmwL6lBTl10H6ZZyntXpLnV1qkQ== X-Received: by 2002:adf:90e7:: with SMTP id i94mr15452355wri.224.1565263848442; Thu, 08 Aug 2019 04:30:48 -0700 (PDT) Return-Path: Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id 4sm219844223wro.78.2019.08.08.04.30.47 (version=TLS1_3 cipher=AEAD-AES256-GCM-SHA384 bits=256/256); Thu, 08 Aug 2019 04:30:47 -0700 (PDT) Date: Thu, 8 Aug 2019 12:30:46 +0100 From: "Leif Lindholm" To: "Gao, Liming" Cc: "devel@edk2.groups.io" , "Zhang, Shenglei" , "Feng, Bob C" , Ard Biesheuvel , Eugene Cohen Subject: Re: [edk2-devel] [PATCH 1/1] BaseTools: Remove tool chain in tools_def.template Message-ID: <20190808113046.GW25813@bivouac.eciton.net> References: <20190808080918.744-1-shenglei.zhang@intel.com> <20190808095518.GT25813@bivouac.eciton.net> <4A89E2EF3DFEDB4C8BFDE51014F606A14E4CCBDA@SHSMSX104.ccr.corp.intel.com> MIME-Version: 1.0 In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E4CCBDA@SHSMSX104.ccr.corp.intel.com> User-Agent: Mutt/1.10.1 (2018-07-13) Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, Aug 08, 2019 at 10:51:54AM +0000, Gao, Liming wrote: > Leif: > > >-----Original Message----- > >From: Leif Lindholm [mailto:leif.lindholm@linaro.org] > >Sent: Thursday, August 08, 2019 5:55 PM > >To: devel@edk2.groups.io; Zhang, Shenglei > >Cc: Feng, Bob C ; Gao, Liming > >; Ard Biesheuvel ; > >Eugene Cohen > >Subject: Re: [edk2-devel] [PATCH 1/1] BaseTools: Remove tool chain in > >tools_def.template > > > >Hi Shenglei, > > > >On Thu, Aug 08, 2019 at 04:09:18PM +0800, Zhang, Shenglei wrote: > >> Remove definition of RVCT, RVCTLINUX, RVCTCYGWIN and CLANG35 > >> in tools_def.template. These tool chains are for ARM and AARCH64 only. > >> There is no change recently and they are not used. > >> https://bugzilla.tianocore.org/show_bug.cgi?id=1750 > > > >This still does not address my comment in the BZ that deleting all > >RVCT profiles before full VS support is enabled for (32-bit) ARM, we > >orphan an awful lot of .asm files. > > > > How about submit another BZ for VS tool chain ARM fully support? > When there is real request, this support can be added in future. Good point. I have raised https://bugzilla.tianocore.org/show_bug.cgi?id=2065 and assigned it to myself. > >This may not have much of a practical effect, since I doubt anyone is > >using these toolchains today - but it does prevent someone from > >actively going through and testing future updates (where before, they > >may just have neglected to do so). > > > >This point needs discussing rather than ignoring, and I think we're > >getting too close to the freeze to consider the patch to go in as is > >at this point. > > > > Agree for more discussion. > > >Whenever this patch does go in should be in the week after a stable > >tag is made, to give plenty of time for anyone affected to shout > >before the next stable tag is made. > > > >After the 2019.08 stable tag has been made, I am happy for a patch > >going in that deletes CLANG35, RVCTCYGWIN and *one*of* RVCT/RVCTLINUX. > > If no one maintain or use it, this tool chain may not work now. > If so, do we still need to keep it? So there are two questions here, really: The first - "why can't we delete all of these now?", I think I have already explained above. (I am not suggesting you did not understand, but I want to clarify that we also need agreement on the timing of this patch in general.) For the second: the Visual Studio assembler (for ARM/AArch64) shares the .asm syntax (and source file name) with the RVCT assembler. If we delete the whole RVCT family of profiles, we are left with a bunch of .asm files that are defined to be assembled by a non-existing toolchain family. Whilst still being of the name and syntax that we will need when enabling the MSFT family. A not exactly precise execution of find * -name "*.inf" -exec grep -H "RVCT" {} \; | grep "|" | grep "\.asm" | wc -l suggests 50 source files are affected in edk2. A further 6 in edk2-platforms. Hence, my preferred obsoletion path for RVCT would mean the family (and at least one toolchain profile) remaining in the tree until the Visual Studio enablement has switched the source files to MSFT. Best Regards, Leif