From: Markus Armbruster <armbru@redhat.com>
To: Laszlo Ersek <lersek@redhat.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
QEMU Developers <qemu-devel@nongnu.org>,
Zheng Xiang <zhengxiang9@huawei.com>,
qemu-arm <qemu-arm@nongnu.org>, Heyi Guo <guoheyi@huawei.com>,
wanghaibin.wang@huawei.com
Subject: Re: [Qemu-devel] [RFC PATCH] hw/arm/virt: use variable size of flash device to save memory
Date: Tue, 26 Mar 2019 19:36:19 +0100 [thread overview]
Message-ID: <8736n9eb4s.fsf@dusky.pond.sub.org> (raw)
In-Reply-To: <abe1d9cd-85bb-2ef8-630f-af28cae6378c@redhat.com> (Laszlo Ersek's message of "Tue, 26 Mar 2019 18:10:36 +0100")
Laszlo Ersek <lersek@redhat.com> writes:
> On 03/26/19 17:39, Markus Armbruster wrote:
>> Laszlo Ersek <lersek@redhat.com> writes:
>
>>> With the dynamic sizing in QEMU (which, IIRC, I had originally
>>> introduced still in the 1MB times, due to the split between the
>>> executable and varstore parts), both the 1MB->2MB switch, and the
>>> 2MB->4MB switch in the firmware caused zero pain in QEMU. And right now,
>>> 4MB looks like a "sweet spot", with some elbow room left.
>>
>> Explicit configuration would've been exactly as painless. Even with
>> pflash sizes restricted to powers of two.
>
> I wrote the patch that ended up as commit 637a5acb46b3 -- with your R-b
> -- in 2013. I'm unsure if machine type properties existed back then, but
> even if they did, do you think I knew about them? :)
>
> You are right, of course; it's just that we can't tell the future.
True! All we can do is continue to design as well as we can given the
information, experience and resources we have, and when the inevitable
design mistakes become apparent, limit their impact.
Some of the things we now consider mistakes we just didn't see. Others
we saw (e.g. multiple pflash devices, unlike physical hardware), but
underestimated their impact.
[...]
next prev parent reply other threads:[~2019-03-26 18:42 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-25 12:51 [Qemu-arm] [RFC PATCH] hw/arm/virt: use variable size of flash device to save memory Xiang Zheng
2019-03-25 13:11 ` Peter Maydell
2019-03-25 14:03 ` Zheng Xiang
2019-03-26 6:17 ` [Qemu-arm] [Qemu-devel] " Markus Armbruster
2019-03-26 11:03 ` Laszlo Ersek
2019-03-26 16:39 ` Markus Armbruster
2019-03-26 17:10 ` Laszlo Ersek
2019-03-26 18:36 ` Markus Armbruster [this message]
2019-04-03 14:12 ` Xiang Zheng
2019-04-03 14:12 ` Xiang Zheng
2019-04-03 15:35 ` [Qemu-arm] " Laszlo Ersek
2019-04-03 15:35 ` Laszlo Ersek
2019-04-08 13:43 ` [Qemu-arm] " Xiang Zheng
2019-04-08 13:43 ` Xiang Zheng
2019-04-08 16:14 ` [Qemu-arm] " Laszlo Ersek
2019-04-08 16:14 ` Laszlo Ersek
2019-04-09 3:39 ` [Qemu-arm] " Xiang Zheng
2019-04-09 3:39 ` Xiang Zheng
2019-04-09 6:01 ` [Qemu-arm] " Markus Armbruster
2019-04-09 6:01 ` Markus Armbruster
2019-04-09 6:01 ` Markus Armbruster
2019-04-09 8:28 ` [Qemu-arm] " Kevin Wolf
2019-04-09 8:28 ` Kevin Wolf
2019-04-09 8:28 ` Kevin Wolf
2019-04-10 8:36 ` [Qemu-arm] " Xiang Zheng
2019-04-10 8:36 ` Xiang Zheng
2019-04-10 8:36 ` Xiang Zheng
2019-04-11 7:15 ` [Qemu-arm] " Markus Armbruster
2019-04-11 7:15 ` Markus Armbruster
2019-04-12 9:26 ` [Qemu-arm] " Xiang Zheng
2019-04-12 9:26 ` Xiang Zheng
2019-04-11 12:22 ` [Qemu-arm] " Kevin Wolf
2019-04-11 12:22 ` Kevin Wolf
2019-04-11 12:22 ` Kevin Wolf
2019-04-12 1:52 ` [Qemu-arm] " Xiang Zheng
2019-04-12 1:52 ` Xiang Zheng
2019-04-12 1:52 ` Xiang Zheng
2019-04-12 9:50 ` [Qemu-arm] " Xiang Zheng
2019-04-12 9:50 ` Xiang Zheng
2019-04-12 9:50 ` Xiang Zheng
2019-04-12 10:57 ` [Qemu-arm] " Kevin Wolf
2019-04-12 10:57 ` Kevin Wolf
2019-04-12 10:57 ` Kevin Wolf
2019-04-15 2:39 ` [Qemu-arm] " Xiang Zheng
2019-04-15 2:39 ` Xiang Zheng
2019-04-15 2:39 ` Xiang Zheng
2019-04-22 1:37 ` [Qemu-arm] " Xiang Zheng
2019-04-22 1:37 ` Xiang Zheng
2019-04-22 1:37 ` Xiang Zheng
2019-03-25 14:07 ` Laszlo Ersek
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-all 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=8736n9eb4s.fsf@dusky.pond.sub.org \
--to=armbru@redhat.com \
--cc=ard.biesheuvel@linaro.org \
--cc=guoheyi@huawei.com \
--cc=lersek@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-arm@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=wanghaibin.wang@huawei.com \
--cc=zhengxiang9@huawei.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.