[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg
From: |
Alexander Graf |
Subject: |
[Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg |
Date: |
Mon, 25 Jan 2010 17:58:26 +0100 |
On 25.01.2010, at 17:52, Jes Sorensen wrote:
> Hi,
>
> This is the QEMU patch for providing the e820-reserve space through
> qemu-cfg.
Howdy. Congratulations to the new mail address - looks neat ;-).
Two comments:
1) I don't see how passing a single region is any help. I'd rather like to see
a device tree like table structure
You'd get one variable for len of the table, one with the contents. So for a
universal reserved region specifier you'd get:
<u64 base><u64 len>
Then have len=2 and put data in the table:
<u64 base1><u64 len1><u64 base2><u64 len2>
That way we'd get 2 entries and the chance to enhance them later on. In fact,
it might even make sense to pass the whole table in such a form. That way qemu
generates all of the e820 tables and we can declare whatever we want. Just add
a type field in the table.
2) Please inline patches. They showed up as attachments here, making them
really hard to comment on.
Alex
- [Qemu-devel] Re: [PATCH] Seabios - read e820 reserve from qemu_cfg, (continued)
- [Qemu-devel] Re: [PATCH] Seabios - read e820 reserve from qemu_cfg, Kevin O'Connor, 2010/01/25
- [Qemu-devel] [PATCH] Seabios - read e820 table from qemu_cfg, Jes Sorensen, 2010/01/26
- [Qemu-devel] Re: [PATCH] Seabios - read e820 table from qemu_cfg, Kevin O'Connor, 2010/01/27
- [Qemu-devel] Re: [PATCH] Seabios - read e820 table from qemu_cfg, Jes Sorensen, 2010/01/29
- [Qemu-devel] Re: [PATCH] Seabios - read e820 table from qemu_cfg, Gleb Natapov, 2010/01/29
- [Qemu-devel] Re: [PATCH] Seabios - read e820 table from qemu_cfg, Kevin O'Connor, 2010/01/29
[Qemu-devel] [PATCH] QEMU-KVM - provide e820 reserve through qemu_cfg, Jes Sorensen, 2010/01/25
- [Qemu-devel] [PATCH] QEMU - provide e820 reserve through qemu_cfg, Jes Sorensen, 2010/01/25
- [Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg,
Alexander Graf <=
- [Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg, Jes Sorensen, 2010/01/25
- [Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg, Alexander Graf, 2010/01/25
- [Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg, Jes Sorensen, 2010/01/25
- [Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg, Alexander Graf, 2010/01/25
- [Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg, Anthony Liguori, 2010/01/25
- [Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg, Jes Sorensen, 2010/01/25
- [Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg, Alexander Graf, 2010/01/25
- [Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg, Jes Sorensen, 2010/01/25
[Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg, Gleb Natapov, 2010/01/26
[Qemu-devel] Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg, Jes Sorensen, 2010/01/26