From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:4001:c0b::230; helo=mail-it0-x230.google.com; envelope-from=ard.biesheuvel@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-it0-x230.google.com (mail-it0-x230.google.com [IPv6:2607:f8b0:4001:c0b::230]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 674A621F3882E for ; Thu, 12 Oct 2017 08:43:26 -0700 (PDT) Received: by mail-it0-x230.google.com with SMTP id n195so7353259itg.2 for ; Thu, 12 Oct 2017 08:46:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=257f9gvWSvoJuWVw7WcHZoNz/hRRPZeDF3pw/tZ7JkU=; b=WqY+iFkQEiW4/P1Yhjir+8bDGjFgyK9+mASeNJV+P+2qOVYiXXGOra1uq95a9UI2Nk CX+4lz3tUGH7Ad1IXTTfsgWqIrZUDMhP3zmEMS2MSmp1mdlufbkVyzb7b0+ttfe5rYmg zKnIQNBdI2kEgBiF65n1sQssIaZpTWxce2AhE= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=257f9gvWSvoJuWVw7WcHZoNz/hRRPZeDF3pw/tZ7JkU=; b=WvlYMCJ1gOoQFU1lLNneqVbs7AOG5AYHvmmMXVkxnx/GtDiX4MLz4L2UEq9lwF7xfU yBM3kz0+GtLiU/2gCQqiGWEm37hWjIpPMXUHf4w3Yj3/8HcVTt2dV0WJoSvnQiG10MXN VD8j0Ntx4iLKsFkhbovRSe/AV6j9ZtRy+uUiqsSmAsx5JmJFGYoCmKzkIqLYI9XW928e 6DPICrzxfBBqwGwFtF7VY7g+PhuX8CR6uj5S9NvOYDet/BLkNvKT92iSa97TA3aq1KMI p8SeQpGLXVHYqaeCRrhTqxHsxgdgyjFkQ4ZNpdMZPqsnhdLEt0KgOV16Bp6gyrmrcwdy jRlA== X-Gm-Message-State: AMCzsaXiBUJyikmciDfHm6fnQo7YOIdkFX5Bj0cG18jhiRLKr43NMmyw 7XFI5yKMyNpnhitRWGDZJXM+3miYq8j/FV4YXVNgDg== X-Google-Smtp-Source: AOwi7QCNaKw8icu8BoP7RzF5dRCNpXAwZOM5r/BTEaHosTc8MfEMcTTcBxY5WmYkp/vtLAegYEWmfN1pOyrFIi7UkqM= X-Received: by 10.36.107.21 with SMTP id v21mr3502595itc.43.1507823215485; Thu, 12 Oct 2017 08:46:55 -0700 (PDT) MIME-Version: 1.0 Received: by 10.107.131.167 with HTTP; Thu, 12 Oct 2017 08:46:54 -0700 (PDT) In-Reply-To: <20171012154346.g3pw5asmfgbv2ist@bivouac.eciton.net> References: <20171002194753.4316-1-evan.lloyd@arm.com> <74D8A39837DF1E4DA445A8C0B3885C503A9D7016@shsmsx102.ccr.corp.intel.com> <20171012154346.g3pw5asmfgbv2ist@bivouac.eciton.net> From: Ard Biesheuvel Date: Thu, 12 Oct 2017 16:46:54 +0100 Message-ID: To: Leif Lindholm Cc: Evan Lloyd , "Yao, Jiewen" , "edk2-devel@lists.01.org" , Matteo Carlini , Stephanie.Hughes-Fitt@arm.com, thomas.abraham@arm.com, Arvind.Chauhan@arm.com, Daniil Egranov Subject: Re: [PATCH 0/2] Dynamic Tables X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 12 Oct 2017 15:43:26 -0000 Content-Type: text/plain; charset="UTF-8" On 12 October 2017 at 16:43, Leif Lindholm wrote: > On Tue, Oct 10, 2017 at 06:52:02PM +0000, Evan Lloyd wrote: >> > I notice you only defined ARM namespace in this patch, and implemented >> > ARM library instance. >> > Also most consumers of ConfigurationManager are from ARM platform >> > package. So if it urgent from ARM platform, you may consider to check into >> > ArmPkg at first. >> >> [[Evan Lloyd]] This sounds sensible, and I will discuss it with >> Leif. Another option might be to start up a new module. > > My preference would be for this to start as a branch in edk2-staging. > > While there, we can bikeshed over the naming and placement (but I will > admit to a certain affinity for Sean's suggestion to keep it separate). > > I do not think it is a good fit for ArmPkg. > +1 We are working very hard to reduce ArmPkg to pieces that are actually specific to the ARM architecture. The fact that the Intel engineers may not be interested in using this new dynamic table feature for their platforms does not make it an ARM thing. > If Evan can send out a request for creation as described in > https://github.com/tianocore/edk2-staging/blob/about/README, > I am happy to do the mechanics. >