From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 93CCD82257 for ; Wed, 22 Feb 2017 18:49:44 -0800 (PST) Received: from orsmga004.jf.intel.com ([10.7.209.38]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Feb 2017 18:49:44 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.35,197,1484035200"; d="scan'208";a="61321133" Received: from fmsmsx103.amr.corp.intel.com ([10.18.124.201]) by orsmga004.jf.intel.com with ESMTP; 22 Feb 2017 18:49:43 -0800 Received: from FMSMSX109.amr.corp.intel.com (10.18.116.9) by FMSMSX103.amr.corp.intel.com (10.18.124.201) with Microsoft SMTP Server (TLS) id 14.3.248.2; Wed, 22 Feb 2017 18:49:43 -0800 Received: from shsmsx103.ccr.corp.intel.com (10.239.4.69) by fmsmsx109.amr.corp.intel.com (10.18.116.9) with Microsoft SMTP Server (TLS) id 14.3.248.2; Wed, 22 Feb 2017 18:49:43 -0800 Received: from shsmsx101.ccr.corp.intel.com ([169.254.1.177]) by SHSMSX103.ccr.corp.intel.com ([10.239.4.69]) with mapi id 14.03.0248.002; Thu, 23 Feb 2017 10:49:41 +0800 From: "Wei, David" To: "Richardson, Brian" , Rebecca Cran , "Gao, Liming" , "edk2-devel@lists.01.org" CC: "Lu, ShifeiX A" , "Zimmer, Vincent" , Andrew Fish , "Wei, David" Thread-Topic: [edk2] Improvements to build system etc. for edk2-platforms devel-MinnowBoard3? Thread-Index: AQHSjKnl3dfLZL1X/ke01pZKUJR7uKFz7w2AgAAQCoCAALSrgIAABPeAgAAme4CAAQHe4A== Date: Thu, 23 Feb 2017 02:49:40 +0000 Message-ID: <89954A0B46707A448411A627AD4EEE3468EFDFFF@SHSMSX101.ccr.corp.intel.com> References: <4A89E2EF3DFEDB4C8BFDE51014F606A14D6E27B3@shsmsx102.ccr.corp.intel.com> <95a8d930-6bb2-e666-1957-8d78a25efd10@bluestop.org> <80AC2BAA3152784F98F581129E5CF5AF6655EC20@ORSMSX114.amr.corp.intel.com> <2d7462e9-de6c-67b2-ffd7-a3fa194ce06b@bluestop.org> <80AC2BAA3152784F98F581129E5CF5AF6655F06F@ORSMSX114.amr.corp.intel.com> In-Reply-To: <80AC2BAA3152784F98F581129E5CF5AF6655F06F@ORSMSX114.amr.corp.intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Subject: Re: Improvements to build system etc. for edk2-platforms devel-MinnowBoard3? X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 23 Feb 2017 02:49:44 -0000 Content-Language: en-US Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Yes, as Brian said, at this stage, patches are welcomed, including the pyt= hon script. You can also file bugs on https://bugzilla.tianocore.org/ for = issue track and discussion. Please remember to CC stakeholders.=20 Specifically for the multi-thread building , I think maybe NUMBER_OF_PROCE= SSORS Windows environment variable could be used as the proper thread numbe= r.=20 Thanks, David Wei =20 -----Original Message----- From: Richardson, Brian=20 Sent: Thursday, February 23, 2017 3:10 AM To: Rebecca Cran ; Gao, Liming = ; edk2-devel@lists.01.org Cc: Lu, ShifeiX A ; Zimmer, Vincent ; Andrew Fish ; Wei, David Subject: RE: [edk2] Improvements to build system etc. for edk2-platforms de= vel-MinnowBoard3? We're ok with help fixing issues (yay open source), so thanks for the help.= Patches are welcome at this time. But we do track them in Bugzilla, so ope= ning an issue there is the first step to a solution.=20 For the processor thread setting, note that we have historically disabled p= rocessor threading by default because we don't know the build system config= uration. While we at Intel prefer everyone own an Intel(R) Core(TM) i7 or I= ntel(R) Xeon processor, we know that's not the case ... so keeping it disab= led has been seen as a safe option. We should definitely do a better job of= documenting that setting change, but I think we need to consider generical= ly changing that setting by default in target.txt (which requires a Bugzill= a entry). The .bat/.sh files are required to trigger post-build tools, which are OS d= ependent. Even if the build was triggered by a Python script, we still need= to do some verification to make sure there are no functional differences w= hen we build in a Windows environment versus Linux (work in progress). Thanks ... br --- Brian Richardson, Senior Technical Marketing Engineer, Intel Software brian.richardson@intel.com -- @intel_Brian (Twitter & WeChat) https://software.intel.com/en-us/meet-the-developers/evangelists/team/brian= -richardson=20 -----Original Message----- From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Rebe= cca Cran Sent: Wednesday, February 22, 2017 11:53 AM To: Richardson, Brian ; Gao, Liming ; edk2-devel@lists.01.org Cc: Lu, ShifeiX A ; Zimmer, Vincent ; Andrew Fish ; Wei, David Subject: Re: [edk2] Improvements to build system etc. for edk2-platforms de= vel-MinnowBoard3? On 2/22/2017 9:34 AM, Richardson, Brian wrote: > Thanks for the input. For future reference, you can use the TianoCore=20 > Bugzilla to report issues on any EDK II feature/platform.=20 > https://github.com/tianocore/tianocore.github.io/wiki/Reporting-Issues > > I agree the readme.md should be present, and use markup instead of plain = text to work better in github. You can open an issue on this in Bugzilla. > > Normally, we ask folks to change the number of processor threads based on= their system configuration. We don't add a larger thread number by default= , but it might be good to set it '5' by default (assuming a dual core proce= ssor with hyperthreading) instead of '1' (assuming a single core system w/o= threading). I don't know if this will cause any compatibility issues on ol= der systems, but it's worth a check. > > At this time, MinnowBoard 3 build is only validated in Windows. That's wh= y there is no equivalent .sh file for BuildBIOS yet, but it will be added o= nce Linux build is verified and checked in. I'm more about _fixing_ issues I find rather than reporting them! Are you s= aying that patches wouldn't be welcome just now? Is there a reason why you= don't want to make full use of the CPU while building? And I understand th= at MinnowBoard 3 only builds under Windows at the moment, but if more of it= built using python (and python is already listed as a prerequisite in the = ReadMe file) the porting might be simpler. -- Rebecca _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel