mbox series

[SRU,jammy/linux-kvm,0/1] UBUNTU: [Config] kvm: Add support for dummy net driver

Message ID 20230124183139.83310-1-jacob.martin@canonical.com
Headers show
Series UBUNTU: [Config] kvm: Add support for dummy net driver | expand

Message

Jacob Martin Jan. 24, 2023, 6:31 p.m. UTC
https://bugs.launchpad.net/bugs/1998244

Note: This issue was originally triaged and fixed by ~joseogando for lunar/linux-kvm. I am applying the fix to kinetic and jammy as training.

SRU Justification:

[Impact]

* Without Kernel support for the Dummy net driver, it is not possible to successfully run selftests that use it. This includes as per this bug net/fib_nexthop_nongw.sh, but also could potentially include other such as rtnetlink.sh, fib_rule_tests.sh, forwarding/* etc.

[Fix]

* Custom patch to enable CONFIG_DUMMY=m

[Test Plan]

* Before the fix, the net/fib_nexthop_nongw.sh script will fail in the specific line which does

$ ip -n h1 link add name eth0 type dummy
Error: Unknown device type.

After the fix it does not fail.

[Where problems could occur]

* The regression can be considered as low, since we are adding functionality not previously present.

[Other Info]

* There does not seem to be a historical reason for removing it from linux-kvm.

Jacob Martin (1):
  UBUNTU: [Config] kvm: Add support for dummy net driver

 debian.kvm/config/annotations          | 3 +++
 debian.kvm/config/config.common.ubuntu | 2 +-
 2 files changed, 4 insertions(+), 1 deletion(-)

Comments

Tim Gardner Jan. 24, 2023, 7:08 p.m. UTC | #1
On 1/24/23 11:31, Jacob Martin wrote:
> https://bugs.launchpad.net/bugs/1998244
> 
> Note: This issue was originally triaged and fixed by ~joseogando for lunar/linux-kvm. I am applying the fix to kinetic and jammy as training.
> 
> SRU Justification:
> 
> [Impact]
> 
> * Without Kernel support for the Dummy net driver, it is not possible to successfully run selftests that use it. This includes as per this bug net/fib_nexthop_nongw.sh, but also could potentially include other such as rtnetlink.sh, fib_rule_tests.sh, forwarding/* etc.
> 
> [Fix]
> 
> * Custom patch to enable CONFIG_DUMMY=m
> 
> [Test Plan]
> 
> * Before the fix, the net/fib_nexthop_nongw.sh script will fail in the specific line which does
> 
> $ ip -n h1 link add name eth0 type dummy
> Error: Unknown device type.
> 
> After the fix it does not fail.
> 
> [Where problems could occur]
> 
> * The regression can be considered as low, since we are adding functionality not previously present.
> 
> [Other Info]
> 
> * There does not seem to be a historical reason for removing it from linux-kvm.
> 
> Jacob Martin (1):
>    UBUNTU: [Config] kvm: Add support for dummy net driver
> 
>   debian.kvm/config/annotations          | 3 +++
>   debian.kvm/config/config.common.ubuntu | 2 +-
>   2 files changed, 4 insertions(+), 1 deletion(-)
> 
Acked-by: Tim Gardner <tim.gardner@canonical.com>
Luke Nowakowski-Krijger Jan. 27, 2023, 7:01 p.m. UTC | #2
Acked-by: Luke Nowakowski-Krijger <luke.nowakowskikrijger@canonical.com

On Tue, Jan 24, 2023 at 10:32 AM Jacob Martin <jacob.martin@canonical.com>
wrote:

> https://bugs.launchpad.net/bugs/1998244
>
> Note: This issue was originally triaged and fixed by ~joseogando for
> lunar/linux-kvm. I am applying the fix to kinetic and jammy as training.
>
> SRU Justification:
>
> [Impact]
>
> * Without Kernel support for the Dummy net driver, it is not possible to
> successfully run selftests that use it. This includes as per this bug
> net/fib_nexthop_nongw.sh, but also could potentially include other such as
> rtnetlink.sh, fib_rule_tests.sh, forwarding/* etc.
>
> [Fix]
>
> * Custom patch to enable CONFIG_DUMMY=m
>
> [Test Plan]
>
> * Before the fix, the net/fib_nexthop_nongw.sh script will fail in the
> specific line which does
>
> $ ip -n h1 link add name eth0 type dummy
> Error: Unknown device type.
>
> After the fix it does not fail.
>
> [Where problems could occur]
>
> * The regression can be considered as low, since we are adding
> functionality not previously present.
>
> [Other Info]
>
> * There does not seem to be a historical reason for removing it from
> linux-kvm.
>
> Jacob Martin (1):
>   UBUNTU: [Config] kvm: Add support for dummy net driver
>
>  debian.kvm/config/annotations          | 3 +++
>  debian.kvm/config/config.common.ubuntu | 2 +-
>  2 files changed, 4 insertions(+), 1 deletion(-)
>
> --
> 2.25.1
>
>
> --
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
>
Luke Nowakowski-Krijger Jan. 31, 2023, 6:02 p.m. UTC | #3
Applied to jammy:linux-kvm master-next

Thanks!
- Luke

On Tue, Jan 24, 2023 at 10:32 AM Jacob Martin <jacob.martin@canonical.com>
wrote:

> https://bugs.launchpad.net/bugs/1998244
>
> Note: This issue was originally triaged and fixed by ~joseogando for
> lunar/linux-kvm. I am applying the fix to kinetic and jammy as training.
>
> SRU Justification:
>
> [Impact]
>
> * Without Kernel support for the Dummy net driver, it is not possible to
> successfully run selftests that use it. This includes as per this bug
> net/fib_nexthop_nongw.sh, but also could potentially include other such as
> rtnetlink.sh, fib_rule_tests.sh, forwarding/* etc.
>
> [Fix]
>
> * Custom patch to enable CONFIG_DUMMY=m
>
> [Test Plan]
>
> * Before the fix, the net/fib_nexthop_nongw.sh script will fail in the
> specific line which does
>
> $ ip -n h1 link add name eth0 type dummy
> Error: Unknown device type.
>
> After the fix it does not fail.
>
> [Where problems could occur]
>
> * The regression can be considered as low, since we are adding
> functionality not previously present.
>
> [Other Info]
>
> * There does not seem to be a historical reason for removing it from
> linux-kvm.
>
> Jacob Martin (1):
>   UBUNTU: [Config] kvm: Add support for dummy net driver
>
>  debian.kvm/config/annotations          | 3 +++
>  debian.kvm/config/config.common.ubuntu | 2 +-
>  2 files changed, 4 insertions(+), 1 deletion(-)
>
> --
> 2.25.1
>
>
> --
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
>