qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Laszlo Ersek <lersek@redhat.com>
To: "Michael S. Tsirkin" <mst@redhat.com>,
	Igor Mammedov <imammedo@redhat.com>
Cc: Xiao Guangrong <guangrong.xiao@linux.intel.com>,
	ehabkost@redhat.com, kvm@vger.kernel.org, gleb@kernel.org,
	mtosatti@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com,
	pbonzini@redhat.com, dan.j.williams@intel.com, rth@twiddle.net
Subject: Re: [Qemu-devel] How to reserve guest physical region for ACPI
Date: Thu, 7 Jan 2016 18:08:02 +0100	[thread overview]
Message-ID: <568E9B72.6000304@redhat.com> (raw)
In-Reply-To: <20160107123214-mutt-send-email-mst@redhat.com>

On 01/07/16 11:54, Michael S. Tsirkin wrote:
> On Thu, Jan 07, 2016 at 11:30:25AM +0100, Igor Mammedov wrote:
>> On Tue, 5 Jan 2016 18:43:02 +0200
>> "Michael S. Tsirkin" <mst@redhat.com> wrote:
>>
>>> On Tue, Jan 05, 2016 at 05:30:25PM +0100, Igor Mammedov wrote:

...

>>>>>>> An alternative is to add an XSDT, XP ignores that.
>>>>>>> XSDT at the moment breaks OVMF (because it loads both
>>>>>>> the RSDT and the XSDT, which is wrong), but I think
>>>>>>> Laszlo was working on a fix for that.    
>>>>>> Using XSDT would increase ACPI tables occupied RAM
>>>>>> as it would duplicate DSDT + non XP supported AML
>>>>>> at global namespace.    
>>>>>
>>>>> Not at all - I posted patches linking to same
>>>>> tables from both RSDT and XSDT at some point.
>>>>> Only the list of pointers would be different.  
>>>> if you put XP incompatible AML in separate SSDT and link it
>>>> only from XSDT than that would work but if incompatibility
>>>> is in DSDT, one would have to provide compat DSDT for RSDT
>>>> an incompat DSDT for XSDT.  
>>>
>>> So don't do this.
>> well spec says "An ACPI-compatible OS must use the XSDT if present",
>> which I read as tables pointed by RSDT MUST be pointed by XSDT
>> as well and RSDT MUST NOT not be used.
>>
>> so if we put incompatible changes in a separate SSDT and put
>> it only in XSDT that might work. Showstopper here is OVMF which
>> has issues with it as Laszlo pointed out.
> 
> But that's just a bug.

Yes, but the bug (actually: lack of feature) is in the UEFI
specification. The current EFI_ACPI_TABLE_PROTOCOL implementation in
edk2 conforms to the specification. In order to expose the functionality
that the above trick needs, the UEFI spec has to be changed. In my
(limited, admittedly) experience, that's an uphill battle.

[...]

      parent reply	other threads:[~2016-01-07 17:08 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-02  7:20 [Qemu-devel] [PATCH v9 0/5] implement vNVDIMM Xiao Guangrong
2015-12-02  7:20 ` [Qemu-devel] [PATCH v9 1/5] nvdimm: implement NVDIMM device abstract Xiao Guangrong
2015-12-02  7:20 ` [Qemu-devel] [PATCH v9 2/5] acpi: support specified oem table id for build_header Xiao Guangrong
2015-12-02  7:20 ` [Qemu-devel] [PATCH v9 3/5] nvdimm acpi: build ACPI NFIT table Xiao Guangrong
2015-12-02  7:20 ` [Qemu-devel] [PATCH v9 4/5] nvdimm acpi: build ACPI nvdimm devices Xiao Guangrong
2015-12-02  7:21 ` [Qemu-devel] [PATCH v9 5/5] nvdimm: add maintain info Xiao Guangrong
2015-12-10  3:11 ` [Qemu-devel] [PATCH v9 0/5] implement vNVDIMM Xiao Guangrong
2015-12-21 14:13   ` Xiao Guangrong
2015-12-28  2:39 ` [Qemu-devel] How to reserve guest physical region for ACPI Xiao Guangrong
2015-12-28 12:50   ` Michael S. Tsirkin
2015-12-30 15:55     ` Igor Mammedov
2015-12-30 19:52       ` Michael S. Tsirkin
2016-01-04 20:17         ` Laszlo Ersek
2016-01-05 17:08           ` Igor Mammedov
2016-01-05 17:22             ` Laszlo Ersek
2016-01-06 13:39               ` Igor Mammedov
2016-01-06 14:43                 ` Laszlo Ersek
2016-01-07 13:51           ` Igor Mammedov
2016-01-07 17:33             ` Laszlo Ersek
2016-01-05 16:30         ` Igor Mammedov
2016-01-05 16:43           ` Michael S. Tsirkin
2016-01-05 17:07             ` Laszlo Ersek
2016-01-05 17:07             ` Xiao Guangrong
2016-01-07  9:21               ` Igor Mammedov
2016-01-08  4:21                 ` Xiao Guangrong
2016-01-08  9:42                   ` Laszlo Ersek
2016-01-08 15:59                   ` Igor Mammedov
2016-01-07 10:30             ` Igor Mammedov
2016-01-07 10:54               ` Michael S. Tsirkin
2016-01-07 13:42                 ` Igor Mammedov
2016-01-07 17:11                   ` Laszlo Ersek
2016-01-07 17:08                 ` Laszlo Ersek [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=568E9B72.6000304@redhat.com \
    --to=lersek@redhat.com \
    --cc=dan.j.williams@intel.com \
    --cc=ehabkost@redhat.com \
    --cc=gleb@kernel.org \
    --cc=guangrong.xiao@linux.intel.com \
    --cc=imammedo@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=mst@redhat.com \
    --cc=mtosatti@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).