Gleb Natapov wrote:
On Mon, Feb 01, 2010 at 10:31:30PM +0800, Liu, Jinsong wrote:
Kevin O'Connor wrote:
On Sat, Jan 30, 2010 at 11:10:43PM +0800, Jiang, Yunhong wrote:
For how to setup the memory as NVS, you can refer to src/memmap.c, maybe using add_e820().
Adding to the e820 is simple. Just align the madt to 4k:
madt_size = ALIGN(madt_size, PAGE_SIZE); struct multiple_apic_table *madt = memalign_high(PAGE_SIZE, madt_size);
and then add it to the e820:
add_e820((u64)madt, madt_size, E820_NVS);
It's not strictly necessary to align to 4k, but I think it makes sense to do that.
-Kevin
Gleb, Yunhong, and Connor,
Thank you all for help me review and give me valuable suggestion! I read ACPI spec 4.0 and think Gleb is correct. I update my patch accordingly as attached, please help me to review it.
You left build_ssdt() but removed code that create its content, so now empty SSDT is created. Just remove the whole function (and don't forget to update you sources since you patch applies with errors).
Kevin, while testing this patch I noticed that Seabios actually doesn't mark memory ACPI tables reside in as ACPI DATA!
Thanks for review! I removed build_ssdt().
BTW, I just pull seabios from git://git.kernel.org/pub/scm/virt/kvm/seabios.git, and find the latest patch is still 5da68339ecf44677b8f4f115cdf3cb1da46a9f6c. Maybe the error when apply my last patch comes from it included some middle-aml code of src/acpi-dsdt.hex? In the new patch attached, I cancel the middle-aml code (generated at compile stage) of src/acpi-dsdt.hex.
Best Regards, Jinsong