From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: None (no SPF record) identity=mailfrom; client-ip=2607:f8b0:4864:20::d44; helo=mail-io1-xd44.google.com; envelope-from=mw@semihalf.com; receiver=edk2-devel@lists.01.org Received: from mail-io1-xd44.google.com (mail-io1-xd44.google.com [IPv6:2607:f8b0:4864:20::d44]) (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 8D64C2117D75A for ; Wed, 23 Jan 2019 00:28:52 -0800 (PST) Received: by mail-io1-xd44.google.com with SMTP id l14so1034141ioj.5 for ; Wed, 23 Jan 2019 00:28:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=semihalf-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=SEZ7pf56O6WuuV9Xce6PIbK8q8spBDKxVn+SuNuyLRE=; b=vJQJLdJDy0UYF/7GayYRVLXsjvs83Rb08kQd77Qvjv+sBBDDciI/Hr2ptmtscIcbjC KRF0803Tm7M1Inyg3fT3IDHhNJiMoSVwGFBa8YpGp3PfBZ45QORDHBDRodSzHW3iabJF JxmTWkB+pIMLSzcZNXcdA3mjneGl+nuUU534dklmO5ywcUmDczHfyq4XYeixd7HG2jN6 uS+kLDHYnXV/VBZndVSMkuvJZrj3h40gTo2wAw4bnWt/dAQGdKoOYWQAWTTz6NV//zj4 dNqi43B8XQLMGNhq7OtV9sJyXxoyJaa0URZP4L5lhL+rUqUua221yzaqnxZ7GvxTIyuv U5Ig== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=SEZ7pf56O6WuuV9Xce6PIbK8q8spBDKxVn+SuNuyLRE=; b=GUEBUu62uH512yqSiJphAmT3piXYeKxmFRuCKY/on1hXnDHJ2hydqxPWMmQbUw/jT6 un3675xJGnKdnj5lbuA7R6KzQ2wGZ1dfjH/VgsuP4Q5TGMt7F7jQngftRwgEWEHiMbem E6/2rlRZdg7gj6v3NEhQOmZMhhKca4JnoSBkGuBhdp70FmFpoG0MVYq6wlQ3SzsD/HJ3 c+TbtM5LwK7VCOb4tjCVIk3UYK8DcChS0+rJPo+RVz6HPKtF4AemIq4zMv3g3Js27cTv aPyPEl6281v+1/bkmquUYYsk1XiBL37+2tFZdf2b/37Eb56ptgFCBGSGxIWVfGBj8AGS iCsQ== X-Gm-Message-State: AHQUAubNRBc+lTPS5UiBtWh6fw5X5yUIjnvKaQ7Ob8S+WUtdQ5XcGMK6 aUitYYqGK3OgOmfVweRR49+eQqApSIFr156/RvrO9w== X-Google-Smtp-Source: ALg8bN41j8szHvD1mU1jeRjmU0K97ZdCJKj6NnuVWKm1/p9DK7ipgg1yD3v+GMrHm0PQB6aXJW1Mv4VfOWGCwB5i4WA= X-Received: by 2002:a6b:92d6:: with SMTP id u205mr691413iod.221.1548232131355; Wed, 23 Jan 2019 00:28:51 -0800 (PST) MIME-Version: 1.0 References: <1548120742-11928-1-git-send-email-mw@semihalf.com> <1548120742-11928-2-git-send-email-mw@semihalf.com> <20190122172643.i6newphbfjiertsb@bivouac.eciton.net> <20190122190649.x2bh7gd5szxmfxy5@bivouac.eciton.net> <20190122202627.ccu3koeskydsvjus@bivouac.eciton.net> <20190122210937.rf44pvv6gaudlme7@bivouac.eciton.net> In-Reply-To: <20190122210937.rf44pvv6gaudlme7@bivouac.eciton.net> From: Marcin Wojtas Date: Wed, 23 Jan 2019 09:28:40 +0100 Message-ID: To: Leif Lindholm Cc: edk2-devel-01 , Ard Biesheuvel , nadavh@marvell.com, "jsd@semihalf.com" , Grzegorz Jaszczyk , Kostya Porotchkin Subject: Re: [platforms: PATCH v2 1/4] Marvell/Armada7k8k: Shift PEI stack base X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 23 Jan 2019 08:28:52 -0000 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi Leif, wt., 22 sty 2019 o 22:10 Leif Lindholm napisa=C5= =82(a): > > On Tue, Jan 22, 2019 at 09:56:14PM +0100, Marcin Wojtas wrote: > > > > > I think I gave my suggestion for the resolution of this problem (= with > > > > > moving StackBase to 0x05400000 as the alternative) in my previous > > > > > reply. > > > > > > > > > > > > > Yes, and I answered, presenting the alternative memory map with > > > > additional 64kB "cut out" on top of 20MB "hole" of memory, which I'= m > > > > not fancy, given available space inside the 20MB chunk. > > > > > > Please go back and reread my first and my second email. > > > Then please point out where I have, other than as an alternative > > > solution, suggested growing the cutout size. > > > > > > Then perhaps we can rewind this conversation and try again? > > > > Ok. So would it be sufficient to replace > > gMarvellTokenSpaceGuid.PcdSecureRegionBase with two sets of separate > > PCDs for ARM-TF runtime services and OPTEE leaving the PEI stack base > > @0x43f0000? > > That would be lovely, thank you :) > > (Although your reference to wanting to keep the PEI stack area out of > the hands of the operating system might mean that you want three? I'll > leave that to your discretion.) > PEI stack has its own PCDs: gArmPlatformTokenSpaceGuid.PcdCPUCoresStackBase gArmPlatformTokenSpaceGuid.PcdCPUCorePrimaryStackSize I want to keep it simple (and btw aligned with U-Boot booting the mainline DTB with single 20MB reserved memory area), so what I intend to do is to limit reserved region in Armada7k8kMemoryInitPeiLib.c with PcdArmTFRegionBase (@0x4000000) up to PcdOpteeRegionBase + PcdOpteeRegionSize (@0x5400000). Best regards, Marcin