diff mbox series

[SRU,J,1/1] UBUNTU: [Config] update tracking bug for CONFIG_INTEL_IOMMU_DEFAULT_ON

Message ID 20240209085250.36609-1-andrea.righi@canonical.com
State New
Headers show
Series [SRU,J,1/1] UBUNTU: [Config] update tracking bug for CONFIG_INTEL_IOMMU_DEFAULT_ON | expand

Commit Message

Andrea Righi Feb. 9, 2024, 8:52 a.m. UTC
BugLink: https://bugs.launchpad.net/bugs/1971699

In commit ("UBUNTU: [Config] disable Intel DMA remapping options by
default") we disabled Intel IOMMU by default, but the entry in
annotations is missing a direct reference to the tracking bug.

Fix it by adding the tracker bug to the note.

Signed-off-by: Andrea Righi <andrea.righi@canonical.com>
---
 debian.master/config/annotations | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Andrei Gherzan Feb. 9, 2024, 10:34 a.m. UTC | #1
On 24/02/09 09:52AM, Andrea Righi wrote:
> BugLink: https://bugs.launchpad.net/bugs/1971699
> 
> In commit ("UBUNTU: [Config] disable Intel DMA remapping options by
> default") we disabled Intel IOMMU by default, but the entry in
> annotations is missing a direct reference to the tracking bug.
> 
> Fix it by adding the tracker bug to the note.
> 
> Signed-off-by: Andrea Righi <andrea.righi@canonical.com>
> ---
>  debian.master/config/annotations | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/debian.master/config/annotations b/debian.master/config/annotations
> index f27bbeca498b..1a2eded51d47 100644
> --- a/debian.master/config/annotations
> +++ b/debian.master/config/annotations
> @@ -289,7 +289,7 @@ CONFIG_INTEGRITY_PLATFORM_KEYRING               policy<{'amd64': 'y', 'arm64': '
>  CONFIG_INTEGRITY_PLATFORM_KEYRING               note<'Required for lockdown'>
>  
>  CONFIG_INTEL_IOMMU_DEFAULT_ON                   policy<{'amd64': 'n'}>
> -CONFIG_INTEL_IOMMU_DEFAULT_ON                   note<'the IOMMU can trigger boot failures'>
> +CONFIG_INTEL_IOMMU_DEFAULT_ON                   note<'LP: #1971699'>
>  
>  CONFIG_IOMMU_DEBUGFS                            policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'ppc64el': 'n', 's390x': 'n'}>
>  CONFIG_IOMMU_DEBUGFS                            note<'Should not be enabled in production, LP #1861057'>

Acked-by: Andrei Gherzan <andrei.gherzan@canonical.com>
Manuel Diewald Feb. 9, 2024, 12:52 p.m. UTC | #2
On Fri, Feb 09, 2024 at 09:52:50AM +0100, Andrea Righi wrote:
> BugLink: https://bugs.launchpad.net/bugs/1971699
> 
> In commit ("UBUNTU: [Config] disable Intel DMA remapping options by
> default") we disabled Intel IOMMU by default, but the entry in
> annotations is missing a direct reference to the tracking bug.
> 
> Fix it by adding the tracker bug to the note.
> 
> Signed-off-by: Andrea Righi <andrea.righi@canonical.com>
> ---
>  debian.master/config/annotations | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/debian.master/config/annotations b/debian.master/config/annotations
> index f27bbeca498b..1a2eded51d47 100644
> --- a/debian.master/config/annotations
> +++ b/debian.master/config/annotations
> @@ -289,7 +289,7 @@ CONFIG_INTEGRITY_PLATFORM_KEYRING               policy<{'amd64': 'y', 'arm64': '
>  CONFIG_INTEGRITY_PLATFORM_KEYRING               note<'Required for lockdown'>
>  
>  CONFIG_INTEL_IOMMU_DEFAULT_ON                   policy<{'amd64': 'n'}>
> -CONFIG_INTEL_IOMMU_DEFAULT_ON                   note<'the IOMMU can trigger boot failures'>
> +CONFIG_INTEL_IOMMU_DEFAULT_ON                   note<'LP: #1971699'>
>  
>  CONFIG_IOMMU_DEBUGFS                            policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'ppc64el': 'n', 's390x': 'n'}>
>  CONFIG_IOMMU_DEBUGFS                            note<'Should not be enabled in production, LP #1861057'>
> -- 
> 2.43.0
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team

Acked-by: Manuel Diewald <manuel.diewald@canonical.com>
Roxana Nicolescu Feb. 12, 2024, 8:45 a.m. UTC | #3
On 09/02/2024 09:52, Andrea Righi wrote:
> BugLink: https://bugs.launchpad.net/bugs/1971699
>
> In commit ("UBUNTU: [Config] disable Intel DMA remapping options by
> default") we disabled Intel IOMMU by default, but the entry in
> annotations is missing a direct reference to the tracking bug.
>
> Fix it by adding the tracker bug to the note.
>
> Signed-off-by: Andrea Righi <andrea.righi@canonical.com>
> ---
>   debian.master/config/annotations | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/debian.master/config/annotations b/debian.master/config/annotations
> index f27bbeca498b..1a2eded51d47 100644
> --- a/debian.master/config/annotations
> +++ b/debian.master/config/annotations
> @@ -289,7 +289,7 @@ CONFIG_INTEGRITY_PLATFORM_KEYRING               policy<{'amd64': 'y', 'arm64': '
>   CONFIG_INTEGRITY_PLATFORM_KEYRING               note<'Required for lockdown'>
>   
>   CONFIG_INTEL_IOMMU_DEFAULT_ON                   policy<{'amd64': 'n'}>
> -CONFIG_INTEL_IOMMU_DEFAULT_ON                   note<'the IOMMU can trigger boot failures'>
> +CONFIG_INTEL_IOMMU_DEFAULT_ON                   note<'LP: #1971699'>
>   
>   CONFIG_IOMMU_DEBUGFS                            policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'ppc64el': 'n', 's390x': 'n'}>
>   CONFIG_IOMMU_DEBUGFS                            note<'Should not be enabled in production, LP #1861057'>
Applied to jammy master-next branch. Thanks!
diff mbox series

Patch

diff --git a/debian.master/config/annotations b/debian.master/config/annotations
index f27bbeca498b..1a2eded51d47 100644
--- a/debian.master/config/annotations
+++ b/debian.master/config/annotations
@@ -289,7 +289,7 @@  CONFIG_INTEGRITY_PLATFORM_KEYRING               policy<{'amd64': 'y', 'arm64': '
 CONFIG_INTEGRITY_PLATFORM_KEYRING               note<'Required for lockdown'>
 
 CONFIG_INTEL_IOMMU_DEFAULT_ON                   policy<{'amd64': 'n'}>
-CONFIG_INTEL_IOMMU_DEFAULT_ON                   note<'the IOMMU can trigger boot failures'>
+CONFIG_INTEL_IOMMU_DEFAULT_ON                   note<'LP: #1971699'>
 
 CONFIG_IOMMU_DEBUGFS                            policy<{'amd64': 'n', 'arm64': 'n', 'armhf': 'n', 'ppc64el': 'n', 's390x': 'n'}>
 CONFIG_IOMMU_DEBUGFS                            note<'Should not be enabled in production, LP #1861057'>