[PATCH 0/4] bhyve: implement NVMe device support

Roman Bogorodskiy posted 4 patches 1 month, 2 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20251025081558.18715-1-bogorodskiy@gmail.com
src/bhyve/bhyve_capabilities.c                | 14 ++++++
src/bhyve/bhyve_capabilities.h                |  1 +
src/bhyve/bhyve_command.c                     | 47 ++++++++++++++++++-
src/bhyve/bhyve_device.c                      |  1 +
src/bhyve/bhyve_domain.c                      | 42 +++++++++++++++--
.../bhyvexml2argv-2-nvme-2-controllers.args   | 10 ++++
.../bhyvexml2argv-2-nvme-2-controllers.ldargs |  4 ++
.../bhyvexml2argv-2-nvme-2-controllers.xml    | 21 +++++++++
.../bhyvexml2argv-2-nvme-same-controller.args | 10 ++++
...hyvexml2argv-2-nvme-same-controller.ldargs |  4 ++
.../bhyvexml2argv-2-nvme-same-controller.xml  | 21 +++++++++
.../bhyvexml2argvdata/bhyvexml2argv-nvme.args |  9 ++++
.../bhyvexml2argv-nvme.ldargs                 |  4 ++
.../bhyvexml2argvdata/bhyvexml2argv-nvme.xml  | 16 +++++++
tests/bhyvexml2argvtest.c                     |  9 +++-
.../bhyvexml2xmlout-2-nvme-2-controllers.xml  | 36 ++++++++++++++
.../bhyvexml2xmlout-nvme.xml                  | 27 +++++++++++
tests/bhyvexml2xmltest.c                      |  2 +
18 files changed, 272 insertions(+), 6 deletions(-)
create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.args
create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.ldargs
create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.xml
create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.args
create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.ldargs
create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.xml
create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.args
create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.ldargs
create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.xml
create mode 100644 tests/bhyvexml2xmloutdata/bhyvexml2xmlout-2-nvme-2-controllers.xml
create mode 100644 tests/bhyvexml2xmloutdata/bhyvexml2xmlout-nvme.xml
[PATCH 0/4] bhyve: implement NVMe device support
Posted by Roman Bogorodskiy 1 month, 2 weeks ago
Roman Bogorodskiy (4):
  bhyve: implement NVMe device support
  bhyve: tests: cover 2 NVMe devices on 2 controllers case
  bhyve: do not allow more than one NVMe device per controller
  bhyve: nvme: check if NVMe is supported by bhyve

 src/bhyve/bhyve_capabilities.c                | 14 ++++++
 src/bhyve/bhyve_capabilities.h                |  1 +
 src/bhyve/bhyve_command.c                     | 47 ++++++++++++++++++-
 src/bhyve/bhyve_device.c                      |  1 +
 src/bhyve/bhyve_domain.c                      | 42 +++++++++++++++--
 .../bhyvexml2argv-2-nvme-2-controllers.args   | 10 ++++
 .../bhyvexml2argv-2-nvme-2-controllers.ldargs |  4 ++
 .../bhyvexml2argv-2-nvme-2-controllers.xml    | 21 +++++++++
 .../bhyvexml2argv-2-nvme-same-controller.args | 10 ++++
 ...hyvexml2argv-2-nvme-same-controller.ldargs |  4 ++
 .../bhyvexml2argv-2-nvme-same-controller.xml  | 21 +++++++++
 .../bhyvexml2argvdata/bhyvexml2argv-nvme.args |  9 ++++
 .../bhyvexml2argv-nvme.ldargs                 |  4 ++
 .../bhyvexml2argvdata/bhyvexml2argv-nvme.xml  | 16 +++++++
 tests/bhyvexml2argvtest.c                     |  9 +++-
 .../bhyvexml2xmlout-2-nvme-2-controllers.xml  | 36 ++++++++++++++
 .../bhyvexml2xmlout-nvme.xml                  | 27 +++++++++++
 tests/bhyvexml2xmltest.c                      |  2 +
 18 files changed, 272 insertions(+), 6 deletions(-)
 create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.args
 create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.ldargs
 create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.xml
 create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.args
 create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.ldargs
 create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.xml
 create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.args
 create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.ldargs
 create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.xml
 create mode 100644 tests/bhyvexml2xmloutdata/bhyvexml2xmlout-2-nvme-2-controllers.xml
 create mode 100644 tests/bhyvexml2xmloutdata/bhyvexml2xmlout-nvme.xml

-- 
2.51.0
Re: [PATCH 0/4] bhyve: implement NVMe device support
Posted by Michal Prívozník via Devel 1 month, 2 weeks ago
On 10/25/25 10:15, Roman Bogorodskiy wrote:
> Roman Bogorodskiy (4):
>   bhyve: implement NVMe device support
>   bhyve: tests: cover 2 NVMe devices on 2 controllers case
>   bhyve: do not allow more than one NVMe device per controller
>   bhyve: nvme: check if NVMe is supported by bhyve
> 
>  src/bhyve/bhyve_capabilities.c                | 14 ++++++
>  src/bhyve/bhyve_capabilities.h                |  1 +
>  src/bhyve/bhyve_command.c                     | 47 ++++++++++++++++++-
>  src/bhyve/bhyve_device.c                      |  1 +
>  src/bhyve/bhyve_domain.c                      | 42 +++++++++++++++--
>  .../bhyvexml2argv-2-nvme-2-controllers.args   | 10 ++++
>  .../bhyvexml2argv-2-nvme-2-controllers.ldargs |  4 ++
>  .../bhyvexml2argv-2-nvme-2-controllers.xml    | 21 +++++++++
>  .../bhyvexml2argv-2-nvme-same-controller.args | 10 ++++
>  ...hyvexml2argv-2-nvme-same-controller.ldargs |  4 ++
>  .../bhyvexml2argv-2-nvme-same-controller.xml  | 21 +++++++++
>  .../bhyvexml2argvdata/bhyvexml2argv-nvme.args |  9 ++++
>  .../bhyvexml2argv-nvme.ldargs                 |  4 ++
>  .../bhyvexml2argvdata/bhyvexml2argv-nvme.xml  | 16 +++++++
>  tests/bhyvexml2argvtest.c                     |  9 +++-
>  .../bhyvexml2xmlout-2-nvme-2-controllers.xml  | 36 ++++++++++++++
>  .../bhyvexml2xmlout-nvme.xml                  | 27 +++++++++++
>  tests/bhyvexml2xmltest.c                      |  2 +
>  18 files changed, 272 insertions(+), 6 deletions(-)
>  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.args
>  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.ldargs
>  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.xml
>  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.args
>  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.ldargs
>  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.xml
>  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.args
>  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.ldargs
>  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.xml
>  create mode 100644 tests/bhyvexml2xmloutdata/bhyvexml2xmlout-2-nvme-2-controllers.xml
>  create mode 100644 tests/bhyvexml2xmloutdata/bhyvexml2xmlout-nvme.xml
> 

This deserves mentioning in NEWS.rst ;-)

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal
Re: [PATCH 0/4] bhyve: implement NVMe device support
Posted by Jiri Denemark via Devel 1 month, 2 weeks ago
On Wed, Oct 29, 2025 at 10:47:25 +0100, Michal Prívozník wrote:
> On 10/25/25 10:15, Roman Bogorodskiy wrote:
> > Roman Bogorodskiy (4):
> >   bhyve: implement NVMe device support
> >   bhyve: tests: cover 2 NVMe devices on 2 controllers case
> >   bhyve: do not allow more than one NVMe device per controller
> >   bhyve: nvme: check if NVMe is supported by bhyve
> > 
> >  src/bhyve/bhyve_capabilities.c                | 14 ++++++
> >  src/bhyve/bhyve_capabilities.h                |  1 +
> >  src/bhyve/bhyve_command.c                     | 47 ++++++++++++++++++-
> >  src/bhyve/bhyve_device.c                      |  1 +
> >  src/bhyve/bhyve_domain.c                      | 42 +++++++++++++++--
> >  .../bhyvexml2argv-2-nvme-2-controllers.args   | 10 ++++
> >  .../bhyvexml2argv-2-nvme-2-controllers.ldargs |  4 ++
> >  .../bhyvexml2argv-2-nvme-2-controllers.xml    | 21 +++++++++
> >  .../bhyvexml2argv-2-nvme-same-controller.args | 10 ++++
> >  ...hyvexml2argv-2-nvme-same-controller.ldargs |  4 ++
> >  .../bhyvexml2argv-2-nvme-same-controller.xml  | 21 +++++++++
> >  .../bhyvexml2argvdata/bhyvexml2argv-nvme.args |  9 ++++
> >  .../bhyvexml2argv-nvme.ldargs                 |  4 ++
> >  .../bhyvexml2argvdata/bhyvexml2argv-nvme.xml  | 16 +++++++
> >  tests/bhyvexml2argvtest.c                     |  9 +++-
> >  .../bhyvexml2xmlout-2-nvme-2-controllers.xml  | 36 ++++++++++++++
> >  .../bhyvexml2xmlout-nvme.xml                  | 27 +++++++++++
> >  tests/bhyvexml2xmltest.c                      |  2 +
> >  18 files changed, 272 insertions(+), 6 deletions(-)
> >  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.args
> >  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.ldargs
> >  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-2-controllers.xml
> >  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.args
> >  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.ldargs
> >  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-2-nvme-same-controller.xml
> >  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.args
> >  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.ldargs
> >  create mode 100644 tests/bhyvexml2argvdata/bhyvexml2argv-nvme.xml
> >  create mode 100644 tests/bhyvexml2xmloutdata/bhyvexml2xmlout-2-nvme-2-controllers.xml
> >  create mode 100644 tests/bhyvexml2xmloutdata/bhyvexml2xmlout-nvme.xml
> > 
> 
> This deserves mentioning in NEWS.rst ;-)
> 
> Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

I pushed the series since I'm preparing for a freeze. Please, send NEWS
update as a separate patch.

Jirka