Re: [PATCH] QEMU - provide e820 reserve through qemu_cfg

[Alexander Graf <agraf@suse.de>]

Am 25.01.2010 um 18:13 schrieb Jes Sorensen <Jes.Sorensen@redhat.com>:

> On 01/25/10 17:58, Alexander Graf wrote:
>> 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.
>
> I am fine with having QEMU build the e820 tables completely if there  
> is
> a consensus to take that path.

I agree. We better get this right :-). I don't want to maintain 5  
versions of an 380 fw_cfg interface.

>
>> 2) Please inline patches. They showed up as attachments here,  
>> making them really hard to comment on.
>
> Sorry Thunderbug doesn't do that well, but they should be attached as
> txt?

I suppose so, but Apple Mail still doesn't show it and definitely  
doesn't let me comment on it.

The easiest way for me so far has been to use git-send-email and a  
bash alias to fill in login information.


Alex
>