Bug 1305915
| Summary: | [RFE] create a storage "domain" for OVMF non-volatile storage | ||
|---|---|---|---|
| Product: | [oVirt] ovirt-engine | Reporter: | Martin Polednik <mpoledni> |
| Component: | BLL.Storage | Assignee: | Tal Nisan <tnisan> |
| Status: | CLOSED DEFERRED | QA Contact: | Avihai <aefrat> |
| Severity: | high | Docs Contact: | |
| Priority: | high | ||
| Version: | 4.0.0 | CC: | bugs, lersek, michal.skrivanek, mprivozn, tnisan |
| Target Milestone: | --- | Keywords: | FutureFeature, Improvement |
| Target Release: | --- | Flags: | ykaul:
ovirt-future?
rule-engine: planning_ack? rule-engine: devel_ack? rule-engine: testing_ack? |
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Enhancement | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2020-04-01 14:45:28 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | Storage | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
| Bug Depends On: | 1305922, 1305942 | ||
| Bug Blocks: | 1112275 | ||
|
Description
Martin Polednik
2016-02-09 15:00:47 UTC
(
Small correction: the room that is directly usable for non-volatile variables is actually 56KB. Quoting the OVMF whitepaper:
Description Compression type Size
------------------------------ ---------------------- -------
Non-volatile data storage open-coded binary data 128 KB
Variable store 56 KB
Event log 4 KB
Working block 4 KB
Spare area 64 KB
Other than this, the description seems correct. Thanks.
)
I also recommend to inquire the libvirt developers (esp. Michal Privoznik) about whether, and how, libvirtd instantiates the nvram file for a brand new VM, from the varstore *template* that accompanies the firmware binary. (In reply to Laszlo Ersek from comment #2) > I also recommend to inquire the libvirt developers (esp. Michal Privoznik) > about whether, and how, libvirtd instantiates the nvram file for a brand new > VM, from the varstore *template* that accompanies the firmware binary. Libvirt does nothing more than equivalent of 'dd if=template of=domain_specific'. It's done in C completely rather than spawning dd process though. To my understanding this feature was removed from the 4.1 list, since q38 will not be supported in 4.1. Moving out. (In reply to Yaniv Lavi from comment #6) > To my understanding this feature was removed from the 4.1 list, since q35 > will not be supported in 4.1. Moving out. Same for 4.2, so deferring it. The size of the varstore file has changed due to fixing bug 1443351; the new size is 528KB. For details, please refer to the commit message on <https://github.com/tianocore/edk2/commit/b24fca05751f> ("OvmfPkg: introduce 4MB flash image (mainly) for Windows HCK", 2017-04-29): Description Compression type Size [KB] ------------------------- ----------------- ---------------------- Non-volatile data storage open-coded binary 128 -> 528 ( +400) data Variable store 56 -> 256 ( +200) Event log 4 -> 4 ( +0) Working block 4 -> 4 ( +0) Spare area 64 -> 264 ( +200) I understand it was determined this is not needed for q35 support. Can this be closed then? (In reply to Yaniv Lavi from comment #9) > I understand it was determined this is not needed for q35 support. Can this > be closed then? It is not needed for q35 with Seabios. It is needed for q35 with UEFI and Secure Boot This bug didn't get any attention for a while, we didn't have the capacity to make any progress. If you deeply care about it or want to work on it please assign/target accordingly This bug didn't get any attention for a while, we didn't have the capacity to make any progress. If you deeply care about it or want to work on it please assign/target accordingly ok, closing. Please reopen if still relevant/you want to work on it. ok, closing. Please reopen if still relevant/you want to work on it. |