linaro-media-create not backward compatible

Bug #1050349 reported by viresh kumar
8
Affects Status Importance Assigned to Milestone
Linaro Image Tools
Won't Fix
High
Unassigned

Bug Description

I was trying http://releases.linaro.org/11.11/android/leb-panda/ release for panda board with latest linaro-media-create tool.

It created boot.txt in /boot partition, which had boot with DTB option. DTB's weren't supported in 11.11 Android release.
Should such backward compatibility be supported by this tool?

Tags: regression
Revision history for this message
Alexander Sack (asac) wrote :

release critical. should get fixed. Please reroll release with fix

Changed in linaro-image-tools:
assignee: nobody → Данило Шеган (danilo)
status: New → Confirmed
importance: Undecided → Critical
Revision history for this message
Alexander Sack (asac) wrote :

have no .09 milestone for the infrastructure project anymore... will reopen that milestone for the update.

Changed in linaro-ubuntu:
milestone: none → 12.09
Changed in linaro-image-tools:
milestone: none → 2012.09
Revision history for this message
Alexander Sack (asac) wrote :

reopened and targetted for 2012.09... needs to happen for release next week (deadline: available tuesday, validated wednesday).

Also infra: please setup automation to continue testing all old releases. this isn't so hard.

Changed in linaro-ubuntu:
status: New → Invalid
Changed in linaro-image-tools:
status: Confirmed → Triaged
Fathi Boudra (fboudra)
no longer affects: linaro-ubuntu
tags: added: regression
Revision history for this message
Данило Шеган (danilo) wrote :

When you say automation, I suspect you also mean boot testing? (this problem would not have been detected without it, or at least I suspect that since I don't have a panda to test it on, though I can confirm boot.txt contains board.dtb among the boot parameters)

At this time, I am also unsure how can we easily ensure backwards compatibility regarding boot parameters for Android, considering they are simply static strings made part of the code as well. Should we block any changes to existing code in android_boards.py, because they can easily lead to compatibility problems (other than eg. increasing memory limits, though even those are not completely safe)?

FWIW, this problem was present in 2012.08 release as well (and likely 2012.07, as this change was introduced by Vishal right after 2012.06 was rolled out: https://code.launchpad.net/~vishalbhoj/linaro-image-tools/panda-dt/+merge/111352). Considering it was discovered this late, I'd say this is not a blocker for 2012.09 release, so I'd argue we can have this fixed next cycle as well (iow, no need for a re-roll, especially since the instructions which fail refer to fetching the code from bzr—if they recommended use of the matching release, they'd continue to work).

To clarify, I still consider this a critical regression, but since it is not one introduced by this release, and has been lurking for 3 months, I'd rather us not rush a fix in. My take would be that non-trivial changes are needed to android_boards.py and linaro-android-media-create to allow us to selectively tack dtb boot parameters on while ensuring that we haven't broken compatibility with anything over the last ~10 or so releases.

Revision history for this message
Alexander Sack (asac) wrote :

right. if the problem is not new, it doesnt need to get fixed for this release. should still be tracked as critical and getting infrastructure in place needed to test this should get high prio.

Changed in linaro-image-tools:
milestone: 2012.09 → 2012.10
Fathi Boudra (fboudra)
Changed in linaro-image-tools:
importance: Critical → High
Changed in linaro-image-tools:
milestone: 2012.10 → 2012.11
Changed in linaro-image-tools:
milestone: 2012.11 → 2012.12
Milo Casagrande (milo)
Changed in linaro-image-tools:
milestone: 2012.12 → 2013.01
Revision history for this message
Milo Casagrande (milo) wrote :

With the introduction of Android hwpacks, already in l-i-t trunk and available with 2013.01, I guess this issue might be closed, or do we want to adress it in another way?

Revision history for this message
viresh kumar (viresh.kumar) wrote : Re: [Bug 1050349] Re: linaro-media-create not backward compatible

On 10 January 2013 14:49, Milo Casagrande <email address hidden> wrote:
> With the introduction of Android hwpacks, already in l-i-t trunk and
> available with 2013.01, I guess this issue might be closed, or do we
> want to adress it in another way?

Atleast i am not sure about it :(

Revision history for this message
Данило Шеган (danilo) wrote :

Before we can close it, we should make sure that we extract the bits
that did break compatibility in the past (the ones why this bug was
opened) into a separate "hwpack". Then, our trunk will be able to create
bootable older images as well.

We'll be in an interesting position where it can't boot newer images
without external hwpack configs, but I guess that's ok since it's been
in since 2012.08 I think (i.e. smaller number of releases that'd need
changes).

David Zinman (dzinman)
Changed in linaro-image-tools:
milestone: 2013.01 → 2013.02
Changed in linaro-image-tools:
assignee: Данило Шеган (danilo) → Milo Casagrande (milo)
David Zinman (dzinman)
Changed in linaro-image-tools:
milestone: 2013.02 → backlog
Changed in linaro-image-tools:
assignee: Milo Casagrande (milo) → nobody
Revision history for this message
Milo Casagrande (milo) wrote :

Due to the age of this issue, we are acknowledging that this issue will likely not be fixed, is no longer applicable, or was already fixed by an indirect change. If this issue is still important, please add details and reopen the issue.

Changed in linaro-image-tools:
status: Triaged → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.