From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wen Congyang Subject: Re: [SeaBIOS] [PATCH RFC] acpi: add ssdt for cpi hotplug Date: Wed, 02 Nov 2011 15:20:04 +0800 Message-ID: <4EB0EF24.3020002@cn.fujitsu.com> References: <20111101191139.GA18317@redhat.com> <4EB0DE45.1010200@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: seabios@seabios.org, kvm@vger.kernel.org To: "Michael S. Tsirkin" Return-path: Received: from cn.fujitsu.com ([222.73.24.84]:55592 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1751450Ab1KBHSZ (ORCPT ); Wed, 2 Nov 2011 03:18:25 -0400 In-Reply-To: <4EB0DE45.1010200@cn.fujitsu.com> Sender: kvm-owner@vger.kernel.org List-ID: At 11/02/2011 02:08 PM, Wen Congyang Write: > At 11/02/2011 03:11 AM, Michael S. Tsirkin Write: >> So here's the plan: move all hotplug handling out >> to ssdt, this way it'll keep working even with a >> user-supplied dsdt. Next step we can patch >> this ssdt at runtime. >> >> There's little point in this change alone, so posting as RFC, >> will repost with the patching part when it's ready, >> posting now to present opportunity for early feedback. >> >> Compiled only. > > Hot plug on PCI bus 0 can not work with this patch. > It can works without this patch. I think that I may find the reason: ================================ # ls -l /sys/firmware/acpi/tables/ total 0 -r--------. 1 root root 0 Nov 2 14:23 APIC -r--------. 1 root root 0 Nov 2 14:23 DSDT drwxr-xr-x. 2 root root 0 Nov 2 14:23 dynamic -r--------. 1 root root 0 Nov 2 14:23 FACP -r--------. 1 root root 0 Nov 2 14:23 FACS -r--------. 1 root root 0 Nov 2 14:23 HPET -r--------. 1 root root 0 Nov 2 14:23 SSDT # ls -l /sys/firmware/acpi/tables/dynamic/ total 0 ================================ The kernel only loads one SSDT. I use iasl to look at the content of SSDT, and find that it is ssdt-proc.dsl. The kernel does not load ssdt-pcihp.dsl. Thanks Wen Congyang