From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-il1-f172.google.com (mail-il1-f172.google.com [209.85.166.172]) by mx.groups.io with SMTP id smtpd.web09.68656.1638225209321093447 for ; Mon, 29 Nov 2021 14:33:29 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=qzpBcOWc; spf=pass (domain: linaro.org, ip: 209.85.166.172, mailfrom: masami.hiramatsu@linaro.org) Received: by mail-il1-f172.google.com with SMTP id e8so19099113ilu.9 for ; Mon, 29 Nov 2021 14:33:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=i+BbY0cItzn5UH0TANVEUbObNbq4ma3mdgcM8WId9XE=; b=qzpBcOWc9XoqGnl8AFzM5K5oGyk2L2WtRQiaA+li7YCMyvrGJCBclJEpcwnepMCWFi vg8HME/nTejj8X3hio1Ic/u9hF50PZvNP/cEd5vd9+VnhyID2/LGoFO6pTilm458xF8w U5IPkXQGgxNN3OMmlDgmhXCuo5m+HDlwAmKWOKaYVM60BT1FtgWidBYA9UnsfZfz9Rm6 Umm5FDmKqdXD64IZF5ePAseRCogTJA3fP0c4/qAu05J8GDjP89N9EwK/GWwFScQRcKHN DZFx69u+LFLECEbGQf/5NziFu6WDXDCMeBC0RJx4M691/M696ZpKdPkiZJ/B8rNXdVbl 19tQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=i+BbY0cItzn5UH0TANVEUbObNbq4ma3mdgcM8WId9XE=; b=1BclKWTecoqDdpBQp63elL9zt6CQmSXm2bJiQagzWvXkSuI6nvMcKgOvl/+0g0d8yL HD7mvwpkk3ppaMrozz2/V2QsTQG+9N4UOnHUuv1BBP3+uk/Q/R3fPKYduNFq2l7lbTdY 2gF9MQv/kdT7sLX+9I77spZqZW21iZmj+J4svo1m6AS1ZtadBxWgH6sKNmNK/01Zlfj/ 9aBTOddqFYBFAO15Ai6j1OxfI14rcC1rPRIXdIrx9lhbbXPOpOomY0nqGEEQc4ilJc2b uLHm0WjTFNsR1+mnKXOV+T9+CyFzDF5Kh1BEEcXeaIe2AYVDc/ixTZaZROpJ6BAXNA1+ T9Dw== X-Gm-Message-State: AOAM533xd7q9i7hURZa3TmcZbFz4qpI0KWq7hEqVujZaJqRzt/kYnBdL Omie5jbUDlfKMWRgdR1O8/+5gmC2w4Q8sqocwRb0kw== X-Google-Smtp-Source: ABdhPJz1BGHab3NQCAlpt9VvFxp9XbcfczuODausAZE4eDOPSxfyYPI0QRoRUMKk6T+O3z8k+KRkPnJ1vSkprU7dzLc= X-Received: by 2002:a05:6e02:1b8a:: with SMTP id h10mr50830070ili.14.1638225208630; Mon, 29 Nov 2021 14:33:28 -0800 (PST) MIME-Version: 1.0 References: <163610419943.391624.9289897029386201296.stgit@localhost> <163610424153.391624.15870608392900932158.stgit@localhost> In-Reply-To: From: "Masami Hiramatsu" Date: Tue, 30 Nov 2021 07:33:17 +0900 Message-ID: Subject: Re: [PATCH 5/5] [edk2-platforms] Platform/DeveloperBox: Expand NvStorage sizes To: Leif Lindholm Cc: Ard Biesheuvel , devel@edk2.groups.io, Kazuhiko Sakamoto , Masahisa Kojima Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Hi Leif, 2021=E5=B9=B411=E6=9C=8829=E6=97=A5(=E6=9C=88) 22:43 Leif Lindholm : > > On Sat, Nov 27, 2021 at 16:48:45 +0900, Masami Hiramatsu wrote: > > > On Fri, Nov 05, 2021 at 18:24:01 +0900, Masami Hiramatsu wrote: > > > > Expand NvStorage Variable size and FTW spare/working size > > > > for the DeveloperBox platform. > > > > > > > > Since the size of the NvStorage VariableSize is not enough > > > > large, FWTS uefirttime test, which updates the NV > > > > variables in runtime, failes. This expands the size to fix > > > > this issue. > > > > > > Does this change erase all existing variables? > > > > Ah, indeed. It may need to erase all variables. > > That is quite likely to lead to upset users. OK. > > > If so, I think it is worth introducing this as a non-default build > > > option, in order to not wreck existing installations on a firmware > > > update. > > > > > > I think it would also be worth considering whether to update > > > PcdLowestSupportedFirmwareVersion. PcdFirmwareRevision > > > should definitely be updated. > > > > I'm not sure about this point. > > You meant we should have 2 different revisions like a branch? > > - Branch A(current version): keep the variable area size the same. > > - Branch B(new version): expand the variable area. > > And a build option will change the branch by updating the > > PcdFirmwareRevision? > > Not a branch - just that you need to explicitly build for the size of > flash area you want to use, and if you provide pre-built downloadable > ones - provide two variants. I got it. > This becomes a bit of a maintenance nightmare over time. Actually, I'm considering a kind of "leap" firmware release, which involves all firmware update by manual (not automatic), because the SCP-firmware is too old anymore and the new SCP firmware (OSS version) requires to update TF-A, which is not compatible with old ones. Obviously, this must be done by manual. So, afterwards, we will not release old version anymore. Anyway, the old firmware snapshot image is not updated in one year (since source repository has not been updated). The firmware on LVFS is released in 2019. (BTW, can I change the UUID which fwupd detects too?) I will provide a build option for the users who update EDK2 by themselves. > A better solution would be for the firmware to (somehow) resize the > parameter area - retaining existing values - if it encounters the > smaller version. I don't think we have an example of that. Hmm, I rather like to erase it while the "leap" update, since the backward compatibility is not guaranteed. And after the update, user will be able to choose the U-Boot on the DeveloperBox. > PcdLowestSupportedFirmwareVersion still needs to be set, to the same > value as the new PcdFirmwareRevision, to prevent downgrading to a > version that does not support the larger size. OK, so this is for protecting rollback. But this means, do I need to make it optional (switched by build option) too? > > > Also PcdLowestSupportedFirmwareVersion you meant is > > in the capsule file? > > I meant to change the Pcd value. That implements the change in > SystemFirmwareDescriptorTable.aslc. OK. Thank you, > > Regards, > > Leif > > > Thank you, > > > > > > > > > > / > > > Leif > > > > > > > Signed-off-by: Masami Hiramatsu > > > > Reported-by: Kazuhiko Sakamoto > > > > --- > > > > .../Socionext/DeveloperBox/DeveloperBox.dsc.inc | 10 +++++---= -- > > > > 1 file changed, 5 insertions(+), 5 deletions(-) > > > > > > > > diff --git a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc b= /Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc > > > > index 0a364bc457..3baf97ecc0 100644 > > > > --- a/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc > > > > +++ b/Platform/Socionext/DeveloperBox/DeveloperBox.dsc.inc > > > > @@ -280,11 +280,11 @@ > > > > gFip006DxeTokenSpaceGuid.PcdFip006DxeMemBaseAddress|0x08000000 > > > > > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableBase|0x0= 8400000 > > > > - gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableSize|0x0= 0010000 > > > > - gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingBase|0= x08410000 > > > > - gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingSize|0= x00010000 > > > > - gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareBase|0x0= 8420000 > > > > - gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareSize|0x0= 0010000 > > > > + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageVariableSize|0x0= 0080000 > > > > + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingBase|0= x08480000 > > > > + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwWorkingSize|0= x00080000 > > > > + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareBase|0x0= 8500000 > > > > + gEfiMdeModulePkgTokenSpaceGuid.PcdFlashNvStorageFtwSpareSize|0x0= 0080000 > > > > > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultOemId|"SNI " > > > > gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultOemTableId|0x524341= 55514e5953 # SYNQUACR > > > > > > > > > > > > -- > > Masami Hiramatsu --=20 Masami Hiramatsu