ACK/Cmnt: [lunar:linux][lunar:linux-unstable][SRU][PATCH v3] UBUNTU: [Config] Mark CONFIG_SCSI_VIRTIO built-in

[Andrei Gherzan]

On 23/04/18 02:26PM, Dimitri John Ledkov wrote:
> BugLink: https://bugs.launchpad.net/bugs/1685291
> 
> 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.
> 
> Other related bugs in related projects, that this change alone
> improves, but doesn't resolve in full alone:
> Reference: https://bugs.launchpad.net/bugs/1931841
> Reference: https://bugs.launchpad.net/bugs/2006488
> 
> Signed-off-by: Dimitri John Ledkov <dimitri.ledkov at canonical.com>
> Acked-by: Tim Gardner <tim.gardner at canonical.com>
> Acked-by: Andrea Righi <andrea.righi at canonical.com>
> ---
>  Changes:
>  - add subject headers of target kernels
>  - bump v2 to v3
>  - add only one buglink, downgrade others to References
>  - add two acks
>  - addjust note workding and add BugLink to it
> 
>  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..04d2917c6e 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, LP: #1685291'>
>  
>  CONFIG_SECURITY_SAFESETID                       policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 'riscv64': 'y', 's390x': 'y'}>
>  CONFIG_SECURITY_SAFESETID                       note<'LP: #1845391'>
> -- 
> 2.34.1

This looks very useful indeed and I've wished for it a couple of times
in the past when switching kernels in LXC VMs.

Acked-by: Andrei Gherzan <andrei.gherzan at canonical.com>

-- 
Andrei Gherzan