From: Stefan Berger <stefanb@linux.vnet.ibm.com>
To: Kevin O'Connor <kevin@koconnor.net>, Igor Mammedov <imammedo@redhat.com>
Cc: safford@watson.ibm.com, qemu-devel@nongnu.org, quan.xu@intel.com,
mst@redhat.com
Subject: Re: [Qemu-devel] [PATCH 0/5] Extend TPM support with a QEMU-external TPM
Date: Tue, 05 May 2015 13:42:33 -0400 [thread overview]
Message-ID: <55490109.4010603@linux.vnet.ibm.com> (raw)
In-Reply-To: <20150505025003.GA20966@morn.localdomain>
On 05/04/2015 10:50 PM, Kevin O'Connor wrote:
> On Mon, May 04, 2015 at 11:41:54PM +0200, Igor Mammedov wrote:
>> It's not really split implementation, it's installing an additional
>> table which is purely SeaBIOS specific since QEMU doesn't care about
>> PPID data at all nor does it need to do so.
> If both SeaBIOS and QEMU provide acpi tables, it can lead to
> confusion. For example, if seabios adds tables, what acpi version
> should go into the acpi table header, if Windows BSODs what project
> should the bug report go to, how do we ensure no namespace collisions
> between SSDTs, etc.
>
> So, it's technically not difficult, but the support is painful.
>
>> But I'm fine with paravirt implementation if above is considered too
>> hard to implement. If you go this route look at build_ssdt() in QEMU
>> and a new aml_*() API that helps to create AML objects dynamically.
> I think this approach is simpler all around.
I did that now and will repost the series here and those for SeaBIOS.
Stefan
next prev parent reply other threads:[~2015-05-05 17:42 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-15 22:38 [Qemu-devel] [PATCH 0/5] Extend TPM support with a QEMU-external TPM Stefan Berger
2015-04-15 22:38 ` [Qemu-devel] [PATCH 1/5] Provide support for the CUSE TPM Stefan Berger
2015-04-15 22:38 ` [Qemu-devel] [PATCH 2/5] Support Physical Presence Interface Spec Stefan Berger
2015-04-15 22:38 ` [Qemu-devel] [PATCH 3/5] Introduce condition to notifiy waiters of completed command Stefan Berger
2015-04-15 22:38 ` [Qemu-devel] [PATCH 4/5] Introduce condition in TPM backend for notification Stefan Berger
2015-04-15 22:38 ` [Qemu-devel] [PATCH 5/5] Add support for VM suspend/resume for TPM TIS Stefan Berger
2015-04-16 13:35 ` [Qemu-devel] [PATCH 0/5] Extend TPM support with a QEMU-external TPM Igor Mammedov
2015-04-16 14:05 ` Stefan Berger
2015-04-22 7:00 ` Igor Mammedov
2015-04-22 18:18 ` Stefan Berger
2015-04-29 9:06 ` Igor Mammedov
2015-04-29 16:42 ` Stefan Berger
2015-05-04 9:16 ` Igor Mammedov
2015-05-04 15:22 ` Stefan Berger
2015-05-04 16:16 ` Kevin O'Connor
2015-05-04 18:39 ` Stefan Berger
2015-05-04 21:41 ` Igor Mammedov
2015-05-05 2:50 ` Kevin O'Connor
2015-05-05 17:42 ` Stefan Berger [this message]
2015-04-16 18:55 ` Michael S. Tsirkin
2015-04-16 19:21 ` Stefan Berger
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=55490109.4010603@linux.vnet.ibm.com \
--to=stefanb@linux.vnet.ibm.com \
--cc=imammedo@redhat.com \
--cc=kevin@koconnor.net \
--cc=mst@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=quan.xu@intel.com \
--cc=safford@watson.ibm.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).