From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [PATCH 0/4] acpi: fix up EJ0 in DSDT Date: Tue, 27 Sep 2011 16:04:45 +0300 Message-ID: <20110927130445.GA11673@redhat.com> References: <20110922043513.GA488@morn.localdomain> <20110922060948.GA29819@redhat.com> <20110926044018.GB16938@morn.localdomain> <20110926070412.GA5860@redhat.com> <20110927000424.GA32368@morn.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Amos Kong , seabios@seabios.org, Gleb Natapov , kvm@vger.kernel.org, jasowang@redhat.com, alex williamson , Marcelo Tosatti To: "Kevin O'Connor" Return-path: Received: from mx1.redhat.com ([209.132.183.28]:57302 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751609Ab1I0NDu (ORCPT ); Tue, 27 Sep 2011 09:03:50 -0400 Content-Disposition: inline In-Reply-To: <20110927000424.GA32368@morn.localdomain> Sender: kvm-owner@vger.kernel.org List-ID: On Mon, Sep 26, 2011 at 08:04:24PM -0400, Kevin O'Connor wrote: > On Mon, Sep 26, 2011 at 10:04:13AM +0300, Michael S. Tsirkin wrote: > > On Mon, Sep 26, 2011 at 12:40:18AM -0400, Kevin O'Connor wrote: > > > On Thu, Sep 22, 2011 at 09:09:49AM +0300, Michael S. Tsirkin wrote: > > > > On Thu, Sep 22, 2011 at 12:35:13AM -0400, Kevin O'Connor wrote: > > > > > The code to generate basic SSDT code isn't that difficult (see > > > > > build_ssdt and src/ssdt-proc.dsl). Is there a compelling reason to > > > > > patch the DSDT versus just generating the necessary blocks in an SSDT? > > > > > > > > I don't really care whether the code is in DSDT or SSDT, > > > > IMO there isn't much difference between build_ssdt and patching: > > > > main reason is build_ssdt uses offsets hardcoded to a specific binary > > > > (ssdt_proc and SD_OFFSET_* ) while I used > > > > a script to extract offsets. > > > > > > Yes - your script to extract the offsets is nice. > > > > If you still have doubts, > > it might make sense to merge just patch 1 - > > acpi: generate and parse mixed asl/aml listing > > - as the first step. > > With the infrastructure in place it will be > > easier to discuss the best way to use it. > > I'm okay with your first patch. BTW, any more comments with the rest of the patchset? If you just need to think about it, I understand. > However, I wish to tag a release > before committing ACPI changes. Sure. So you'll take this patchset from here or want me to ping you later? > There was a concern raised with > two-pass PCI initialization that I need to follow up on before > tagging. The isa bridge? I thought that got fixed ... > > -Kevin -- MST