[Ubuntu18.10][Backport] Crypto VMX Fixes

Bug #1841488 reported by bugproxy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
The Ubuntu-power-systems project
Invalid
Low
Unassigned
linux (Ubuntu)
Invalid
Undecided
Ubuntu on IBM Power Systems Bug Triage

Bug Description

Cosmic is missing the fix for "5c380d623ed3 - crypto: vmx - Add support for VMS instructions by ASM (4 years, 5 months ago) <Leonidas S. Barbosa>" ... which is fixed by 'crypto: vmx - fix copy-paste error in CTR mode', it fixes a couple of incorrect branch targets, and usage of an incorrect vector assembly instruction.

The others two patches ("crypto: vmx - CTR: always increment IV as quadword ", "crypto: vmx - ghash: do nosimd fallback manually ") fixes for ghash replaces the fallback mechanism that is used on powerpc an operation is required to operate in interrupt context.

These fixes are important to avoid a potential data integrity issue.

Revision history for this message
bugproxy (bugproxy) wrote : crypto: vmx - fix copy-paste error in CTR mode

Default Comment by Bridge

tags: added: architecture-ppc64le bugnameltc-178285 severity-high targetmilestone-inin1810
Revision history for this message
bugproxy (bugproxy) wrote : crypto: vmx - CTR: always increment IV as quadword

Default Comment by Bridge

Revision history for this message
bugproxy (bugproxy) wrote : crypto: vmx - ghash: do nosimd fallback manually

Default Comment by Bridge

Changed in ubuntu:
assignee: nobody → Ubuntu on IBM Power Systems Bug Triage (ubuntu-power-triage)
affects: ubuntu → linux (Ubuntu)
Revision history for this message
Andrew Cloke (andrew-cloke) wrote :

Cosmic (18.10) is now end of life. It has been replaced by Disco (19.04). See https://wiki.ubuntu.com/Releases for further details.

Can this issue be closed?

Changed in ubuntu-power-systems:
importance: Undecided → Low
status: New → Incomplete
Changed in linux (Ubuntu):
status: New → Incomplete
Revision history for this message
Frank Heimes (fheimes) wrote :

As alreday mentioned in comment #4 18.10/cosmic reached it's EOL and one needs to migrate to disco:

So I just checked the situation on 19.04/disco.
In the current disco master two patches are in and two are out (as of today):
$ git log --oneline | grep -m 1 "Add support for VMS instructions by ASM"
5c380d6 crypto: vmx - Add support for VMS instructions by ASM
$ git log --oneline | grep -m 1 "fix copy-paste error in CTR mode"
6cef65e crypto: vmx - fix copy-paste error in CTR mode
$ git log --oneline | grep -m 1 "always increment IV as quadword"
$
$ git log --oneline | grep -m 1 "do nosimd fallback manually"
$

However disco master-next includes all patches (probably via a stable upstream release update):
$ git log --oneline | grep -m 1 "Add support for VMS instructions by ASM"
5c380d6 crypto: vmx - Add support for VMS instructions by ASM
$ git log --oneline | grep -m 1 "fix copy-paste error in CTR mode"
6cef65e crypto: vmx - fix copy-paste error in CTR mode
$ git log --oneline | grep -m 1 "always increment IV as quadword"
b5df7b3 crypto: vmx - CTR: always increment IV as quadword
$ git log --oneline | grep -m 1 "do nosimd fallback manually"
c438f2b crypto: vmx - ghash: do nosimd fallback manually

Hence I suggest to wait for the next and upcoming disco/19.10 kernel and close this LP once it got rolled out to the main pocket.

Revision history for this message
Frank Heimes (fheimes) wrote :

Closing this ticket (setting to Invalid, since it's about cosmic).
A new ticket got opened that addresses this issue specially for disco: LP 1841984
https://bugs.launchpad.net/ubuntu-power-systems/+bug/1841984

Changed in linux (Ubuntu):
status: Incomplete → Invalid
Changed in ubuntu-power-systems:
status: Incomplete → Invalid
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.