[PATCH v2] UBUNTU: [Config] Mark CONFIG_SCSI_VIRTIO built-in

Stefan Bader stefan.bader at canonical.com
Wed Mar 29 07:17:23 UTC 2023


On 28.03.23 16:51, Dimitri John Ledkov wrote:
> BugLink: https://bugs.launchpad.net/bugs/1685291
> BugLink: https://bugs.launchpad.net/bugs/1931841
> BugLink: https://bugs.launchpad.net/bugs/2006488
> 
> This enables initrd-less boot on generic kernel, enabling any Ubuntu
> installation to be lifted from any hardware and converted into a VM
> which will, even with corrupted initrd and missing modules, be able to
> boot by default. This is an extremly useful property to have on all
> ubuntu kernels. Separately, this also removes the need for linux-kvm
> kernel flavour, as all the changes it has are now incorporated into
> generic flavour & cloud-images.
> 
> Signed-off-by: Dimitri John Ledkov <dimitri.ledkov at canonical.com>
> ---

This could be more clear by saying _WHICH_ series. Given the content is 
annotation only it likely can be assumed Lunar. But I would rather 
prefer explicit. Also, instead of adding a ton of bug references, which 
become multiple mentions in the changelog, I would rather choose ONE bug 
report that is generic and good enough and duplicate the others against 
it. Then one also needs only one justification in one report. At least 
to me that sounds much simpler...

-Stefan
> 
>   Changes since v1:
>   - add buglinks
>   - improve annotations comment
> 
>   debian.master/config/annotations | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/debian.master/config/annotations b/debian.master/config/annotations
> index bc466d7318..4017f1f651 100644
> --- a/debian.master/config/annotations
> +++ b/debian.master/config/annotations
> @@ -528,8 +528,8 @@ CONFIG_SCSI_IPR_DUMP                            note<'LP: #1343109'>
>   CONFIG_SCSI_IPR_TRACE                           policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 'riscv64': 'y'}>
>   CONFIG_SCSI_IPR_TRACE                           note<'LP: #1343109'>
>   
> -CONFIG_SCSI_VIRTIO                              policy<{'amd64': 'm', 'arm64': 'm', 'armhf': 'm', 'ppc64el': 'm', 'riscv64': 'm', 's390x': 'm'}>
> -CONFIG_SCSI_VIRTIO                              note<'tech preview of new feature'>
> +CONFIG_SCSI_VIRTIO                              policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 'riscv64': 'y', 's390x': 'y'}>
> +CONFIG_SCSI_VIRTIO                              note<'enable initrdless boot in VMs for ease of use, cloud-kernel parity and disaster recovery'>
>   
>   CONFIG_SECURITY_SAFESETID                       policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 'riscv64': 'y', 's390x': 'y'}>
>   CONFIG_SECURITY_SAFESETID                       note<'LP: #1845391'>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0xE8675DEECBEECEA3.asc
Type: application/pgp-keys
Size: 44613 bytes
Desc: OpenPGP public key
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20230329/1cc4e3b5/attachment-0001.key>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20230329/1cc4e3b5/attachment-0001.sig>


More information about the kernel-team mailing list