From: Xiao Guangrong <guangrong.xiao@linux.intel.com>
To: pbonzini@redhat.com, imammedo@redhat.com
Cc: gleb@kernel.org, mtosatti@redhat.com, stefanha@redhat.com,
mst@redhat.com, rth@twiddle.net, ehabkost@redhat.com,
dan.j.williams@intel.com, kvm@vger.kernel.org,
qemu-devel@nongnu.org,
Xiao Guangrong <guangrong.xiao@linux.intel.com>
Subject: [PATCH v3 0/8] NVDIMM ACPI: introduce the framework of QEMU emulated DSM
Date: Sun, 14 Feb 2016 16:50:57 +0800 [thread overview]
Message-ID: <1455439865-75784-1-git-send-email-guangrong.xiao@linux.intel.com> (raw)
This patchset is against commit a407644079c86390 (net: set endianness on all
backend devices) on pci branch of Michael's git tree
and can be found at:
https://github.com/xiaogr/qemu.git nvdimm-acpi-v3
Changelog in v3:
Changes addressing Michael's comment:
- rebase the patchset against current code
Changes addressing Igor's comment:
- rename the parameters of aml_create_field() to reflect the ACPI spec
- fix the issue that the @target operand can not be optional in
aml_concatenate() that is also cleaned up by using build_opcode_2arg_dst()
Others:
- separate the test patches to the single set and will be posted on later
These changes are based on Igor's comments:
- drop ssdt.rev2 support as the memory address allocated by BIOS/OVMF
are always 32 bits
- support to test NVDIMM tables (NFIT and NVDIMM SSDT)
- add _CRS to report its operation region
- make AML APIs change be the separated patches
This is the second part of vNVDIMM implementation which implements the
BIOS patched dsm memory and introduces the framework that allows QEMU
to emulate DSM method
Thanks to Michael's idea, we do not reserve any memory for NVDIMM ACPI,
instead we let BIOS allocate the memory and patch the address to the
offset we want
IO port is still enabled as it plays as the way to notify QEMU and pass
the patched dsm memory address, so that IO port region, 0x0a18 - 0xa20,
is reserved and it is divided into two 32 bits ports and used to pass
the low 32 bits and high 32 bits of dsm memory address to QEMU
Thanks Igor's idea, this patchset also extends DSDT/SSDT to revision 2
to apply 64 bit operations, in order to keeping compatibility, old
version (<= 2.5) still uses revision 1. Since 64 bit operations breaks
old guests (such as windows XP), we should keep the 64 bits stuff in
the private place where common ACPI operation does not touch it
Xiao Guangrong (8):
acpi: add aml_create_field()
acpi: add aml_concatenate()
acpi: allow using object as offset for OperationRegion
nvdimm acpi: initialize the resource used by NVDIMM ACPI
nvdimm acpi: introduce patched dsm memory
nvdimm acpi: let qemu handle _DSM method
nvdimm acpi: emulate dsm method
nvdimm acpi: add _CRS
hw/acpi/Makefile.objs | 2 +-
hw/acpi/aml-build.c | 28 ++++-
hw/acpi/nvdimm.c | 255 +++++++++++++++++++++++++++++++++++++++++++-
hw/i386/acpi-build.c | 41 +++----
hw/i386/pc.c | 6 +-
hw/i386/pc_piix.c | 5 +
hw/i386/pc_q35.c | 8 +-
include/hw/acpi/aml-build.h | 6 +-
include/hw/i386/pc.h | 4 +-
include/hw/mem/nvdimm.h | 36 ++++++-
10 files changed, 351 insertions(+), 40 deletions(-)
--
1.8.3.1
next reply other threads:[~2016-02-14 8:59 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-14 8:50 Xiao Guangrong [this message]
2016-02-14 8:50 ` [PATCH v3 1/8] acpi: add aml_create_field() Xiao Guangrong
2016-02-14 8:50 ` [PATCH v3 2/8] acpi: add aml_concatenate() Xiao Guangrong
2016-02-15 9:19 ` Igor Mammedov
2016-02-14 8:51 ` [PATCH v3 3/8] acpi: allow using object as offset for OperationRegion Xiao Guangrong
2016-02-14 8:51 ` [PATCH v3 4/8] nvdimm acpi: initialize the resource used by NVDIMM ACPI Xiao Guangrong
2016-02-14 8:51 ` [PATCH v3 5/8] nvdimm acpi: introduce patched dsm memory Xiao Guangrong
2016-02-29 9:38 ` Michael S. Tsirkin
2016-03-01 8:42 ` Xiao Guangrong
2016-03-01 8:53 ` Xiao Guangrong
2016-03-01 9:08 ` Michael S. Tsirkin
2016-03-01 9:18 ` Xiao Guangrong
2016-02-14 8:51 ` [PATCH v3 6/8] nvdimm acpi: let qemu handle _DSM method Xiao Guangrong
2016-02-14 8:51 ` [PATCH v3 7/8] nvdimm acpi: emulate dsm method Xiao Guangrong
2016-02-14 8:51 ` [PATCH v3 8/8] nvdimm acpi: add _CRS Xiao Guangrong
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=1455439865-75784-1-git-send-email-guangrong.xiao@linux.intel.com \
--to=guangrong.xiao@linux.intel.com \
--cc=dan.j.williams@intel.com \
--cc=ehabkost@redhat.com \
--cc=gleb@kernel.org \
--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).