Merge lp://qastaging/~vicamo/ubuntu/vivid/openssl/switch-to-cntvct into lp://qastaging/ubuntu/vivid/openssl

Proposed by You-Sheng Yang
Status: Superseded
Proposed branch: lp://qastaging/~vicamo/ubuntu/vivid/openssl/switch-to-cntvct
Merge into: lp://qastaging/ubuntu/vivid/openssl
Diff against target: 38 lines (+26/-0)
2 files modified
debian/patches/armv4cpuid.S-switch-to-CNTVCT-tick-counter.patch (+25/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp://qastaging/~vicamo/ubuntu/vivid/openssl/switch-to-cntvct
Reviewer Review Type Date Requested Status
Marc Deslauriers Needs Fixing
Review via email: mp+296749@code.qastaging.launchpad.net

This proposal has been superseded by a proposal from 2016-06-14.

Description of the change

Use cntvct virtual timer for feature probing on ARM arch. This fixes following error found on Ubuntu Phone devices:

  Jan 1 00:00:47 ubuntu-phablet kernel: [ 38.026652] (2)[2316:ntpdate]USR_MODE / SVC_MODE Undefined Instruction Address curr:ffffffc05596c000 pc=00000000f716f3a8: (null), instr: 0xee190f1d / compat: yes

The patch was formated from https://github.com/openssl/openssl/commit/723463282fab13a686a30779c401b98b82134a62, which has been included in openssl-1.1.0-pre1. Already applied in openssl-1.0.2d-0ubuntu1 in Xenial, too.

To post a comment you must log in.
Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Hi,

The patch looks OK, but you also need to add an entry to debian/changelog.

Thanks!

review: Needs Fixing

Unmerged revisions

112. By You-Sheng Yang

Switch to CNTVCT tick counter on arm

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches