From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2 1/6] eal: eal stub to add windows support Date: Thu, 07 Mar 2019 09:59:42 +0100 Message-ID: <1823219.Mqzco52y5N@xps> References: <20190306041634.12976-1-anand.rawat@intel.com> <59AF69C657FD0841A61C55336867B5B072749EA4@IRSMSX103.ger.corp.intel.com> <56df497b-5854-6693-8577-87413b647e5a@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, "Richardson, Bruce" , "Kadam, Pallavi" , "Menon, Ranjit" , "Shaw, Jeffrey B" To: Anand Rawat Return-path: Received: from wout2-smtp.messagingengine.com (wout2-smtp.messagingengine.com [64.147.123.25]) by dpdk.org (Postfix) with ESMTP id 94B452C28 for ; Thu, 7 Mar 2019 09:59:46 +0100 (CET) In-Reply-To: <56df497b-5854-6693-8577-87413b647e5a@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 07/03/2019 02:04, Anand Rawat: > On 3/6/2019 3:52 AM, Richardson, Bruce wrote: > > From: Thomas Monjalon [mailto:thomas@monjalon.net] > >> 06/03/2019 12:20, Bruce Richardson: > >>> On Wed, Mar 06, 2019 at 11:03:24AM +0100, Thomas Monjalon wrote: > >>>> 06/03/2019 05:16, Anand Rawat: > >>>>> +if host_machine.system() != 'windows' > >>>>> + common_sources = files( > >>>> > >>>> The definitive solution should be to compile all common EAL files. > >>>> Please explain what are the issues in the common files. > >>>> I think we should not remove them and fix them one by one. > >>>> You could provide a separate patch to skip some files for making > >>>> helloworld working. > >>>> > >>> > >>> I believe that is exactly what this patch is trying to do - it's > >>> skipping the files unneeded to get helloworld working, and the > >>> intention is to fix them one by one and add them back in later. > >>> Perhaps this sort of change should be a separate (precursor) patch > >>> where the cover letter can call this out explicitly? > >>> > >>>>> -deps += 'kvargs' > >>>>> +if host_machine.system() != 'windows' > >>>>> + deps += 'kvargs' > >>>>> +endif > >>>> > >>>> Why kvargs is removed? > >>> > >>> Again, I believe these actions are to disable the parts of DPDK that > >>> are not needed to enable helloworld, allowing later patches to come in > >>> and fix them. > >> > >> They are workarounds to build helloworld. > >> It is good to have progress in the draft tree, but I see no point in > >> merging this in master. > >> I think we should separate patches which are doing definitive changes from > >> temporary workaround patches disabling some files. > >> It is not an issue to merge some patches for Windows which are not > >> compiling. > > Bruce is right, we only compile required header and source files in > order to avoid compatibility errors on windows. Without these > change helloworld on windows would fail to compile. Adding windows > specific implementations of the common headers and sources would > bloat up individual patches as well the number of patches. kvargs is > removed as a dependency to have minimum viable product for helloworld. > If required for lcore mask, it'll added back in v3. Please make separate patches for workarounds. I am interested to push some patches which are really required in master, but the workarounds should stay in the Windows draft repository.