From mboxrd@z Thu Jan  1 00:00:00 1970
From: Fabio Fantoni <fabio.fantoni@m2r.biz>
Subject: Re: [PATCH] tools: ovmf debug build only if tools debug
	is enabled
Date: Thu, 31 Oct 2013 15:56:06 +0100
Message-ID: <52726F86.7000803@m2r.biz>
References: <1383228171-5132-1-git-send-email-fabio.fantoni@m2r.biz>
	<20131031142321.GD29775@zion.uk.xensource.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <20131031142321.GD29775@zion.uk.xensource.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Wei Liu <wei.liu2@citrix.com>
Cc: xen-devel@lists.xensource.com, Ian.Jackson@eu.citrix.com, Ian.Campbell@citrix.com
List-Id: xen-devel@lists.xenproject.org

Il 31/10/2013 15:23, Wei Liu ha scritto:
> Does it make a big difference in terms of speed when running DEBUG build
> and RELEASE build?

In the working test (few months ago) the difference was noticeble - if I 
remember correctly - while now, without a uefi domu full working I don't 
know.
I should do this patch after that one with the gcc!=44 but in the 
meantime a was working on other tasks so i forgot it, sorry.

>
> On Thu, Oct 31, 2013 at 03:02:50PM +0100, Fabio Fantoni wrote:
>> Signed-off-by: Fabio Fantoni <fabio.fantoni@m2r.biz>
>> ---
>>   tools/firmware/ovmf-makefile |   13 +++++++++++--
>>   1 file changed, 11 insertions(+), 2 deletions(-)
>>
>> diff --git a/tools/firmware/ovmf-makefile b/tools/firmware/ovmf-makefile
>> index 073ed44..efb4fb1 100644
>> --- a/tools/firmware/ovmf-makefile
>> +++ b/tools/firmware/ovmf-makefile
>> @@ -1,3 +1,6 @@
>> +XEN_ROOT = $(CURDIR)/../../..
>> +include $(XEN_ROOT)/tools/Rules.mk
>> +
>>   # OVMF building system is not ready yet to run in parallel.
>>   # Force it to be serial in order to exploit parallelism for neighbors.
>>   
>> @@ -9,8 +12,14 @@ all: ovmf.bin
>>   
>>   .PHONY: ovmf.bin
>>   ovmf.bin:
>> -	OvmfPkg/build.sh -a X64
>> -	cp Build/OvmfX64/DEBUG_GCC*/FV/OVMF.fd ovmf.bin
>> +	if [ "$(debug)" == y ]; then \
>> +		OvmfPkg/build.sh -a X64; \
>> +		cp Build/OvmfX64/DEBUG_GCC*/FV/OVMF.fd ovmf.bin; \
>> +	else \
>> +		OvmfPkg/build.sh -a X64 -b RELEASE; \
>> +		cp Build/OvmfX64/RELEASE_GCC*/FV/OVMF.fd ovmf.bin; \
>> +	fi
>> +
>>   
>>   .PHONY: clean
>>   clean:
>> -- 
>> 1.7.9.5