virdevmapper: Deal with kernels without DM support

[PATCH 0/2] virdevmapper: Deal with kernels without DM support

Michal Privoznik posted 2 patches 3 years, 8 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1597674163.git.mprivozn@redhat.com

src/util/virdevmapper.c | 59 ++++++++++++++++++++++++++++++++---------
1 file changed, 47 insertions(+), 12 deletions(-)

Expand all Fold all

[PATCH 0/2] virdevmapper: Deal with kernels without DM support

Posted by Michal Privoznik 3 years, 8 months ago

*** BLURB HERE ***

Michal Prívozník (2):
  virdevmapper: Don't error on kernels without DM support
  virdevmapper: Deal with unloading dm module

 src/util/virdevmapper.c | 59 ++++++++++++++++++++++++++++++++---------
 1 file changed, 47 insertions(+), 12 deletions(-)

-- 
2.26.2

Re: [PATCH 0/2] virdevmapper: Deal with kernels without DM support

Posted by Christian Ehrhardt 3 years, 8 months ago

On Mon, Aug 17, 2020 at 4:27 PM Michal Privoznik <mprivozn@redhat.com> wrote:
>
> *** BLURB HERE ***
>
> Michal Prívozník (2):
>   virdevmapper: Don't error on kernels without DM support
>   virdevmapper: Deal with unloading dm module

Hi Michal,
I know from the discussions on the patches that there likely will be
some changes before this is complete.
Nevertheless I did some testing with them already.

In particular since I was reporting this to also affect system
containers that have device-mapper
loaded as module and in /proc/devices but no /dev/mapper/control
exposed. I wanted to
give your patches a try for this use case as well since I wasn't sure
anyone else would.

I can confirm that functionally your patches applied on top of 6.6 (as
we work on it for
Debian&Ubuntu) make it work again. Therefore:

Tested-by: Christian Ehrhardt <christian.ehrhardt@canonical.com>

>  src/util/virdevmapper.c | 59 ++++++++++++++++++++++++++++++++---------
>  1 file changed, 47 insertions(+), 12 deletions(-)
>
> --
> 2.26.2
>

-- 
Christian Ehrhardt
Staff Engineer, Ubuntu Server
Canonical Ltd