public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: Jeremy Linton <jeremy.linton@arm.com>
To: Ard Biesheuvel <ard.biesheuvel@linaro.org>,
	Udit Kumar <udit.kumar@nxp.com>
Cc: "grant.likely@linaro.org." <grant.likely@linaro.org>,
	"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
	"Olivier.Martin@arm.com" <Olivier.Martin@arm.com>
Subject: Re: Storing Non volatile variables on SD/NAND
Date: Mon, 18 Sep 2017 15:47:45 -0500	[thread overview]
Message-ID: <a4205448-eb9c-93b1-0a2d-e8d79bbd4437@arm.com> (raw)
In-Reply-To: <CAKv+Gu8PzYUtmQdfCOQxeY_g_98YUhn9Mg=B=Ksvk-kn+tHBOQ@mail.gmail.com>

On 09/18/2017 10:43 AM, Ard Biesheuvel wrote:
> On 18 September 2017 at 06:52, Udit Kumar <udit.kumar@nxp.com> wrote:
>> Hi EDK-2 Experts,
>> I am looking to store NV variables on SD/NAND device.
>>
>> While browsing, I came across some old post at link,
>> http://feishare.com/efimail/messages/20130319-1700-Re__edk2__Regarding_storing_Boot_Device_Config_in_persistent_memory-Olivier_Martin.html
>>
>> Looks like, this is possible easily.
> 
> That's a bold statement dude :-)
> 
>>>> What you need to support Non-Volatile UEFI variables is a Non-Volatile Memory. And also a driver that implements the EFI Firmware Volume Block protocol for this NVM device.
>>
>> But MdeModulePkg does Copymem from NV variable start memory to some allocated buffers.  With SD/NAND Copymem is not possible, Is this something changes since 2013 or there are some other way to use SD/NAND
>>
> 
> No, SD/MMC cannot currently be used as the backing store for the EFI
> variable store. The problem is that the variable protocols are
> architectural protocols in PI that need to be present before any
> driver model drivers are dispatched, and so putting the variable store
> on block devices is not something that the PI software architecture
> currently supports (unless you reimplement the whole driver stack as
> DXE drivers).
> 
> On top of that, it is almost impossible to share a block device that
> sits behind a controller between the firmware and the OS at runtime
> (i.e., for SetVariable() calls made by efibootmgr under Linux),
> because only a single agent can take ownership of the controller at
> any given time. (You /could/ dedicate the SD/MMC to the firmware
> entirely, and boot from SATA or USB, but this is out of the question
> on most platforms that need to use SD/MMC for that variable backing
> store, i.e., mobile platforms)
> 
> The best thing would be for you to convince the hardware architects in
> your company to design and implement dual-ported SD/MMC controllers
> that allow a single SD/MMC to have two logical views that are
> independent (although I'm unsure if that is even possible in the
> context of the SD/MMC specifications)

Which still has the problems of selecting "use whole disk" during an OS 
install bricking the machine. Or similarly if the emmc layout isn't just 
right having gparted automatically "fix" the partition table (as it does 
with many of the hikey images) again bricking the machine.

Having the firmware/variable store and OS root/boot on the same device 
is fundamentally flawed. I've went down the path of simply disabling the 
hikey/emmc for use beyond the firmware/variable storage on the hikey. Of 
course I ran smack into the problem of making the block device DXE's 
run-time safe which I've about concluded is far harder than simply 
writing a monolithic variablestore->emmc driver.

The ideal situation for the Hikey, is probably to solder a SPI flash to 
the SPI controller and put the firmware/variable store on that, and 
leave the eMMC entirely for linux's use.



  parent reply	other threads:[~2017-09-18 20:44 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-18 13:52 Storing Non volatile variables on SD/NAND Udit Kumar
2017-09-18 15:43 ` Ard Biesheuvel
2017-09-18 16:52   ` Vladimir Olovyannikov
2017-09-19  5:28     ` Udit Kumar
2017-09-19 16:38       ` Ard Biesheuvel
2017-09-20  4:27         ` Udit Kumar
2017-09-20  4:47           ` Andrew Fish
2017-09-20  5:09             ` Udit Kumar
2017-09-20  5:17               ` Andrew Fish
2017-09-20 14:51                 ` Pankaj Bansal
2017-09-20 17:34                   ` Udit Kumar
2017-09-20 17:39                     ` Ard Biesheuvel
2017-09-20 17:39                       ` Ard Biesheuvel
2017-09-20 17:46                         ` Andrew Fish
2017-10-27  9:35                           ` Udit Kumar
2017-10-27  9:37                       ` Udit Kumar
2017-10-27 13:20                         ` Ard Biesheuvel
2017-10-27 17:46                       ` Jeremy Linton
2017-10-28  3:09                         ` Udit Kumar
2017-10-30 23:10                           ` Jeremy Linton
2017-10-31  4:25                             ` Udit Kumar
2017-09-20 17:34                   ` Andrew Fish
2017-09-18 20:47   ` Jeremy Linton [this message]
2017-09-18 20:53     ` Ard Biesheuvel
2017-09-19  5:27       ` Udit Kumar

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=a4205448-eb9c-93b1-0a2d-e8d79bbd4437@arm.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox