mbox

[PULL,yakkety] Switch to compressed Image on arm64

Message ID CALdTtnujkutpaOMwVxYYYaoPLpF0Ey76N8JG+Qm-qjziizO0Pg@mail.gmail.com
State New
Headers show

Pull-request

git://git.launchpad.net/~dannf/ubuntu/+source/linux/+git/yakkety arm64-zimage

Message

dann frazier Aug. 29, 2016, 4:49 p.m. UTC
BugLink: http://bugs.launchpad.net/bugs/1384955

Building a 4.8-rc3+ kernel with the current Ubuntu config produces an image
too large for X-Gene/U-Boot systems. The compressed Image is about 1/3 the
size of the uncompressed Image, so switching to that will resolve the issue
with some growing room. And, obviously, it'll save disk space on all arm64
platforms.

UEFI can handle the compressed image just fine. xgene-uboot systems
(Mustang/McDivitt) can as well, as long as the uImage header has
the correct compression flags. Until recently, this was not the case.
uImages were always generated w/ compression=none flag. However, both
flash-kernel and debian-installer in yakkety have recently been updated to
dynamically detect the compression type and set the uImage flag correctly.
After some bake time, I plan to SRU that back into xenial to be ready
for hwe kernels.

The following changes since commit 7b77e0d5f133eddf7da7e042dbd4b91fbdb25a6c:

  UBUNTU: [Config] CONFIG_ARCH_ROCKCHIP=y for arm64 (2016-08-25 09:02:50 -0600)

are available in the git repository at:

  git://git.launchpad.net/~dannf/ubuntu/+source/linux/+git/yakkety arm64-zimage

for you to fetch changes up to ac7a01f271a4da2092735a0fe2308d3f5e41617a:

  UBUNTU: [Config] Switch to compressed Image on arm64 (2016-08-29
09:40:11 -0600)

----------------------------------------------------------------
dann frazier (1):
      UBUNTU: [Config] Switch to compressed Image on arm64

 debian.master/control.d/flavour-control.stub | 1 +
 debian.master/rules.d/arm64.mk               | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

Comments

Paolo Pisati Aug. 30, 2016, 10:10 a.m. UTC | #1
+1, makes sense.

On Mon, Aug 29, 2016 at 6:49 PM, Dann Frazier
<dann.frazier@canonical.com> wrote:
> BugLink: http://bugs.launchpad.net/bugs/1384955
>
> Building a 4.8-rc3+ kernel with the current Ubuntu config produces an image
> too large for X-Gene/U-Boot systems. The compressed Image is about 1/3 the
> size of the uncompressed Image, so switching to that will resolve the issue
> with some growing room. And, obviously, it'll save disk space on all arm64
> platforms.
>
> UEFI can handle the compressed image just fine. xgene-uboot systems
> (Mustang/McDivitt) can as well, as long as the uImage header has
> the correct compression flags. Until recently, this was not the case.
> uImages were always generated w/ compression=none flag. However, both
> flash-kernel and debian-installer in yakkety have recently been updated to
> dynamically detect the compression type and set the uImage flag correctly.
> After some bake time, I plan to SRU that back into xenial to be ready
> for hwe kernels.
>
> The following changes since commit 7b77e0d5f133eddf7da7e042dbd4b91fbdb25a6c:
>
>   UBUNTU: [Config] CONFIG_ARCH_ROCKCHIP=y for arm64 (2016-08-25 09:02:50 -0600)
>
> are available in the git repository at:
>
>   git://git.launchpad.net/~dannf/ubuntu/+source/linux/+git/yakkety arm64-zimage
>
> for you to fetch changes up to ac7a01f271a4da2092735a0fe2308d3f5e41617a:
>
>   UBUNTU: [Config] Switch to compressed Image on arm64 (2016-08-29
> 09:40:11 -0600)
>
> ----------------------------------------------------------------
> dann frazier (1):
>       UBUNTU: [Config] Switch to compressed Image on arm64
>
>  debian.master/control.d/flavour-control.stub | 1 +
>  debian.master/rules.d/arm64.mk               | 4 ++--
>  2 files changed, 3 insertions(+), 2 deletions(-)
>
> --
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
Tim Gardner Aug. 30, 2016, 4:52 p.m. UTC | #2
On 08/29/2016 10:49 AM, Dann Frazier wrote:
> git://git.launchpad.net/~dannf/ubuntu/+source/linux/+git/yakkety arm64-zimage

Why a conflict instead of a 'Depends:' ?

+Conflicts: flash-kernel (<< 3.0~rc.4ubuntu64) [arm64]

rtg
dann frazier Aug. 30, 2016, 10:03 p.m. UTC | #3
On Tue, Aug 30, 2016 at 10:52 AM, Tim Gardner <tim.gardner@canonical.com> wrote:
> On 08/29/2016 10:49 AM, Dann Frazier wrote:
>> git://git.launchpad.net/~dannf/ubuntu/+source/linux/+git/yakkety arm64-zimage
>
> Why a conflict instead of a 'Depends:' ?
>
> +Conflicts: flash-kernel (<< 3.0~rc.4ubuntu64) [arm64]

linux-image doesn't depend on flash-kernel. Nor should it - many
systems (e.g. UEFI) don't need it. What this is saying is, "if you
require flash-kernel, it needs to be at least this version".

  -dann
Tim Gardner Aug. 31, 2016, 3:53 p.m. UTC | #4
On 08/30/2016 04:03 PM, Dann Frazier wrote:
> On Tue, Aug 30, 2016 at 10:52 AM, Tim Gardner <tim.gardner@canonical.com> wrote:
>> On 08/29/2016 10:49 AM, Dann Frazier wrote:
>>> git://git.launchpad.net/~dannf/ubuntu/+source/linux/+git/yakkety arm64-zimage
>>
>> Why a conflict instead of a 'Depends:' ?
>>
>> +Conflicts: flash-kernel (<< 3.0~rc.4ubuntu64) [arm64]
> 
> linux-image doesn't depend on flash-kernel. Nor should it - many
> systems (e.g. UEFI) don't need it. What this is saying is, "if you
> require flash-kernel, it needs to be at least this version".
> 
>   -dann
> 

duh! I guess that makes sense.