qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
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: [Qemu-devel] [PATCH v5 0/5] nvdimm: hotplug support
Date: Fri,  4 Nov 2016 02:36:21 +0800	[thread overview]
Message-ID: <1478198186-45204-1-git-send-email-guangrong.xiao@linux.intel.com> (raw)

Hi Michael,

This patchset can replace the patches from [PULL 36/47] to [PULL 39/47]
in your pull request:
[PULL 36/47] nvdimm acpi: prebuild nvdimm devices for available slots
[PULL 37/47] nvdimm acpi: introduce fit buffer
[PULL 38/47] nvdimm acpi: introduce _FIT
[PULL 39/47] pc: memhp: enable nvdimm device hotplug

Thanks for your patience also thank Igor and Stefan for their review.

Chanelog in v5:
   1) make definition of dsm return values be a separate patch
   2) drop stopping nvdimm hot remove in pc_dimm_unplug()
   3) drop length field in the output of Read_Fit
   4) doc fixes & improvement
   5) better naming some functions
   6) code cleanup   
   
Changelog in v4:
   1) drop fit lock and post_hotplug_cb
   2) move nvdimm hotplug code to hw/acpi/nvdimm.c
   3) introduce length field to indicate the fit size
   4) nvdimm acpi cleanup
   5) doc typo fixes

Changelog in v3:
   1) use a dedicated interrupt for nvdimm device hotplug
   2) stop nvdimm device hot unplug
   3) reserve UUID and handle for QEMU internally used QEMU
   5) redesign fit buffer to avoid OSPM reading incomplete fit info
   6) bug fixes and cleanups

Changelog in v2:
   Fixed signed integer overflow pointed out by Stefan Hajnoczi

This patchset enables nvdimm hotplug support, it is used as pc-dimm hotplug,
for example, a new nvdimm device can be plugged as follows:
object_add memory-backend-file,id=mem3,size=10G,mem-path=/home/eric/nvdimm3
device_add nvdimm,id=nvdimm3,memdev=mem3

Xiao Guangrong (5):
  nvdimm acpi: prebuild nvdimm devices for available slots
  nvdimm acpi: introduce fit buffer
  nvdimm acpi: define DSM return codes
  nvdimm acpi: introduce _FIT method
  pc: memhp: enable nvdimm device hotplug

 default-configs/mips-softmmu-common.mak |   1 +
 docs/specs/acpi_nvdimm.txt              |  66 ++++++-
 hw/acpi/ich9.c                          |   8 +-
 hw/acpi/nvdimm.c                        | 321 +++++++++++++++++++++++++++-----
 hw/acpi/piix4.c                         |   7 +-
 hw/i386/acpi-build.c                    |   9 +-
 hw/i386/pc.c                            |  10 +
 hw/mem/nvdimm.c                         |   4 -
 include/hw/acpi/acpi_dev_interface.h    |   1 +
 include/hw/mem/nvdimm.h                 |  23 ++-
 10 files changed, 390 insertions(+), 60 deletions(-)

-- 
1.8.3.1

             reply	other threads:[~2016-11-03 18:51 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-03 18:36 Xiao Guangrong [this message]
2016-11-03 18:36 ` [Qemu-devel] [PATCH v5 1/5] nvdimm acpi: prebuild nvdimm devices for available slots Xiao Guangrong
2016-11-03 18:36 ` [Qemu-devel] [PATCH v5 2/5] nvdimm acpi: introduce fit buffer Xiao Guangrong
2016-11-04 10:08   ` Igor Mammedov
2016-11-03 18:36 ` [Qemu-devel] [PATCH v5 3/5] nvdimm acpi: define DSM return codes Xiao Guangrong
2016-11-03 18:36 ` [Qemu-devel] [PATCH v5 4/5] nvdimm acpi: introduce _FIT method Xiao Guangrong
2016-11-04 10:24   ` Igor Mammedov
2016-11-03 18:36 ` [Qemu-devel] [PATCH v5 5/5] pc: memhp: enable nvdimm device hotplug Xiao Guangrong
2016-11-04  3:50 ` [Qemu-devel] [PATCH v5 0/5] nvdimm: hotplug support Xiao Guangrong
2016-11-04  4:01   ` Michael S. Tsirkin
2016-11-04  4:22     ` Michael S. Tsirkin
2016-11-04  7:51       ` Xiao Guangrong
2016-11-04  9:15       ` Stefan Hajnoczi
2016-11-04  9:51         ` Igor Mammedov
2016-11-04 10:01           ` Xiao Guangrong
2016-11-07 13:04           ` Stefan Hajnoczi

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=1478198186-45204-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).