| Message ID | 20210824153203.51572-1-dimitri.ledkov@canonical.com |
|---|---|
| State | New |
| Headers | show |
| Series | [UNSTABLE,IMPISH] UBUNTU: [Config] enforce ZSTD compression | expand |
Acked-by: Tim Gardner <tim.gardner@canonical.com> On 8/24/21 9:32 AM, Dimitri John Ledkov wrote: > BugLink: https://bugs.launchpad.net/bugs/1931725 > > Previously v5.13+ based kernels enabled ZSTD kernel image compression > on amd64 & s390x. Due to significant bootspeed impact this change > should be enforced for all derivatives as well. Not all derivatives > currently inherit and enforce all annotations from parent config. > > Fixes: f88bdfaeaf ("UBUNTU: [Packaging] use ZSTD to compress s390 kernels") > Signed-off-by: Dimitri John Ledkov <dimitri.ledkov@canonical.com> > --- > debian.master/config/annotations | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/debian.master/config/annotations b/debian.master/config/annotations > index 22d07b3661..c9c451e95f 100644 > --- a/debian.master/config/annotations > +++ b/debian.master/config/annotations > @@ -11371,6 +11371,8 @@ CONFIG_KERNEL_LZO policy<{'amd64': 'n', 'armhf': ' > CONFIG_KERNEL_LZ4 policy<{'amd64': 'n', 'armhf': 'n', 's390x': 'n'}> > CONFIG_KERNEL_ZSTD policy<{'amd64': 'y', 's390x': 'y'}> > CONFIG_KERNEL_UNCOMPRESSED policy<{'s390x': 'n'}> > +# > +CONFIG_KERNEL_ZSTD mark<ENFORCED> note<LP:#1931725> > > # Menu: General setup >> Namespaces support > CONFIG_NAMESPACES policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 's390x': 'y'}> >
On 24/08/2021 17:32, Dimitri John Ledkov wrote: > BugLink: https://bugs.launchpad.net/bugs/1931725 > > Previously v5.13+ based kernels enabled ZSTD kernel image compression > on amd64 & s390x. Due to significant bootspeed impact this change > should be enforced for all derivatives as well. Not all derivatives > currently inherit and enforce all annotations from parent config. > > Fixes: f88bdfaeaf ("UBUNTU: [Packaging] use ZSTD to compress s390 kernels") > Signed-off-by: Dimitri John Ledkov <dimitri.ledkov@canonical.com> > --- > debian.master/config/annotations | 2 ++ > 1 file changed, 2 insertions(+) Best regards, Krzysztof
On 24/08/2021 20:50, Krzysztof Kozlowski wrote: > On 24/08/2021 17:32, Dimitri John Ledkov wrote: >> BugLink: https://bugs.launchpad.net/bugs/1931725 >> >> Previously v5.13+ based kernels enabled ZSTD kernel image compression >> on amd64 & s390x. Due to significant bootspeed impact this change >> should be enforced for all derivatives as well. Not all derivatives >> currently inherit and enforce all annotations from parent config. >> >> Fixes: f88bdfaeaf ("UBUNTU: [Packaging] use ZSTD to compress s390 kernels") >> Signed-off-by: Dimitri John Ledkov <dimitri.ledkov@canonical.com> >> --- >> debian.master/config/annotations | 2 ++ >> 1 file changed, 2 insertions(+) > Fat finger. So one more time: Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> Best regards, Krzysztof
Applied and pushed to unstable & impish-next.
diff --git a/debian.master/config/annotations b/debian.master/config/annotations index 22d07b3661..c9c451e95f 100644 --- a/debian.master/config/annotations +++ b/debian.master/config/annotations @@ -11371,6 +11371,8 @@ CONFIG_KERNEL_LZO policy<{'amd64': 'n', 'armhf': ' CONFIG_KERNEL_LZ4 policy<{'amd64': 'n', 'armhf': 'n', 's390x': 'n'}> CONFIG_KERNEL_ZSTD policy<{'amd64': 'y', 's390x': 'y'}> CONFIG_KERNEL_UNCOMPRESSED policy<{'s390x': 'n'}> +# +CONFIG_KERNEL_ZSTD mark<ENFORCED> note<LP:#1931725> # Menu: General setup >> Namespaces support CONFIG_NAMESPACES policy<{'amd64': 'y', 'arm64': 'y', 'armhf': 'y', 'ppc64el': 'y', 's390x': 'y'}>
BugLink: https://bugs.launchpad.net/bugs/1931725 Previously v5.13+ based kernels enabled ZSTD kernel image compression on amd64 & s390x. Due to significant bootspeed impact this change should be enforced for all derivatives as well. Not all derivatives currently inherit and enforce all annotations from parent config. Fixes: f88bdfaeaf ("UBUNTU: [Packaging] use ZSTD to compress s390 kernels") Signed-off-by: Dimitri John Ledkov <dimitri.ledkov@canonical.com> --- debian.master/config/annotations | 2 ++ 1 file changed, 2 insertions(+)