From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:55895) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SUhOV-0008Hp-3n for qemu-devel@nongnu.org; Wed, 16 May 2012 12:48:12 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SUhOM-0000Zw-Lm for qemu-devel@nongnu.org; Wed, 16 May 2012 12:48:10 -0400 Received: from mx1.redhat.com ([209.132.183.28]:13746) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SUhOM-0000Zm-Dl for qemu-devel@nongnu.org; Wed, 16 May 2012 12:48:02 -0400 Message-ID: <4FB3DA3B.7000006@redhat.com> Date: Wed, 16 May 2012 18:47:55 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <1336998923-30144-1-git-send-email-gleb@redhat.com> <1336998923-30144-2-git-send-email-gleb@redhat.com> <20120515014319.GA21845@morn.localdomain> <20120515080605.GD32036@redhat.com> <20120515231810.GA674@morn.localdomain> <20120516134657.GU32036@redhat.com> <4FB3CCC7.4070701@redhat.com> <20120516164028.GC28798@redhat.com> In-Reply-To: <20120516164028.GC28798@redhat.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 2/2] Get system state configuration from QEMU and patcth DSDT with it. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gleb Natapov Cc: Kevin O'Connor , seabios@seabios.org, qemu-devel@nongnu.org Il 16/05/2012 18:40, Gleb Natapov ha scritto: > On Wed, May 16, 2012 at 05:50:31PM +0200, Paolo Bonzini wrote: >> Il 16/05/2012 15:46, Gleb Natapov ha scritto: >>> I saw that, but I don't get why doing it this way instead of defining >>> the object in AML and patching it? I can define Name(S4VL, 0x2) and path >>> 0x2 to whatever QEMU wants me to use, or I can patch Package directly >>> like I did. >>> >> >> Can we build an SSDT that includes the contents of fw_cfg (e.g. >> FW_CFG_SIGNATURE at offset 0, FW_CFG_UUID at offset 4, FW_CFG_NOGRAPHIC >> at offset 16... the entry <-> offset mapping and the defaults would be >> part of SeaBIOS), and then read that data from normal DSDT methods? > > Kevin does not want to use offsets any more :) He wants to use files, so > this will not work for new entries. Then we can have: - a table in SeaBIOS with (filenames, expected length) pairs - an ACPI table with a list of offsets for each file (-1 if file not found or length < expected length), with the same indices as the previous table - and another blob with all the files concatenated The idea is the same, just pass the fw_cfg data to the DSDT and read it from there. As long as Windows and Linux can cope with the more complex AML, there is no need to do complicated patching IMO... >> >> That would be similar to Gerd's patch, but without letting the OSPM use >> the real fw_cfg device. >> > Latest Gerd's patch does not use fw_cfg device. Yes, I meant the same as his first patch, not really the machanics of creating the BDAT. Paolo