ACK/cmnt: [trusty] [PULL] UBUNTU: fix no console in case of cloud environment

Stefan Bader stefan.bader at canonical.com
Wed May 13 09:04:13 UTC 2015


On 12.05.2015 09:03, Ming Lei wrote:
> In current arm64 Trusty cloud image, the 'console=' parameter
> has been removed from grub config, and always resort to 'stdout'
> DT property which is setup from QEMU.
> 
> Unfortunately parsing 'stdout' DT property is just merged to 3.19
> kernel, so there is no console usable any more for arm64 Trusty
> VM when booting via uefi/grub.
> 
> 9 of 10 patches are cherry picked from 3.19 to trusty, and the
> other one is backported by just dropping the dt self-test code, and
> with these 10 patches, arm64 Trusty VM can be booted with console
> usable via uefi/grub.
> 
> BugLink: http://bugs.launchpad.net/bugs/1438585
> 
> SRU Justification:
>     Impact: no console if arm64 trusty based VM is booted from the cloud image
>     Fix: backport stdout enablement patches from upstream
>     Testcase: boot one arm64 trusty VM via uefi/grub
> 
> So please pull the top 10 patches from below tree/branch:
> 
>      git://kernel.ubuntu.com/git/ming/ubuntu-trusty.git  stdout_support

Wrong URI: "git://kernel.ubuntu.com/ming/ubuntu-trusty.git  stdout_support" is
the correct one. ;)
> 
> 
> 120c28c of: support passing console options with stdout-path
> 1a3d0e7 of: add optional options parameter to of_find_node_by_path()
> f29d72d of: Add bindings for chosen node, stdout-path
> 98cbe35 of: correct of_console_check()'s return value
> ecadb06 of: Enable console on serial ports specified by /chosen/stdout-path
> 30f9087 of: Create of_console_check() for selecting a console specified in /chos
> 0ac7353 of: Make of_find_node_by_path() handle /aliases
> 7c7e998 of: Create unlocked version of for_each_child_of_node()
> 651599f lib: add glibc style strchrnul() variant
> f26ae5f of: Add support for ePAPR "stdout-path" property

The lib change adds a function which should be ok. The rest of the changes
appear to be doing what they claim and are mostly limited to the openfirmware
driver. The regression risk should be limited.

-Stefan
> 
> Thanks,
> Ming Lei
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20150513/cb0092d0/attachment.sig>


More information about the kernel-team mailing list