[v3] nvdimm: fixes for (non-)dax backends

[Qemu-devel] [PATCH-for-2.12 v3 0/3] nvdimm: fixes for (non-)dax backends

Haozhong Zhang posted 3 patches 6 years, 4 months ago

Diff against v2 v4
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20171127043517.22441-1-haozhong.zhang@intel.com

Test checkpatch passed

Test docker failed

Test ppc passed

Test s390x passed

There is a newer version of this series

backends/hostmem-file.c | 41 ++++++++++++++++++++++++++++++++++++++++-
docs/nvdimm.txt         | 31 +++++++++++++++++++++++++++++++
exec.c                  |  8 +++++++-
hw/acpi/nvdimm.c        |  7 +++++++
hw/mem/nvdimm.c         | 28 +++++++++++++++++++++++++++-
include/exec/memory.h   |  3 +++
include/hw/mem/nvdimm.h | 12 ++++++++++++
memory.c                |  2 ++
numa.c                  |  2 +-
9 files changed, 130 insertions(+), 4 deletions(-)

Expand all Fold all

[Qemu-devel] [PATCH-for-2.12 v3 0/3] nvdimm: fixes for (non-)dax backends

Posted by Haozhong Zhang 6 years, 4 months ago

Previous versions can be found at
  v2: https://lists.gnu.org/archive/html/qemu-devel/2017-06/msg01203.html
  v1: https://lists.gnu.org/archive/html/qemu-devel/2017-05/msg05919.html

Changes in v3:
 * Add an option 'align' to 'memory-backend-file' to address the
   failure when mmap device dax (patch 1).
 * Remove device dax check, which needs to access sysfs and may not
   work with SELinux.
 * Add a boolean option 'unarmed' to '-device nvdimm', which allows
   users to control the unarmed flag in guest ACPI NFIT. I don't make
   it as OnOffAuto, because of the remove of device dax check.
 * Document new options added by this patch series.

Haozhong Zhang (3):
  hostmem-file: add "align" option
  nvdimm: add a macro for property "label-size"
  nvdimm: add 'unarmed' option

 backends/hostmem-file.c | 41 ++++++++++++++++++++++++++++++++++++++++-
 docs/nvdimm.txt         | 31 +++++++++++++++++++++++++++++++
 exec.c                  |  8 +++++++-
 hw/acpi/nvdimm.c        |  7 +++++++
 hw/mem/nvdimm.c         | 28 +++++++++++++++++++++++++++-
 include/exec/memory.h   |  3 +++
 include/hw/mem/nvdimm.h | 12 ++++++++++++
 memory.c                |  2 ++
 numa.c                  |  2 +-
 9 files changed, 130 insertions(+), 4 deletions(-)

-- 
2.14.1

Re: [Qemu-devel] [PATCH-for-2.12 v3 0/3] nvdimm: fixes for (non-)dax backends

Posted by Michael S. Tsirkin 6 years, 4 months ago

On Mon, Nov 27, 2017 at 12:35:14PM +0800, Haozhong Zhang wrote:
> Previous versions can be found at
>   v2: https://lists.gnu.org/archive/html/qemu-devel/2017-06/msg01203.html
>   v1: https://lists.gnu.org/archive/html/qemu-devel/2017-05/msg05919.html
> 
> Changes in v3:
>  * Add an option 'align' to 'memory-backend-file' to address the
>    failure when mmap device dax (patch 1).
>  * Remove device dax check, which needs to access sysfs and may not
>    work with SELinux.
>  * Add a boolean option 'unarmed' to '-device nvdimm', which allows
>    users to control the unarmed flag in guest ACPI NFIT. I don't make
>    it as OnOffAuto, because of the remove of device dax check.
>  * Document new options added by this patch series.

If you expect me to be the one to merge this, pls copy me
on all patches.


> Haozhong Zhang (3):
>   hostmem-file: add "align" option
>   nvdimm: add a macro for property "label-size"
>   nvdimm: add 'unarmed' option
> 
>  backends/hostmem-file.c | 41 ++++++++++++++++++++++++++++++++++++++++-
>  docs/nvdimm.txt         | 31 +++++++++++++++++++++++++++++++
>  exec.c                  |  8 +++++++-
>  hw/acpi/nvdimm.c        |  7 +++++++
>  hw/mem/nvdimm.c         | 28 +++++++++++++++++++++++++++-
>  include/exec/memory.h   |  3 +++
>  include/hw/mem/nvdimm.h | 12 ++++++++++++
>  memory.c                |  2 ++
>  numa.c                  |  2 +-
>  9 files changed, 130 insertions(+), 4 deletions(-)
> 
> -- 
> 2.14.1